xprtrdma: Remove rpcrdma_buffer_get_req_locked()

Clean up. There is only one call-site for this helper, and it can be
simplified by using list_first_entry_or_null().

Signed-off-by: Chuck Lever <chuck.lever@oracle.com>
Signed-off-by: Anna Schumaker <Anna.Schumaker@Netapp.com>

diff --git a/net/sunrpc/xprtrdma/verbs.c b/net/sunrpc/xprtrdma/verbs.c
index 601b78b6448323c28aa5fdfc3e6fad85303fa89f..b9a6e5d8d8c7f82bf616101b2c5510f6a8bb043f 100644 (file)
--- a/net/sunrpc/xprtrdma/verbs.c
+++ b/net/sunrpc/xprtrdma/verbs.c
@@ -1175,17 +1175,6 @@ out:
        return rc;
 }
 
-static struct rpcrdma_req *
-rpcrdma_buffer_get_req_locked(struct rpcrdma_buffer *buf)
-{
-       struct rpcrdma_req *req;
-
-       req = list_first_entry(&buf->rb_send_bufs,
-                              struct rpcrdma_req, rl_list);
-       list_del_init(&req->rl_list);
-       return req;
-}
-
 static struct rpcrdma_rep *
 rpcrdma_buffer_get_rep_locked(struct rpcrdma_buffer *buf)
 {
@@ -1351,15 +1340,12 @@ rpcrdma_buffer_get(struct rpcrdma_buffer *buffers)
        struct rpcrdma_req *req;
 
        spin_lock(&buffers->rb_lock);
-       if (unlikely(list_empty(&buffers->rb_send_bufs)))
-               goto out_noreqs;
-       req = rpcrdma_buffer_get_req_locked(buffers);
+       req = list_first_entry_or_null(&buffers->rb_send_bufs,
+                                      struct rpcrdma_req, rl_list);
+       if (req)
+               list_del_init(&req->rl_list);
        spin_unlock(&buffers->rb_lock);
        return req;
-
-out_noreqs:
-       spin_unlock(&buffers->rb_lock);
-       return NULL;
 }
 
 /**