[PATCH 13/14] sched: update the cpu_power

Morten Rasmussen morten.rasmussen at arm.com
Wed May 22 11:46:39 EDT 2013


On Thu, Apr 25, 2013 at 06:23:29PM +0100, Vincent Guittot wrote:
> @@ -4596,8 +4597,12 @@ static void update_cpu_power(struct sched_domain *sd, int cpu)
>  	cpu_rq(cpu)->cpu_available = power;
>  	sdg->sgp->power_available = power;
>  
> +	if (!is_my_buddy(cpu, cpu))
> +		power = 1;
> +

Using your fix for is_my_buddy() for patch 11 the above will always be
true when sched_packing_mode is to anything else than
SCHED_PACKING_FULL. So cpu_power for all cpus in
SCHED_PACKING_{DEFAULT,NONE} is 1.

As far is I understand, this is not the intention?

Cheers,
Morten

>  	cpu_rq(cpu)->cpu_power = power;
>  	sdg->sgp->power = power;
> +
>  }
>  
>  void update_group_power(struct sched_domain *sd, int cpu)
> -- 
> 1.7.9.5
> 
> 




More information about the linux-arm-kernel mailing list