x86: fix code paths used by update_mptable
authorJan Beulich <jbeulich@novell.com>
Thu, 12 Mar 2009 13:07:23 +0000 (13:07 +0000)
committerIngo Molnar <mingo@elte.hu>
Fri, 13 Mar 2009 01:37:19 +0000 (02:37 +0100)
commit5c0e6f035df983210e4d22213aed624ced502d3d
tree6986ed92d5f62f421d3b3be776132994c8c05d6b
parent82034d6f59b4772f4233bbb61c670290803a9960
x86: fix code paths used by update_mptable

Impact: fix crashes under Xen due to unrobust e820 code

find_e820_area_size() must return a properly distinguishable and
out-of-bounds value when it fails, and -1UL does not meet that
criteria on i386/PAE. Additionally, callers of the function must
check against that value.

early_reserve_e820() should be prepared for the region found to be
outside of the addressable range on 32-bits.

e820_update_range_map() should not blindly update e820, but should do
all it work on the map it got a pointer passed for (which in 50% of the
cases is &e820_saved). It must also not call e820_add_region(), as that
again acts on e820 unconditionally.

The issues were found when trying to make this option work in our Xen
kernel (i.e. where some of the silent assumptions made in the code
would not hold).

Signed-off-by: Jan Beulich <jbeulich@novell.com>
LKML-Reference: <49B9171B.76E4.0078.0@novell.com>
Signed-off-by: Ingo Molnar <mingo@elte.hu>
arch/x86/kernel/check.c
arch/x86/kernel/e820.c