Kexec command line length

[H. Peter Anvin]

Vivek Goyal wrote:
> Hi Neil,
> 
> I had a closer look at the code and following are my thoughts.
> 
> If I look at the 2.6.24 x86 boot code, I think this code does not expect a
> boot loader to put 2048 size command line in 4K page (so called zero page).
> I think what it expects is that a boot loader puts command line somewhere
> outside the zero page and just pass the pointer to that command line in zero
> page.

No, and it never has to the best of my knowledge.

> So far kexec has been putting command line in zero page. I think it worked
> because command line was small (256) and zero page had lots of free
> reserved area. (E820MAX was 32). 
> 
> Now in latest kernel code E820MAX has been increased to 128 and I don't see
> lot of free space in bootparam where we can put the 2048 size command line.
> 
> If we just continue to do what we are doing and just extend the command
> line size to 2048 in kexec-tools, i think this will overlap with some other
> area, either with EDD or E820 map etc and real mode code will overwrite part
> of command line as passed by kexec, on some systems.
> 
> So I think we should modify kexec-tools and start putting the 2048
> size command line outside the setup/zero page.
> 
> CCing HPA and Eric. They should be able to guide us better.

You should put it outside struct bootparams.  Furthermore, you should 
expect an increasing amount of information to be needed beyond struct 
bootparams in the future; the intent is to have a tagged linked list of 
expansion information in the future.

We are already out of e820 *and* EDD information space...

	-hpa