Re: [PATCH v10 3/9] cpuset: Simulate auto-off of sched.domain_root at cgroup removal

Waiman Long
2018-06-21T08:22:23Z

On 06/20/2018 10:11 PM, Peter Zijlstra wrote:
> On Mon, Jun 18, 2018 at 12:14:02PM +0800, Waiman Long wrote:
>> @@ -1058,7 +1060,12 @@ static int update_reserved_cpumask(struct cpuset *cpuset,
>> * Check if any CPUs in addmask or delmask are in the effective_cpus
>> * of a sibling cpuset. The implied cpu_exclusive of a scheduling
>> * domain root will ensure there are no overlap in cpus_allowed.
>> + *
>> + * This check is skipped if the cpuset is dying.
> Comments that state what the code does are mostly useless; please
> explain _why_ if anything.

I am adding more restrictions on where the domain_root can be turned on
to make sure that there will be no surprise.

I have a script to test the new cpuset v2 functionality and found that
cgroup deletion may sometime failed if there was not enough time for the
previous operation to complete. That is the reason why I relax the
checking for dying cgroup to make my test script pass.

Cheers,
Longman