From: "H. Peter Anvin" <hpa@zytor.com>
To: Vivek Goyal <vgoyal@redhat.com>
Cc: Neil Horman <nhorman@redhat.com>,
kexec@lists.infradead.org,
"Eric W. Biederman" <ebiederm@xmission.com>,
Neil Horman <nhorman@tuxdriver.com>
Subject: Re: Kexec command line length
Date: Fri, 25 Jan 2008 07:44:29 -0800 [thread overview]
Message-ID: <479A03DD.5030308@zytor.com> (raw)
In-Reply-To: <20080125153924.GA13287@redhat.com>
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
_______________________________________________
kexec mailing list
kexec@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/kexec
next prev parent reply other threads:[~2008-01-25 15:48 UTC|newest]
Thread overview: 35+ messages / expand[flat|nested] mbox.gz Atom feed top
2008-01-14 13:43 Kexec command line length Neil Horman
2008-01-14 14:50 ` Bernhard Walle
2008-01-14 15:40 ` Neil Horman
2008-01-15 15:27 ` Vivek Goyal
2008-01-15 17:09 ` Neil Horman
2008-01-15 17:37 ` Vivek Goyal
2008-01-25 12:35 ` Neil Horman
2008-01-25 15:39 ` Vivek Goyal
2008-01-25 15:44 ` H. Peter Anvin [this message]
2008-01-25 19:50 ` Neil Horman
2008-01-25 19:54 ` H. Peter Anvin
2008-01-25 20:11 ` Neil Horman
2008-01-25 20:13 ` Vivek Goyal
2008-01-25 20:54 ` Neil Horman
2008-01-28 17:08 ` Neil Horman
2008-01-28 19:37 ` Vivek Goyal
2008-01-28 20:07 ` Neil Horman
2008-01-28 20:20 ` Vivek Goyal
2008-01-28 20:53 ` Neil Horman
2008-01-28 21:09 ` Vivek Goyal
2008-01-28 21:29 ` Bernhard Walle
2008-01-29 1:01 ` Neil Horman
2008-01-29 15:41 ` Vivek Goyal
2008-01-29 18:17 ` Bernhard Walle
2008-01-29 18:52 ` Neil Horman
2008-01-29 19:57 ` Neil Horman
2008-01-30 20:53 ` Vivek Goyal
2008-01-30 20:59 ` Neil Horman
2008-01-30 21:08 ` Vivek Goyal
2008-01-30 21:18 ` Neil Horman
2008-01-30 21:45 ` Vivek Goyal
2008-01-31 0:10 ` Neil Horman
2008-01-31 7:16 ` Bernhard Walle
2008-02-12 21:11 ` Neil Horman
2008-02-18 3:14 ` Simon Horman
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=479A03DD.5030308@zytor.com \
--to=hpa@zytor.com \
--cc=ebiederm@xmission.com \
--cc=kexec@lists.infradead.org \
--cc=nhorman@redhat.com \
--cc=nhorman@tuxdriver.com \
--cc=vgoyal@redhat.com \
/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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.