diff options
| author |   Yinghai Lu <yhlu.kernel.send@gmail.com> | 2008-03-23 00:16:49 -0700 |
|---|---|---|
| committer |   Ingo Molnar <mingo@elte.hu> | 2008-03-26 22:23:41 +0100 |
| commit | 76c324182bbd29dfe4298ca65efb15be18055df1 (patch) | |
| tree | 7dcdd3aded048ab8613f532140d1d5bad14e4621 /arch/x86/mm/discontig_32.c | |
| parent | x86: GEODE: add missing module.h include (diff) | |
| download | linux-dev-76c324182bbd29dfe4298ca65efb15be18055df1.tar.xz linux-dev-76c324182bbd29dfe4298ca65efb15be18055df1.zip | |
x86: fix trim mtrr not to setup_memory two times
we could call find_max_pfn() directly instead of setup_memory() to get
max_pfn needed for mtrr trimming.
otherwise setup_memory() is called two times... that is duplicated...
[ mingo@elte.hu: both Thomas and me simulated a double call to
setup_bootmem_allocator() and can confirm that it is a real bug
which can hang in certain configs. It's not been reported yet but
that is probably due to the relatively scarce nature of
MTRR-trimming systems. ]
Signed-off-by: Yinghai Lu <yhlu.kernel@gmail.com>
Signed-off-by: Ingo Molnar <mingo@elte.hu>
Diffstat (limited to '')
| -rw-r--r-- | arch/x86/mm/discontig_32.c | 1 |
1 files changed, 0 insertions, 1 deletions
diff --git a/arch/x86/mm/discontig_32.c b/arch/x86/mm/discontig_32.c index c394ca0720b8..8e25e06ff730 100644 --- a/arch/x86/mm/discontig_32.c +++ b/arch/x86/mm/discontig_32.c @@ -324,7 +324,6 @@ unsigned long __init setup_memory(void) * this space and use it to adjust the boundary between ZONE_NORMAL * and ZONE_HIGHMEM. */ - find_max_pfn(); get_memcfg_numa(); kva_pages = calculate_numa_remap_pages(); |