summaryrefslogtreecommitdiffstats
path: root/arch/x86/include/asm/mmzone_32.h
diff options
context:
space:
mode:
authorTejun Heo <tj@kernel.org>2009-03-01 16:06:56 +0900
committerTejun Heo <tj@kernel.org>2009-03-01 16:06:56 +0900
commitd0c4f570276cb4d2dc4215b90eb7cb6e2bdd4a15 (patch)
tree5cf4f85082ed720df7d815299782e25e44d75c9d /arch/x86/include/asm/mmzone_32.h
parentaf6326d72c95d6e0bbc88c92185c654f57acef3b (diff)
downloadop-kernel-dev-d0c4f570276cb4d2dc4215b90eb7cb6e2bdd4a15.zip
op-kernel-dev-d0c4f570276cb4d2dc4215b90eb7cb6e2bdd4a15.tar.gz
bootmem, x86: further fixes for arch-specific bootmem wrapping
Impact: fix new breakages introduced by previous fix Commit c132937556f56ee4b831ef4b23f1846e05fde102 tried to clean up bootmem arch wrapper but it wasn't quite correct. Before the commit, the followings were broken. * Low level interface functions prefixed with __ ignored arch preference. * reserve_bootmem(...) can't be mapped into reserve_bootmem_node(NODE_DATA(0)->bdata, ...) because the node is not preference here. The region specified MUST fall into the specified region; otherwise, it will panic. After the commit, * If allocation fails for the arch preferred node, it should fallback to whatever is available. Instead, it simply failed allocation. There are too many internal details to allow generic wrapping and still keep things simple for archs. Plus, all that arch wants is a way to prefer certain node over another. This patch drops the generic wrapping around alloc_bootmem_core() and add alloc_bootmem_core() instead. If necessary, arch can define bootmem_arch_referred_node() macro or function which takes all allocation information and returns the preferred node. bootmem generic code will always try the preferred node first and then fallback to other nodes as usual. Breakages noted and changes reviewed by Johannes Weiner. Signed-off-by: Tejun Heo <tj@kernel.org> Acked-by: Johannes Weiner <hannes@cmpxchg.org>
Diffstat (limited to 'arch/x86/include/asm/mmzone_32.h')
-rw-r--r--arch/x86/include/asm/mmzone_32.h8
1 files changed, 2 insertions, 6 deletions
diff --git a/arch/x86/include/asm/mmzone_32.h b/arch/x86/include/asm/mmzone_32.h
index eeacf67..ede6998 100644
--- a/arch/x86/include/asm/mmzone_32.h
+++ b/arch/x86/include/asm/mmzone_32.h
@@ -92,12 +92,8 @@ static inline int pfn_valid(int pfn)
#ifdef CONFIG_NEED_MULTIPLE_NODES
/* always use node 0 for bootmem on this numa platform */
-#define alloc_bootmem_core(__bdata, size, align, goal, limit) \
-({ \
- bootmem_data_t __maybe_unused * __abm_bdata_dummy = (__bdata); \
- __alloc_bootmem_core(NODE_DATA(0)->bdata, \
- (size), (align), (goal), (limit)); \
-})
+#define bootmem_arch_preferred_node(__bdata, size, align, goal, limit) \
+ (NODE_DATA(0)->bdata)
#endif /* CONFIG_NEED_MULTIPLE_NODES */
#endif /* _ASM_X86_MMZONE_32_H */
OpenPOWER on IntegriCloud