diff options
author | Andrew Morton <akpm@linux-foundation.org> | 2008-04-28 05:12:39 -0400 |
---|---|---|
committer | Linus Torvalds <torvalds@linux-foundation.org> | 2008-04-28 11:58:20 -0400 |
commit | b45445684198a946b587732265692e6495993abf (patch) | |
tree | 0ffe64205f396bb00f6dfd3cbee630a1d0a975e7 | |
parent | ac6aadb24b7d4f0e54246732e221c102073412bf (diff) |
mm: make early_pfn_to_nid() a C function
Fix this (sparc64)
mm/sparse-vmemmap.c: In function `vmemmap_verify':
mm/sparse-vmemmap.c:64: warning: unused variable `pfn'
by switching to a C function which touches its arg.
(reason 3,555 why macros are bad)
Also, the `nid' arg was misnamed.
Reviewed-by: Christoph Lameter <clameter@sgi.com>
Acked-by: Andy Whitcroft <apw@shadowen.org>
Cc: Mel Gorman <mel@csn.ul.ie>
Cc: Andi Kleen <ak@suse.de>
Cc: KAMEZAWA Hiroyuki <kamezawa.hiroyu@jp.fujitsu.com>
Signed-off-by: Andrew Morton <akpm@linux-foundation.org>
Signed-off-by: Linus Torvalds <torvalds@linux-foundation.org>
-rw-r--r-- | include/linux/mmzone.h | 5 |
1 files changed, 4 insertions, 1 deletions
diff --git a/include/linux/mmzone.h b/include/linux/mmzone.h index 498d6ceff2f4..0aece6d8937e 100644 --- a/include/linux/mmzone.h +++ b/include/linux/mmzone.h | |||
@@ -841,7 +841,10 @@ static inline struct zoneref *first_zones_zonelist(struct zonelist *zonelist, | |||
841 | 841 | ||
842 | #if !defined(CONFIG_HAVE_ARCH_EARLY_PFN_TO_NID) && \ | 842 | #if !defined(CONFIG_HAVE_ARCH_EARLY_PFN_TO_NID) && \ |
843 | !defined(CONFIG_ARCH_POPULATES_NODE_MAP) | 843 | !defined(CONFIG_ARCH_POPULATES_NODE_MAP) |
844 | #define early_pfn_to_nid(nid) (0UL) | 844 | static inline unsigned long early_pfn_to_nid(unsigned long pfn) |
845 | { | ||
846 | return 0; | ||
847 | } | ||
845 | #endif | 848 | #endif |
846 | 849 | ||
847 | #ifdef CONFIG_FLATMEM | 850 | #ifdef CONFIG_FLATMEM |