diff options
| author | Chengming Zhou <zhouchengming@bytedance.com> | 2022-04-08 19:53:09 +0800 |
|---|---|---|
| committer | Peter Zijlstra <peterz@infradead.org> | 2022-04-22 12:14:07 +0200 |
| commit | 0a00a354644ee1800d31c47cf5927b9b50272fac (patch) | |
| tree | 9bd2909864610b2678df51c2fbabb662b37d6ea0 /kernel/sched | |
| parent | 64eaf50731ac0a8c76ce2fedd50ef6652aabc5ff (diff) | |
| download | linux-0a00a354644ee1800d31c47cf5927b9b50272fac.tar.gz linux-0a00a354644ee1800d31c47cf5927b9b50272fac.tar.bz2 linux-0a00a354644ee1800d31c47cf5927b9b50272fac.zip | |
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 <zhouchengming@bytedance.com>
Signed-off-by: Peter Zijlstra (Intel) <peterz@infradead.org>
Reviewed-by: Ben Segall <bsegall@google.com>
Reviewed-by: Vincent Guittot <vincent.guittot@linaro.org>
Link: https://lore.kernel.org/r/20220408115309.81603-2-zhouchengming@bytedance.com
Diffstat (limited to 'kernel/sched')
| -rw-r--r-- | kernel/sched/fair.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/kernel/sched/fair.c b/kernel/sched/fair.c index f74b34080c9a..3eba0dcc4962 100644 --- a/kernel/sched/fair.c +++ b/kernel/sched/fair.c @@ -4850,7 +4850,7 @@ static int tg_unthrottle_up(struct task_group *tg, void *data) cfs_rq->throttled_clock_pelt; /* Add cfs_rq with load or one or more already running entities to the list */ - if (!cfs_rq_is_decayed(cfs_rq) || cfs_rq->nr_running) + if (!cfs_rq_is_decayed(cfs_rq)) list_add_leaf_cfs_rq(cfs_rq); } |