Re: [PATCH] Add 2 more E820 to ACPI specification

[Robert Millan <rmh@aybabtu.com>]

On Wed, Jan 20, 2010 at 12:17:26AM +0100, Vladimir 'φ-coder/phcoder' Serbinenko wrote:
> Robert Millan wrote:
> > On Fri, Jan 15, 2010 at 06:18:43PM +0100, Vladimir 'φ-coder/phcoder' Serbinenko wrote:
> >   
> >> Hello. This defines 2 more types for mmap_type in a way to match E820
> >> values. As grub legacy and grub2 both just pass E820 values through it
> >> shouldn't be any problem.
> >>     
> >
> > Note that although Multiboot's memory map format is modelled after e820,
> > it is defined by us, not by BIOS vendors.
> >
> >   
> Yes, but this information is needed and I prefer to provide it in
> grub-legacy-compatible way.
> >> -value of 1 indicates available @sc{ram}, and all other values currently
> >> +value of 1 indicates available @sc{ram}, value of 3 indicates usable memory
> >> +holding ACPI information,
> >>     
> >
> > I'm concerned about increasing the amount of arch-specific references; what do
> > Multiboot kernels that use ACPI currently do?
> >
> >   
> According to a post by Bernard Trotter many of them rely on multiboot
> loader passing e820 through

I just checked GRUB Legacy;  it seems that it is passing through the whole
thing.  I think this was a mistake, but it's too late to fix it now.

Feel free to commit this.

-- 
Robert Millan

  "Be the change you want to see in the world" -- Gandhi