linux-kernel - Re: [PATCH 2/5] init/Kconfig: Split expert menu into a separate file, init/Kconfig.expert

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Message-ID: <1432805068.8508.40.camel@x220>
Date:	Thu, 28 May 2015 11:24:28 +0200
From:	Paul Bolle <pebolle@...cali.nl>
To:	Josh Triplett <josh@...htriplett.org>,
	Andrea Arcangeli <aarcange@...hat.com>,
	Pavel Emelyanov <xemul@...allels.com>,
	Andrew Morton <akpm@...ux-foundation.org>
Cc:	Ingo Molnar <mingo@...nel.org>,
	"Paul E. McKenney" <paulmck@...ux.vnet.ibm.com>,
	Michal Hocko <mhocko@...e.cz>,
	Vladimir Davydov <vdavydov@...allels.com>,
	Johannes Weiner <hannes@...xchg.org>,
	Geert Uytterhoeven <geert@...ux-m68k.org>,
	Andy Lutomirski <luto@...capital.net>,
	Bertrand Jacquin <beber@...eeweb.net>,
	"Luis R. Rodriguez" <mcgrof@...e.com>,
	Iulia Manda <iulia.manda21@...il.com>,
	Pranith Kumar <bobby.prani@...il.com>,
	Clark Williams <clark.williams@...il.com>,
	Mel Gorman <mgorman@...e.de>,
	Randy Dunlap <rdunlap@...radead.org>,
	Michal Marek <mmarek@...e.cz>, linux-kbuild@...r.kernel.org,
	linux-kernel@...r.kernel.org
Subject: Re: [PATCH 2/5] init/Kconfig: Split expert menu into a separate
 file, init/Kconfig.expert

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 <josh@...htriplett.org>

> --- 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 <pebolle@...cali.nl>

Thanks,


Paul Bolle

--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/