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

["Huang, Ying" <ying.huang@intel.com>]

On Mon, 2008-06-23 at 03:47 -0500, Paul Jackson wrote:
> Huang Ying wrote:
> > So, I think it is better to remove "EFI memmap based code".
> 
> You give good reasons for -adding- E820 EXT code.  Fine.
> 
> You give no reason for -removing- the EFI memmap based code,
> except the implicit (unstated) reason that we should only
> support a single mechanism.
> 
> However the kernel routinely supports a variety of mechanisms
> for various BIOS firmware, as it should.
> 
> Internally, within the kernel, when it is entirely within the
> kernels control and when there is no externally visible kernel
> interface affected, we routinely strive to minimize redundant
> mechanisms, as we should.
> 
> But externally, such as in supporting various boot firmware
> protocols, we routinely support multiple useful interfaces.
> 
> If that EFI memmap based code for > 128 nodes is causing you
> no problem, then please leave it be.  It is providing us good
> benefit.

Please fix the following issue, if it is agreed to keep this redundant
code in kernel:

4. Current EFI memmap based code does not work properly in all
situation, for example it can not works with kernel parameter:
"memmap=exactmap, memmap=<xxx>, ...", "mem=<xxx>" or "noefi".

Best Regards,
Huang Ying