* [RFC 4/4] sched: add per group cpu_power_orig
@ 2014-03-28 13:27 ` Vincent Guittot
0 siblings, 0 replies; 6+ messages in thread
From: Vincent Guittot @ 2014-03-28 13:27 UTC (permalink / raw)
To: peterz, mingo, linux-kernel, linux, linux-arm-kernel
Cc: preeti, Morten.Rasmussen, efault, linaro-kernel, Vincent Guittot
This new field cpu_power_orig reflects the available capacity of a CPUs unlike
the cpu_power which reflects the current capacity that can be altered by
frequency and rt tasks.
Signed-off-by: Vincent Guittot <vincent.guittot@linaro.org>
---
kernel/sched/core.c | 2 +-
kernel/sched/fair.c | 1 +
kernel/sched/sched.h | 1 +
3 files changed, 3 insertions(+), 1 deletion(-)
diff --git a/kernel/sched/core.c b/kernel/sched/core.c
index 5b20b27..8513f83 100644
--- a/kernel/sched/core.c
+++ b/kernel/sched/core.c
@@ -6911,7 +6911,7 @@ void __init sched_init(void)
#ifdef CONFIG_SMP
rq->sd = NULL;
rq->rd = NULL;
- rq->cpu_power = SCHED_POWER_SCALE;
+ rq->cpu_power = rq->cpu_power_orig = SCHED_POWER_SCALE;
rq->post_schedule = 0;
rq->active_balance = 0;
rq->next_balance = jiffies;
diff --git a/kernel/sched/fair.c b/kernel/sched/fair.c
index 7387c05..f1e4db6 100644
--- a/kernel/sched/fair.c
+++ b/kernel/sched/fair.c
@@ -5611,6 +5611,7 @@ static void update_cpu_power(struct sched_domain *sd, int cpu)
power >>= SCHED_POWER_SHIFT;
+ cpu_rq(cpu)->cpu_power_orig = power;
sdg->sgp->power_orig = power;
if (sched_feat(ARCH_POWER))
diff --git a/kernel/sched/sched.h b/kernel/sched/sched.h
index 46c3784..2abdf5c 100644
--- a/kernel/sched/sched.h
+++ b/kernel/sched/sched.h
@@ -578,6 +578,7 @@ struct rq {
struct sched_domain *sd;
unsigned long cpu_power;
+ unsigned long cpu_power_orig;
unsigned char idle_balance;
/* For active balancing */
--
1.9.0
^ permalink raw reply related [flat|nested] 6+ messages in thread
* [RFC 4/4] sched: add per group cpu_power_orig
@ 2014-03-28 13:27 ` Vincent Guittot
0 siblings, 0 replies; 6+ messages in thread
From: Vincent Guittot @ 2014-03-28 13:27 UTC (permalink / raw)
To: linux-arm-kernel
This new field cpu_power_orig reflects the available capacity of a CPUs unlike
the cpu_power which reflects the current capacity that can be altered by
frequency and rt tasks.
Signed-off-by: Vincent Guittot <vincent.guittot@linaro.org>
---
kernel/sched/core.c | 2 +-
kernel/sched/fair.c | 1 +
kernel/sched/sched.h | 1 +
3 files changed, 3 insertions(+), 1 deletion(-)
diff --git a/kernel/sched/core.c b/kernel/sched/core.c
index 5b20b27..8513f83 100644
--- a/kernel/sched/core.c
+++ b/kernel/sched/core.c
@@ -6911,7 +6911,7 @@ void __init sched_init(void)
#ifdef CONFIG_SMP
rq->sd = NULL;
rq->rd = NULL;
- rq->cpu_power = SCHED_POWER_SCALE;
+ rq->cpu_power = rq->cpu_power_orig = SCHED_POWER_SCALE;
rq->post_schedule = 0;
rq->active_balance = 0;
rq->next_balance = jiffies;
diff --git a/kernel/sched/fair.c b/kernel/sched/fair.c
index 7387c05..f1e4db6 100644
--- a/kernel/sched/fair.c
+++ b/kernel/sched/fair.c
@@ -5611,6 +5611,7 @@ static void update_cpu_power(struct sched_domain *sd, int cpu)
power >>= SCHED_POWER_SHIFT;
+ cpu_rq(cpu)->cpu_power_orig = power;
sdg->sgp->power_orig = power;
if (sched_feat(ARCH_POWER))
diff --git a/kernel/sched/sched.h b/kernel/sched/sched.h
index 46c3784..2abdf5c 100644
--- a/kernel/sched/sched.h
+++ b/kernel/sched/sched.h
@@ -578,6 +578,7 @@ struct rq {
struct sched_domain *sd;
unsigned long cpu_power;
+ unsigned long cpu_power_orig;
unsigned char idle_balance;
/* For active balancing */
--
1.9.0
^ permalink raw reply related [flat|nested] 6+ messages in thread
* Re: [RFC 4/4] sched: add per group cpu_power_orig
2014-03-28 13:27 ` Vincent Guittot
@ 2014-04-01 10:41 ` Preeti U Murthy
-1 siblings, 0 replies; 6+ messages in thread
From: Preeti U Murthy @ 2014-04-01 10:41 UTC (permalink / raw)
To: Vincent Guittot
Cc: peterz, mingo, linux-kernel, linux, linux-arm-kernel,
Morten.Rasmussen, efault, linaro-kernel
Hi Vincent,
On 03/28/2014 06:57 PM, Vincent Guittot wrote:
> This new field cpu_power_orig reflects the available capacity of a CPUs unlike
> the cpu_power which reflects the current capacity that can be altered by
> frequency and rt tasks.
>
> Signed-off-by: Vincent Guittot <vincent.guittot@linaro.org>
Should not the subject be "add per rq cpu_power_orig"?
Regards
Preeti U Murthy
> ---
> kernel/sched/core.c | 2 +-
> kernel/sched/fair.c | 1 +
> kernel/sched/sched.h | 1 +
> 3 files changed, 3 insertions(+), 1 deletion(-)
>
> diff --git a/kernel/sched/core.c b/kernel/sched/core.c
> index 5b20b27..8513f83 100644
> --- a/kernel/sched/core.c
> +++ b/kernel/sched/core.c
> @@ -6911,7 +6911,7 @@ void __init sched_init(void)
> #ifdef CONFIG_SMP
> rq->sd = NULL;
> rq->rd = NULL;
> - rq->cpu_power = SCHED_POWER_SCALE;
> + rq->cpu_power = rq->cpu_power_orig = SCHED_POWER_SCALE;
> rq->post_schedule = 0;
> rq->active_balance = 0;
> rq->next_balance = jiffies;
> diff --git a/kernel/sched/fair.c b/kernel/sched/fair.c
> index 7387c05..f1e4db6 100644
> --- a/kernel/sched/fair.c
> +++ b/kernel/sched/fair.c
> @@ -5611,6 +5611,7 @@ static void update_cpu_power(struct sched_domain *sd, int cpu)
>
> power >>= SCHED_POWER_SHIFT;
>
> + cpu_rq(cpu)->cpu_power_orig = power;
> sdg->sgp->power_orig = power;
>
> if (sched_feat(ARCH_POWER))
> diff --git a/kernel/sched/sched.h b/kernel/sched/sched.h
> index 46c3784..2abdf5c 100644
> --- a/kernel/sched/sched.h
> +++ b/kernel/sched/sched.h
> @@ -578,6 +578,7 @@ struct rq {
> struct sched_domain *sd;
>
> unsigned long cpu_power;
> + unsigned long cpu_power_orig;
>
> unsigned char idle_balance;
> /* For active balancing */
>
^ permalink raw reply [flat|nested] 6+ messages in thread
* [RFC 4/4] sched: add per group cpu_power_orig
@ 2014-04-01 10:41 ` Preeti U Murthy
0 siblings, 0 replies; 6+ messages in thread
From: Preeti U Murthy @ 2014-04-01 10:41 UTC (permalink / raw)
To: linux-arm-kernel
Hi Vincent,
On 03/28/2014 06:57 PM, Vincent Guittot wrote:
> This new field cpu_power_orig reflects the available capacity of a CPUs unlike
> the cpu_power which reflects the current capacity that can be altered by
> frequency and rt tasks.
>
> Signed-off-by: Vincent Guittot <vincent.guittot@linaro.org>
Should not the subject be "add per rq cpu_power_orig"?
Regards
Preeti U Murthy
> ---
> kernel/sched/core.c | 2 +-
> kernel/sched/fair.c | 1 +
> kernel/sched/sched.h | 1 +
> 3 files changed, 3 insertions(+), 1 deletion(-)
>
> diff --git a/kernel/sched/core.c b/kernel/sched/core.c
> index 5b20b27..8513f83 100644
> --- a/kernel/sched/core.c
> +++ b/kernel/sched/core.c
> @@ -6911,7 +6911,7 @@ void __init sched_init(void)
> #ifdef CONFIG_SMP
> rq->sd = NULL;
> rq->rd = NULL;
> - rq->cpu_power = SCHED_POWER_SCALE;
> + rq->cpu_power = rq->cpu_power_orig = SCHED_POWER_SCALE;
> rq->post_schedule = 0;
> rq->active_balance = 0;
> rq->next_balance = jiffies;
> diff --git a/kernel/sched/fair.c b/kernel/sched/fair.c
> index 7387c05..f1e4db6 100644
> --- a/kernel/sched/fair.c
> +++ b/kernel/sched/fair.c
> @@ -5611,6 +5611,7 @@ static void update_cpu_power(struct sched_domain *sd, int cpu)
>
> power >>= SCHED_POWER_SHIFT;
>
> + cpu_rq(cpu)->cpu_power_orig = power;
> sdg->sgp->power_orig = power;
>
> if (sched_feat(ARCH_POWER))
> diff --git a/kernel/sched/sched.h b/kernel/sched/sched.h
> index 46c3784..2abdf5c 100644
> --- a/kernel/sched/sched.h
> +++ b/kernel/sched/sched.h
> @@ -578,6 +578,7 @@ struct rq {
> struct sched_domain *sd;
>
> unsigned long cpu_power;
> + unsigned long cpu_power_orig;
>
> unsigned char idle_balance;
> /* For active balancing */
>
^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [RFC 4/4] sched: add per group cpu_power_orig
2014-04-01 10:41 ` Preeti U Murthy
@ 2014-04-01 11:11 ` Vincent Guittot
-1 siblings, 0 replies; 6+ messages in thread
From: Vincent Guittot @ 2014-04-01 11:11 UTC (permalink / raw)
To: Preeti U Murthy
Cc: Peter Zijlstra, Ingo Molnar, linux-kernel,
Russell King - ARM Linux, LAK, Morten Rasmussen, Mike Galbraith,
linaro-kernel
On 1 April 2014 12:41, Preeti U Murthy <preeti@linux.vnet.ibm.com> wrote:
> Hi Vincent,
>
> On 03/28/2014 06:57 PM, Vincent Guittot wrote:
>> This new field cpu_power_orig reflects the available capacity of a CPUs unlike
>> the cpu_power which reflects the current capacity that can be altered by
>> frequency and rt tasks.
>>
>> Signed-off-by: Vincent Guittot <vincent.guittot@linaro.org>
>
> Should not the subject be "add per rq cpu_power_orig"?
yes you're right :-)
I don't know how i came to that subject... a quick and dirty
copy/paste of something else
thanks
Vincent
>
> Regards
> Preeti U Murthy
>> ---
>> kernel/sched/core.c | 2 +-
>> kernel/sched/fair.c | 1 +
>> kernel/sched/sched.h | 1 +
>> 3 files changed, 3 insertions(+), 1 deletion(-)
>>
>> diff --git a/kernel/sched/core.c b/kernel/sched/core.c
>> index 5b20b27..8513f83 100644
>> --- a/kernel/sched/core.c
>> +++ b/kernel/sched/core.c
>> @@ -6911,7 +6911,7 @@ void __init sched_init(void)
>> #ifdef CONFIG_SMP
>> rq->sd = NULL;
>> rq->rd = NULL;
>> - rq->cpu_power = SCHED_POWER_SCALE;
>> + rq->cpu_power = rq->cpu_power_orig = SCHED_POWER_SCALE;
>> rq->post_schedule = 0;
>> rq->active_balance = 0;
>> rq->next_balance = jiffies;
>> diff --git a/kernel/sched/fair.c b/kernel/sched/fair.c
>> index 7387c05..f1e4db6 100644
>> --- a/kernel/sched/fair.c
>> +++ b/kernel/sched/fair.c
>> @@ -5611,6 +5611,7 @@ static void update_cpu_power(struct sched_domain *sd, int cpu)
>>
>> power >>= SCHED_POWER_SHIFT;
>>
>> + cpu_rq(cpu)->cpu_power_orig = power;
>> sdg->sgp->power_orig = power;
>>
>> if (sched_feat(ARCH_POWER))
>> diff --git a/kernel/sched/sched.h b/kernel/sched/sched.h
>> index 46c3784..2abdf5c 100644
>> --- a/kernel/sched/sched.h
>> +++ b/kernel/sched/sched.h
>> @@ -578,6 +578,7 @@ struct rq {
>> struct sched_domain *sd;
>>
>> unsigned long cpu_power;
>> + unsigned long cpu_power_orig;
>>
>> unsigned char idle_balance;
>> /* For active balancing */
>>
>
^ permalink raw reply [flat|nested] 6+ messages in thread
* [RFC 4/4] sched: add per group cpu_power_orig
@ 2014-04-01 11:11 ` Vincent Guittot
0 siblings, 0 replies; 6+ messages in thread
From: Vincent Guittot @ 2014-04-01 11:11 UTC (permalink / raw)
To: linux-arm-kernel
On 1 April 2014 12:41, Preeti U Murthy <preeti@linux.vnet.ibm.com> wrote:
> Hi Vincent,
>
> On 03/28/2014 06:57 PM, Vincent Guittot wrote:
>> This new field cpu_power_orig reflects the available capacity of a CPUs unlike
>> the cpu_power which reflects the current capacity that can be altered by
>> frequency and rt tasks.
>>
>> Signed-off-by: Vincent Guittot <vincent.guittot@linaro.org>
>
> Should not the subject be "add per rq cpu_power_orig"?
yes you're right :-)
I don't know how i came to that subject... a quick and dirty
copy/paste of something else
thanks
Vincent
>
> Regards
> Preeti U Murthy
>> ---
>> kernel/sched/core.c | 2 +-
>> kernel/sched/fair.c | 1 +
>> kernel/sched/sched.h | 1 +
>> 3 files changed, 3 insertions(+), 1 deletion(-)
>>
>> diff --git a/kernel/sched/core.c b/kernel/sched/core.c
>> index 5b20b27..8513f83 100644
>> --- a/kernel/sched/core.c
>> +++ b/kernel/sched/core.c
>> @@ -6911,7 +6911,7 @@ void __init sched_init(void)
>> #ifdef CONFIG_SMP
>> rq->sd = NULL;
>> rq->rd = NULL;
>> - rq->cpu_power = SCHED_POWER_SCALE;
>> + rq->cpu_power = rq->cpu_power_orig = SCHED_POWER_SCALE;
>> rq->post_schedule = 0;
>> rq->active_balance = 0;
>> rq->next_balance = jiffies;
>> diff --git a/kernel/sched/fair.c b/kernel/sched/fair.c
>> index 7387c05..f1e4db6 100644
>> --- a/kernel/sched/fair.c
>> +++ b/kernel/sched/fair.c
>> @@ -5611,6 +5611,7 @@ static void update_cpu_power(struct sched_domain *sd, int cpu)
>>
>> power >>= SCHED_POWER_SHIFT;
>>
>> + cpu_rq(cpu)->cpu_power_orig = power;
>> sdg->sgp->power_orig = power;
>>
>> if (sched_feat(ARCH_POWER))
>> diff --git a/kernel/sched/sched.h b/kernel/sched/sched.h
>> index 46c3784..2abdf5c 100644
>> --- a/kernel/sched/sched.h
>> +++ b/kernel/sched/sched.h
>> @@ -578,6 +578,7 @@ struct rq {
>> struct sched_domain *sd;
>>
>> unsigned long cpu_power;
>> + unsigned long cpu_power_orig;
>>
>> unsigned char idle_balance;
>> /* For active balancing */
>>
>
^ permalink raw reply [flat|nested] 6+ messages in thread
end of thread, other threads:[~2014-04-01 11:11 UTC | newest]
Thread overview: 6+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2014-03-28 13:27 [RFC 4/4] sched: add per group cpu_power_orig Vincent Guittot
2014-03-28 13:27 ` Vincent Guittot
2014-04-01 10:41 ` Preeti U Murthy
2014-04-01 10:41 ` Preeti U Murthy
2014-04-01 11:11 ` Vincent Guittot
2014-04-01 11:11 ` Vincent Guittot
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.