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

Tejun Heo
2024-07-08T18:56:05Z

On Mon, Jun 24, 2024 at 10:51:49AM -1000, Tejun Heo wrote:
> From: Tejun Heo <tj@kernel.org>
>
> 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@kernel.org>
> Suggested-by: Thomas Gleixner <tglx@linutronix.de>
> Cc: Ingo Molnar <mingo@redhat.com>
> Cc: Peter Zijlstra <peterz@infradead.org>
> Cc: David Vernet <void@manifault.com>

Applying to sched_ext/for-6.11.

Thanks.

--
tejun