Re: [PATCH sched_ext/for-6.11] sched, sched_ext: Simplify dl_prio() case handling in sched_fork()

From: Tejun Heo
Date: Mon Jul 08 2024 - 14:56:11 EST


On Mon, Jun 24, 2024 at 10:51:49AM -1000, Tejun Heo wrote:
> From: Tejun Heo <tj@xxxxxxxxxx>
>
> sched_fork() returns with -EAGAIN if dl_prio(@p). a7a9fc549293 ("sched_ext:
> Add boilerplate for extensible scheduler class") added scx_pre_fork() call
> before it and then scx_cancel_fork() on the exit path. This is silly as the
> dl_prio() block can just be moved above the scx_pre_fork() call.
>
> Move the dl_prio() block above the scx_pre_fork() call and remove the now
> unnecessary scx_cancel_fork() invocation.
>
> Signed-off-by: Tejun Heo <tj@xxxxxxxxxx>
> Suggested-by: Thomas Gleixner <tglx@xxxxxxxxxxxxx>
> Cc: Ingo Molnar <mingo@xxxxxxxxxx>
> Cc: Peter Zijlstra <peterz@xxxxxxxxxxxxx>
> Cc: David Vernet <void@xxxxxxxxxxxxx>

Applying to sched_ext/for-6.11.

Thanks.

--
tejun