Merge branch 'for-linus' of git://git.kernel.org/pub/scm/linux/kernel/git/viro/vfs

Pull  more VFS bits from Al Viro:
 "Unfortunately, it looks like xattr series will have to wait until the
  next cycle ;-/

  This pile contains 9p cleanups and fixes (races in v9fs_fid_add()
  etc), fixup for nommu breakage in shmem.c, several cleanups and a bit
  more file_inode() work"

* 'for-linus' of git://git.kernel.org/pub/scm/linux/kernel/git/viro/vfs:
  constify path_get/path_put and fs_struct.c stuff
  fix nommu breakage in shmem.c
  cache the value of file_inode() in struct file
  9p: if v9fs_fid_lookup() gets to asking server, it'd better have hashed dentry
  9p: make sure ->lookup() adds fid to the right dentry
  9p: untangle ->lookup() a bit
  9p: double iput() in ->lookup() if d_materialise_unique() fails
  9p: v9fs_fid_add() can't fail now
  v9fs: get rid of v9fs_dentry
  9p: turn fid->dlist into hlist
  9p: don't bother with private lock in ->d_fsdata; dentry->d_lock will do just fine
  more file_inode() open-coded instances
  selinux: opened file can't have NULL or negative ->f_path.dentry

(In the meantime, the hlist traversal macros have changed, so this
required a semantic conflict fixup for the newly hlistified fid->dlist)

1 files changed, 7 insertions, 12 deletions

diff --git a/fs/9p/vfs_inode_dotl.c b/fs/9p/vfs_inode_dotl.c
index 61e4fa70a6fa..53687bbf2296 100644
--- a/fs/9p/vfs_inode_dotl.c
+++ b/fs/9p/vfs_inode_dotl.c
@@ -333,9 +333,7 @@ v9fs_vfs_atomic_open_dotl(struct inode *dir, struct dentry *dentry,
         /* Now set the ACL based on the default value */
         v9fs_set_create_acl(inode, fid, dacl, pacl);
 
-        err = v9fs_fid_add(dentry, fid);
-        if (err < 0)
-                goto error;
+        v9fs_fid_add(dentry, fid);
         d_instantiate(dentry, inode);
 
         v9inode = V9FS_I(inode);
@@ -453,12 +451,11 @@ static int v9fs_vfs_mkdir_dotl(struct inode *dir,
                                  err);
                         goto error;
                 }
-                err = v9fs_fid_add(dentry, fid);
-                if (err < 0)
-                        goto error;
+                v9fs_fid_add(dentry, fid);
                 v9fs_set_create_acl(inode, fid, dacl, pacl);
                 d_instantiate(dentry, inode);
                 fid = NULL;
+                err = 0;
         } else {
                 /*
                  * Not in cached mode. No need to populate
@@ -747,11 +744,10 @@ v9fs_vfs_symlink_dotl(struct inode *dir, struct dentry *dentry,
                                  err);
                         goto error;
                 }
-                err = v9fs_fid_add(dentry, fid);
-                if (err < 0)
-                        goto error;
+                v9fs_fid_add(dentry, fid);
                 d_instantiate(dentry, inode);
                 fid = NULL;
+                err = 0;
         } else {
                 /* Not in cached mode. No need to populate inode with stat */
                 inode = v9fs_get_inode(dir->i_sb, S_IFLNK, 0);
@@ -900,11 +896,10 @@ v9fs_vfs_mknod_dotl(struct inode *dir, struct dentry *dentry, umode_t omode,
                         goto error;
                 }
                 v9fs_set_create_acl(inode, fid, dacl, pacl);
-                err = v9fs_fid_add(dentry, fid);
-                if (err < 0)
-                        goto error;
+                v9fs_fid_add(dentry, fid);
                 d_instantiate(dentry, inode);
                 fid = NULL;
+                err = 0;
         } else {
                 /*
                  * Not in cached mode. No need to populate inode with stat.