aboutsummaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorSebastian Andrzej Siewior <bigeasy@linutronix.de>2016-11-26 18:13:35 -0500
committerThomas Gleixner <tglx@linutronix.de>2016-12-01 18:52:35 -0500
commit76f290935b1a395ee6c9a14697f16593591d1fe8 (patch)
tree53802f66d64f5ff31f69df029ed05c579e9ac43d
parentb32614c03413f8a6025d8677c2b7c0ee976e63d4 (diff)
mm/vmstat: Drop get_online_cpus() from init_cpu_node_state/vmstat_cpu_dead()
Both functions are called with protection against cpu hotplug already so *_online_cpus() could be dropped. Signed-off-by: Sebastian Andrzej Siewior <bigeasy@linutronix.de> Acked-by: Michal Hocko <mhocko@suse.com> Cc: linux-mm@kvack.org Cc: rt@linutronix.de Cc: Johannes Weiner <hannes@cmpxchg.org> Cc: Andrew Morton <akpm@linux-foundation.org> Cc: Mel Gorman <mgorman@techsingularity.net> Cc: Vlastimil Babka <vbabka@suse.cz> Link: http://lkml.kernel.org/r/20161126231350.10321-8-bigeasy@linutronix.de Signed-off-by: Thomas Gleixner <tglx@linutronix.de>
-rw-r--r--mm/vmstat.c7
1 files changed, 1 insertions, 6 deletions
diff --git a/mm/vmstat.c b/mm/vmstat.c
index 604f26a4f696..0b63ffb5c407 100644
--- a/mm/vmstat.c
+++ b/mm/vmstat.c
@@ -1722,24 +1722,19 @@ static void __init init_cpu_node_state(void)
1722{ 1722{
1723 int cpu; 1723 int cpu;
1724 1724
1725 get_online_cpus();
1726 for_each_online_cpu(cpu) 1725 for_each_online_cpu(cpu)
1727 node_set_state(cpu_to_node(cpu), N_CPU); 1726 node_set_state(cpu_to_node(cpu), N_CPU);
1728 put_online_cpus();
1729} 1727}
1730 1728
1731static void vmstat_cpu_dead(int node) 1729static void vmstat_cpu_dead(int node)
1732{ 1730{
1733 int cpu; 1731 int cpu;
1734 1732
1735 get_online_cpus();
1736 for_each_online_cpu(cpu) 1733 for_each_online_cpu(cpu)
1737 if (cpu_to_node(cpu) == node) 1734 if (cpu_to_node(cpu) == node)
1738 goto end; 1735 return;
1739 1736
1740 node_clear_state(node, N_CPU); 1737 node_clear_state(node, N_CPU);
1741end:
1742 put_online_cpus();
1743} 1738}
1744 1739
1745/* 1740/*