Skip to content

Commit 0a00a35

Browse files
Chengming ZhouPeter Zijlstra
authored andcommitted
sched/fair: Delete useless condition in tg_unthrottle_up()
We have tested cfs_rq->load.weight in cfs_rq_is_decayed(), the first condition "!cfs_rq_is_decayed(cfs_rq)" is enough to cover the second condition "cfs_rq->nr_running". Signed-off-by: Chengming Zhou <[email protected]> Signed-off-by: Peter Zijlstra (Intel) <[email protected]> Reviewed-by: Ben Segall <[email protected]> Reviewed-by: Vincent Guittot <[email protected]> Link: https://lore.kernel.org/r/[email protected]
1 parent 64eaf50 commit 0a00a35

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
@@ -4850,7 +4850,7 @@ static int tg_unthrottle_up(struct task_group *tg, void *data)
48504850
cfs_rq->throttled_clock_pelt;
48514851

48524852
/* Add cfs_rq with load or one or more already running entities to the list */
4853-
if (!cfs_rq_is_decayed(cfs_rq) || cfs_rq->nr_running)
4853+
if (!cfs_rq_is_decayed(cfs_rq))
48544854
list_add_leaf_cfs_rq(cfs_rq);
48554855
}
48564856

0 commit comments

Comments
 (0)