Re: [PATCH 1/2] x86: Sanity check the e820 against the SRAT table using e820 map only

Ingo Molnar
2009-05-11T09:37:43Z

* Yinghai Lu <yinghai@kernel.org> wrote:

> - e820ram = max_pfn - absent_pages_in_range(0, max_pfn);
> + e820ram = max_pfn - (e820_hole_size(0, max_pfn<<PAGE_SHIFT)>>PAGE_SHIFT);

btw., it would be nice to have a debug check that prints a warning
if the SRAT does not cover all RAM, or if it covers RAM that is not
present in the e820 map. Such a warning might alert us to double
check all the PXM settings in the SRAT and could uncover more quirks
like the above ...

Ingo