]> www.infradead.org Git - users/jedix/linux-maple.git/commit
sched_ext: initialize built-in idle state before ops.init()
authorAndrea Righi <arighi@nvidia.com>
Tue, 25 Mar 2025 09:32:12 +0000 (10:32 +0100)
committerTejun Heo <tj@kernel.org>
Wed, 26 Mar 2025 20:47:50 +0000 (10:47 -1000)
commitf0c6eab5e45c529f449fbc595873719e00de6d79
treeab7c526f10b6332cad81421b49a8da00b5dba891
parenta8897ed8523d4c9d782e282b18005a3779c92714
sched_ext: initialize built-in idle state before ops.init()

A BPF scheduler may want to use the built-in idle cpumasks in ops.init()
before the scheduler is fully initialized, either directly or through a
BPF timer for example.

However, this would result in an error, since the idle state has not
been properly initialized yet.

This can be easily verified by modifying scx_simple to call
scx_bpf_get_idle_cpumask() in ops.init():

$ sudo scx_simple

DEBUG DUMP
===========================================================================

scx_simple[121] triggered exit kind 1024:
  runtime error (built-in idle tracking is disabled)
...

Fix this by properly initializing the idle state before ops.init() is
called. With this change applied:

$ sudo scx_simple
local=2 global=0
local=19 global=11
local=23 global=11
...

Fixes: d73249f88743d ("sched_ext: idle: Make idle static keys private")
Signed-off-by: Andrea Righi <arighi@nvidia.com>
Reviewed-by: Joel Fernandes <joelagnelf@nvidia.com>
Signed-off-by: Tejun Heo <tj@kernel.org>
kernel/sched/ext.c
kernel/sched/ext_idle.c