diff options
| author | Cody P Schafer <cody@linux.vnet.ibm.com> | 2013-07-03 15:02:10 -0700 |
|---|---|---|
| committer | Chanho Park <chanho61.park@samsung.com> | 2014-04-16 16:13:04 +0900 |
| commit | 5bbfb3e16925287bf2bd0f73cb8170c2cb468248 (patch) | |
| tree | 7deade518b3472d80e98ced4d63f471b0ba04e11 | |
| parent | 8e78224d8780105801f815e61ae00b55e46f9530 (diff) | |
| download | linux-3.10-5bbfb3e16925287bf2bd0f73cb8170c2cb468248.tar.gz linux-3.10-5bbfb3e16925287bf2bd0f73cb8170c2cb468248.tar.bz2 linux-3.10-5bbfb3e16925287bf2bd0f73cb8170c2cb468248.zip | |
memory_hotplug: use pgdat_resize_lock() in online_pages()
mmzone.h documents node_size_lock (which pgdat_resize_lock() locks) as
follows:
* Must be held any time you expect node_start_pfn, node_present_pages
* or node_spanned_pages stay constant. [...]
So actually hold it when we update node_present_pages in online_pages().
Signed-off-by: Cody P Schafer <cody@linux.vnet.ibm.com>
Cc: David Rientjes <rientjes@google.com>
Signed-off-by: Andrew Morton <akpm@linux-foundation.org>
Signed-off-by: Linus Torvalds <torvalds@linux-foundation.org>
| -rw-r--r-- | mm/memory_hotplug.c | 5 |
1 files changed, 5 insertions, 0 deletions
diff --git a/mm/memory_hotplug.c b/mm/memory_hotplug.c index 1ad92b46753..527c51084bb 100644 --- a/mm/memory_hotplug.c +++ b/mm/memory_hotplug.c @@ -918,6 +918,7 @@ static void node_states_set_node(int node, struct memory_notify *arg) int __ref online_pages(unsigned long pfn, unsigned long nr_pages, int online_type) { + unsigned long flags; unsigned long onlined_pages = 0; struct zone *zone; int need_zonelists_rebuild = 0; @@ -996,7 +997,11 @@ int __ref online_pages(unsigned long pfn, unsigned long nr_pages, int online_typ zone->managed_pages += onlined_pages; zone->present_pages += onlined_pages; + + pgdat_resize_lock(zone->zone_pgdat, &flags); zone->zone_pgdat->node_present_pages += onlined_pages; + pgdat_resize_unlock(zone->zone_pgdat, &flags); + if (onlined_pages) { node_states_set_node(zone_to_nid(zone), &arg); if (need_zonelists_rebuild) |