Re: [PATCH v3 05/12] ARM: topology: use new cpu_power interface

2014-07-09 Thread Vincent Guittot
On 9 July 2014 09:49, Amit Kucheria  wrote:
> On Mon, Jun 30, 2014 at 6:05 PM, Vincent Guittot
>  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 
>> ---
>>  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-ker...@lists.linaro.org
>> http://lists.linaro.org/mailman/listinfo/linaro-kernel
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/


Re: [PATCH v3 05/12] ARM: topology: use new cpu_power interface

2014-07-09 Thread Amit Kucheria
On Mon, Jun 30, 2014 at 6:05 PM, Vincent Guittot
 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 
> ---
>  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-ker...@lists.linaro.org
> http://lists.linaro.org/mailman/listinfo/linaro-kernel
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/


Re: [PATCH v3 05/12] ARM: topology: use new cpu_power interface

2014-07-09 Thread Amit Kucheria
On Mon, Jun 30, 2014 at 6:05 PM, Vincent Guittot
vincent.guit...@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.guit...@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-ker...@lists.linaro.org
 http://lists.linaro.org/mailman/listinfo/linaro-kernel
--
To unsubscribe from this list: send the line unsubscribe linux-kernel in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/


Re: [PATCH v3 05/12] ARM: topology: use new cpu_power interface

2014-07-09 Thread Vincent Guittot
On 9 July 2014 09:49, Amit Kucheria amit.kuche...@linaro.org wrote:
 On Mon, Jun 30, 2014 at 6:05 PM, Vincent Guittot
 vincent.guit...@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.guit...@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-ker...@lists.linaro.org
 http://lists.linaro.org/mailman/listinfo/linaro-kernel
--
To unsubscribe from this list: send the line unsubscribe linux-kernel in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/


Re: [PATCH v3 05/12] ARM: topology: use new cpu_power interface

2014-07-08 Thread Rik van Riel
-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 

Acked-by: Rik van Riel 


- -- 
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-
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/


Re: [PATCH v3 05/12] ARM: topology: use new cpu_power interface

2014-07-08 Thread Rik van Riel
-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.guit...@linaro.org

Acked-by: Rik van Riel r...@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-
--
To unsubscribe from this list: send the line unsubscribe linux-kernel in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/


[PATCH v3 05/12] ARM: topology: use new cpu_power interface

2014-06-30 Thread Vincent Guittot
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 
---
 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

--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/


[PATCH v3 05/12] ARM: topology: use new cpu_power interface

2014-06-30 Thread Vincent Guittot
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.guit...@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

--
To unsubscribe from this list: send the line unsubscribe linux-kernel in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/