NFS: Merge nfs41_free_stateid() with _nfs41_free_stateid()
authorAnna Schumaker <Anna.Schumaker@Netapp.com>
Tue, 15 May 2018 17:03:39 +0000 (13:03 -0400)
committerTrond Myklebust <trond.myklebust@hammerspace.com>
Thu, 31 May 2018 19:02:16 +0000 (15:02 -0400)
Having these exist as two functions doesn't seem to add anything useful,
and I think merging them together makes this easier to follow.

Signed-off-by: Anna Schumaker <Anna.Schumaker@Netapp.com>
Signed-off-by: Trond Myklebust <trond.myklebust@hammerspace.com>
fs/nfs/nfs4proc.c

index 6c749ce7587eb63fe50c8c70299d87e54b1d87de..553cb8f9819463b4f6593997ddffb0d3c55b2307 100644 (file)
@@ -9335,7 +9335,17 @@ static const struct rpc_call_ops nfs41_free_stateid_ops = {
        .rpc_release = nfs41_free_stateid_release,
 };
 
-static struct rpc_task *_nfs41_free_stateid(struct nfs_server *server,
+/**
+ * nfs41_free_stateid - perform a FREE_STATEID operation
+ *
+ * @server: server / transport on which to perform the operation
+ * @stateid: state ID to release
+ * @cred: credential
+ * @is_recovery: set to true if this call needs to be privileged
+ *
+ * Note: this function is always asynchronous.
+ */
+static int nfs41_free_stateid(struct nfs_server *server,
                const nfs4_stateid *stateid,
                struct rpc_cred *cred,
                bool privileged)
@@ -9351,6 +9361,7 @@ static struct rpc_task *_nfs41_free_stateid(struct nfs_server *server,
                .flags = RPC_TASK_ASYNC,
        };
        struct nfs_free_stateid_data *data;
+       struct rpc_task *task;
 
        nfs4_state_protect(server->nfs_client, NFS_SP4_MACH_CRED_STATEID,
                &task_setup.rpc_client, &msg);
@@ -9358,7 +9369,7 @@ static struct rpc_task *_nfs41_free_stateid(struct nfs_server *server,
        dprintk("NFS call  free_stateid %p\n", stateid);
        data = kmalloc(sizeof(*data), GFP_NOFS);
        if (!data)
-               return ERR_PTR(-ENOMEM);
+               return -ENOMEM;
        data->server = server;
        nfs4_stateid_copy(&data->args.stateid, stateid);
 
@@ -9367,27 +9378,7 @@ static struct rpc_task *_nfs41_free_stateid(struct nfs_server *server,
        msg.rpc_argp = &data->args;
        msg.rpc_resp = &data->res;
        nfs4_init_sequence(&data->args.seq_args, &data->res.seq_res, 1, privileged);
-       return rpc_run_task(&task_setup);
-}
-
-/**
- * nfs41_free_stateid - perform a FREE_STATEID operation
- *
- * @server: server / transport on which to perform the operation
- * @stateid: state ID to release
- * @cred: credential
- * @is_recovery: set to true if this call needs to be privileged
- *
- * Note: this function is always asynchronous.
- */
-static int nfs41_free_stateid(struct nfs_server *server,
-               const nfs4_stateid *stateid,
-               struct rpc_cred *cred,
-               bool is_recovery)
-{
-       struct rpc_task *task;
-
-       task = _nfs41_free_stateid(server, stateid, cred, is_recovery);
+       task = rpc_run_task(&task_setup);
        if (IS_ERR(task))
                return PTR_ERR(task);
        rpc_put_task(task);