Commit-ID:  4c501327b4c67f2b874219f20c41d2619b49c78c
Gitweb:     http://git.kernel.org/tip/4c501327b4c67f2b874219f20c41d2619b49c78c
Author:     Sebastian Andrzej Siewior <[email protected]>
AuthorDate: Tue, 29 Nov 2016 15:51:14 +0100
Committer:  Thomas Gleixner <[email protected]>
CommitDate: Fri, 2 Dec 2016 00:52:35 +0100

mm/vmstat: Avoid on each online CPU loops

Both iterations over online cpus can be replaced by the proper node
specific functions.

Signed-off-by: Sebastian Andrzej Siewior <[email protected]>
Acked-by: Michal Hocko <[email protected]>
Cc: [email protected]
Cc: [email protected]
Cc: Johannes Weiner <[email protected]>
Cc: Andrew Morton <[email protected]>
Cc: Mel Gorman <[email protected]>
Cc: Vlastimil Babka <[email protected]>
Link: http://lkml.kernel.org/r/[email protected]
Signed-off-by: Thomas Gleixner <[email protected]>

---
 mm/vmstat.c | 16 +++++++++-------
 1 file changed, 9 insertions(+), 7 deletions(-)

diff --git a/mm/vmstat.c b/mm/vmstat.c
index 0b63ffb..5152cd1 100644
--- a/mm/vmstat.c
+++ b/mm/vmstat.c
@@ -1720,19 +1720,21 @@ static void __init start_shepherd_timer(void)
 
 static void __init init_cpu_node_state(void)
 {
-       int cpu;
+       int node;
 
-       for_each_online_cpu(cpu)
-               node_set_state(cpu_to_node(cpu), N_CPU);
+       for_each_online_node(node) {
+               if (cpumask_weight(cpumask_of_node(node)) > 0)
+                       node_set_state(node, N_CPU);
+       }
 }
 
 static void vmstat_cpu_dead(int node)
 {
-       int cpu;
+       const struct cpumask *node_cpus;
 
-       for_each_online_cpu(cpu)
-               if (cpu_to_node(cpu) == node)
-                       return;
+       node_cpus = cpumask_of_node(node);
+       if (cpumask_weight(node_cpus) > 0)
+               return;
 
        node_clear_state(node, N_CPU);
 }

Reply via email to