Skip to content
Snippets Groups Projects
Commit f5717c93 authored by Chuyi Zhou's avatar Chuyi Zhou Committed by Tejun Heo
Browse files

sched_ext: Use SCX_CALL_OP_TASK in task_tick_scx


Now when we use scx_bpf_task_cgroup() in ops.tick() to get the cgroup of
the current task, the following error will occur:

scx_foo[3795244] triggered exit kind 1024:
  runtime error (called on a task not being operated on)

The reason is that we are using SCX_CALL_OP() instead of SCX_CALL_OP_TASK()
when calling ops.tick(), which triggers the error during the subsequent
scx_kf_allowed_on_arg_tasks() check.

SCX_CALL_OP_TASK() was first introduced in commit 36454023 ("sched_ext:
Track tasks that are subjects of the in-flight SCX operation") to ensure
task's rq lock is held when accessing task's sched_group. Since ops.tick()
is marked as SCX_KF_TERMINAL and task_tick_scx() is protected by the rq
lock, we can use SCX_CALL_OP_TASK() to avoid the above issue. Similarly,
the same changes should be made for ops.disable() and ops.exit_task(), as
they are also protected by task_rq_lock() and it's safe to access the
task's task_group.

Fixes: 36454023 ("sched_ext: Track tasks that are subjects of the in-flight SCX operation")
Signed-off-by: default avatarChuyi Zhou <zhouchuyi@bytedance.com>
Signed-off-by: default avatarTejun Heo <tj@kernel.org>
parent 2e2006c9
No related branches found
No related tags found
Loading
Loading
0% Loading or .
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment