lockd: Add helper for *_RES callbacks

Signed-off-by: Trond Myklebust <Trond.Myklebust@netapp.com>

1 files changed, 55 insertions, 95 deletions

diff --git a/fs/lockd/svc4proc.c b/fs/lockd/svc4proc.c
index cb51c7025825..a2dd9ccb9b32 100644
--- a/fs/lockd/svc4proc.c
+++ b/fs/lockd/svc4proc.c
@@ -21,10 +21,6 @@
 
 #define NLMDBG_FACILITY         NLMDBG_CLIENT
 
-static u32      nlm4svc_callback(struct svc_rqst *, u32, struct nlm_res *);
-
-static const struct rpc_call_ops nlm4svc_callback_ops;
-
 /*
  * Obtain client and file from arguments
  */
@@ -234,83 +230,89 @@ nlm4svc_proc_granted(struct svc_rqst *rqstp, struct nlm_args *argp,
 }
 
 /*
+ * This is the generic lockd callback for async RPC calls
+ */
+static void nlm4svc_callback_exit(struct rpc_task *task, void *data)
+{
+        dprintk("lockd: %4d callback returned %d\n", task->tk_pid,
+                        -task->tk_status);
+}
+
+static void nlm4svc_callback_release(void *data)
+{
+        nlm_release_call(data);
+}
+
+static const struct rpc_call_ops nlm4svc_callback_ops = {
+        .rpc_call_done = nlm4svc_callback_exit,
+        .rpc_release = nlm4svc_callback_release,
+};
+
+/*
  * `Async' versions of the above service routines. They aren't really,
  * because we send the callback before the reply proper. I hope this
  * doesn't break any clients.
  */
-static int
-nlm4svc_proc_test_msg(struct svc_rqst *rqstp, struct nlm_args *argp,
-                                             void            *resp)
+static int nlm4svc_callback(struct svc_rqst *rqstp, u32 proc, struct nlm_args *argp,
+                int (*func)(struct svc_rqst *, struct nlm_args *, struct nlm_res  *))
 {
-        struct nlm_res  res;
-        u32             stat;
+        struct nlm_host *host;
+        struct nlm_rqst *call;
+        int stat;
 
-        dprintk("lockd: TEST_MSG      called\n");
-        memset(&res, 0, sizeof(res));
+        host = nlmsvc_lookup_host(rqstp);
+        if (host == NULL)
+                return rpc_system_err;
+
+        call = nlm_alloc_call(host);
+        if (call == NULL)
+                return rpc_system_err;
+
+        stat = func(rqstp, argp, &call->a_res);
+        if (stat != 0) {
+                nlm_release_call(call);
+                return stat;
+        }
 
-        if ((stat = nlm4svc_proc_test(rqstp, argp, &res)) == 0)
-                stat = nlm4svc_callback(rqstp, NLMPROC_TEST_RES, &res);
-        return stat;
+        call->a_flags = RPC_TASK_ASYNC;
+        if (nlm_async_reply(call, proc, &nlm4svc_callback_ops) < 0)
+                return rpc_system_err;
+        return rpc_success;
 }
 
