Re: [PATCH 4/7] sched_ext: Add a DL server for sched_ext tasks
From: Andrea Righi
Date: Wed Jan 21 2026 - 12:27:13 EST
On Wed, Jan 21, 2026 at 04:52:45PM +0100, Peter Zijlstra wrote:
> On Wed, Jan 21, 2026 at 01:49:38PM +0100, Andrea Righi wrote:
> > On Wed, Jan 21, 2026 at 01:29:01PM +0100, Peter Zijlstra wrote:
> > > On Tue, Jan 20, 2026 at 10:50:35PM +0100, Andrea Righi wrote:
> > >
> > > > @@ -1813,6 +1813,7 @@ void dl_server_stop(struct sched_dl_entity *dl_se)
> > > > hrtimer_try_to_cancel(&dl_se->dl_timer);
> > > > dl_se->dl_defer_armed = 0;
> > > > dl_se->dl_throttled = 0;
> > > > + dl_se->dl_defer_running = 0;
> > > > dl_se->dl_defer_idle = 0;
> > > > dl_se->dl_server_active = 0;
> > > > }
> > >
> > > This should definitely not be in this patch. Why was this added? Were
> > > you trying to do the same as:
> > >
> > > ca1e8eede4fc ("sched/deadline: Fix server stopping with runnable tasks")
> > >
> >
> > The problem is that if remove this, RT can completely stall EXT tasks, also
> > with ca1e8eede4fc applied.
>
> But that's not something ext specific, right? Can you pull this change
> out and write a sane Changelog for it, describing the problem and so?
Yeah, this should happen with fair as well. I'll try to reproduce the
problem without ext and try to get more info.
Thanks,
-Andrea