lookup_dcache(): lift d_alloc() into callers
authorAl Viro <viro@zeniv.linux.org.uk>
Sun, 6 Mar 2016 01:09:32 +0000 (20:09 -0500)
committerAl Viro <viro@zeniv.linux.org.uk>
Sun, 6 Mar 2016 01:09:32 +0000 (20:09 -0500)
... and kill need_lookup thing

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

index 5bd9e078685596fa92bf707be80cb86a1105472a..50020b168f151fa43e59c0684565e9430f006f4a 100644 (file)
@@ -1448,12 +1448,11 @@ static int follow_dotdot(struct nameidata *nd)
  * dir->d_inode->i_mutex must be held
  */
 static struct dentry *lookup_dcache(struct qstr *name, struct dentry *dir,
-                                   unsigned int flags, bool *need_lookup)
+                                   unsigned int flags)
 {
        struct dentry *dentry;
        int error;
 
-       *need_lookup = false;
        dentry = d_lookup(dir, name);
        if (dentry) {
                if (dentry->d_flags & DCACHE_OP_REVALIDATE) {
@@ -1470,14 +1469,6 @@ static struct dentry *lookup_dcache(struct qstr *name, struct dentry *dir,
                        }
                }
        }
-
-       if (!dentry) {
-               dentry = d_alloc(dir, name);
-               if (unlikely(!dentry))
-                       return ERR_PTR(-ENOMEM);
-
-               *need_lookup = true;
-       }
        return dentry;
 }
 
@@ -1509,13 +1500,15 @@ static struct dentry *lookup_real(struct inode *dir, struct dentry *dentry,
 static struct dentry *__lookup_hash(struct qstr *name,
                struct dentry *base, unsigned int flags)
 {
-       bool need_lookup;
-       struct dentry *dentry;
+       struct dentry *dentry = lookup_dcache(name, base, flags);
 
-       dentry = lookup_dcache(name, base, flags, &need_lookup);
-       if (!need_lookup)
+       if (dentry)
                return dentry;
 
+       dentry = d_alloc(base, name);
+       if (unlikely(!dentry))
+               return ERR_PTR(-ENOMEM);
+
        return lookup_real(base->d_inode, dentry, flags);
 }
 
@@ -3018,16 +3011,22 @@ static int lookup_open(struct nameidata *nd, struct path *path,
        struct inode *dir_inode = dir->d_inode;
        struct dentry *dentry;
        int error;
-       bool need_lookup;
+       bool need_lookup = false;
 
        *opened &= ~FILE_CREATED;
-       dentry = lookup_dcache(&nd->last, dir, nd->flags, &need_lookup);
+       dentry = lookup_dcache(&nd->last, dir, nd->flags);
        if (IS_ERR(dentry))
                return PTR_ERR(dentry);
 
-       /* Cached positive dentry: will open in f_op->open */
-       if (!need_lookup && dentry->d_inode)
+       if (!dentry) {
+               dentry = d_alloc(dir, &nd->last);
+               if (unlikely(!dentry))
+                       return -ENOMEM;
+               need_lookup = true;
+       } else if (dentry->d_inode) {
+               /* Cached positive dentry: will open in f_op->open */
                goto out_no_open;
+       }
 
        if ((nd->flags & LOOKUP_OPEN) && dir_inode->i_op->atomic_open) {
                return atomic_open(nd, dentry, path, file, op, got_write,