Re: [patch v5 11/15] sched: add power/performance balance allow flag

[Alex Shi]

>>> (power_lb == 1 && perf_lb == 1) is incorrect and impossible to have.
>>>
>>> (power_lb == 0 && perf_lb == 0) is possible and it means there is no any
>>> balance on this cpu.
>>>
>>> So, enumeration is not enough.
>>
>> Huh.. both 0 doesn't make any sense either. If there's no balancing, we
>> shouldn't be here to begin with.
> 
> Also, why is this in the lb_env at all, shouldn't we simply use the
> global sched_balance_policy all over the place? Its not like we want to
> change power/perf on a finer granularity.

they are in lb_env, since we need to set them according to each group
status, mostly in update_sd_lb_power_stats().

Even the sched_balance_policy is powersaving, the domain may also need
performance balance since there are maybe too much tasks or much
imbalance in domain.

when we find the domain is not suitable for power balance, we will set
lb_perf = 1, then we don't need go through other groups for power info
collection.
> 


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