From mboxrd@z Thu Jan  1 00:00:00 1970
Return-Path: <linux-kernel-owner@vger.kernel.org>
Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand
	id S1752400AbbEKWPW (ORCPT <rfc822;w@1wt.eu>);
	Mon, 11 May 2015 18:15:22 -0400
Received: from lb3-smtp-cloud3.xs4all.net ([194.109.24.30]:35725 "EHLO
	lb3-smtp-cloud3.xs4all.net" rhost-flags-OK-OK-OK-OK)
	by vger.kernel.org with ESMTP id S1751413AbbEKWPU (ORCPT
	<rfc822;linux-kernel@vger.kernel.org>);
	Mon, 11 May 2015 18:15:20 -0400
Message-ID: <1431382517.2398.127.camel@x220>
Subject: Re: [PATCH 2/1] init/Kconfig: Split expert menu into a separate
 file, init/Kconfig.expert
From: Paul Bolle <pebolle@tiscali.nl>
To: Josh Triplett <josh@joshtriplett.org>
Cc: Ingo Molnar <mingo@kernel.org>, Andrew Morton <akpm@linux-foundation.org>,
        "Paul E. McKenney" <paulmck@linux.vnet.ibm.com>,
        Michal Hocko <mhocko@suse.cz>,
        Vladimir Davydov <vdavydov@parallels.com>,
        Johannes Weiner <hannes@cmpxchg.org>,
        Geert Uytterhoeven <geert@linux-m68k.org>,
        Andy Lutomirski <luto@amacapital.net>,
        Bertrand Jacquin <beber@meleeweb.net>,
        "Luis R. Rodriguez" <mcgrof@suse.com>,
        Iulia Manda <iulia.manda21@gmail.com>,
        Pranith Kumar <bobby.prani@gmail.com>,
        Clark Williams <clark.williams@gmail.com>,
        Mel Gorman <mgorman@suse.de>, Randy Dunlap <rdunlap@infradead.org>,
        linux-kernel@vger.kernel.org
Date: Tue, 12 May 2015 00:15:17 +0200
In-Reply-To: <20150511220419.GB15791@jtriplet-mobl1>
References: <20150511181327.GA1884@jtriplet-mobl1>
	 <20150511202301.GA11897@jtriplet-mobl1> <1431381021.2398.111.camel@x220>
	 <20150511220419.GB15791@jtriplet-mobl1>
Content-Type: text/plain; charset="UTF-8"
X-Mailer: Evolution 3.10.4 (3.10.4-4.fc20) 
Mime-Version: 1.0
Content-Transfer-Encoding: 7bit
Sender: linux-kernel-owner@vger.kernel.org
List-ID: <linux-kernel.vger.kernel.org>
X-Mailing-List: linux-kernel@vger.kernel.org

On Mon, 2015-05-11 at 15:04 -0700, Josh Triplett wrote:
> On Mon, May 11, 2015 at 11:50:21PM +0200, Paul Bolle wrote:
> > On Mon, 2015-05-11 at 13:23 -0700, Josh Triplett wrote:
> > > +config KALLSYMS_ALL
> > > +	bool "Include all symbols in kallsyms"
> > 
> > (For some reason this entry doesn't have if EXPERT but it seems to
> > behave as expected. Odd.)
> 
> Because it depends on KALLSYMS.  Magic!

Welcome to the land of Kconfig!

> > > +	depends on DEBUG_KERNEL && KALLSYMS
> > > +	help
> > > +	   Normally kallsyms only contains the symbols of functions for nicer
> > > +	   OOPS messages and backtraces (i.e., symbols from the text and inittext
> > > +	   sections). This is sufficient for most cases. And only in very rare
> > > +	   cases (e.g., when a debugger is used) all symbols are required (e.g.,
> > > +	   names of variables from the data sections, etc).
> > > +
> > > +	   This option makes sure that all symbols are loaded into the kernel
> > > +	   image (i.e., symbols from all sections) in cost of increased kernel
> > > +	   size (depending on the kernel configuration, it may be 300KiB or
> > > +	   something like this).
> > > +
> > > +	   Say N unless you really need all symbols.
> > > +
> > > +config PRINTK
> > > +	default y
> > > +	bool "Enable support for printk" if EXPERT
> > 
> > Now you're touching this: bool [...] as the first line, please.
> 
> I'd like the commit moving these to their own file to very obviously
> look like it's moving these lines unmodified, rather than making changes
> in the process.  If you want this (and all the subsequent instances you
> flagged) cleaned up, let's do that as a subsequent patch separate from
> the move.

That was one of my pet peeves leaving it's cage. Still, not sure if we'd
really notice if it's not a simple move. But a separate patch would
mainly add noise, so let's not do that.

Thanks,


Paul Bolle