sched/deadline: Fix dl_server_stopped()

auldp · auldp · commit f02ab9e83f03 · 2025-09-11T16:22:54.000Z
JIRA: https://issues.redhat.com/browse/RHEL-110301 commit 4717432 Author: Huacai Chen <chenhuacai@kernel.org> Date: Sat Aug 9 21:04:19 2025 +0800 sched/deadline: Fix dl_server_stopped() Commit cccb45d ("sched/deadline: Less agressive dl_server handling") introduces dl_server_stopped(). But it is obvious that dl_server_stopped() should return true if dl_se->dl_server_active is 0. Fixes: cccb45d ("sched/deadline: Less agressive dl_server handling") Signed-off-by: Huacai Chen <chenhuacai@loongson.cn> Signed-off-by: Peter Zijlstra (Intel) <peterz@infradead.org> Link: https://lkml.kernel.org/r/20250809130419.1980742-1-chenhuacai@loongson.cn Signed-off-by: Phil Auld <pauld@redhat.com>
diff --git a/kernel/sched/deadline.c b/kernel/sched/deadline.c
@@ -1611,7 +1611,7 @@ void dl_server_stop(struct sched_dl_entity *dl_se)
 static bool dl_server_stopped(struct sched_dl_entity *dl_se)
 {
 	if (!dl_se->dl_server_active)
-		return false;
+		return true;
 
 	if (dl_se->dl_server_idle) {
 		dl_server_stop(dl_se);

Original file line number	Diff line number	Diff line change
`@@ -1611,7 +1611,7 @@ void dl_server_stop(struct sched_dl_entity *dl_se)`
`1611`	`1611`	`static bool dl_server_stopped(struct sched_dl_entity *dl_se)`
`1612`	`1612`	`{`
`1613`	`1613`	`if (!dl_se->dl_server_active)`
`1614`		`- return false;`
	`1614`	`+ return true;`
`1615`	`1615`
`1616`	`1616`	`if (dl_se->dl_server_idle) {`
`1617`	`1617`	`dl_server_stop(dl_se);`