summaryrefslogtreecommitdiffstats
path: root/kernel/cgroup.c
diff options
context:
space:
mode:
authorLi Zefan <lizefan@huawei.com>2013-01-24 14:31:27 +0800
committerTejun Heo <tj@kernel.org>2013-01-24 12:05:18 -0800
commit86a3db5643c7d29bb36ca85c7a4bb67ad4d88d77 (patch)
tree62fb73cc15fbcb751ee434a7bd9d1546851a4556 /kernel/cgroup.c
parent2a73991b76cbd38c4a0c6704449ccc08c89c3ff3 (diff)
downloadlinux-86a3db5643c7d29bb36ca85c7a4bb67ad4d88d77.tar.gz
linux-86a3db5643c7d29bb36ca85c7a4bb67ad4d88d77.tar.bz2
linux-86a3db5643c7d29bb36ca85c7a4bb67ad4d88d77.zip
cgroup: remove duplicate RCU free on struct cgroup
When destroying a cgroup, though in cgroup_diput() we've called synchronize_rcu(), we then still have to free it via call_rcu(). The story is, long ago to fix a race between reading /proc/sched_debug and freeing cgroup, the code was changed to utilize call_rcu(). See commit a47295e6bc42ad35f9c15ac66f598aa24debd4e2 ("cgroups: make cgroup_path() RCU-safe") As we've fixed cpu cgroup that cpu_cgroup_offline_css() is used to unregister a task_group so there won't be concurrent access to this task_group after synchronize_rcu() in diput(). Now we can just kfree(cgrp). Signed-off-by: Li Zefan <lizefan@huawei.com> Signed-off-by: Tejun Heo <tj@kernel.org>
Diffstat (limited to 'kernel/cgroup.c')
-rw-r--r--kernel/cgroup.c2
1 files changed, 1 insertions, 1 deletions
diff --git a/kernel/cgroup.c b/kernel/cgroup.c
index a04932281bc9..af993919aa04 100644
--- a/kernel/cgroup.c
+++ b/kernel/cgroup.c
@@ -892,7 +892,7 @@ static void cgroup_diput(struct dentry *dentry, struct inode *inode)
simple_xattrs_free(&cgrp->xattrs);
ida_simple_remove(&cgrp->root->cgroup_ida, cgrp->id);
- kfree_rcu(cgrp, rcu_head);
+ kfree(cgrp);
} else {
struct cfent *cfe = __d_cfe(dentry);
struct cgroup *cgrp = dentry->d_parent->d_fsdata;