[PATCH] sched_ext: Clarify sched_ext_ops table for userland scheduler

Changwoo Min posted 1 patch 2 weeks ago
kernel/sched/ext.c | 6 ++++--
1 file changed, 4 insertions(+), 2 deletions(-)
[PATCH] sched_ext: Clarify sched_ext_ops table for userland scheduler
Posted by Changwoo Min 2 weeks ago
Update the comments in sched_ext_ops to clarify this table is for
a BPF scheduler and a userland scheduler should also rely on the
sched_ext_ops table through the BPF scheduler.

Signed-off-by: Changwoo Min <changwoo@igalia.com>
---
 kernel/sched/ext.c | 6 ++++--
 1 file changed, 4 insertions(+), 2 deletions(-)

diff --git a/kernel/sched/ext.c b/kernel/sched/ext.c
index 02f39314ef8a..f1a185ab5fbc 100644
--- a/kernel/sched/ext.c
+++ b/kernel/sched/ext.c
@@ -199,8 +199,10 @@ struct scx_dump_ctx {
 /**
  * struct sched_ext_ops - Operation table for BPF scheduler implementation
  *
- * Userland can implement an arbitrary scheduling policy by implementing and
- * loading operations in this table.
+ * A BPF scheduler can implement an arbitrary scheduling policy by
+ * implementing and loading operations in this table. Note that a userland
+ * scheduling policy can also be implemented using the BPF scheduler
+ * as a shim layer.
  */
 struct sched_ext_ops {
 	/**
-- 
2.47.0
Re: [PATCH] sched_ext: Clarify sched_ext_ops table for userland scheduler
Posted by Tejun Heo 2 weeks ago
On Sat, Nov 09, 2024 at 11:25:14AM +0900, Changwoo Min wrote:
> Update the comments in sched_ext_ops to clarify this table is for
> a BPF scheduler and a userland scheduler should also rely on the
> sched_ext_ops table through the BPF scheduler.
> 
> Signed-off-by: Changwoo Min <changwoo@igalia.com>

Applied to sched_ext/for-6.13.

Thanks.

-- 
tejun