diff options
author | Tejun Heo <tj@kernel.org> | 2012-11-19 11:13:39 -0500 |
---|---|---|
committer | Tejun Heo <tj@kernel.org> | 2012-11-19 11:13:39 -0500 |
commit | 033fa1c5f5f73833598a0beb022c0048fb769dad (patch) | |
tree | ec6c34da43d3d71ec3dbfc116ae1c3b8197edc89 /kernel/cgroup.c | |
parent | 2260e7fc1f18ad815324605c1ce7d5c6fd9b19a2 (diff) |
cgroup, cpuset: remove cgroup_subsys->post_clone()
Currently CGRP_CPUSET_CLONE_CHILDREN triggers ->post_clone(). Now
that clone_children is cpuset specific, there's no reason to have this
rather odd option activation mechanism in cgroup core. cpuset can
check the flag from its ->css_allocate() and take the necessary
action.
Move cpuset_post_clone() logic to the end of cpuset_css_alloc() and
remove cgroup_subsys->post_clone().
Loosely based on Glauber's "generalize post_clone into post_create"
patch.
Signed-off-by: Tejun Heo <tj@kernel.org>
Original-patch-by: Glauber Costa <glommer@parallels.com>
Original-patch: <1351686554-22592-2-git-send-email-glommer@parallels.com>
Acked-by: Serge E. Hallyn <serge.hallyn@ubuntu.com>
Acked-by: Li Zefan <lizefan@huawei.com>
Cc: Glauber Costa <glommer@parallels.com>
Diffstat (limited to 'kernel/cgroup.c')
-rw-r--r-- | kernel/cgroup.c | 4 |
1 files changed, 0 insertions, 4 deletions
diff --git a/kernel/cgroup.c b/kernel/cgroup.c index 2895880e6800..96719f73e95d 100644 --- a/kernel/cgroup.c +++ b/kernel/cgroup.c | |||
@@ -4142,10 +4142,6 @@ static long cgroup_create(struct cgroup *parent, struct dentry *dentry, | |||
4142 | if (err) | 4142 | if (err) |
4143 | goto err_free_all; | 4143 | goto err_free_all; |
4144 | } | 4144 | } |
4145 | /* At error, ->css_free() callback has to free assigned ID. */ | ||
4146 | if (test_bit(CGRP_CPUSET_CLONE_CHILDREN, &parent->flags) && | ||
4147 | ss->post_clone) | ||
4148 | ss->post_clone(cgrp); | ||
4149 | 4145 | ||
4150 | if (ss->broken_hierarchy && !ss->warned_broken_hierarchy && | 4146 | if (ss->broken_hierarchy && !ss->warned_broken_hierarchy && |
4151 | parent->parent) { | 4147 | parent->parent) { |