RE: [PATCH V2] cgroup: Introduce cgroup_detach_task().
From: Ni, Xun
Date: Mon Aug 25 2014 - 22:22:38 EST
Maybe your point can be added to the README or FAQ for cgroup. Try to shoot a patch for that, it will be more effective...
From: cgroups-owner@xxxxxxxxxxxxxxx [mailto:cgroups-owner@xxxxxxxxxxxxxxx] On Behalf Of Dongsheng Yang
Sent: Tuesday, August 26, 2014 10:16 AM
To: Li Zefan
Cc: Tejun Heo; Dongsheng Yang; cgroups@xxxxxxxxxxxxxxx; linux-kernel@xxxxxxxxxxxxxxx
Subject: Re: [PATCH V2] cgroup: Introduce cgroup_detach_task().
On Tue, Aug 26, 2014 at 9:35 AM, Li Zefan <lizefan@xxxxxxxxxx> wrote:
> On 2014/8/25 23:00, Dongsheng Yang wrote:
>> On Mon, Aug 25, 2014 at 10:47 PM, Tejun Heo <tj@xxxxxxxxxx> wrote:
>>> On Mon, Aug 25, 2014 at 10:46:03PM +0800, Dongsheng Yang wrote:
>>>> My point here is that attaching and detaching are a pair of operations.
>>> There is no detaching from a cgroup. A task is always attached to a
>>> cgroup whether that's a root or non-root cgroup.
>> Okey, I should not think it as attaching and detaching. Just treat
>> them as a move between root and non-root cgroup.
>> It sounds reasonable to me now.
> I from time to time have to explain this to other people.
Ha, we usually want to find a detach function when we saw a function named as cgroup_attach_task().
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