[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <CA+qeAOq7hYq-TJMh8jQpJy=nWwk2FsUMSS1Hd=-JkhLLXbweEQ@mail.gmail.com>
Date: Mon, 25 Aug 2014 22:46:03 +0800
From: Dongsheng Yang <dongsheng081251@...il.com>
To: Tejun Heo <tj@...nel.org>
Cc: Dongsheng Yang <yangds.fnst@...fujitsu.com>, lizefan@...wei.com,
cgroups@...r.kernel.org, linux-kernel@...r.kernel.org
Subject: Re: [PATCH V2] cgroup: Introduce cgroup_detach_task().
Hi tj.
On Mon, Aug 25, 2014 at 10:12 PM, Tejun Heo <tj@...nel.org> 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.
Thanx
> Thanks.
>
> --
> tejun
> --
> To unsubscribe from this list: send the line "unsubscribe cgroups" in
> the body of a message to majordomo@...r.kernel.org
> 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@...r.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/
Powered by blists - more mailing lists