Re: [PATCH] x86: tidy e820 output

David Rientjes
2010-09-22T23:27:25Z

On Wed, 22 Sep 2010, H. Peter Anvin wrote:

> > I'm referring to using [start, start + addr - 1] in the output, like
> > /sys/firmware/memmap does, as opposed to [start, start + addr]. Is it not
> > valuable to include the actual e820 map in some way, especially when you
> > have your own BIOS? We've always used this output since it isn't
> > available later.
>
> "The actual e820 map" contains (start, length, type) -- the end bracket
> is not part of it at all. Either way, /sys/firmware/memmap does provide
> the memory map as provided by the firmware through whatever means.
>

Ok, I agree that fixing the off-by-1 problem that's been there for ages is
better and worth modifying our scripts when they break. Thanks!