Re: [PATCH 1/2] sched_ext: Initialize idle masks before ops.init()
From: Kuba Piecuch
Date: Fri Jul 31 2026 - 06:57:34 EST
Hi Andrea,
On Fri Jul 31, 2026 at 8:59 AM UTC, Andrea Righi wrote:
> @@ -59,11 +59,13 @@ static inline void init_sched_ext_class(void) {}
> #endif /* CONFIG_SCHED_CLASS_EXT */
>
> #ifdef CONFIG_SCHED_CLASS_EXT
> +DECLARE_STATIC_KEY_FALSE(scx_idle_tracking_enabled);
> +
I was originally thinking about reusing scx_builtin_idle_enabled here,
apologies if I wasn't clear enough.
My reasoning is: If the user is doing their own idle CPU tracking,
in which case scx_builtin_idle_enable will be disabled, what's the point
of SCX tracking idle CPUs?
Do you see a scenario where using one static branch is problematic?
Thanks,
Kuba