login
Header Space

 
 

Re: [PATCH 2.6.24] chroot= as a new kernel parameter

Score:
Previous message: [thread] [date] [author]
Next message: [thread] [date] [author]
To: Pavel Machek <pavel@...>
Cc: H. Peter Anvin <hpa@...>, Pawel Plociennik <paplociennik@...>, <linux-kernel@...>
Date: Thursday, March 6, 2008 - 5:42 pm

* Pavel Machek <pavel@ucw.cz> wrote:


ouch ...


agreed ...

i really find it so disheartening at times that people fight trivial 
usability additions tooth and nail in a _9 million lines of code_ kernel 
with a ... "bloat" argument.

Lets face it: Linux is _still_ hard and a pain to administer, our kernel 
boot parameters are ad-hoc, they dont match up to the .config parameters 
and it is all a total mess. There's absolutely no design behind them 
(look at all the inconsistent parameter forms for turning off smp, acpi, 
hpet, nohz, etc.).

if RAM overhead of a new boot option would really be an issue on smaller 
setups then the right solution is to make a new .config option that 
hardcodes a specific command line and _disable_ all the commandline 
parsing. That would also be a nice security feature for certain setups 
and would save _a lot more_ RAM than another rejected boot parameter. 
Really, all the 'bloat' based objections are totally, utterly silly.

i had a similar experience when i added the relatime boot option:

  http://people.redhat.com/mingo/relatime-patches/improve-relatime.patch

Look back the lkml discussion for all the "bloat" and "use /etc/fstab" 
clowning around that happened when i sent that patch ... and we still 
have no good configuration vectors to turn atime off. I'd rate it good 
comedy that happened around that patch: "Kernel hackers shoot in their 
own foot and are proud of it".

multiple, consistent vectors for configurability are _GOOD_. That was 
the success story behind Apache. Forcing everyone into a "you must use 
an initrd for this" idea is 80's thinking and actively harmful to Linux.

	Ingo
--
Previous message: [thread] [date] [author]
Next message: [thread] [date] [author]

Messages in current thread:
[PATCH 2.6.24] chroot= as a new kernel parameter, Pawel Plociennik, (Thu Mar 6, 1:01 am)
Re: [PATCH 2.6.24] chroot= as a new kernel parameter, Chris Wedgwood, (Thu Mar 6, 6:34 am)
Re: [PATCH 2.6.24] chroot= as a new kernel parameter, Ingo Molnar, (Thu Mar 6, 6:44 am)
Re: [PATCH 2.6.24] chroot= as a new kernel parameter, Chris Wedgwood, (Thu Mar 6, 7:22 am)
Re: [PATCH 2.6.24] chroot= as a new kernel parameter, Ingo Molnar, (Thu Mar 6, 7:37 am)
Re: [PATCH 2.6.24] chroot= as a new kernel parameter, Chris Wedgwood, (Fri Mar 7, 4:23 am)
Re: [PATCH 2.6.24] chroot= as a new kernel parameter, Pavel Machek, (Thu Mar 6, 7:53 am)
Re: [PATCH 2.6.24] chroot= as a new kernel parameter, Peter Zijlstra, (Thu Mar 6, 3:46 pm)
Re: [PATCH 2.6.24] chroot= as a new kernel parameter, Willy Tarreau, (Thu Mar 6, 4:57 pm)
Re: [PATCH 2.6.24] chroot= as a new kernel parameter, H. Peter Anvin, (Thu Mar 6, 12:58 am)
Re: [PATCH 2.6.24] chroot= as a new kernel parameter, Ingo Molnar, (Thu Mar 6, 6:27 am)
Re: [PATCH 2.6.24] chroot= as a new kernel parameter, H. Peter Anvin, (Thu Mar 6, 11:17 am)
Re: [PATCH 2.6.24] chroot= as a new kernel parameter, Pavel Machek, (Thu Mar 6, 5:20 pm)
Re: [PATCH 2.6.24] chroot= as a new kernel parameter, Ingo Molnar, (Thu Mar 6, 5:42 pm)
Re: [PATCH 2.6.24] chroot= as a new kernel parameter, H. Peter Anvin, (Thu Mar 6, 5:50 pm)
Re: [PATCH 2.6.24] chroot= as a new kernel parameter, Ingo Molnar, (Thu Mar 6, 6:22 pm)
Re: [PATCH 2.6.24] chroot= as a new kernel parameter, H. Peter Anvin, (Thu Mar 6, 5:30 pm)
Re: [PATCH 2.6.24] chroot= as a new kernel parameter, Bernd Petrovitsch, (Fri Mar 7, 6:36 am)
Re: [PATCH 2.6.24] chroot= as a new kernel parameter, H. Peter Anvin, (Fri Mar 7, 8:28 am)
Re: [PATCH 2.6.24] chroot= as a new kernel parameter, Matthias Schniedermeyer, (Thu Mar 6, 6:05 pm)
Re: [PATCH 2.6.24] chroot= as a new kernel parameter, Ingo Molnar, (Thu Mar 6, 4:57 pm)
Re: [PATCH 2.6.24] chroot= as a new kernel parameter, H. Peter Anvin, (Thu Mar 6, 5:05 pm)
Re: [PATCH 2.6.24] chroot= as a new kernel parameter, Pawel Plociennik, (Thu Mar 6, 12:54 pm)
Re: [PATCH 2.6.24] chroot= as a new kernel parameter, Pawel Plociennik, (Thu Mar 6, 11:47 am)
speck-geostationary