Re: [PATCH] sched_ext: Provide a per-scheduler unique sequence counter

From: Andrea Righi
Date: Tue Oct 01 2024 - 09:29:25 EST


On Mon, Sep 30, 2024 at 08:33:17AM -1000, Tejun Heo wrote:
> Hello, Andrea.
>
> On Fri, Sep 27, 2024 at 06:59:01PM +0200, Andrea Righi wrote:
> ...
> > @@ -662,6 +662,13 @@ struct sched_ext_ops {
> > */
> > u64 hotplug_seq;
> >
> > + /*
> > + * enable_seq - unique per-scheduler counter that can be accessed from
> > + * user-space to determine if a scheduler (within a specific hierarchy)
> > + * has been restarted.
> > + */
> > + s64 enable_seq;
>
> Let's just make it a global variable for now. When we package up context for
> each scheduler instance into a struct, it will get packaged up together.
> It's a bit odd to add enable_seq to ops as userspace can't do anything with
> it (note that hotplug_seq is different in that it's provided by the
> userspace on load).

Yep, makes sense, I just sent it because it was mentioned in the other
thread about enable_seq, but we don't really need it right now, so we
can just ignore it for now.

Thanks,
-Andrea