SUNRPC: Clean up scheduling of autoclose

commit e26d9972720e2484f44cdd94ca4e31cc372ed2ed upstream.

Consolidate duplicated code in xprt_force_disconnect() and
xprt_conditional_disconnect().

Signed-off-by: Trond Myklebust <trond.myklebust@hammerspace.com>
Signed-off-by: Anna Schumaker <Anna.Schumaker@Netapp.com>
Signed-off-by: Meena Shanmugam <meenashanmugam@google.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>

diff --git a/net/sunrpc/xprt.c b/net/sunrpc/xprt.c
index 6bc225d64d23fe74db610bfd92e638d42d4b0d06..a6bb957084f79679305327201b5b6c8aa922f4c2 100644 (file)
--- a/net/sunrpc/xprt.c
+++ b/net/sunrpc/xprt.c
@@ -731,6 +731,20 @@ void xprt_disconnect_done(struct rpc_xprt *xprt)
 }
 EXPORT_SYMBOL_GPL(xprt_disconnect_done);
 
+/**
+ * xprt_schedule_autoclose_locked - Try to schedule an autoclose RPC call
+ * @xprt: transport to disconnect
+ */
+static void xprt_schedule_autoclose_locked(struct rpc_xprt *xprt)
+{
+       set_bit(XPRT_CLOSE_WAIT, &xprt->state);
+       if (test_and_set_bit(XPRT_LOCKED, &xprt->state) == 0)
+               queue_work(xprtiod_workqueue, &xprt->task_cleanup);
+       else if (xprt->snd_task && !test_bit(XPRT_SND_IS_COOKIE, &xprt->state))
+               rpc_wake_up_queued_task_set_status(&xprt->pending,
+                                                  xprt->snd_task, -ENOTCONN);
+}
+
 /**
  * xprt_force_disconnect - force a transport to disconnect
  * @xprt: transport to disconnect
@@ -742,13 +756,7 @@ void xprt_force_disconnect(struct rpc_xprt *xprt)
 
        /* Don't race with the test_bit() in xprt_clear_locked() */
        spin_lock(&xprt->transport_lock);
-       set_bit(XPRT_CLOSE_WAIT, &xprt->state);
-       /* Try to schedule an autoclose RPC call */
-       if (test_and_set_bit(XPRT_LOCKED, &xprt->state) == 0)
-               queue_work(xprtiod_workqueue, &xprt->task_cleanup);
-       else if (xprt->snd_task && !test_bit(XPRT_SND_IS_COOKIE, &xprt->state))
-               rpc_wake_up_queued_task_set_status(&xprt->pending,
-                                                  xprt->snd_task, -ENOTCONN);
+       xprt_schedule_autoclose_locked(xprt);
        spin_unlock(&xprt->transport_lock);
 }
 EXPORT_SYMBOL_GPL(xprt_force_disconnect);
@@ -788,11 +796,7 @@ void xprt_conditional_disconnect(struct rpc_xprt *xprt, unsigned int cookie)
                goto out;
        if (test_bit(XPRT_CLOSING, &xprt->state))
                goto out;
-       set_bit(XPRT_CLOSE_WAIT, &xprt->state);
-       /* Try to schedule an autoclose RPC call */
-       if (test_and_set_bit(XPRT_LOCKED, &xprt->state) == 0)
-               queue_work(xprtiod_workqueue, &xprt->task_cleanup);
-       xprt_wake_pending_tasks(xprt, -EAGAIN);
+       xprt_schedule_autoclose_locked(xprt);
 out:
        spin_unlock(&xprt->transport_lock);
 }