diff options
author | Yinghai Lu <yhlu.kernel@gmail.com> | 2008-06-24 17:57:29 -0400 |
---|---|---|
committer | Ingo Molnar <mingo@elte.hu> | 2008-07-08 07:10:36 -0400 |
commit | d86623a0d55a14e7295e8ca1bd258e0c7f8dcb31 (patch) | |
tree | c036b459354461d5bcf975d125ea338a213f2177 /arch/x86/mm | |
parent | 1a0db38e5fa50eeb885194b1f4e494d4de55b918 (diff) |
x86: add table_top check for alloc_low_page in 64 bit
that range is from find_e820_area, so don't try to use end_pfn to see
if out of boundary...use table_top instead to avoid possible strange
result while cross the boundary...
also change early_printk to printk, because init_memory_mapping is after
early param parsing, and console=uart8250 already working at that time.
Signed-off-by: Yinghai Lu <yhlu.kernel@gmail.com>
Signed-off-by: Ingo Molnar <mingo@elte.hu>
Diffstat (limited to 'arch/x86/mm')
-rw-r--r-- | arch/x86/mm/init_64.c | 9 |
1 files changed, 5 insertions, 4 deletions
diff --git a/arch/x86/mm/init_64.c b/arch/x86/mm/init_64.c index 359e3afaaa6b..6eced2f10734 100644 --- a/arch/x86/mm/init_64.c +++ b/arch/x86/mm/init_64.c | |||
@@ -227,6 +227,7 @@ void __init cleanup_highmap(void) | |||
227 | 227 | ||
228 | static unsigned long __initdata table_start; | 228 | static unsigned long __initdata table_start; |
229 | static unsigned long __meminitdata table_end; | 229 | static unsigned long __meminitdata table_end; |
230 | static unsigned long __meminitdata table_top; | ||
230 | 231 | ||
231 | static __meminit void *alloc_low_page(unsigned long *phys) | 232 | static __meminit void *alloc_low_page(unsigned long *phys) |
232 | { | 233 | { |
@@ -240,7 +241,7 @@ static __meminit void *alloc_low_page(unsigned long *phys) | |||
240 | return adr; | 241 | return adr; |
241 | } | 242 | } |
242 | 243 | ||
243 | if (pfn >= end_pfn) | 244 | if (pfn >= table_top) |
244 | panic("alloc_low_page: ran out of memory"); | 245 | panic("alloc_low_page: ran out of memory"); |
245 | 246 | ||
246 | adr = early_ioremap(pfn * PAGE_SIZE, PAGE_SIZE); | 247 | adr = early_ioremap(pfn * PAGE_SIZE, PAGE_SIZE); |
@@ -372,10 +373,10 @@ static void __init find_early_table_space(unsigned long end) | |||
372 | 373 | ||
373 | table_start >>= PAGE_SHIFT; | 374 | table_start >>= PAGE_SHIFT; |
374 | table_end = table_start; | 375 | table_end = table_start; |
376 | table_top = table_start + (tables >> PAGE_SHIFT); | ||
375 | 377 | ||
376 | early_printk("kernel direct mapping tables up to %lx @ %lx-%lx\n", | 378 | printk(KERN_DEBUG "kernel direct mapping tables up to %lx @ %lx-%lx\n", |
377 | end, table_start << PAGE_SHIFT, | 379 | end, table_start << PAGE_SHIFT, table_top << PAGE_SHIFT); |
378 | (table_start << PAGE_SHIFT) + tables); | ||
379 | } | 380 | } |
380 | 381 | ||
381 | static void __init init_gbpages(void) | 382 | static void __init init_gbpages(void) |