handle_mounts(): pass dentry in, turn path into a pure out argument

All callers are equivalent to
	path->dentry = dentry;
	path->mnt = nd->path.mnt;
	err = handle_mounts(path, ...)
Pass dentry as an explicit argument, fill *path in handle_mounts()
itself.

Signed-off-by: Al Viro <viro@zeniv.linux.org.uk>
This commit is contained in:
Al Viro 2020-01-09 14:41:00 -05:00
parent e73cabff59
commit db3c9ade50

View File

@ -1385,11 +1385,15 @@ static bool __follow_mount_rcu(struct nameidata *nd, struct path *path,
!(path->dentry->d_flags & DCACHE_NEED_AUTOMOUNT); !(path->dentry->d_flags & DCACHE_NEED_AUTOMOUNT);
} }
static inline int handle_mounts(struct path *path, struct nameidata *nd, static inline int handle_mounts(struct nameidata *nd, struct dentry *dentry,
struct inode **inode, unsigned int *seqp) struct path *path, struct inode **inode,
unsigned int *seqp)
{ {
int ret = follow_managed(path, nd); int ret;
path->mnt = nd->path.mnt;
path->dentry = dentry;
ret = follow_managed(path, nd);
if (likely(ret >= 0)) { if (likely(ret >= 0)) {
*inode = d_backing_inode(path->dentry); *inode = d_backing_inode(path->dentry);
*seqp = 0; /* out of RCU mode, so the value doesn't matter */ *seqp = 0; /* out of RCU mode, so the value doesn't matter */
@ -1685,10 +1689,7 @@ static int lookup_fast(struct nameidata *nd,
dput(dentry); dput(dentry);
return status; return status;
} }
return handle_mounts(nd, dentry, path, inode, seqp);
path->mnt = mnt;
path->dentry = dentry;
return handle_mounts(path, nd, inode, seqp);
} }
/* Fast lookup failed, do it the slow way */ /* Fast lookup failed, do it the slow way */
@ -1859,6 +1860,7 @@ static inline int step_into(struct nameidata *nd, struct path *path,
static int walk_component(struct nameidata *nd, int flags) static int walk_component(struct nameidata *nd, int flags)
{ {
struct path path; struct path path;
struct dentry *dentry;
struct inode *inode; struct inode *inode;
unsigned seq; unsigned seq;
int err; int err;
@ -1877,13 +1879,11 @@ static int walk_component(struct nameidata *nd, int flags)
if (unlikely(err <= 0)) { if (unlikely(err <= 0)) {
if (err < 0) if (err < 0)
return err; return err;
path.dentry = lookup_slow(&nd->last, nd->path.dentry, dentry = lookup_slow(&nd->last, nd->path.dentry, nd->flags);
nd->flags); if (IS_ERR(dentry))
if (IS_ERR(path.dentry)) return PTR_ERR(dentry);
return PTR_ERR(path.dentry);
path.mnt = nd->path.mnt; err = handle_mounts(nd, dentry, &path, &inode, &seq);
err = handle_mounts(&path, nd, &inode, &seq);
if (unlikely(err < 0)) if (unlikely(err < 0))
return err; return err;
} }
@ -2355,7 +2355,7 @@ static inline int lookup_last(struct nameidata *nd)
static int handle_lookup_down(struct nameidata *nd) static int handle_lookup_down(struct nameidata *nd)
{ {
struct path path = nd->path; struct path path;
struct inode *inode = nd->inode; struct inode *inode = nd->inode;
unsigned seq = nd->seq; unsigned seq = nd->seq;
int err; int err;
@ -2366,11 +2366,12 @@ static int handle_lookup_down(struct nameidata *nd)
* at the very beginning of walk, so we lose nothing * at the very beginning of walk, so we lose nothing
* if we simply redo everything in non-RCU mode * if we simply redo everything in non-RCU mode
*/ */
path = nd->path;
if (unlikely(!__follow_mount_rcu(nd, &path, &inode, &seq))) if (unlikely(!__follow_mount_rcu(nd, &path, &inode, &seq)))
return -ECHILD; return -ECHILD;
} else { } else {
dget(path.dentry); dget(nd->path.dentry);
err = handle_mounts(&path, nd, &inode, &seq); err = handle_mounts(nd, nd->path.dentry, &path, &inode, &seq);
if (unlikely(err < 0)) if (unlikely(err < 0))
return err; return err;
} }
@ -3393,9 +3394,7 @@ static int do_last(struct nameidata *nd,
got_write = false; got_write = false;
} }
path.mnt = nd->path.mnt; error = handle_mounts(nd, dentry, &path, &inode, &seq);
path.dentry = dentry;
error = handle_mounts(&path, nd, &inode, &seq);
if (unlikely(error < 0)) if (unlikely(error < 0))
return error; return error;
finish_lookup: finish_lookup: