Re: [PATCH v2 0/3] cgroup/cpuset: Fix some more bugs in cpuset partition code

[Waiman Long]

On 11/2/26 3:04 AM, Tejun Heo wrote:
> >    cgroup/cpuset: Fix incorrect CPU accounting with parent's
> >      effective_xcpus expansion
> >    cgroup/cpuset: Recompute effective cpumask in update_cpumasks_hier()
> >      after calling update_parent_effective_cpumask()
> >    cgroup/cpuset: Do housekeeping check when isolated child with root
> >      parent may get more xcpus
> Applied 1-3 to cgroup/for-7.4-fixes with the following tags added:
>
>    Reviewed-by: Guopeng Zhang <zhangguopeng@xxxxxxxxxx>
>    Tested-by: Guopeng Zhang <zhangguopeng@xxxxxxxxxx>
>
> Two issues I noticed:
>
> - Three of patch 1's new TEST_MATRIX rows end with A1 as a single-CPU
>    root partition, which has no sched domain, so they fail the ISOLCPUS
>    check when the script runs with -v.
>
> - The comment at the end of the "partcmd_update w/o newmask" block in
>    update_parent_effective_cpumask() still says there is no need to
>    adjust the parent's effective_cpus. Patch 1's new expansion branch now
>    does exactly that.

Thanks for noticing that. Will fix that in the next set of fixes that I 
am working on.

Cheers,
Longman