diff options
author | Tejun Heo <tj@kernel.org> | 2011-07-12 03:58:07 -0400 |
---|---|---|
committer | H. Peter Anvin <hpa@linux.intel.com> | 2011-07-13 19:35:58 -0400 |
commit | 15fb09722df32b7685be1cbcac198bb556ddaffe (patch) | |
tree | cb010e80d2ecbfee08a38cf9e65c6ff5b1fa32bb /mm/memblock.c | |
parent | 53348f27168534561c0c814843bbf181314374f4 (diff) |
memblock: Use MEMBLOCK_ALLOC_ACCESSIBLE instead of ANYWHERE in memblock_alloc_try_nid()
After node affine allocation fails, memblock_alloc_try_nid() calls
memblock_alloc_base() with @max_addr set to MEMBLOCK_ALLOC_ANYWHERE.
This is inconsistent with memblock_alloc() and what the function's
sole user - sparc/mm/init_64 - expects, although it doesn't make any
difference as sparc64 doesn't have highmem and ACCESSIBLE equals
ANYWHERE.
This patch makes memblock_alloc_try_nid() use ACCESSIBLE instead of
ANYWHERE. This isn't complete as node affine allocation doesn't
consider memblock.current_limit. It will be handled with future
changes.
This patch doesn't introduce any behavior difference.
Signed-off-by: Tejun Heo <tj@kernel.org>
Link: http://lkml.kernel.org/r/1310457490-3356-4-git-send-email-tj@kernel.org
Cc: Yinghai Lu <yinghai@kernel.org>
Cc: Benjamin Herrenschmidt <benh@kernel.crashing.org>
Signed-off-by: H. Peter Anvin <hpa@linux.intel.com>
Diffstat (limited to 'mm/memblock.c')
-rw-r--r-- | mm/memblock.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/mm/memblock.c b/mm/memblock.c index a0562d1a6ad4..87e512dc8018 100644 --- a/mm/memblock.c +++ b/mm/memblock.c | |||
@@ -612,7 +612,7 @@ phys_addr_t __init memblock_alloc_try_nid(phys_addr_t size, phys_addr_t align, i | |||
612 | 612 | ||
613 | if (res) | 613 | if (res) |
614 | return res; | 614 | return res; |
615 | return memblock_alloc_base(size, align, MEMBLOCK_ALLOC_ANYWHERE); | 615 | return memblock_alloc_base(size, align, MEMBLOCK_ALLOC_ACCESSIBLE); |
616 | } | 616 | } |
617 | 617 | ||
618 | 618 | ||