Re: [PATCHSET for-4.13] cgroup: implement cgroup2 thread mode, v2

From	Waiman Long <longman@redhat.com>
Newsgroups	linux.kernel
Subject	Re: [PATCHSET for-4.13] cgroup: implement cgroup2 thread mode, v2
Date	2017-07-12 16:10 +0200
Message-ID	<u2quZ-LL-13@gated-at.bofh.it> (permalink)
References	(6 earlier) <u22iZ-29B-1@gated-at.bofh.it> <u24b8-3nP-25@gated-at.bofh.it> <u26FX-4O2-1@gated-at.bofh.it> <u2aJA-7t8-11@gated-at.bofh.it> <u2kzf-5ja-11@gated-at.bofh.it>
Organization	Red Hat

Show all headers | View raw

On 07/12/2017 03:45 AM, Peter Zijlstra wrote:
> On Tue, Jul 11, 2017 at 05:12:39PM -0400, Waiman Long wrote:
>> On 07/11/2017 12:52 PM, Peter Zijlstra wrote:
>>> On Tue, Jul 11, 2017 at 10:14:42AM -0400, Waiman Long wrote:
>>>
>>>> The "join" was a special op for the children of cgroup root to join the
>>>> root as part of a threaded subtree. The children can instead use the
>>>> "enable" option to become a thread root which was the configuration
>>>> shown above.  This behavior applied only to children of root. Down the
>>>> hierarchy, you can't have configuration like:
>>>>
>>>>      R (t=0)
>>>>     / \
>>>>        D (t=1)
>>>>       / \
>>>>      T   D (t=1)
>>> Why not?
>>>
>>> First you create:
>>>
>>>       R (t=0)
>>>      / \
>>>         D (t=1)
>>>        / \
>>>       T   T (t=1)
>>>
>>> Then you flip t=0 like:
>>>
>>>       R (t=0)
>>>      / \
>>>         D (t=1)
>>>        / \
>>>       T   D (t=0)
>>>
>>> And then you flip t=1 again:
>>>
>>>       R (t=0)
>>>      / \
>>>         D (t=1)
>>>        / \
>>>       T   D (t=1)
>> Tejun's thread mode patch has constraints on what operations are allowed
>> and what aren't. For a threaded subtree, thread mode cannot be disabled
>> in the middle of the tree.
> Where in that scenario did I change anything in the middle? All
> operations were on a leaf group.

What I mean is that you can't disable thread mode if not at the thread
root with no children left.

Cheers,
Longman

Back to linux.kernel | Previous | Next — Previous in thread | Find similar | Unroll thread

Thread

Re: [PATCHSET for-4.13] cgroup: implement cgroup2 thread mode, v2 Peter Zijlstra <peterz@infradead.org> - 2017-07-10 10:40 +0200
  Re: [PATCHSET for-4.13] cgroup: implement cgroup2 thread mode, v2 Waiman Long <longman@redhat.com> - 2017-07-10 23:10 +0200
    Re: [PATCHSET for-4.13] cgroup: implement cgroup2 thread mode, v2 Peter Zijlstra <peterz@infradead.org> - 2017-07-11 14:20 +0200
      Re: [PATCHSET for-4.13] cgroup: implement cgroup2 thread mode, v2 Waiman Long <longman@redhat.com> - 2017-07-11 16:20 +0200
        Re: [PATCHSET for-4.13] cgroup: implement cgroup2 thread mode, v2 Peter Zijlstra <peterz@infradead.org> - 2017-07-11 19:00 +0200
          Re: [PATCHSET for-4.13] cgroup: implement cgroup2 thread mode, v2 Waiman Long <longman@redhat.com> - 2017-07-11 23:20 +0200
            Re: [PATCHSET for-4.13] cgroup: implement cgroup2 thread mode, v2 Peter Zijlstra <peterz@infradead.org> - 2017-07-12 09:50 +0200
              Re: [PATCHSET for-4.13] cgroup: implement cgroup2 thread mode, v2 Waiman Long <longman@redhat.com> - 2017-07-12 16:10 +0200

csiph-web