Re: [PATCH V2] cgroup: Introduce cgroup_detach_task().
From: Dongsheng Yang
Date: Mon Aug 25 2014 - 10:46:11 EST
On Mon, Aug 25, 2014 at 10:12 PM, Tejun Heo <tj@xxxxxxxxxx> wrote:
> On Mon, Aug 25, 2014 at 07:32:10PM +0800, Dongsheng Yang wrote:
>> Currently, the only method to detach a task from a cgroup is moving
>> it to others. It looks not natrual to me.
> Ummm... how is this different from moving it to the root cgroup?
> "just because" usually isn't a good enough rationale.
Thanx for your reply :).
My point here is that attaching and detaching are a pair of operations.
And the things involved in these operations should be a process
and a cgroup. But currently, when we want to detach a process from
a cgroup (A), we have to attach it to another cgroup (B). I think it is not
easy to understand. People would say: "Why I want to detach a process
from A, I have to care about another cgroup."
So I created this patch here wanting to make these operations more easy to
understand. And make attaching and detaching to be antonymous in action.
I think maybe we can use it in the Cgroup V2 which is in developing. :)
That's all my reason for this patch. But you could think it makes no sense.
It's okey to me.
> 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
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/