On Sun, Jul 20, 2014 at 9:32 AM, Vasily Averin <vvs@xxxxxxxxxxxxx> wrote: > v2: mntget(path->mnt) should not be called if mountpoint_last() returns 1. > > Currently umount on symlink blocks following umount: > > /vz is separate mount > > # ls /vz/ -al | grep test > drwxr-xr-x. 2 root root 4096 Jul 19 01:14 testdir > lrwxrwxrwx. 1 root root 11 Jul 19 01:16 testlink -> /vz/testdir > > # umount -l /vz/testlink > umount: /vz/testlink: not mounted (expected) > > # lsof /vz > # umount /vz > umount: /vz: device is busy. (unexpected) > > In this case mountpoint_last() gets an extra refcount on path->mnt > > Signed-off-by: Vasily Averin <vvs@xxxxxxxxxx> Sounds reasonable to me. Can you please send the patch inline and not as attachment? -- Thanks, //richard -- To unsubscribe from this list: send the line "unsubscribe linux-fsdevel" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html