From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1753514AbbE1JYm (ORCPT ); Thu, 28 May 2015 05:24:42 -0400 Received: from lb1-smtp-cloud2.xs4all.net ([194.109.24.21]:47058 "EHLO lb1-smtp-cloud2.xs4all.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751548AbbE1JYd (ORCPT ); Thu, 28 May 2015 05:24:33 -0400 Message-ID: <1432805068.8508.40.camel@x220> Subject: Re: [PATCH 2/5] init/Kconfig: Split expert menu into a separate file, init/Kconfig.expert From: Paul Bolle To: Josh Triplett , Andrea Arcangeli , Pavel Emelyanov , Andrew Morton Cc: Ingo Molnar , "Paul E. McKenney" , Michal Hocko , Vladimir Davydov , Johannes Weiner , Geert Uytterhoeven , Andy Lutomirski , Bertrand Jacquin , "Luis R. Rodriguez" , Iulia Manda , Pranith Kumar , Clark Williams , Mel Gorman , Randy Dunlap , Michal Marek , linux-kbuild@vger.kernel.org, linux-kernel@vger.kernel.org Date: Thu, 28 May 2015 11:24:28 +0200 In-Reply-To: References: 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: X-Mailing-List: linux-kernel@vger.kernel.org On Thu, 2015-05-14 at 08:35 -0700, Josh Triplett wrote: > The expert menu frequently gets broken by a config item in the middle > that leaves off the "if EXPERT" from its prompt. This results in the > remainder of the menu spilling out into the parent "General setup" menu. > Move the entire expert menu into a separate Kconfig file, > init/Kconfig.expert, to make this harder to do accidentally, and to > break up the exceedingly long init/Kconfig a bit. > > Signed-off-by: Josh Triplett > --- a/init/Kconfig > +++ b/init/Kconfig > -config ADVISE_SYSCALLS > - bool "Enable madvise/fadvise syscalls" if EXPERT > - default y > - help > - This option enables the madvise and fadvise syscalls, used by > - applications to advise the kernel about their future memory or file > - usage, improving performance. If building an embedded system where no > - applications use these syscalls, you can disable this option to save > - space. > - In the meantime commit d6c6ae9b71ad ("userfaultfd: buildsystem activation") added config USERFAULTFD here in linux-next, pushing PCI_QUIRKS out of the menu. USERFAULTFD defaults to 'y'. Perhaps it was intended to make (its prompt) visible only if EXPERT is set. Andrea, Pavel, Andrew? > -config PCI_QUIRKS > - default y > - bool "Enable PCI quirk workarounds" if EXPERT > - depends on PCI > - help > - This enables workarounds for various PCI chipset > - bugs/quirks. Disable this only if your target machine is > - unaffected by PCI quirks. Anyhow, applied cleanly with d6c6ae9b71ad reverted. I had to jump through some hoops to convince myself the patch really only moves some lines around (except for the added source statement). There's no git diff option that could have helped me here, is there? I'm sure we'll figure out a way to handle config USERFAULTFD, so Acked-by: Paul Bolle Thanks, Paul Bolle