nfsd: eliminate __nfs4_get_fd
authorJeff Layton <jlayton@kernel.org>
Wed, 18 Jan 2023 17:31:39 +0000 (12:31 -0500)
committerChuck Lever <chuck.lever@oracle.com>
Mon, 20 Feb 2023 14:20:52 +0000 (09:20 -0500)
This is wrapper is pointless, and just obscures what's going on.

Signed-off-by: Jeff Layton <jlayton@kernel.org>
Signed-off-by: Chuck Lever <chuck.lever@oracle.com>
fs/nfsd/nfs4state.c

index 7d8bd0bb0a0e13bd1a623d8511bdc48abb9425a8..2143d0b50d63a00d8403028c477f8a3fea086ee3 100644 (file)
@@ -599,12 +599,6 @@ put_nfs4_file(struct nfs4_file *fi)
        }
 }
 
-static struct nfsd_file *
-__nfs4_get_fd(struct nfs4_file *f, int oflag)
-{
-       return nfsd_file_get(f->fi_fds[oflag]);
-}
-
 static struct nfsd_file *
 find_writeable_file_locked(struct nfs4_file *f)
 {
@@ -612,9 +606,9 @@ find_writeable_file_locked(struct nfs4_file *f)
 
        lockdep_assert_held(&f->fi_lock);
 
-       ret = __nfs4_get_fd(f, O_WRONLY);
+       ret = nfsd_file_get(f->fi_fds[O_WRONLY]);
        if (!ret)
-               ret = __nfs4_get_fd(f, O_RDWR);
+               ret = nfsd_file_get(f->fi_fds[O_RDWR]);
        return ret;
 }
 
@@ -637,9 +631,9 @@ find_readable_file_locked(struct nfs4_file *f)
 
        lockdep_assert_held(&f->fi_lock);
 
-       ret = __nfs4_get_fd(f, O_RDONLY);
+       ret = nfsd_file_get(f->fi_fds[O_RDONLY]);
        if (!ret)
-               ret = __nfs4_get_fd(f, O_RDWR);
+               ret = nfsd_file_get(f->fi_fds[O_RDWR]);
        return ret;
 }
 
@@ -663,11 +657,11 @@ find_any_file(struct nfs4_file *f)
        if (!f)
                return NULL;
        spin_lock(&f->fi_lock);
-       ret = __nfs4_get_fd(f, O_RDWR);
+       ret = nfsd_file_get(f->fi_fds[O_RDWR]);
        if (!ret) {
-               ret = __nfs4_get_fd(f, O_WRONLY);
+               ret = nfsd_file_get(f->fi_fds[O_WRONLY]);
                if (!ret)
-                       ret = __nfs4_get_fd(f, O_RDONLY);
+                       ret = nfsd_file_get(f->fi_fds[O_RDONLY]);
        }
        spin_unlock(&f->fi_lock);
        return ret;