namei: lift nameidata into filename_mountpoint()
authorAl Viro <viro@zeniv.linux.org.uk>
Sat, 2 May 2015 02:08:30 +0000 (22:08 -0400)
committerAl Viro <viro@zeniv.linux.org.uk>
Mon, 11 May 2015 02:18:33 +0000 (22:18 -0400)
when we go for on-demand allocation of saved state in
link_path_walk(), we'll want nameidata to stay around
for all 3 calls of path_mountpoint().

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

index 293300c1a84eaaead9f38ede535c21c27665983c..ab2bcbdbd68346d48b9a99df1074bab953756040 100644 (file)
@@ -2344,31 +2344,28 @@ out:
  */
 static int
 path_mountpoint(int dfd, const struct filename *name, struct path *path,
-               unsigned int flags)
+               struct nameidata *nd, unsigned int flags)
 {
-       struct nameidata nd;
-       int err;
-
-       err = path_init(dfd, name, flags, &nd);
+       int err = path_init(dfd, name, flags, nd);
        if (unlikely(err))
                goto out;
 
-       err = mountpoint_last(&nd, path);
+       err = mountpoint_last(nd, path);
        while (err > 0) {
                void *cookie;
                struct path link = *path;
-               err = may_follow_link(&link, &nd);
+               err = may_follow_link(&link, nd);
                if (unlikely(err))
                        break;
-               nd.flags |= LOOKUP_PARENT;
-               err = follow_link(&link, &nd, &cookie);
+               nd->flags |= LOOKUP_PARENT;
+               err = follow_link(&link, nd, &cookie);
                if (err)
                        break;
-               err = mountpoint_last(&nd, path);
-               put_link(&nd, &link, cookie);
+               err = mountpoint_last(nd, path);
+               put_link(nd, &link, cookie);
        }
 out:
-       path_cleanup(&nd);
+       path_cleanup(nd);
        return err;
 }
 
@@ -2376,14 +2373,15 @@ static int
 filename_mountpoint(int dfd, struct filename *name, struct path *path,
                        unsigned int flags)
 {
+       struct nameidata nd;
        int error;
        if (IS_ERR(name))
                return PTR_ERR(name);
-       error = path_mountpoint(dfd, name, path, flags | LOOKUP_RCU);
+       error = path_mountpoint(dfd, name, path, &nd, flags | LOOKUP_RCU);
        if (unlikely(error == -ECHILD))
-               error = path_mountpoint(dfd, name, path, flags);
+               error = path_mountpoint(dfd, name, path, &nd, flags);
        if (unlikely(error == -ESTALE))
-               error = path_mountpoint(dfd, name, path, flags | LOOKUP_REVAL);
+               error = path_mountpoint(dfd, name, path, &nd, flags | LOOKUP_REVAL);
        if (likely(!error))
                audit_inode(name, path->dentry, 0);
        putname(name);