Re: [PATCH] x86: Make e820_remove_range to handle all covered case

From: Rusty Russell
Date: Wed Mar 31 2010 - 19:17:13 EST


On Tue, 30 Mar 2010 04:08:29 pm Yinghai Lu wrote:
>
> Rusty found on lguest with trim_bios_range, max_pfn is not right anymore, and
> looks e820_remove_range does not work right.
>
> [ 0.000000] BIOS-provided physical RAM map:
> [ 0.000000] LGUEST: 0000000000000000 - 0000000004000000 (usable)
> [ 0.000000] Notice: NX (Execute Disable) protection missing in CPU or disabled in BIOS!
> [ 0.000000] DMI not present or invalid.
> [ 0.000000] last_pfn = 0x3fa0 max_arch_pfn = 0x100000
> [ 0.000000] init_memory_mapping: 0000000000000000-0000000003fa0000
>
> root cause is: the e820_remove_range doesn't handle that all cover case.
> e820_remove_range(BIOS_START, BIOS_END - BIOS_start, ,,)
> get miss one left range.
>
> Make that is corresponding to e820_update_range() to handle that case too.

Tested-by: Rusty Russell <rusty@xxxxxxxxxxxxxxx>

Please put in -urgent...

Thanks,
Rusty.
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/