diff options
| author | Youssef Esmat <[email protected]> | 2024-05-27 14:06:49 +0200 |
|---|---|---|
| committer | Peter Zijlstra <[email protected]> | 2024-07-29 12:22:35 +0200 |
| commit | a741b82423f41501e301eb6f9820b45ca202e877 (patch) | |
| tree | 2d399097164b6e696f65eee8a507ce3ae40c8a09 | |
| parent | c245910049d04fbfa85bb2f5acd591c24e9907c7 (diff) | |
sched/core: Clear prev->dl_server in CFS pick fast path
In case the previous pick was a DL server pick, ->dl_server might be
set. Clear it in the fast path as well.
Fixes: 63ba8422f876 ("sched/deadline: Introduce deadline servers")
Signed-off-by: Youssef Esmat <[email protected]>
Signed-off-by: Daniel Bristot de Oliveira <[email protected]>
Signed-off-by: Peter Zijlstra (Intel) <[email protected]>
Tested-by: Juri Lelli <[email protected]>
Cc: [email protected]
Link: https://lore.kernel.org/r/7f7381ccba09efcb4a1c1ff808ed58385eccc222.1716811044.git.bristot@kernel.org
| -rw-r--r-- | kernel/sched/core.c | 7 |
1 files changed, 7 insertions, 0 deletions
diff --git a/kernel/sched/core.c b/kernel/sched/core.c index e61da3b88be3..1074ae804480 100644 --- a/kernel/sched/core.c +++ b/kernel/sched/core.c @@ -5840,6 +5840,13 @@ __pick_next_task(struct rq *rq, struct task_struct *prev, struct rq_flags *rf) } /* + * This is a normal CFS pick, but the previous could be a DL pick. + * Clear it as previous is no longer picked. + */ + if (prev->dl_server) + prev->dl_server = NULL; + + /* * This is the fast path; it cannot be a DL server pick; * therefore even if @p == @prev, ->dl_server must be NULL. */ |