diff options
author | Yinghai Lu <yinghai@kernel.org> | 2010-02-27 12:29:38 -0500 |
---|---|---|
committer | H. Peter Anvin <hpa@zytor.com> | 2010-03-01 20:59:24 -0500 |
commit | 81d0d950e5037a26b71e568ff235ff9e998f4ab3 (patch) | |
tree | c9f6d2ca8dd5ddacc0bbb0fbc67f30449dbf29ff | |
parent | fb90ef93df654f2678933efbbf864adac0ae490e (diff) |
sparsemem: Fix compilation on PowerPC
Stephen reported:
build (powerpc
ppc64_defconfig) produced these warnings:
mm/sparse.c: In function 'sparse_init':
mm/sparse.c:488: warning: unused variable 'map_count'
mm/sparse.c:484: warning: unused variable 'size2'
mm/sparse.c:481: warning: unused variable 'map_map'
mm/sparse.c: At top level:
mm/sparse.c:442: warning: 'sparse_early_mem_maps_alloc_node' defined but not used
Introduced by commit 9bdac914240759457175ac0d6529a37d2820bc4d
("sparsemem: Put mem map for one node together").
Conditionalize the bits appropriately based on the setting of
CONFIG_SPARSEMEM_ALLOC_MEM_MAP_TOGETHER.
Reported-by: Stephen Rothwell <sfr@canb.auug.org.au>
Tested-by: Stephen Rothwell <sfr@canb.auug.org.au>
Signed-off-by: Yinghai Lu <yinghai@kernel.org>
LKML-Reference: <4B895682.1080706@kernel.org>
Signed-off-by: H. Peter Anvin <hpa@zytor.com>
-rw-r--r-- | mm/sparse.c | 11 |
1 files changed, 7 insertions, 4 deletions
diff --git a/mm/sparse.c b/mm/sparse.c index 9b6b93a4d78d..22896d589133 100644 --- a/mm/sparse.c +++ b/mm/sparse.c | |||
@@ -439,6 +439,7 @@ void __init sparse_mem_maps_populate_node(struct page **map_map, | |||
439 | } | 439 | } |
440 | #endif /* !CONFIG_SPARSEMEM_VMEMMAP */ | 440 | #endif /* !CONFIG_SPARSEMEM_VMEMMAP */ |
441 | 441 | ||
442 | #ifdef CONFIG_SPARSEMEM_ALLOC_MEM_MAP_TOGETHER | ||
442 | static void __init sparse_early_mem_maps_alloc_node(struct page **map_map, | 443 | static void __init sparse_early_mem_maps_alloc_node(struct page **map_map, |
443 | unsigned long pnum_begin, | 444 | unsigned long pnum_begin, |
444 | unsigned long pnum_end, | 445 | unsigned long pnum_end, |
@@ -447,8 +448,7 @@ static void __init sparse_early_mem_maps_alloc_node(struct page **map_map, | |||
447 | sparse_mem_maps_populate_node(map_map, pnum_begin, pnum_end, | 448 | sparse_mem_maps_populate_node(map_map, pnum_begin, pnum_end, |
448 | map_count, nodeid); | 449 | map_count, nodeid); |
449 | } | 450 | } |
450 | 451 | #else | |
451 | #ifndef CONFIG_SPARSEMEM_ALLOC_MEM_MAP_TOGETHER | ||
452 | static struct page __init *sparse_early_mem_map_alloc(unsigned long pnum) | 452 | static struct page __init *sparse_early_mem_map_alloc(unsigned long pnum) |
453 | { | 453 | { |
454 | struct page *map; | 454 | struct page *map; |
@@ -478,14 +478,17 @@ void __init sparse_init(void) | |||
478 | { | 478 | { |
479 | unsigned long pnum; | 479 | unsigned long pnum; |
480 | struct page *map; | 480 | struct page *map; |
481 | struct page **map_map; | ||
482 | unsigned long *usemap; | 481 | unsigned long *usemap; |
483 | unsigned long **usemap_map; | 482 | unsigned long **usemap_map; |
484 | int size, size2; | 483 | int size; |
485 | int nodeid_begin = 0; | 484 | int nodeid_begin = 0; |
486 | unsigned long pnum_begin = 0; | 485 | unsigned long pnum_begin = 0; |
487 | unsigned long usemap_count; | 486 | unsigned long usemap_count; |
487 | #ifdef CONFIG_SPARSEMEM_ALLOC_MEM_MAP_TOGETHER | ||
488 | unsigned long map_count; | 488 | unsigned long map_count; |
489 | int size2; | ||
490 | struct page **map_map; | ||
491 | #endif | ||
489 | 492 | ||
490 | /* | 493 | /* |
491 | * map is using big page (aka 2M in x86 64 bit) | 494 | * map is using big page (aka 2M in x86 64 bit) |