From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1752772AbcDEPTz (ORCPT ); Tue, 5 Apr 2016 11:19:55 -0400 Received: from 8bytes.org ([81.169.241.247]:51113 "EHLO theia.8bytes.org" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751615AbcDEPTy (ORCPT ); Tue, 5 Apr 2016 11:19:54 -0400 Date: Tue, 5 Apr 2016 17:19:51 +0200 From: Joerg Roedel To: Paul Gortmaker Cc: linux-kernel@vger.kernel.org, "Steven Rostedt (Red Hat)" , iommu@lists.linux-foundation.org Subject: Re: [PATCH] drivers/iommu: don't select DEBUG_FS for AMD_IOMMU_STATS Message-ID: <20160405151951.GG11519@8bytes.org> References: <1459041524-30929-1-git-send-email-paul.gortmaker@windriver.com> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <1459041524-30929-1-git-send-email-paul.gortmaker@windriver.com> User-Agent: Mutt/1.5.21 (2010-09-15) Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Sat, Mar 26, 2016 at 09:18:44PM -0400, Paul Gortmaker wrote: > 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. That reasoning sounds a bit odd, as most production kernels have DEBUG_FS enabled anyway, and I see no problem with that. Disabling all debug features in 'production kernels' is a bit overkill. Only if the feature has any runtime impact (performance, memory consumption, security, ...) it makes sense to disable it for production kernels. Other features could stay enabled, and DEBUG_FS is one of them. For some debug features we even don't offer a way to disable them, see BUG_ON, WARN_ON and friends. Joerg