SUNRPC: Fix possible autodisconnect during connect due to old last_used
authorDave Wysochanski <dwysocha@redhat.com>
Wed, 26 Jun 2019 20:30:24 +0000 (16:30 -0400)
committerTrond Myklebust <trond.myklebust@hammerspace.com>
Sat, 6 Jul 2019 18:54:53 +0000 (14:54 -0400)
Ensure last_used is updated before calling mod_timer inside
xprt_schedule_autodisconnect.  This avoids a possible xprt_autoclose
firing immediately after a successful connect when xprt_unlock_connect
calls xprt_schedule_autodisconnect with an old value of last_used.

Signed-off-by: Dave Wysochanski <dwysocha@redhat.com>
Signed-off-by: Trond Myklebust <trond.myklebust@hammerspace.com>
net/sunrpc/xprt.c

index c4d138202abb672a02855784a2abceaef2ee030d..70d6a1f10db9f0caf0f56d2a3f02d59a7a042ab0 100644 (file)
@@ -750,6 +750,7 @@ static void
 xprt_schedule_autodisconnect(struct rpc_xprt *xprt)
        __must_hold(&xprt->transport_lock)
 {
+       xprt->last_used = jiffies;
        if (RB_EMPTY_ROOT(&xprt->recv_queue) && xprt_has_timer(xprt))
                mod_timer(&xprt->timer, xprt->last_used + xprt->idle_timeout);
 }
@@ -1765,7 +1766,6 @@ void xprt_release(struct rpc_task *task)
        xprt->ops->release_xprt(xprt, task);
        if (xprt->ops->release_request)
                xprt->ops->release_request(task);
-       xprt->last_used = jiffies;
        xprt_schedule_autodisconnect(xprt);
        spin_unlock(&xprt->transport_lock);
        if (req->rq_buffer)