From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1754710AbcC0BTe (ORCPT ); Sat, 26 Mar 2016 21:19:34 -0400 Received: from mail5.windriver.com ([192.103.53.11]:41194 "EHLO mail5.wrs.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1753924AbcC0BTd (ORCPT ); Sat, 26 Mar 2016 21:19:33 -0400 From: Paul Gortmaker To: CC: Paul Gortmaker , Joerg Roedel , "Steven Rostedt (Red Hat)" , Subject: [PATCH] drivers/iommu: don't select DEBUG_FS for AMD_IOMMU_STATS Date: Sat, 26 Mar 2016 21:18:44 -0400 Message-ID: <1459041524-30929-1-git-send-email-paul.gortmaker@windriver.com> X-Mailer: git-send-email 2.6.1 MIME-Version: 1.0 Content-Type: text/plain Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org We have at least one big banner telling people that they should not deploy production kernels with DEBUG options enabled, but at the same time, we make it hard for people to turn DEBUG_FS off when we select (vs. depend on) the CONFIG_DEBUG_FS option. Since we actively discourage people using debug-like features on any builds that are production oriented (see trace_printk banner for one example), so a generic sounding option should not select DEBUG_FS. Since this is not a system critical option, convert it from a select to a depend -- this makes it nicer for us old school folk who edit .config and then run "make oldconfig" as a (bad?) habit. Cc: Joerg Roedel Cc: "Steven Rostedt (Red Hat)" Cc: iommu@lists.linux-foundation.org Signed-off-by: Paul Gortmaker --- drivers/iommu/Kconfig | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-) diff --git a/drivers/iommu/Kconfig b/drivers/iommu/Kconfig index dd1dc39f84ff..959032bdbe12 100644 --- a/drivers/iommu/Kconfig +++ b/drivers/iommu/Kconfig @@ -126,8 +126,7 @@ config AMD_IOMMU config AMD_IOMMU_STATS bool "Export AMD IOMMU statistics to debugfs" - depends on AMD_IOMMU - select DEBUG_FS + depends on AMD_IOMMU && DEBUG_FS ---help--- This option enables code in the AMD IOMMU driver to collect various statistics about whats happening in the driver and exports that -- 2.6.1