Resend the patch for more people to review On some single node x64 system with huge amount of physical memory e.g > 64G. the memmap size maybe very big.
If the memmap is allocated from low pages, it may occupies too much memory below 4G. then swiotlb could fail to reserve bounce buffer under 4G which will lead to boot failure. This patch will first try to allocate memmap memory above 4G in sparse vmemmap code. If it failed, it will allocate memmap above MAX_DMA_ADDRESS. This patch is against 2.6.24-rc1-git14 Signed-off-by: Zou Nan hai <[EMAIL PROTECTED]> Signed-off-by: Suresh Siddha <[EMAIL PROTECTED]> diff -Nraup a/arch/x86/mm/init_64.c b/arch/x86/mm/init_64.c --- a/arch/x86/mm/init_64.c 2007-11-06 15:16:12.000000000 +0800 +++ b/arch/x86/mm/init_64.c 2007-11-06 15:55:50.000000000 +0800 @@ -448,6 +448,13 @@ void online_page(struct page *page) num_physpages++; } +void * __meminit alloc_bootmem_high_node(pg_data_t *pgdat, unsigned long size, + unsigned long align) +{ + return __alloc_bootmem_core(pgdat->bdata, size, + align, (4UL*1024*1024*1024), 0, 1); +} + #ifdef CONFIG_MEMORY_HOTPLUG /* * Memory is added always to NORMAL zone. This means you will never get diff -Nraup a/include/linux/bootmem.h b/include/linux/bootmem.h --- a/include/linux/bootmem.h 2007-11-06 16:06:31.000000000 +0800 +++ b/include/linux/bootmem.h 2007-11-06 15:50:36.000000000 +0800 @@ -61,6 +61,10 @@ extern void *__alloc_bootmem_core(struct unsigned long limit, int strict_goal); +extern void *alloc_bootmem_high_node(pg_data_t *pgdat, + unsigned long size, + unsigned long align); + #ifndef CONFIG_HAVE_ARCH_BOOTMEM_NODE extern void reserve_bootmem(unsigned long addr, unsigned long size); #define alloc_bootmem(x) \ diff -Nraup a/mm/bootmem.c b/mm/bootmem.c --- a/mm/bootmem.c 2007-11-06 16:06:31.000000000 +0800 +++ b/mm/bootmem.c 2007-11-06 15:49:20.000000000 +0800 @@ -492,3 +492,11 @@ void * __init __alloc_bootmem_low_node(p return __alloc_bootmem_core(pgdat->bdata, size, align, goal, ARCH_LOW_ADDRESS_LIMIT, 0); } + +__attribute__((weak)) __meminit +void *alloc_bootmem_high_node(pg_data_t *pgdat, unsigned long size, + unsigned long align) +{ + return NULL; +} + diff -Nraup a/mm/sparse-vmemmap.c b/mm/sparse-vmemmap.c --- a/mm/sparse-vmemmap.c 2007-11-06 15:16:12.000000000 +0800 +++ b/mm/sparse-vmemmap.c 2007-11-06 16:08:52.000000000 +0800 @@ -43,9 +43,13 @@ void * __meminit vmemmap_alloc_block(uns if (page) return page_address(page); return NULL; - } else + } else { + void *p = alloc_bootmem_high_node(NODE_DATA(node), size, size); + if (p) + return p; return __alloc_bootmem_node(NODE_DATA(node), size, size, __pa(MAX_DMA_ADDRESS)); + } } void __meminit vmemmap_verify(pte_t *pte, int node, - To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to [EMAIL PROTECTED] More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/