diff options
| author | David Carlier <devnexen@gmail.com> | 2026-04-30 10:27:47 +0100 |
|---|---|---|
| committer | Tejun Heo <tj@kernel.org> | 2026-05-04 11:01:04 -1000 |
| commit | b34c82777a2c0648ee053595f4b290fd5249b093 (patch) | |
| tree | 6019ada06d513ea7b06da91f69286325cac6bb0f /rust | |
| parent | ff9eda4ea906b1f02fc260ddc42d2d9bd736a49c (diff) | |
sched_ext: idle: Recheck prev_cpu after narrowing allowed mask
scx_select_cpu_dfl() narrows @allowed to @cpus_allowed & @p->cpus_ptr
when the BPF caller supplies a @cpus_allowed that differs from
@p->cpus_ptr and @p doesn't have full affinity. However,
@is_prev_allowed was computed against the original (wider)
@cpus_allowed, so the prev_cpu fast paths could pick a @prev_cpu that
is in @cpus_allowed but not in @p->cpus_ptr, violating the intended
invariant that the returned CPU is always usable by @p. The kernel
masks this via the SCX_EV_SELECT_CPU_FALLBACK fallback, but the
behavior contradicts the documented contract.
Move the @is_prev_allowed evaluation past the narrowing block so it
tests against the final @allowed mask.
Fixes: ee9a4e92799d ("sched_ext: idle: Properly handle invalid prev_cpu during idle selection")
Cc: stable@vger.kernel.org # v6.16+
Assisted-by: Claude <noreply@anthropic.com>
Signed-off-by: David Carlier <devnexen@gmail.com>
Reviewed-by: Andrea Righi <arighi@nvidia.com>
Signed-off-by: Tejun Heo <tj@kernel.org>
Diffstat (limited to 'rust')
0 files changed, 0 insertions, 0 deletions
