linux-kernel - Re: [PATCH][RFC] Create a top-level "Space-critical features" menu.

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: <20070508202215.GV11166@waste.org>
Date:	Tue, 8 May 2007 15:22:15 -0500
From:	Matt Mackall <mpm@...enic.com>
To:	"Robert P. J. Day" <rpjday@...dspring.com>
Cc:	Thomas Gleixner <tglx@...utronix.de>,
	Linux Kernel Mailing List <linux-kernel@...r.kernel.org>
Subject: Re: [PATCH][RFC] Create a top-level "Space-critical features" menu.

On Tue, May 08, 2007 at 04:41:58AM -0400, Robert P. J. Day wrote:
> On Tue, 8 May 2007, Thomas Gleixner wrote:
> 
> > On Tue, 2007-05-08 at 04:06 -0400, Robert P. J. Day wrote:
> > >   i've always hated that lower-level menu under "General setup":
> >
> > Good reason to break everything which depends on CONFIG_EMBEDDED.
> >
> > 	tglx
> >
> > > -menuconfig EMBEDDED
> > > -	bool "Configure standard kernel features (for small systems)"
> 
> ah, now i see what you're getting at here:
> 
> $ grep EMBEDDED $(find . -name Kconfig\*)
> ./drivers/pcmcia/Kconfig:       select CARDBUS if !EMBEDDED
> ./drivers/pcmcia/Kconfig:       bool "Special initialization for O2Micro bridges" if EMBEDDED
> ./drivers/pcmcia/Kconfig:       bool "Special initialization for Ricoh bridges" if EMBEDDED
> ./drivers/pcmcia/Kconfig:       bool "Special initialization for TI and EnE bridges" if EMBEDDED
> ./drivers/pcmcia/Kconfig:       bool "Auto-tune EnE bridges for CB cards" if EMBEDDED
> ./drivers/pcmcia/Kconfig:       bool "Special initialization for Toshiba ToPIC bridges" if EMBEDDED
> ./drivers/ata/Kconfig:  depends on EMBEDDED || ARCH_RPC
> ./drivers/acpi/Kconfig: bool "Power Management Timer Support" if EMBEDDED
> ./drivers/ide/Kconfig:  depends on ALPHA || SUPERH || IA64 || EMBEDDED
> ... snip ...
> 
>   in my opinion, the config option "EMBEDDED" is just plain silly.  as
> i mentioned earlier, there may be other reasons that people want to
> de-activate normally selected features rather than just for strictly
> embedded systems.
> 
>   here's an example:
> 
> drivers/input/keyboard/Kconfig:
> ...
> menuconfig INPUT_KEYBOARD
>         bool "Keyboards" if EMBEDDED || !X86
>         default y
> ...
> 
>   so i should only be allowed to de-activate keyboard support if i
> claim i'm working on an "embedded" system?  that's absurd.  maybe it's
> a 64-CPU monster server that just happens to be running headless and
> needs no keyboard.  making that selection dependent on EMBEDDED is
> just silly in that case.

No, turning off keyboard on your 64-CPU monster server to save ~10k is
the thing that's absurd. More importantly, it's a support headache
because invariably someone will try to plug a keyboard in and complain
to us when it mysteriously doesn't work. If memory serves, Linus
himself did exactly that shortly after the current input layer was
merged.

-- 
Mathematics is the supreme nostalgia of our time.
-
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/