svcrdma: Clean up read chunk counting

The byte_count argument is not used, and the function is called
only from one place.

Signed-off-by: Chuck Lever <chuck.lever@oracle.com>
Reviewed-by: Steve Wise <swise@opengridcomputing.com>
Signed-off-by: J. Bruce Fields <bfields@redhat.com>

2 files changed, 12 insertions, 19 deletions

diff --git a/net/sunrpc/xprtrdma/svc_rdma_marshal.c b/net/sunrpc/xprtrdma/svc_rdma_marshal.c
index 65b146297f5a..b681855cf970 100644
--- a/net/sunrpc/xprtrdma/svc_rdma_marshal.c
+++ b/net/sunrpc/xprtrdma/svc_rdma_marshal.c
@@ -71,22 +71,6 @@ static u32 *decode_read_list(u32 *va, u32 *vaend)
 }
 
 /*
- * Determine number of chunks and total bytes in chunk list. The chunk
- * list has already been verified to fit within the RPCRDMA header.
- */
-void svc_rdma_rcl_chunk_counts(struct rpcrdma_read_chunk *ch,
-                               int *ch_count, int *byte_count)
-{
-        /* compute the number of bytes represented by read chunks */
-        *byte_count = 0;
-        *ch_count = 0;
-        for (; ch->rc_discrim != 0; ch++) {
-                *byte_count = *byte_count + ntohl(ch->rc_target.rs_length);
-                *ch_count = *ch_count + 1;
-        }
-}
-
-/*
  * Decodes a write chunk list. The expected format is as follows:
  *    descrim  : xdr_one
  *    nchunks  : <count>
diff --git a/net/sunrpc/xprtrdma/svc_rdma_recvfrom.c b/net/sunrpc/xprtrdma/svc_rdma_recvfrom.c
index 2c67de032009..b3b7bb85844d 100644
--- a/net/sunrpc/xprtrdma/svc_rdma_recvfrom.c
+++ b/net/sunrpc/xprtrdma/svc_rdma_recvfrom.c
@@ -365,12 +365,22 @@ static int rdma_read_chunk_frmr(struct svcxprt_rdma *xprt,
         return ret;
 }
 
+static unsigned int
+rdma_rcl_chunk_count(struct rpcrdma_read_chunk *ch)
+{
+        unsigned int count;
+
+        for (count = 0; ch->rc_discrim != xdr_zero; ch++)
+                count++;
+        return count;
+}
+
 static int rdma_read_chunks(struct svcxprt_rdma *xprt,
                             struct rpcrdma_msg *rmsgp,
                             struct svc_rqst *rqstp,
                             struct svc_rdma_op_ctxt *head)
 {
-        int page_no, ch_count, ret;
+        int page_no, ret;
         struct rpcrdma_read_chunk *ch;
         u32 page_offset, byte_count;
         u64 rs_offset;
@@ -381,8 +391,7 @@ static int rdma_read_chunks(struct svcxprt_rdma *xprt,
         if (!ch)
                 return 0;
 
-        svc_rdma_rcl_chunk_counts(ch, &ch_count, &byte_count);
-        if (ch_count > RPCSVC_MAXPAGES)
+        if (rdma_rcl_chunk_count(ch) > RPCSVC_MAXPAGES)
                 return -EINVAL;
 
         /* The request is completed when the RDMA_READs complete. The