]> git.baikalelectronics.ru Git - kernel.git/commit
cgroup: superblock can't be released with active dentries
authorTejun Heo <tj@kernel.org>
Thu, 24 May 2012 15:24:39 +0000 (08:24 -0700)
committerTejun Heo <tj@kernel.org>
Mon, 28 May 2012 00:22:56 +0000 (17:22 -0700)
commit49e407277138149cc4776a781f9dc59ff9749212
tree4288f6d27c8a2e30c1fa3b8a9a0f95c5fd6d48c6
parentd16cb33fbeaac4b15d2d63130945f7959a0d1182
cgroup: superblock can't be released with active dentries

a574ecbde6 "cgroup: make css->refcnt clearing on cgroup removal
optional" allowed a css to linger after the associated cgroup is
removed.  As a css holds a reference on the cgroup's dentry, it means
that cgroup dentries may linger for a while.

cgroup_create() does grab an active reference on the superblock to
prevent it from going away while there are !root cgroups; however, the
reference is put from cgroup_diput() which is invoked on cgroup
removal, so cgroup dentries which are removed but persisting due to
lingering csses already have released their superblock active refs
allowing superblock to be killed while those dentries are around.

Given the right condition, this makes cgroup_kill_sb() call
kill_litter_super() with dentries with non-zero d_count leading to
BUG() in shrink_dcache_for_umount_subtree().

Fix it by adding cgroup_dops->d_release() operation and moving
deactivate_super() to it.  cgroup_diput() now marks dentry->d_fsdata
with itself if superblock should be deactivated and cgroup_d_release()
deactivates the superblock on dentry release.

Signed-off-by: Tejun Heo <tj@kernel.org>
Reported-by: Sasha Levin <levinsasha928@gmail.com>
Tested-by: Sasha Levin <levinsasha928@gmail.com>
LKML-Reference: <CA+1xoqe5hMuxzCRhMy7J0XchDk2ZnuxOHJKikROk1-ReAzcT6g@mail.gmail.com>
Acked-by: Li Zefan <lizefan@huawei.com>
kernel/cgroup.c