diff options
author | Mel Gorman <mgorman@suse.de> | 2015-06-30 17:56:55 -0400 |
---|---|---|
committer | Linus Torvalds <torvalds@linux-foundation.org> | 2015-06-30 22:44:56 -0400 |
commit | 8a942fdea560d4ac0e9d9fabcd5201ad20e0c382 (patch) | |
tree | 452c0f5acccf96ffc54748a44ab5bc788f7dbffc /include/linux/mm.h | |
parent | d70ddd7a5d9aa335f9b4b0c3d879e1e70ee1e4e3 (diff) |
mm: meminit: make __early_pfn_to_nid SMP-safe and introduce meminit_pfn_in_nid
__early_pfn_to_nid() use static variables to cache recent lookups as
memblock lookups are very expensive but it assumes that memory
initialisation is single-threaded. Parallel initialisation of struct
pages will break that assumption so this patch makes __early_pfn_to_nid()
SMP-safe by requiring the caller to cache recent search information.
early_pfn_to_nid() keeps the same interface but is only safe to use early
in boot due to the use of a global static variable. meminit_pfn_in_nid()
is an SMP-safe version that callers must maintain their own state for.
Signed-off-by: Mel Gorman <mgorman@suse.de>
Tested-by: Nate Zimmer <nzimmer@sgi.com>
Tested-by: Waiman Long <waiman.long@hp.com>
Tested-by: Daniel J Blueman <daniel@numascale.com>
Acked-by: Pekka Enberg <penberg@kernel.org>
Cc: Robin Holt <robinmholt@gmail.com>
Cc: Nate Zimmer <nzimmer@sgi.com>
Cc: Dave Hansen <dave.hansen@intel.com>
Cc: Waiman Long <waiman.long@hp.com>
Cc: Scott Norton <scott.norton@hp.com>
Cc: "Luck, Tony" <tony.luck@intel.com>
Cc: Ingo Molnar <mingo@elte.hu>
Cc: "H. Peter Anvin" <hpa@zytor.com>
Cc: Thomas Gleixner <tglx@linutronix.de>
Signed-off-by: Andrew Morton <akpm@linux-foundation.org>
Signed-off-by: Linus Torvalds <torvalds@linux-foundation.org>
Diffstat (limited to 'include/linux/mm.h')
-rw-r--r-- | include/linux/mm.h | 6 |
1 files changed, 4 insertions, 2 deletions
diff --git a/include/linux/mm.h b/include/linux/mm.h index d662af2d0d01..2e872f92dbac 100644 --- a/include/linux/mm.h +++ b/include/linux/mm.h | |||
@@ -1726,7 +1726,8 @@ extern void sparse_memory_present_with_active_regions(int nid); | |||
1726 | 1726 | ||
1727 | #if !defined(CONFIG_HAVE_MEMBLOCK_NODE_MAP) && \ | 1727 | #if !defined(CONFIG_HAVE_MEMBLOCK_NODE_MAP) && \ |
1728 | !defined(CONFIG_HAVE_ARCH_EARLY_PFN_TO_NID) | 1728 | !defined(CONFIG_HAVE_ARCH_EARLY_PFN_TO_NID) |
1729 | static inline int __early_pfn_to_nid(unsigned long pfn) | 1729 | static inline int __early_pfn_to_nid(unsigned long pfn, |
1730 | struct mminit_pfnnid_cache *state) | ||
1730 | { | 1731 | { |
1731 | return 0; | 1732 | return 0; |
1732 | } | 1733 | } |
@@ -1734,7 +1735,8 @@ static inline int __early_pfn_to_nid(unsigned long pfn) | |||
1734 | /* please see mm/page_alloc.c */ | 1735 | /* please see mm/page_alloc.c */ |
1735 | extern int __meminit early_pfn_to_nid(unsigned long pfn); | 1736 | extern int __meminit early_pfn_to_nid(unsigned long pfn); |
1736 | /* there is a per-arch backend function. */ | 1737 | /* there is a per-arch backend function. */ |
1737 | extern int __meminit __early_pfn_to_nid(unsigned long pfn); | 1738 | extern int __meminit __early_pfn_to_nid(unsigned long pfn, |
1739 | struct mminit_pfnnid_cache *state); | ||
1738 | #endif | 1740 | #endif |
1739 | 1741 | ||
1740 | extern void set_dma_reserve(unsigned long new_dma_reserve); | 1742 | extern void set_dma_reserve(unsigned long new_dma_reserve); |