[PATCH] sched: Fix the comment error in the wait_task_inactive interface.
From: hupu
Date: Wed Sep 18 2024 - 04:31:09 EST
The previous comment was incorrect because "task_on_cpu" only care about
p->on_cpu and does not care whether the runqueue has changed or not,
especially on SMP systems. In addition, task_on_cpu returns true
instead of false when p is running on a CPU.
Signed-off-by: hupu<hupu.gm@xxxxxxxxx>
---
kernel/sched/core.c | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/kernel/sched/core.c b/kernel/sched/core.c
index f3951e4a55e5..05b231a18440 100644
--- a/kernel/sched/core.c
+++ b/kernel/sched/core.c
@@ -2129,8 +2129,8 @@ unsigned long wait_task_inactive(struct task_struct *p, unsigned int match_state
* NOTE! Since we don't hold any locks, it's not
* even sure that "rq" stays as the right runqueue!
* But we don't care, since "task_on_cpu()" will
- * return false if the runqueue has changed and p
- * is actually now running somewhere else!
+ * return true as long as p is running on a CPU,
+ * regardless of any changes to the runqueue.
*/
while (task_on_cpu(rq, p)) {
if (!task_state_match(p, match_state))
-- 2.17.1