| Message ID | 1404144343-18720-6-git-send-email-vincent.guittot@linaro.org (mailing list archive) |
|---|---|
| State | New, archived |
| Headers | show |
-----BEGIN PGP SIGNED MESSAGE----- Hash: SHA1 On 06/30/2014 12:05 PM, Vincent Guittot wrote: > Use the new arch_scale_cpu_power in order to reflect the original > capacity of a CPU instead of arch_scale_freq_power which is more > linked to a scaling of the capacity linked to the frequency. > > Signed-off-by: Vincent Guittot <vincent.guittot@linaro.org> Acked-by: Rik van Riel <riel@redhat.com> - -- All rights reversed -----BEGIN PGP SIGNATURE----- Version: GnuPG v1 Comment: Using GnuPG with Thunderbird - http://www.enigmail.net/ iQEcBAEBAgAGBQJTvLLgAAoJEM553pKExN6DZ40H/38CjOJQjRY4WZ+2UxlXSXfU V7gcMwHCNIJvJQ0fG3IfxVacdlJsK8LsQMgPL5sYCeJpLKYrQHPIQ2dIdkmz2W9I lcpr5dBq1AAB+PAbkywV2wtQBlDd0AMMQNrPqL6FdK92M4ML3iweJluA4qp0K1ip AtZM90ku4SQ52RyQR3ePIb396CeT1TRqihDnpWxbcZt53DkqWW3SCVE68qL1SIEO YKKlZyFXxLJDBZfM2kk90L8NPSmXzTa9xdsu+3CQKiyukqhgJoNe6J9E11CjlKBx BQggFRRTliIUat7IGlkgcyWLhXhhMLrfoXJN/sEdrWG2T2rApnI4u2Q4PM02/zU= =hHGQ -----END PGP SIGNATURE-----
On Mon, Jun 30, 2014 at 6:05 PM, Vincent Guittot <vincent.guittot@linaro.org> wrote: > Use the new arch_scale_cpu_power in order to reflect the original capacity of s/arch_scale_cpu_power/arch_scale_cpu_capacity and similar renames in the commit logs across the entire patchset to take into account all the renames in the code. > a CPU instead of arch_scale_freq_power which is more linked to a scaling of > the capacity linked to the frequency. here too... > Signed-off-by: Vincent Guittot <vincent.guittot@linaro.org> > --- > arch/arm/kernel/topology.c | 4 ++-- > 1 file changed, 2 insertions(+), 2 deletions(-) > > diff --git a/arch/arm/kernel/topology.c b/arch/arm/kernel/topology.c > index d42a7db..2310bfb 100644 > --- a/arch/arm/kernel/topology.c > +++ b/arch/arm/kernel/topology.c > @@ -42,7 +42,7 @@ > */ > static DEFINE_PER_CPU(unsigned long, cpu_scale); > > -unsigned long arch_scale_freq_capacity(struct sched_domain *sd, int cpu) > +unsigned long arch_scale_cpu_capacity(struct sched_domain *sd, int cpu) > { > return per_cpu(cpu_scale, cpu); > } > @@ -166,7 +166,7 @@ static void update_cpu_capacity(unsigned int cpu) > set_capacity_scale(cpu, cpu_capacity(cpu) / middle_capacity); > > printk(KERN_INFO "CPU%u: update cpu_capacity %lu\n", > - cpu, arch_scale_freq_capacity(NULL, cpu)); > + cpu, arch_scale_cpu_capacity(NULL, cpu)); > } > > #else > -- > 1.9.1 > > > _______________________________________________ > linaro-kernel mailing list > linaro-kernel@lists.linaro.org > http://lists.linaro.org/mailman/listinfo/linaro-kernel
On 9 July 2014 09:49, Amit Kucheria <amit.kucheria@linaro.org> wrote: > On Mon, Jun 30, 2014 at 6:05 PM, Vincent Guittot > <vincent.guittot@linaro.org> wrote: >> Use the new arch_scale_cpu_power in order to reflect the original capacity of > > s/arch_scale_cpu_power/arch_scale_cpu_capacity and similar renames in > the commit logs across the entire patchset to take into account all > the renames in the code. good catch, the commit log passes through the find and replace. I will change the other commit message as well > >> a CPU instead of arch_scale_freq_power which is more linked to a scaling of >> the capacity linked to the frequency. > > here too... > >> Signed-off-by: Vincent Guittot <vincent.guittot@linaro.org> >> --- >> arch/arm/kernel/topology.c | 4 ++-- >> 1 file changed, 2 insertions(+), 2 deletions(-) >> >> diff --git a/arch/arm/kernel/topology.c b/arch/arm/kernel/topology.c >> index d42a7db..2310bfb 100644 >> --- a/arch/arm/kernel/topology.c >> +++ b/arch/arm/kernel/topology.c >> @@ -42,7 +42,7 @@ >> */ >> static DEFINE_PER_CPU(unsigned long, cpu_scale); >> >> -unsigned long arch_scale_freq_capacity(struct sched_domain *sd, int cpu) >> +unsigned long arch_scale_cpu_capacity(struct sched_domain *sd, int cpu) >> { >> return per_cpu(cpu_scale, cpu); >> } >> @@ -166,7 +166,7 @@ static void update_cpu_capacity(unsigned int cpu) >> set_capacity_scale(cpu, cpu_capacity(cpu) / middle_capacity); >> >> printk(KERN_INFO "CPU%u: update cpu_capacity %lu\n", >> - cpu, arch_scale_freq_capacity(NULL, cpu)); >> + cpu, arch_scale_cpu_capacity(NULL, cpu)); >> } >> >> #else >> -- >> 1.9.1 >> >> >> _______________________________________________ >> linaro-kernel mailing list >> linaro-kernel@lists.linaro.org >> http://lists.linaro.org/mailman/listinfo/linaro-kernel
diff --git a/arch/arm/kernel/topology.c b/arch/arm/kernel/topology.c index d42a7db..2310bfb 100644 --- a/arch/arm/kernel/topology.c +++ b/arch/arm/kernel/topology.c @@ -42,7 +42,7 @@ */ static DEFINE_PER_CPU(unsigned long, cpu_scale); -unsigned long arch_scale_freq_capacity(struct sched_domain *sd, int cpu) +unsigned long arch_scale_cpu_capacity(struct sched_domain *sd, int cpu) { return per_cpu(cpu_scale, cpu); } @@ -166,7 +166,7 @@ static void update_cpu_capacity(unsigned int cpu) set_capacity_scale(cpu, cpu_capacity(cpu) / middle_capacity); printk(KERN_INFO "CPU%u: update cpu_capacity %lu\n", - cpu, arch_scale_freq_capacity(NULL, cpu)); + cpu, arch_scale_cpu_capacity(NULL, cpu)); } #else
Use the new arch_scale_cpu_power in order to reflect the original capacity of a CPU instead of arch_scale_freq_power which is more linked to a scaling of the capacity linked to the frequency. Signed-off-by: Vincent Guittot <vincent.guittot@linaro.org> --- arch/arm/kernel/topology.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-)