On 23 November 2015 at 15:28, Juri Lelli <juri.lelli@xxxxxxx> wrote: > Instead of looping through all cpus calling set_capacity_scale, we can > initialise cpu_scale per-cpu variables to SCHED_CAPACITY_SCALE with their > definition. > > Cc: Russell King <linux@xxxxxxxxxxxxxxxx> > Signed-off-by: Juri Lelli <juri.lelli@xxxxxxx> > --- > arch/arm/kernel/topology.c | 4 +--- > 1 file changed, 1 insertion(+), 3 deletions(-) > > diff --git a/arch/arm/kernel/topology.c b/arch/arm/kernel/topology.c > index 08b7847..ec279d1 100644 > --- a/arch/arm/kernel/topology.c > +++ b/arch/arm/kernel/topology.c > @@ -40,7 +40,7 @@ > * to run the rebalance_domains for all idle cores and the cpu_capacity can be > * updated during this sequence. > */ > -static DEFINE_PER_CPU(unsigned long, cpu_scale); > +static DEFINE_PER_CPU(unsigned long, cpu_scale) = SCHED_CAPACITY_SCALE; > > unsigned long arch_scale_cpu_capacity(struct sched_domain *sd, int cpu) > { > @@ -306,8 +306,6 @@ void __init init_cpu_topology(void) > cpu_topo->socket_id = -1; > cpumask_clear(&cpu_topo->core_sibling); > cpumask_clear(&cpu_topo->thread_sibling); > - > - set_capacity_scale(cpu, SCHED_CAPACITY_SCALE); > } > smp_wmb(); FWIW, Acked-by: Vincent Guittot <vincent.guittot@xxxxxxxxxx> > > -- > 2.2.2 > -- To unsubscribe from this list: send the line "unsubscribe devicetree" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html