From: "H. Peter Anvin" <hpa@zytor.com>
To: Alon Bar-Lev <alon.barlev@gmail.com>
Cc: Linux Kernel Mailing List <linux-kernel@vger.kernel.org>,
"Barry K. Nathan" <barryn@pobox.com>,
Adrian Bunk <bunk@fs.tum.de>
Subject: Re: [PATCH][TAKE 3] THE LINUX/I386 BOOT PROTOCOL - Breaking the 256 limit
Date: Fri, 14 Apr 2006 11:46:17 -0700 [thread overview]
Message-ID: <443FEDF9.6050203@zytor.com> (raw)
In-Reply-To: <443FE560.6010805@gmail.com>
Alon Bar-Lev wrote:
>
> Hello,
>
> The problem is that boot loader developers did not understand the old
> statement: "A string that is too long will be automatically truncated by
> the kernel, a boot loader may allow a longer command line to be passed
> to permit future kernels to extend this limit."
>
> Most of them handed the same buffer to < 2.02 protocols and >= 2.0.2
> protocols. When I've opened bugs against that they claimed that they
> follow instructions since the 256 limit was explicitly mentioned. I've
> ended up in patching GRUB my-self to allow this.
>
> I thought that this should be made clearer... But maybe I did not write
> it too well.
>
> I've removed the 255+1 limitation from the boot protocol main
> description, so there will be no known limit there... And moved it to
> the <2.02 section notes.
>
> Can you please suggest a different phrasing? Or maybe you think that it
> is not needed at all... But then I have a problem of making boot loader
> fix their code.
>
Well, obviously, since apparently LILO doesn't properly null-terminate
long command line.
Thinking about it a bit, the way to deal with the LILO problem is
probably to actually *usw* the boot loader ID byte we've had in there
since the 2.00 protocol. In other words, if the boot loader ID is 0x1X
where X <= current version (I don't know how LILO manages this ID) then
truncate the command line to 255 bytes; when this is fixed in LILO then
LILO gets to bump its boot loader ID version number.
-hpa
next prev parent reply other threads:[~2006-04-14 18:48 UTC|newest]
Thread overview: 8+ messages / expand[flat|nested] mbox.gz Atom feed top
2006-04-13 23:54 [PATCH][TAKE 3] THE LINUX/I386 BOOT PROTOCOL - Breaking the 256 limit Alon Bar-Lev
2006-04-14 0:38 ` Joshua Hudson
2006-04-14 17:53 ` H. Peter Anvin
2006-04-14 18:09 ` Alon Bar-Lev
2006-04-14 18:46 ` H. Peter Anvin [this message]
2006-04-14 19:01 ` Alon Bar-Lev
2006-04-14 19:04 ` H. Peter Anvin
2006-04-14 19:05 ` H. Peter Anvin
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=443FEDF9.6050203@zytor.com \
--to=hpa@zytor.com \
--cc=alon.barlev@gmail.com \
--cc=barryn@pobox.com \
--cc=bunk@fs.tum.de \
--cc=linux-kernel@vger.kernel.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox