diff options
| author | Li Zefan <lizefan@huawei.com> | 2013-01-24 07:32:02 +0100 |
|---|---|---|
| committer | Tejun Heo <tj@kernel.org> | 2013-01-24 21:05:18 +0100 |
| commit | 9ed8a659703876a9fe96ab86d1b296c2f0084242 (patch) | |
| tree | 03db3def6303814681f19412c524b3951ccde242 /kernel/cgroup.c | |
| parent | cgroup: remove synchronize_rcu() from cgroup_diput() (diff) | |
| download | linux-9ed8a659703876a9fe96ab86d1b296c2f0084242.tar.xz linux-9ed8a659703876a9fe96ab86d1b296c2f0084242.zip | |
cgroup: remove bogus comments in cgroup_diput()
Since commit 48ddbe194623ae089cc0576e60363f2d2e85662a
("cgroup: make css->refcnt clearing on cgroup removal optional"),
each css holds a ref on cgroup's dentry, so cgroup_diput() won't be
called until all css' refs go down to 0, which invalids the comments.
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.c | 6 |
1 files changed, 0 insertions, 6 deletions
diff --git a/kernel/cgroup.c b/kernel/cgroup.c index 02e4f201472e..800852282c21 100644 --- a/kernel/cgroup.c +++ b/kernel/cgroup.c @@ -899,12 +899,6 @@ static void cgroup_diput(struct dentry *dentry, struct inode *inode) struct cgroup *cgrp = dentry->d_fsdata; BUG_ON(!(cgroup_is_removed(cgrp))); - /* It's possible for external users to be holding css - * reference counts on a cgroup; css_put() needs to - * be able to access the cgroup after decrementing - * the reference count in order to know if it needs to - * queue the cgroup to be handled by the release - * agent */ call_rcu(&cgrp->rcu_head, cgroup_free_rcu); } else { struct cfent *cfe = __d_cfe(dentry); |