Commit 77dd4c52 authored by Jan Beulich's avatar Jan Beulich

_csched_cpu_pick(): simplify sched_smt_power_savings dependent condition

At least to me, using ?: instead of the (a && ...) || (!a && ...)
construct is far easier to grok with a single look.
Signed-off-by: default avatarJan Beulich <jbeulich@novell.com>
parent 66ca440f
......@@ -526,10 +526,9 @@ _csched_cpu_pick(const struct scheduler *ops, struct vcpu *vc, bool_t commit)
weight_cpu = cpus_weight(cpu_idlers);
weight_nxt = cpus_weight(nxt_idlers);
/* smt_power_savings: consolidate work rather than spreading it */
if ( ( sched_smt_power_savings
&& (weight_cpu > weight_nxt) )
|| ( !sched_smt_power_savings
&& (weight_cpu * migrate_factor < weight_nxt) ) )
if ( sched_smt_power_savings ?
weight_cpu > weight_nxt :
weight_cpu * migrate_factor < weight_nxt )
{
cpus_and(nxt_idlers, cpus, nxt_idlers);
spc = CSCHED_PCPU(nxt);
......
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment