Re: [PATCH] sched_ext: Documentation: Fix ops table header reference

[Andrea Righi <arighi@nvidia.com>]

On Tue, Jul 07, 2026 at 05:45:38PM +0800, luoliang@kylinos.cn wrote:
> From: Liang Luo <luoliang@kylinos.cn>
> 
> The "Where to Look" and "ABI Instability" sections state that the ops
> table is defined in include/linux/sched/ext.h. However, struct
> sched_ext_ops is actually defined in kernel/sched/ext/internal.h, along
> with the SCX_OPS_* flags; include/linux/sched/ext.h holds the core data
> structures (struct sched_ext_entity, struct scx_dispatch_q, ...) and the
> DSQ constants. Point the ops table references to the correct header.
> 
> Signed-off-by: Liang Luo <luoliang@kylinos.cn>

Looks good to me.

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

Thanks,
-Andrea

> ---
>  Documentation/scheduler/sched-ext.rst | 8 +++++---
>  1 file changed, 5 insertions(+), 3 deletions(-)
> 
> diff --git a/Documentation/scheduler/sched-ext.rst b/Documentation/scheduler/sched-ext.rst
> index 4b1ffd03f516..2771ea4cc14a 100644
> --- a/Documentation/scheduler/sched-ext.rst
> +++ b/Documentation/scheduler/sched-ext.rst
> @@ -493,8 +493,9 @@ a freshly woken up task gets on a CPU.
>  Where to Look
>  =============
>  
> -* ``include/linux/sched/ext.h`` defines the core data structures, ops table
> -  and constants.
> +* ``include/linux/sched/ext.h`` defines the core data structures and
> +  constants, while the ops table (``struct sched_ext_ops``) is defined in
> +  ``kernel/sched/ext/internal.h``.
>  
>  * ``kernel/sched/ext/ext.c`` contains sched_ext core implementation and helpers.
>    The functions prefixed with ``scx_bpf_`` can be called from the BPF
> @@ -555,7 +556,8 @@ ABI Instability
>  ===============
>  
>  The APIs provided by sched_ext to BPF schedulers programs have no stability
> -guarantees. This includes the ops table callbacks and constants defined in
> +guarantees. This includes the ops table callbacks defined in
> +``kernel/sched/ext/internal.h`` and the constants defined in
>  ``include/linux/sched/ext.h``, as well as the ``scx_bpf_`` kfuncs defined in
>  ``kernel/sched/ext/ext.c`` and ``kernel/sched/ext/idle.c``.
>  
> -- 
> 2.43.0
>