rcuwalk: recheck mount_lock after mountpoint crossing attempts
authorAl Viro <viro@zeniv.linux.org.uk>
Thu, 20 Mar 2014 19:18:22 +0000 (15:18 -0400)
committerAl Viro <viro@zeniv.linux.org.uk>
Sun, 23 Mar 2014 04:32:55 +0000 (00:32 -0400)
commitb37199e626b31e1175fb06764c5d1d687723aac2
tree0ac51a84f15fa251800fb40191dcf4a86c4c04c2
parente825196d48d2b89a6ec3a8eff280098d2a78207e
rcuwalk: recheck mount_lock after mountpoint crossing attempts

We can get false negative from __lookup_mnt() if an unrelated vfsmount
gets moved.  In that case legitimize_mnt() is guaranteed to fail,
and we will fall back to non-RCU walk... unless we end up running
into a hard error on a filesystem object we wouldn't have reached
if not for that false negative.  IOW, delaying that check until
the end of pathname resolution is wrong - we should recheck right
after we attempt to cross the mountpoint.  We don't need to recheck
unless we see d_mountpoint() being true - in that case even if
we have just raced with mount/umount, we can simply go on as if
we'd come at the moment when the sucker wasn't a mountpoint; if we
run into a hard error as the result, it was a legitimate outcome.
__lookup_mnt() returning NULL is different in that respect, since
it might've happened due to operation on completely unrelated
mountpoint.

Cc: stable@vger.kernel.org
Signed-off-by: Al Viro <viro@zeniv.linux.org.uk>
fs/namei.c