OSDN Git Service

reimplement path_mountpoint() with less magic
authorAl Viro <viro@zeniv.linux.org.uk>
Fri, 10 Jan 2020 22:17:19 +0000 (17:17 -0500)
committerAl Viro <viro@zeniv.linux.org.uk>
Wed, 15 Jan 2020 06:36:06 +0000 (01:36 -0500)
commitc64cd6e34ea340adbb2a0a2f99cc884b96dcdca5
tree87786ffcaf8408cacc0784ea2bd280f93b26c7e2
parent1edc8eb2e93130e36ac74ac9c80913815a57d413
reimplement path_mountpoint() with less magic

... and get rid of a bunch of bugs in it.  Background:
the reason for path_mountpoint() is that umount() really doesn't
want attempts to revalidate the root of what it's trying to umount.
The thing we want to avoid actually happen from complete_walk();
solution was to do something parallel to normal path_lookupat()
and it both went overboard and got the boilerplate subtly
(and not so subtly) wrong.

A better solution is to do pretty much what the normal path_lookupat()
does, but instead of complete_walk() do unlazy_walk().  All it takes
to avoid that ->d_weak_revalidate() call...  mountpoint_last() goes
away, along with everything it got wrong, and so does the magic around
LOOKUP_NO_REVAL.

Another source of bugs is that when we traverse mounts at the final
location (and we need to do that - umount . expects to get whatever's
overmounting ., if any, out of the lookup) we really ought to take
care of ->d_manage() - as it is, manual umount of autofs automount
in progress can lead to unpleasant surprises for the daemon.  Easily
solved by using handle_lookup_down() instead of follow_mount().

Tested-by: Ian Kent <raven@themaw.net>
Signed-off-by: Al Viro <viro@zeniv.linux.org.uk>
fs/namei.c
fs/nfs/nfstrace.h
include/linux/namei.h