Re: [PATCH 2/2] cgroup: fix cgroup hierarchy umount race

2012-07-16 Thread 'Tejun Heo'
Hello, Al. On Sat, Jul 14, 2012 at 01:08:52PM +0100, Al Viro wrote: > On Sat, Jul 07, 2012 at 04:46:59PM -0700, 'Tejun Heo' wrote: > > Fix it by holding an extra superblock->s_active reference across > > dput() from css release, which is the dput() path added by 48ddbe1946 > > and the only one whi

Re: [PATCH 2/2] cgroup: fix cgroup hierarchy umount race

2012-07-14 Thread Al Viro
On Sat, Jul 07, 2012 at 04:46:59PM -0700, 'Tejun Heo' wrote: > Fix it by holding an extra superblock->s_active reference across > dput() from css release, which is the dput() path added by 48ddbe1946 > and the only one which doesn't hold an extra s_active ref across the > final cgroup dput(). > @@

[PATCH 2/2] cgroup: fix cgroup hierarchy umount race

2012-07-07 Thread 'Tejun Heo'
>From 5db9a4d99b0157a513944e9a44d29c9cec2e91dc Mon Sep 17 00:00:00 2001 From: Tejun Heo Date: Sat, 7 Jul 2012 16:08:18 -0700 48ddbe1946 "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