-static int
-nlm4svc_proc_lock_msg(struct svc_rqst *rqstp, struct nlm_args *argp,
+static int nlm4svc_proc_test_msg(struct svc_rqst *rqstp, struct nlm_args *argp,
                                              void            *resp)
 {
-        struct nlm_res  res;
-        u32             stat;
+        dprintk("lockd: TEST_MSG      called\n");
+        return nlm4svc_callback(rqstp, NLMPROC_TEST_RES, argp, nlm4svc_proc_test);
+}
 
+static int nlm4svc_proc_lock_msg(struct svc_rqst *rqstp, struct nlm_args *argp,
+                                             void            *resp)
+{
         dprintk("lockd: LOCK_MSG      called\n");
-        memset(&res, 0, sizeof(res));
-
-        if ((stat = nlm4svc_proc_lock(rqstp, argp, &res)) == 0)
-                stat = nlm4svc_callback(rqstp, NLMPROC_LOCK_RES, &res);
-        return stat;
+        return nlm4svc_callback(rqstp, NLMPROC_LOCK_RES, argp, nlm4svc_proc_lock);
 }
 
-static int
-nlm4svc_proc_cancel_msg(struct svc_rqst *rqstp, struct nlm_args *argp,
+static int nlm4svc_proc_cancel_msg(struct svc_rqst *rqstp, struct nlm_args *argp,
                                                void            *resp)
 {
-        struct nlm_res  res;
-        u32             stat;
-
         dprintk("lockd: CANCEL_MSG    called\n");
-        memset(&res, 0, sizeof(res));
-
-        if ((stat = nlm4svc_proc_cancel(rqstp, argp, &res)) == 0)
-                stat = nlm4svc_callback(rqstp, NLMPROC_CANCEL_RES, &res);
-        return stat;
+        return nlm4svc_callback(rqstp, NLMPROC_CANCEL_RES, argp, nlm4svc_proc_cancel);
 }
 
-static int
-nlm4svc_proc_unlock_msg(struct svc_rqst *rqstp, struct nlm_args *argp,
+static int nlm4svc_proc_unlock_msg(struct svc_rqst *rqstp, struct nlm_args *argp,
                                                void            *resp)
 {
-        struct nlm_res  res;
-        u32             stat;
-
         dprintk("lockd: UNLOCK_MSG    called\n");
-        memset(&res, 0, sizeof(res));
-
-        if ((stat = nlm4svc_proc_unlock(rqstp, argp, &res)) == 0)
-                stat = nlm4svc_callback(rqstp, NLMPROC_UNLOCK_RES, &res);
-        return stat;
+        return nlm4svc_callback(rqstp, NLMPROC_UNLOCK_RES, argp, nlm4svc_proc_unlock);
 }
 
-static int
-nlm4svc_proc_granted_msg(struct svc_rqst *rqstp, struct nlm_args *argp,
+static int nlm4svc_proc_granted_msg(struct svc_rqst *rqstp, struct nlm_args *argp,
                                                 void            *resp)
 {
-        struct nlm_res  res;
-        u32             stat;
-
         dprintk("lockd: GRANTED_MSG   called\n");
-        memset(&res, 0, sizeof(res));
-
-        if ((stat = nlm4svc_proc_granted(rqstp, argp, &res)) == 0)
-                stat = nlm4svc_callback(rqstp, NLMPROC_GRANTED_RES, &res);
-        return stat;
+        return nlm4svc_callback(rqstp, NLMPROC_GRANTED_RES, argp, nlm4svc_proc_granted);
 }
 
 /*
@@ -472,48 +474,6 @@ nlm4svc_proc_granted_res(struct svc_rqst *rqstp, struct nlm_res  *argp,
 
 
 /*
- * This is the generic lockd callback for async RPC calls
- */
-static u32
-nlm4svc_callback(struct svc_rqst *rqstp, u32 proc, struct nlm_res *resp)
-{
-        struct nlm_host *host;
-        struct nlm_rqst *call;
-
-        host = nlmsvc_lookup_host(rqstp);
-        if (host == NULL)
-                return rpc_system_err;
-
-        call = nlm_alloc_call(host);
-        if (call == NULL)
-                return rpc_system_err;
-
-
-        call->a_flags = RPC_TASK_ASYNC;
-        memcpy(&call->a_args, resp, sizeof(*resp));
-
-        if (nlm_async_call(call, proc, &nlm4svc_callback_ops) < 0)
-                return rpc_system_err;
-        return rpc_success;
-}
-
-static void nlm4svc_callback_exit(struct rpc_task *task, void *data)
-{
-        dprintk("lockd: %4d callback returned %d\n", task->tk_pid,
-                        -task->tk_status);
-}
-
-static void nlm4svc_callback_release(void *data)
-{
-        nlm_release_call(data);
-}
-
-static const struct rpc_call_ops nlm4svc_callback_ops = {
-        .rpc_call_done = nlm4svc_callback_exit,
-        .rpc_release = nlm4svc_callback_release,
-};
-
-/*
  * NLM Server procedures.
  */