Re: [PATCH-next v9 08/11] cgroup/cpuset: Move mpol_rebind_mm/cpuset_migrate_mm() calls inside cpuset_attach_task()

[Waiman Long]

On 6/30/26 10:14 PM, Ridong Chen wrote:
> > @@ -3203,28 +3208,60 @@ static void cpuset_attach_task(struct cpuset 
> > *cs, struct task_struct *task)
> >        */
> >       WARN_ON_ONCE(set_cpus_allowed_ptr(task, cpus_attach));
> >   +    if (cpuset_v2() && !attach_ctx.mems_updated)
> > +        return;
> > +
> >       cpuset_change_task_nodemask(task, &attach_ctx.nodemask_to);
> >       cpuset1_update_task_spread_flags(cs, task);
> > +
> > +    if ((task != task->group_leader) ||
> > +        (!is_memory_migrate(cs) && !attach_ctx.mems_updated))
> > +        return;
> > +
>
> Nit.
>
> IIUC, the !is_memory_migrate(cs) check may be unnecessary. Previously, 
> placing this condition outside could prevent an unnecessary loop, but 
> in its current position, it appears redundant.
>
>     if (task != task->group_leader ||
>         !attach_ctx.mems_updated)

You are right. The is_memory_migrate(cs) check is unnecessary. Will 
remove it in the next version.

Cheers,
Longman