getting rid of 'opened' argument of ->atomic_open() - part 1
authorAl Viro <viro@zeniv.linux.org.uk>
Fri, 8 Jun 2018 15:44:56 +0000 (11:44 -0400)
committerAl Viro <viro@zeniv.linux.org.uk>
Thu, 12 Jul 2018 14:04:19 +0000 (10:04 -0400)
'opened' argument of finish_open() is unused.  Kill it.

Signed-off-by Al Viro <viro@zeniv.linux.org.uk>

fs/9p/vfs_inode.c
fs/9p/vfs_inode_dotl.c
fs/ceph/file.c
fs/cifs/dir.c
fs/fuse/dir.c
fs/gfs2/inode.c
fs/namei.c
fs/nfs/dir.c
fs/open.c
include/linux/fs.h

index 5669297924804e2d1e84b5aa63026a1b62825aaf..7b6ff3275d9c23d65d8b0314aad14f017d57086b 100644 (file)
@@ -917,7 +917,7 @@ v9fs_vfs_atomic_open(struct inode *dir, struct dentry *dentry,
                v9inode->writeback_fid = (void *) inode_fid;
        }
        mutex_unlock(&v9inode->v_mutex);
-       err = finish_open(file, dentry, generic_file_open, opened);
+       err = finish_open(file, dentry, generic_file_open);
        if (err)
                goto error;
 
index ee65db5c7eb051516b0d9d70d159cd44e0e0a66a..c6939b7cb18c842c5f22f6321989df3aab96bce2 100644 (file)
@@ -352,7 +352,7 @@ v9fs_vfs_atomic_open_dotl(struct inode *dir, struct dentry *dentry,
        }
        mutex_unlock(&v9inode->v_mutex);
        /* Since we are opening a file, assign the open fid to the file */
-       err = finish_open(file, dentry, generic_file_open, opened);
+       err = finish_open(file, dentry, generic_file_open);
        if (err)
                goto err_clunk_old_fid;
        file->private_data = ofid;
index 38a63fff7903eac7bfd8421385c603770b8d6ad8..38b28cb2fac171fffa2123dce60b8d1b540b1240 100644 (file)
@@ -509,7 +509,7 @@ int ceph_atomic_open(struct inode *dir, struct dentry *dentry,
                        ceph_init_inode_acls(d_inode(dentry), &acls);
                        file->f_mode |= FMODE_CREATED;
                }
-               err = finish_open(file, dentry, ceph_open, opened);
+               err = finish_open(file, dentry, ceph_open);
        }
 out_req:
        if (!req->r_err && req->r_target_inode)
index 21d7e393900e4a30c5ed837ee3284800695f9acc..891bfd62e67a2e177cbb41bb5cb6859d392b0e7f 100644 (file)
@@ -541,7 +541,7 @@ cifs_atomic_open(struct inode *inode, struct dentry *direntry,
        if ((oflags & (O_CREAT | O_EXCL)) == (O_CREAT | O_EXCL))
                file->f_mode |= FMODE_CREATED;
 
-       rc = finish_open(file, direntry, generic_file_open, opened);
+       rc = finish_open(file, direntry, generic_file_open);
        if (rc) {
                if (server->ops->close)
                        server->ops->close(xid, tcon, &fid);
index d4bdcf51e6cbfcf66d9fd10507de6836170cd3b8..a5b1f5ff8cb761d6ec2e726a7bc7b6f0cd805efb 100644 (file)
@@ -469,7 +469,7 @@ static int fuse_create_open(struct inode *dir, struct dentry *entry,
        d_instantiate(entry, inode);
        fuse_change_entry_timeout(entry, &outentry);
        fuse_invalidate_attr(dir);
-       err = finish_open(file, entry, generic_file_open, opened);
+       err = finish_open(file, entry, generic_file_open);
        if (err) {
                fuse_sync_release(ff, flags);
        } else {
index 4aba00a6004ba5bb24159db7162cb2801b674ce1..59f695e96d6364f70992c34d7734ebe592fbb6cd 100644 (file)
@@ -626,7 +626,7 @@ static int gfs2_create_inode(struct inode *dir, struct dentry *dentry,
                error = 0;
                if (file) {
                        if (S_ISREG(inode->i_mode))
-                               error = finish_open(file, dentry, gfs2_open_common, opened);
+                               error = finish_open(file, dentry, gfs2_open_common);
                        else
                                error = finish_no_open(file, NULL);
                }
@@ -768,7 +768,7 @@ static int gfs2_create_inode(struct inode *dir, struct dentry *dentry,
        d_instantiate(dentry, inode);
        if (file) {
                file->f_mode |= FMODE_CREATED;
-               error = finish_open(file, dentry, gfs2_open_common, opened);
+               error = finish_open(file, dentry, gfs2_open_common);
        }
        gfs2_glock_dq_uninit(ghs);
        gfs2_glock_dq_uninit(ghs + 1);
@@ -866,7 +866,7 @@ static struct dentry *__gfs2_lookup(struct inode *dir, struct dentry *dentry,
                return d;
        }
        if (file && S_ISREG(inode->i_mode))
-               error = finish_open(file, dentry, gfs2_open_common, opened);
+               error = finish_open(file, dentry, gfs2_open_common);
 
        gfs2_glock_dq_uninit(&gh);
        if (error) {
index d2aeb282ed05f855a0108d73d489796e8e30b23c..117b118853f26dc92f3db11305b29aead903e59a 100644 (file)
@@ -3475,7 +3475,7 @@ static int do_tmpfile(struct nameidata *nd, unsigned flags,
        if (error)
                goto out2;
        file->f_path.mnt = path.mnt;
-       error = finish_open(file, child, NULL, opened);
+       error = finish_open(file, child, NULL);
 out2:
        mnt_drop_write(path.mnt);
 out:
index 0ac50983fc4e54d04dcc7c579ad956506dfec520..22176a3818d54ba1353d739f9bb8f8fcb80ffd7c 100644 (file)
@@ -1439,7 +1439,7 @@ static int nfs_finish_open(struct nfs_open_context *ctx,
 {
        int err;
 
-       err = finish_open(file, dentry, do_open, opened);
+       err = finish_open(file, dentry, do_open);
        if (err)
                goto out;
        if (S_ISREG(file->f_path.dentry->d_inode->i_mode))
index d2030a3c5c524c6ab9ab8381bf5947c01b407318..dbaac9efc7fc6b1a2b12274d5f2f0721487c0079 100644 (file)
--- a/fs/open.c
+++ b/fs/open.c
@@ -843,8 +843,7 @@ cleanup_file:
  * Returns zero on success or -errno if the open failed.
  */
 int finish_open(struct file *file, struct dentry *dentry,
-               int (*open)(struct inode *, struct file *),
-               int *opened)
+               int (*open)(struct inode *, struct file *))
 {
        BUG_ON(file->f_mode & FMODE_OPENED); /* once it's opened, it's opened */
 
index ca668c7e48a7cbea5454e478ce869bd96e16ab3b..70be3e4c26ac0ebc9fe68381425e5674a4bd00b3 100644 (file)
@@ -2441,8 +2441,7 @@ enum {
        FILE_OPENED = 2
 };
 extern int finish_open(struct file *file, struct dentry *dentry,
-                       int (*open)(struct inode *, struct file *),
-                       int *opened);
+                       int (*open)(struct inode *, struct file *));
 extern int finish_no_open(struct file *file, struct dentry *dentry);
 
 /* fs/ioctl.c */