linux-kernel - Re: [PATCH 01/26] Dynamic kernel command-line

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] [day] [month] [year] [list]

Date:	Tue, 05 Sep 2006 19:52:52 -0400
From:	Bill Davidsen <davidsen@....com>
To:	Matthew Wilcox <matthew@....cx>
CC:	Andi Kleen <ak@...e.de>, Matt Domsch <Matt_Domsch@...l.com>,
	Andrew Morton <akpm@...l.org>, linux-kernel@...r.kernel.org,
	johninsd@....rr.com, davej@...emonkey.org.uk, Riley@...liams.name,
	trini@...nel.crashing.org, davem@...emloft.net, ecd@...inaid.de,
	jj@...site.ms.mff.cuni.cz, anton@...ba.org, wli@...omorphy.com,
	lethal@...ux-sh.org, rc@....org.uk, spyro@....com, rth@...ddle.net,
	avr32@...el.com, hskinnemoen@...el.com, starvik@...s.com,
	ralf@...ux-mips.org, grundler@...isc-linux.org,
	geert@...ux-m68k.org, zippel@...ux-m68k.org, paulus@...ba.org,
	schwidefsky@...ibm.com, heiko.carstens@...ibm.com,
	uclinux-v850@....nec.co.jp, chris@...kel.net
Subject: Re: [PATCH 01/26] Dynamic kernel command-line - common

Matthew Wilcox wrote:
> On Mon, Sep 04, 2006 at 12:52:14AM +0300, Alon Bar-Lev wrote:
>> @@ -116,8 +116,12 @@ extern void time_init(void);
>>  void (*late_time_init)(void);
>>  extern void softirq_init(void);
>>  
>> -/* Untouched command line (eg. for /proc) saved by 
>> arch-specific code. */
>> -char saved_command_line[COMMAND_LINE_SIZE];
>> +/* Untouched command line saved by arch-specific code. */
>> +char __initdata boot_command_line[COMMAND_LINE_SIZE];
> 
> Your patch is wordwrapped.
> 
> Also, what was the point of all this?  Was there some discussion that
> this would be useful?
> 
Assuming that this works as described, the benefits are obvious. In most 
cases I would expect the memory saved to be more than the patch takes, 
and the patch is init code. Having the possibility of having very long 
command lines is bound to be useful on occasion, particularly when using 
drivers with option-names-way-too-long.

I agree that this wouldn't make my top 200 list of wanted features, but 
it's here, it saves a little memory, it adds a capability someone will 
find useful, why not use it? It appears to be a small win in every case.

-- 
Bill Davidsen <davidsen@....com>
   Obscure bug of 2004: BASH BUFFER OVERFLOW - if bash is being run by a
normal user and is setuid root, with the "vi" line edit mode selected,
and the character set is "big5," an off-by-one errors occurs during
wildcard (glob) expansion.
-
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/