cgroup: mark "tasks" cgroup file as insane
authorTejun Heo <tj@kernel.org>
Tue, 4 Jun 2013 02:13:02 +0000 (19:13 -0700)
committerTejun Heo <tj@kernel.org>
Wed, 5 Jun 2013 19:00:33 +0000 (12:00 -0700)
Some resources controlled by cgroup aren't per-task and cgroup core
allowing threads of a single thread_group to be in different cgroups
forced memcg do explicitly find the group leader and use it.  This is
gonna be nasty when transitioning to unified hierarchy and in general
we don't want and won't support granularity finer than processes.

Mark "tasks" with CFTYPE_INSANE.

Signed-off-by: Tejun Heo <tj@kernel.org>
Acked-by: Li Zefan <lizefan@huawei.com>
Cc: Johannes Weiner <hannes@cmpxchg.org>
Cc: Michal Hocko <mhocko@suse.cz>
Cc: Balbir Singh <bsingharora@gmail.com>
Cc: KAMEZAWA Hiroyuki <kamezawa.hiroyu@jp.fujitsu.com>
Cc: cgroups@vger.kernel.org
Cc: Vivek Goyal <vgoyal@redhat.com>
kernel/cgroup.c

index fefc41c..1e0f445 100644 (file)
@@ -4037,6 +4037,7 @@ static int cgroup_clone_children_write(struct cgroup *cgrp,
 static struct cftype files[] = {
        {
                .name = "tasks",
+               .flags = CFTYPE_INSANE,         /* use "procs" instead */
                .open = cgroup_tasks_open,
                .write_u64 = cgroup_tasks_write,
                .release = cgroup_pidlist_release,