Messages in this thread Patch in this message | | | Date | Fri, 17 Feb 2006 22:29:35 +0900 | From | Yasunori Goto <> | Subject | [PATCH: 009/012] Memory hotplug for new nodes v.2 (pgdat link insert) |
| |
This is to insert pgdat to its link. To be honest, I would like remove this pgdat link, and I posted patches to remove it for a long time ago to -mm.
http://marc.theaimsgroup.com/?l=linux-mm&m=112808582217281&w=2 http://marc.theaimsgroup.com/?l=linux-mm&m=112808582227966&w=2
But, it might have influence against performance for big NUMA box. So, I should check it. However, I'm very hard to access big NUMA box. Our machine is divided as small partitioned for test of each team inside of fujitsu every time. Sigh. :-(.
This is lazy patch instead of them. To use this code for x86_64, this code is moved to generic place.
Signed-off-by: Yasunori Goto <y-goto@jp.fujitsu.com>
Index: pgdat3/arch/ia64/mm/discontig.c =================================================================== --- pgdat3.orig/arch/ia64/mm/discontig.c 2006-02-17 16:52:52.000000000 +0900 +++ pgdat3/arch/ia64/mm/discontig.c 2006-02-17 18:35:33.000000000 +0900 @@ -381,31 +381,6 @@ static void __init *memory_less_node_all } /** - * pgdat_insert - insert the pgdat into global pgdat_list - * @pgdat: the pgdat for a node. - */ -static void __init pgdat_insert(pg_data_t *pgdat) -{ - pg_data_t *prev = NULL, *next; - - for_each_pgdat(next) - if (pgdat->node_id < next->node_id) - break; - else - prev = next; - - if (prev) { - prev->pgdat_next = pgdat; - pgdat->pgdat_next = next; - } else { - pgdat->pgdat_next = pgdat_list; - pgdat_list = pgdat; - } - - return; -} - -/** * memory_less_nodes - allocate and initialize CPU only nodes pernode * information. */ Index: pgdat3/mm/page_alloc.c =================================================================== --- pgdat3.orig/mm/page_alloc.c 2006-02-17 16:52:50.000000000 +0900 +++ pgdat3/mm/page_alloc.c 2006-02-17 16:52:53.000000000 +0900 @@ -1575,6 +1575,31 @@ void show_free_areas(void) show_swap_cache_info(); } +/** + * pgdat_insert - insert the pgdat into global pgdat_list + * @pgdat: the pgdat for a node. + */ +void __meminit pgdat_insert(pg_data_t *pgdat) +{ + pg_data_t *prev = NULL, *next; + + for_each_pgdat(next) + if (pgdat->node_id < next->node_id) + break; + else + prev = next; + + if (prev) { + prev->pgdat_next = pgdat; + pgdat->pgdat_next = next; + } else { + pgdat->pgdat_next = pgdat_list; + pgdat_list = pgdat; + } + + return; +} + /* * Builds allocation fallback zone lists. * Index: pgdat3/mm/memory_hotplug.c =================================================================== --- pgdat3.orig/mm/memory_hotplug.c 2006-02-17 16:52:50.000000000 +0900 +++ pgdat3/mm/memory_hotplug.c 2006-02-17 16:52:53.000000000 +0900 @@ -57,6 +57,7 @@ static int __add_section(struct zone *zo return register_new_memory(__pfn_to_section(phys_start_pfn)); } +extern void __meminit pgdat_insert(pg_data_t *pgdat); extern int kswapd(void *); int new_pgdat_init(int nid, unsigned long start_pfn, unsigned long nr_pages) { @@ -89,6 +90,7 @@ int new_pgdat_init(int nid, unsigned lon pgdat->kswapd = p; node_set_online(nid); + pgdat_insert(pgdat); arch_register_node(nid); return 0; Index: pgdat3/include/linux/mmzone.h =================================================================== --- pgdat3.orig/include/linux/mmzone.h 2006-02-17 16:52:50.000000000 +0900 +++ pgdat3/include/linux/mmzone.h 2006-02-17 18:39:48.000000000 +0900 @@ -650,6 +650,7 @@ void sparse_init(void); void memory_present(int nid, unsigned long start, unsigned long end); unsigned long __init node_memmap_size_bytes(int, unsigned long, unsigned long); +extern void __meminit pgdat_insert(pg_data_t *); #endif /* !__ASSEMBLY__ */ #endif /* __KERNEL__ */ -- Yasunori Goto
- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
| |