Re: [PATCH 5/6] cgroup: implement cgroup v2 thread support

Tejun Heo
2017-07-18T17:10:46Z

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.

Thanks.

--
tejun