Re: [PATCH v2] sched_ext: pass the initial cpu.idle state in scx_cgroup_init_args
From: Tao Cui
Date: Mon Aug 24 2026 - 22:28:57 EST
Hello, Tejun,
在 2026/8/25 00:54, Tejun Heo 写道:
> Hello, Tao.
>
> On Mon, Aug 24, 2026 at 10:28:16PM +0800, Tao Cui wrote:
>> +
>> + /* whether the cgroup is configured idle via cpu.idle */
>> + bool idle;
>
> In sched_ext, a bare "idle" reads as CPU idle state (ops.update_idle(),
> idle cpumasks, scx_bpf_pick_idle_cpu()). cpu.idle is the cgroup analog
> of the SCHED_IDLE policy, so please name the field sched_idle instead.
> Renaming tg->scx.idle to match would be nice too.
>
Thanks for the review. Both are renamed in v3: the new
scx_cgroup_init_args field, and tg->scx.idle -> tg->scx.sched_idle
as a separate second patch in the series, so 1/2 stays minimal for
stable backport.
Thanks,
Tao
> Thanks.
>