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...
Thanks, Xun -----Original Message----- From: [email protected] [mailto:[email protected]] On Behalf Of Dongsheng Yang Sent: Tuesday, August 26, 2014 10:16 AM To: Li Zefan Cc: Tejun Heo; Dongsheng Yang; [email protected]; [email protected] Subject: Re: [PATCH V2] cgroup: Introduce cgroup_detach_task(). On Tue, Aug 26, 2014 at 9:35 AM, Li Zefan <[email protected]> wrote: > On 2014/8/25 23:00, Dongsheng Yang wrote: >> On Mon, Aug 25, 2014 at 10:47 PM, Tejun Heo <[email protected]> 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 [email protected] More majordomo info at http://vger.kernel.org/majordomo-info.html

