diff options
| author | Dietmar Eggemann <dietmar.eggemann@arm.com> | 2019-08-02 15:59:43 +0100 |
|---|---|---|
| committer | Peter Zijlstra <peterz@infradead.org> | 2019-08-06 12:49:18 +0200 |
| commit | f4904815f97a934258445a8f763f6b6c48f007e7 (patch) | |
| tree | 79166e57fd917a0b11c1db51870c1dae50f7a77b /kernel/sched/psi.c | |
| parent | e21a712a9685488f5ce80495b37b9fdbe96c230d (diff) | |
| download | linux-f4904815f97a934258445a8f763f6b6c48f007e7.tar.gz | |
sched/deadline: Fix double accounting of rq/running bw in push & pull
{push,pull}_dl_task() always calls {de,}activate_task() with .flags=0
which sets p->on_rq=TASK_ON_RQ_MIGRATING.
{push,pull}_dl_task()->{de,}activate_task()->{de,en}queue_task()->
{de,en}queue_task_dl() calls {sub,add}_{running,rq}_bw() since
p->on_rq==TASK_ON_RQ_MIGRATING.
So {sub,add}_{running,rq}_bw() in {push,pull}_dl_task() is
double-accounting for that task.
Fix it by removing rq/running bw accounting in [push/pull]_dl_task().
Fixes: 7dd778841164 ("sched/core: Unify p->on_rq updates")
Signed-off-by: Dietmar Eggemann <dietmar.eggemann@arm.com>
Signed-off-by: Peter Zijlstra (Intel) <peterz@infradead.org>
Cc: Valentin Schneider <valentin.schneider@arm.com>
Cc: Ingo Molnar <mingo@kernel.org>
Cc: Luca Abeni <luca.abeni@santannapisa.it>
Cc: Daniel Bristot de Oliveira <bristot@redhat.com>
Cc: Juri Lelli <juri.lelli@redhat.com>
Cc: Qais Yousef <qais.yousef@arm.com>
Link: https://lkml.kernel.org/r/20190802145945.18702-2-dietmar.eggemann@arm.com
Diffstat (limited to 'kernel/sched/psi.c')
0 files changed, 0 insertions, 0 deletions