Re: [PATCH v24 11/11] sched: Migrate whole chain in proxy_migrate_task()

[John Stultz]

On Mon, Dec 29, 2025 at 10:46 PM K Prateek Nayak <kprateek.nayak@xxxxxxx> wrote:
> On 11/25/2025 4:01 AM, John Stultz wrote:
> > diff --git a/include/linux/sched.h b/include/linux/sched.h
> > index 178ed37850470..775cc06f756d0 100644
> > --- a/include/linux/sched.h
> > +++ b/include/linux/sched.h
> > @@ -1243,6 +1243,9 @@ struct task_struct {
> >       struct mutex                    *blocked_on;    /* lock we're blocked on */
> >       struct task_struct              *blocked_donor; /* task that is boosting this task */
> >       raw_spinlock_t                  blocked_lock;
> > +#ifdef CONFIG_SCHED_PROXY_EXEC
> > +     struct list_head                migration_node;
>
> We can just reuse the "p->se.group_node" here instead of adding the
> "migration_node" - it is only used to move tasks during load balancing,
> and once we deactivate a task, it can no longer be picked for load
> balancing so we should be safe (serialized by the rq_lock).
>

Ah, yes. This along with the rest of your suggestion here is a nice
cleanup! Thank you! I'll get them integrated into the series!
-john