Re: [PATCH] x86 boot: Pass E820 memory map entries more than 128 via linked list of setup data

[Paul Jackson <pj@sgi.com>]

Huang Ying wrote:
> With this patch, your previous patch:
> 
> x86 boot: add code to add BIOS provided EFI memory entries to kernel
> 
> is not necessary, so can be reversed.
> 
> What do you think about that?

... the same thing I thought about it when you asked this question
last month, in post http://lkml.org/lkml/2008/5/26/307.

Please see my reply in http://lkml.org/lkml/2008/5/28/59.
It was lengthy and carefully researched, so I won't repeat
it here.

In short, we still need it.  The key kernel routine that adds
additional EFI entries to the existing E820 map is just 20 or so
fairly easy lines.  We agree that it doesn't handle some of what your
more extended work with a linked list of setup data handles (such as
additional EDD entries), but it does handle additional memory map
(node) entries using the existing data structure interface between
the firmware and kernel.

-- 
                  I won't rest till it's the best ...
                  Programmer, Linux Scalability
                  Paul Jackson <pj@sgi.com> 1.940.382.4214