sched: Use cached value of span instead of calling sched_domain_span()
authorViresh Kumar <viresh.kumar@linaro.org>
Tue, 11 Jun 2013 11:02:43 +0000 (16:32 +0530)
committerIngo Molnar <mingo@kernel.org>
Wed, 19 Jun 2013 10:58:46 +0000 (12:58 +0200)
In the beginning of build_sched_groups() we called sched_domain_span() and
cached its return value in span. Few statements later we are calling it again to
get the same pointer.

Lets use the cached value instead as it hasn't changed in between.

Signed-off-by: Viresh Kumar <viresh.kumar@linaro.org>
Signed-off-by: Peter Zijlstra <peterz@infradead.org>
Link: http://lkml.kernel.org/r/834ecd507071ad88aff039352dbc7e063dd996a7.1370948150.git.viresh.kumar@linaro.org
Signed-off-by: Ingo Molnar <mingo@kernel.org>
kernel/sched/core.c

index 547b7d3..3388387 100644 (file)
@@ -5347,7 +5347,7 @@ build_sched_groups(struct sched_domain *sd, int cpu)
        get_group(cpu, sdd, &sd->groups);
        atomic_inc(&sd->groups->ref);
 
-       if (cpu != cpumask_first(sched_domain_span(sd)))
+       if (cpu != cpumask_first(span))
                return 0;
 
        lockdep_assert_held(&sched_domains_mutex);