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 -- To unsubscribe from this list: send the line "unsubscribe cgroups" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
