Re: [PATCHSET v2 sched_ext/for-7.3] sched_ext: Split sub-scheduler implementation into sub.c

[Andrea Righi <arighi@nvidia.com>]

Hi Tejun,

On Wed, Jul 01, 2026 at 08:10:42AM -1000, Tejun Heo wrote:
> Hello,
> 
> v2: Fold the scx_dispatch_sched() sub.h promotion into the split (patch 4) so
>     it is self-contained. v1 left it to a later patch, so the posted split had
>     sub.c call an ext.c file-local static (Andrea). __always_inline kept.
>     Patches 1-3 unchanged.
> 
> v1: https://lore.kernel.org/all/20260701031429.1892218-1-tj@kernel.org
> 
> The sub-scheduler implementation has grown and will keep growing. Move it
> out of ext.c into a new kernel/sched/ext/sub.c. The first three patches are
> mechanical prep (prefix file-local helpers, expose shared internals, inline
> a few trivial helpers) so the move itself stays pure code motion. No
> functional change.

Looks good to me.

Reviewed-by: Andrea Righi <arighi@nvidia.com>

Thanks,
-Andrea

> 
> Based on sched_ext/for-7.3 (5df6a4506d06) with sched_ext/for-7.2-fixes
> (b7d9c359e5cf) assumed merged.
> 
> Tejun Heo (4):
>   sched_ext: Prefix file-local ext.c helpers exposed by the sub.c split
>   sched_ext: Expose the ext.c internals used by the sub.c split
>   sched_ext: Inline small ext.c helpers shared across the sub.c split
>   sched_ext: Split sub-scheduler implementation into sub.c
> 
>  kernel/sched/build_policy.c |    2 +
>  kernel/sched/ext/ext.c      | 1117 +++++--------------------------------------
>  kernel/sched/ext/internal.h |  164 ++++++-
>  kernel/sched/ext/sub.c      |  668 ++++++++++++++++++++++++++
>  kernel/sched/ext/sub.h      |  161 +++++++
>  5 files changed, 1101 insertions(+), 1011 deletions(-)
> 
> Thanks.
> 
> --
> tejun