On Wed, Jan 30, 2008 at 04:18:34PM -0500, Neil Horman wrote: > On Wed, Jan 30, 2008 at 04:08:15PM -0500, Vivek Goyal wrote: > > On Wed, Jan 30, 2008 at 03:59:47PM -0500, Neil Horman wrote: > > > On Wed, Jan 30, 2008 at 03:53:23PM -0500, Vivek Goyal wrote: > > > > On Tue, Jan 29, 2008 at 02:57:53PM -0500, Neil Horman wrote: > > > > > > > > > > Ok, I've got a new patch here. In response to Bernhard and Viveks concerns, > > > > > I've added a few checks. Bernhard, thank you for digging into the history of > > > > > the addition of the 2048 byte command line. As I understand what you've posted > > > > > heres what we need to capture: > > > > > > > > > > On X86: > > > > > Boot loader version >= 2.06 > > > > > error if cmdline_len > setup_header.cmdline_size > > > > > 2.05 > Boot loader version > 2.06 > > > > > warn if cmdline_len > 255 > > > > > Boot loader version < 2.05 > > > > > error if cmdline_len > 255 > > > > > > > > > > on X86_64: > > > > > Boot loader version >= 2.04 > > > > > error if cmdline_len > setup_header.cmdline_size > > > > > > > > cmdline_size was introduced only in version 2.06. So this will run > > > > into trouble on x86_64 version 2.04 and 2.05, isn't it? > > > > > > > Then how did version 2.04 on x86_64 determine its maximum command line length? > > > Or did it just assume a max length of 2048 bytes? > > > > I think it just did not tell. There was no way for a kernel bzImage to > > tell boot-loader what's the supported command line size is (Pre 2.06). > > > So, for 2.04 on x86_64, we should just assume a 2048 byte command line length > then? > No. 2.04 is really old. As per boot.txt, it was introduced in 2.6.14. Anyway, thinking about it more, Large command lines were introduced in 2.6.21 kernel (for both i386 and x86_64). cmdline_size was introduced in 2.6.22 (for both i386 and x86_64). So how about following test condition for both the arch. if (boot_protocol >= 2.06) error message based on cmdline_size field else Warn user if command line is more than 256 but continue to work. Let the user know that it is not known if kernel being loaded supports command line size greater than 256. Command line will be truncated by destination kernel if it does not support large command lines. User is taking a chance I think above will work both for i386 and x86_64. This is not precise message. If we want to do better, then we shall have to rely on linux version string. But If somebody is building custom kernel with their own versioning, then we will run into issues. So I think, for the time being we are good with above simple condition. Thanks Vivek
