Commit 313e924c authored by Grzegorz Nosek's avatar Grzegorz Nosek Committed by Linus Torvalds

cgroups: relax ns_can_attach checks to allow attaching to grandchild cgroups

The ns_proxy cgroup allows moving processes to child cgroups only one
level deep at a time.  This commit relaxes this restriction and makes it
possible to attach tasks directly to grandchild cgroups, e.g.:

($pid is in the root cgroup)
echo $pid > /cgroup/CG1/CG2/tasks

Previously this operation would fail with -EPERM and would have to be
performed as two steps:
echo $pid > /cgroup/CG1/tasks
echo $pid > /cgroup/CG1/CG2/tasks

Also, the target cgroup no longer needs to be empty to move a task there.
Signed-off-by: default avatarGrzegorz Nosek <root@localdomain.pl>
Acked-by: default avatarSerge Hallyn <serue@us.ibm.com>
Reviewed-by: default avatarLi Zefan <lizf@cn.fujitsu.com>
Cc: Paul Menage <menage@google.com>
Signed-off-by: default avatarAndrew Morton <akpm@linux-foundation.org>
Signed-off-by: default avatarLinus Torvalds <torvalds@linux-foundation.org>
parent d20a390a
...@@ -348,8 +348,8 @@ int cgroup_path(const struct cgroup *cgrp, char *buf, int buflen); ...@@ -348,8 +348,8 @@ int cgroup_path(const struct cgroup *cgrp, char *buf, int buflen);
int cgroup_task_count(const struct cgroup *cgrp); int cgroup_task_count(const struct cgroup *cgrp);
/* Return true if the cgroup is a descendant of the current cgroup */ /* Return true if cgrp is a descendant of the task's cgroup */
int cgroup_is_descendant(const struct cgroup *cgrp); int cgroup_is_descendant(const struct cgroup *cgrp, struct task_struct *task);
/* Control Group subsystem type. See Documentation/cgroups.txt for details */ /* Control Group subsystem type. See Documentation/cgroups.txt for details */
......
...@@ -3084,18 +3084,19 @@ int cgroup_clone(struct task_struct *tsk, struct cgroup_subsys *subsys, ...@@ -3084,18 +3084,19 @@ int cgroup_clone(struct task_struct *tsk, struct cgroup_subsys *subsys,
} }
/** /**
* cgroup_is_descendant - see if @cgrp is a descendant of current task's cgrp * cgroup_is_descendant - see if @cgrp is a descendant of @task's cgrp
* @cgrp: the cgroup in question * @cgrp: the cgroup in question
* @task: the task in question
* *
* See if @cgrp is a descendant of the current task's cgroup in * See if @cgrp is a descendant of @task's cgroup in the appropriate
* the appropriate hierarchy. * hierarchy.
* *
* If we are sending in dummytop, then presumably we are creating * If we are sending in dummytop, then presumably we are creating
* the top cgroup in the subsystem. * the top cgroup in the subsystem.
* *
* Called only by the ns (nsproxy) cgroup. * Called only by the ns (nsproxy) cgroup.
*/ */
int cgroup_is_descendant(const struct cgroup *cgrp) int cgroup_is_descendant(const struct cgroup *cgrp, struct task_struct *task)
{ {
int ret; int ret;
struct cgroup *target; struct cgroup *target;
...@@ -3105,7 +3106,7 @@ int cgroup_is_descendant(const struct cgroup *cgrp) ...@@ -3105,7 +3106,7 @@ int cgroup_is_descendant(const struct cgroup *cgrp)
return 1; return 1;
get_first_subsys(cgrp, NULL, &subsys_id); get_first_subsys(cgrp, NULL, &subsys_id);
target = task_cgroup(current, subsys_id); target = task_cgroup(task, subsys_id);
while (cgrp != target && cgrp!= cgrp->top_cgroup) while (cgrp != target && cgrp!= cgrp->top_cgroup)
cgrp = cgrp->parent; cgrp = cgrp->parent;
ret = (cgrp == target); ret = (cgrp == target);
......
...@@ -34,7 +34,7 @@ int ns_cgroup_clone(struct task_struct *task, struct pid *pid) ...@@ -34,7 +34,7 @@ int ns_cgroup_clone(struct task_struct *task, struct pid *pid)
/* /*
* Rules: * Rules:
* 1. you can only enter a cgroup which is a child of your current * 1. you can only enter a cgroup which is a descendant of your current
* cgroup * cgroup
* 2. you can only place another process into a cgroup if * 2. you can only place another process into a cgroup if
* a. you have CAP_SYS_ADMIN * a. you have CAP_SYS_ADMIN
...@@ -45,21 +45,15 @@ int ns_cgroup_clone(struct task_struct *task, struct pid *pid) ...@@ -45,21 +45,15 @@ int ns_cgroup_clone(struct task_struct *task, struct pid *pid)
static int ns_can_attach(struct cgroup_subsys *ss, static int ns_can_attach(struct cgroup_subsys *ss,
struct cgroup *new_cgroup, struct task_struct *task) struct cgroup *new_cgroup, struct task_struct *task)
{ {
struct cgroup *orig;
if (current != task) { if (current != task) {
if (!capable(CAP_SYS_ADMIN)) if (!capable(CAP_SYS_ADMIN))
return -EPERM; return -EPERM;
if (!cgroup_is_descendant(new_cgroup)) if (!cgroup_is_descendant(new_cgroup, current))
return -EPERM; return -EPERM;
} }
if (atomic_read(&new_cgroup->count) != 0) if (!cgroup_is_descendant(new_cgroup, task))
return -EPERM;
orig = task_cgroup(task, ns_subsys_id);
if (orig && orig != new_cgroup->parent)
return -EPERM; return -EPERM;
return 0; return 0;
...@@ -77,7 +71,7 @@ static struct cgroup_subsys_state *ns_create(struct cgroup_subsys *ss, ...@@ -77,7 +71,7 @@ static struct cgroup_subsys_state *ns_create(struct cgroup_subsys *ss,
if (!capable(CAP_SYS_ADMIN)) if (!capable(CAP_SYS_ADMIN))
return ERR_PTR(-EPERM); return ERR_PTR(-EPERM);
if (!cgroup_is_descendant(cgroup)) if (!cgroup_is_descendant(cgroup, current))
return ERR_PTR(-EPERM); return ERR_PTR(-EPERM);
ns_cgroup = kzalloc(sizeof(*ns_cgroup), GFP_KERNEL); ns_cgroup = kzalloc(sizeof(*ns_cgroup), GFP_KERNEL);
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment