Skip to content

Commit a658353

Browse files
Tao ZhouPeter Zijlstra
authored andcommitted
sched/fair: Revise comment about lb decision matrix
If busiest group type is group_misfit_task, the local group type must be group_has_spare according to below code in update_sd_pick_busiest(): if (sgs->group_type == group_misfit_task && (!capacity_greater(capacity_of(env->dst_cpu), sg->sgc->max_capacity) || sds->local_stat.group_type != group_has_spare)) return false; group type imbalanced and overloaded and fully_busy are filtered in here. misfit and asym are filtered before in update_sg_lb_stats(). So, change the decision matrix to: busiest \ local has_spare fully_busy misfit asym imbalanced overloaded has_spare nr_idle balanced N/A N/A balanced balanced fully_busy nr_idle nr_idle N/A N/A balanced balanced misfit_task force N/A N/A N/A *N/A* *N/A* asym_packing force force N/A N/A force force imbalanced force force N/A N/A force force overloaded force force N/A N/A force avg_load Fixes: 0b0695f ("sched/fair: Rework load_balance()") Signed-off-by: Tao Zhou <[email protected]> Signed-off-by: Peter Zijlstra (Intel) <[email protected]> Reviewed-by: Dietmar Eggemann <[email protected]> Reviewed-by: Vincent Guittot <[email protected]> Link: https://lkml.kernel.org/r/[email protected]
1 parent 890d550 commit a658353

File tree

1 file changed

+1
-1
lines changed

1 file changed

+1
-1
lines changed

kernel/sched/fair.c

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -9496,7 +9496,7 @@ static inline void calculate_imbalance(struct lb_env *env, struct sd_lb_stats *s
94969496
* busiest \ local has_spare fully_busy misfit asym imbalanced overloaded
94979497
* has_spare nr_idle balanced N/A N/A balanced balanced
94989498
* fully_busy nr_idle nr_idle N/A N/A balanced balanced
9499-
* misfit_task force N/A N/A N/A force force
9499+
* misfit_task force N/A N/A N/A N/A N/A
95009500
* asym_packing force force N/A N/A force force
95019501
* imbalanced force force N/A N/A force force
95029502
* overloaded force force N/A N/A force avg_load

0 commit comments

Comments
 (0)