[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH] Add 2 more E820 to ACPI specification
From: |
Robert Millan |
Subject: |
Re: [PATCH] Add 2 more E820 to ACPI specification |
Date: |
Wed, 20 Jan 2010 00:33:33 +0100 |
User-agent: |
Mutt/1.5.18 (2008-05-17) |
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