On 07/18/2017 01:10 PM, Tejun Heo wrote: > Hello, Waiman. > > On Tue, Jul 18, 2017 at 10:37:41AM -0400, Waiman Long wrote: >> Thinking about it some more. There is a place for invalid domain. It is >> not the child of a threaded cgroup. It is the siblings of a threaded >> cgroup whose parent is not root. >> >> Root - A (domain) - B (domain) >> \ C (domain) >> >> With "echo threaded > B/cgroup.type": >> >> Root - A (threaded domain) - B (threaded) >> \ C (domain, invalid) > Yes, I noted that when I was replying to Peter. > >> Any children of a threaded cgroup should be threaded. > It's really difficult to discuss if you just declare that something > should be a certain way without giving rationale for thinking so. > > If we could get rid of the invalid state completely that way, I'd > completely agree with you but that isn't the case here as you noted > yourself, so the choice between the two isn't something trivially > clear. Both choices come with their pros and cons. We can absoultely > discuss them comparing the pros and cons. I am not advocating on removing the invalid state now as I note about sibling cgroups. I am just saying that there is no point in not doing an automatic conversion to threaded for newly created children of threaded cgroups (not thread root). I don't see any cons in doing that. 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
