Commit 0197518cd3672029618a16a57597946a094ac7a8

Authored by Tang Chen
Committed by Linus Torvalds
1 parent bbcab8789d

memory-hotplug: remove memmap of sparse-vmemmap

Introduce a new API vmemmap_free() to free and remove vmemmap
pagetables.  Since pagetable implements are different, each architecture
has to provide its own version of vmemmap_free(), just like
vmemmap_populate().

Note: vmemmap_free() is not implemented for ia64, ppc, s390, and sparc.

[mhocko@suse.cz: fix implicit declaration of remove_pagetable]
Signed-off-by: Yasuaki Ishimatsu <isimatu.yasuaki@jp.fujitsu.com>
Signed-off-by: Jianguo Wu <wujianguo@huawei.com>
Signed-off-by: Wen Congyang <wency@cn.fujitsu.com>
Signed-off-by: Tang Chen <tangchen@cn.fujitsu.com>
Cc: KOSAKI Motohiro <kosaki.motohiro@jp.fujitsu.com>
Cc: Jiang Liu <jiang.liu@huawei.com>
Cc: Kamezawa Hiroyuki <kamezawa.hiroyu@jp.fujitsu.com>
Cc: Lai Jiangshan <laijs@cn.fujitsu.com>
Cc: Wu Jianguo <wujianguo@huawei.com>
Cc: Ingo Molnar <mingo@elte.hu>
Cc: Thomas Gleixner <tglx@linutronix.de>
Cc: "H. Peter Anvin" <hpa@zytor.com>
Signed-off-by: Michal Hocko <mhocko@suse.cz>
Signed-off-by: Andrew Morton <akpm@linux-foundation.org>
Signed-off-by: Linus Torvalds <torvalds@linux-foundation.org>

Showing 8 changed files with 31 additions and 1 deletions Side-by-side Diff

... ... @@ -434,5 +434,8 @@
434 434 return 0;
435 435 }
436 436 #endif /* CONFIG_ARM64_64K_PAGES */
  437 +void vmemmap_free(struct page *memmap, unsigned long nr_pages)
  438 +{
  439 +}
437 440 #endif /* CONFIG_SPARSEMEM_VMEMMAP */
arch/ia64/mm/discontig.c
... ... @@ -823,5 +823,8 @@
823 823 return vmemmap_populate_basepages(start_page, size, node);
824 824 }
825 825  
  826 +void vmemmap_free(struct page *memmap, unsigned long nr_pages)
  827 +{
  828 +}
826 829 #endif
arch/powerpc/mm/init_64.c
... ... @@ -298,5 +298,9 @@
298 298 return 0;
299 299 }
300 300  
  301 +void vmemmap_free(struct page *memmap, unsigned long nr_pages)
  302 +{
  303 +}
  304 +
301 305 #endif /* CONFIG_SPARSEMEM_VMEMMAP */
... ... @@ -268,6 +268,10 @@
268 268 return ret;
269 269 }
270 270  
  271 +void vmemmap_free(struct page *memmap, unsigned long nr_pages)
  272 +{
  273 +}
  274 +
271 275 /*
272 276 * Add memory segment to the segment list if it doesn't overlap with
273 277 * an already present segment.
arch/sparc/mm/init_64.c
... ... @@ -2236,6 +2236,10 @@
2236 2236 }
2237 2237 }
2238 2238  
  2239 +void vmemmap_free(struct page *memmap, unsigned long nr_pages)
  2240 +{
  2241 +}
  2242 +
2239 2243 #endif /* CONFIG_SPARSEMEM_VMEMMAP */
2240 2244  
2241 2245 static void prot_init_common(unsigned long page_none,
arch/x86/mm/init_64.c
... ... @@ -1011,6 +1011,14 @@
1011 1011 flush_tlb_all();
1012 1012 }
1013 1013  
  1014 +void __ref vmemmap_free(struct page *memmap, unsigned long nr_pages)
  1015 +{
  1016 + unsigned long start = (unsigned long)memmap;
  1017 + unsigned long end = (unsigned long)(memmap + nr_pages);
  1018 +
  1019 + remove_pagetable(start, end, false);
  1020 +}
  1021 +
1014 1022 static void __meminit
1015 1023 kernel_physical_mapping_remove(unsigned long start, unsigned long end)
1016 1024 {
... ... @@ -1718,6 +1718,9 @@
1718 1718 unsigned long pages, int node);
1719 1719 int vmemmap_populate(struct page *start_page, unsigned long pages, int node);
1720 1720 void vmemmap_populate_print_last(void);
  1721 +#ifdef CONFIG_MEMORY_HOTPLUG
  1722 +void vmemmap_free(struct page *memmap, unsigned long nr_pages);
  1723 +#endif
1721 1724 void register_page_bootmem_memmap(unsigned long section_nr, struct page *map,
1722 1725 unsigned long size);
1723 1726  
... ... @@ -615,10 +615,11 @@
615 615 }
616 616 static void __kfree_section_memmap(struct page *memmap, unsigned long nr_pages)
617 617 {
618   - return; /* XXX: Not implemented yet */
  618 + vmemmap_free(memmap, nr_pages);
619 619 }
620 620 static void free_map_bootmem(struct page *memmap, unsigned long nr_pages)
621 621 {
  622 + vmemmap_free(memmap, nr_pages);
622 623 }
623 624 #else
624 625 static struct page *__kmalloc_section_memmap(unsigned long nr_pages)