diff options
author | Jeremy Fitzhardinge <jeremy@goop.org> | 2008-04-28 05:12:03 -0400 |
---|---|---|
committer | Linus Torvalds <torvalds@linux-foundation.org> | 2008-04-28 11:58:17 -0400 |
commit | 180c06efce691f2b721dd0d965079827bdd7ee03 (patch) | |
tree | 6d5dbde8f341544bbd212e9d376af1e7880a4229 /mm | |
parent | ea01ea937dcae2caa146dea1918cccf2f16ed3c4 (diff) |
hotplug-memory: make online_page() common
All architectures use an effectively identical definition of online_page(), so
just make it common code. x86-64, ia64, powerpc and sh are actually
identical; x86-32 is slightly different.
x86-32's differences arise because it puts its hotplug pages in the highmem
zone. We can handle this in the generic code by inspecting the page to see if
its in highmem, and update the totalhigh_pages count appropriately. This
leaves init_32.c:free_new_highpage with a single caller, so I folded it into
add_one_highpage_init.
I also removed an incorrect comment referring to the NUMA case; any NUMA
details have already been dealt with by the time online_page() is called.
[akpm@linux-foundation.org: fix indenting]
Signed-off-by: Jeremy Fitzhardinge <jeremy.fitzhardinge@citrix.com>
Acked-by: Dave Hansen <dave@linux.vnet.ibm.com>
Reviewed-by: KAMEZAWA Hiroyuki <kamez.hiroyu@jp.fujitsu.com>
Tested-by: KAMEZAWA Hiroyuki <kamez.hiroyu@jp.fujitsu.com>
Cc: Yasunori Goto <y-goto@jp.fujitsu.com>
Cc: Christoph Lameter <clameter@sgi.com>
Acked-by: Ingo Molnar <mingo@elte.hu>
Acked-by: Yasunori Goto <y-goto@jp.fujitsu.com>
Signed-off-by: Andrew Morton <akpm@linux-foundation.org>
Signed-off-by: Linus Torvalds <torvalds@linux-foundation.org>
Diffstat (limited to 'mm')
-rw-r--r-- | mm/memory_hotplug.c | 19 |
1 files changed, 19 insertions, 0 deletions
diff --git a/mm/memory_hotplug.c b/mm/memory_hotplug.c index d5094929766d..c8b3ca79de2d 100644 --- a/mm/memory_hotplug.c +++ b/mm/memory_hotplug.c | |||
@@ -219,6 +219,25 @@ static void grow_pgdat_span(struct pglist_data *pgdat, | |||
219 | pgdat->node_start_pfn; | 219 | pgdat->node_start_pfn; |
220 | } | 220 | } |
221 | 221 | ||
222 | void online_page(struct page *page) | ||
223 | { | ||
224 | totalram_pages++; | ||
225 | num_physpages++; | ||
226 | |||
227 | #ifdef CONFIG_HIGHMEM | ||
228 | if (PageHighMem(page)) | ||
229 | totalhigh_pages++; | ||
230 | #endif | ||
231 | |||
232 | #ifdef CONFIG_FLATMEM | ||
233 | max_mapnr = max(page_to_pfn(page), max_mapnr); | ||
234 | #endif | ||
235 | |||
236 | ClearPageReserved(page); | ||
237 | init_page_count(page); | ||
238 | __free_page(page); | ||
239 | } | ||
240 | |||
222 | static int online_pages_range(unsigned long start_pfn, unsigned long nr_pages, | 241 | static int online_pages_range(unsigned long start_pfn, unsigned long nr_pages, |
223 | void *arg) | 242 | void *arg) |
224 | { | 243 | { |