Re: [PATCH v5 09/14] sched/debug: Add support to change sched_ext server params

From: Tejun Heo
Date: Mon Jun 23 2025 - 18:13:12 EST


On Fri, Jun 20, 2025 at 04:32:24PM -0400, Joel Fernandes wrote:
...
> - if (!runtime)
> - printk_deferred("Fair server disabled in CPU %d, system may crash due to starvation.\n",
> - cpu_of(rq));
> + if (!runtime) {
> + if (server == &rq->fair_server)
> + printk_deferred("Fair server disabled on CPU %d, system may crash due to starvation.\n",
> + cpu_of(rq));
> + else
> + printk_deferred("Ext server disabled on CPU %d, system may crash due to starvation.\n",
> + cpu_of(rq));

It's minor but the above is a bit ugly. Maybe pass in the name as an arg or
put it in dl_se?

Thanks.

--
tejun