Commit 860477d1 authored by Chuck Lever's avatar Chuck Lever Committed by Anna Schumaker

xprtrdma: Count RDMA_NOMSG type calls

RDMA_NOMSG type calls are less efficient than RDMA_MSG. Count NOMSG
calls so administrators can tell if they happen to be used more than
expected.
Signed-off-by: default avatarChuck Lever <chuck.lever@oracle.com>
Tested-by: default avatarDevesh Sharma <devesh.sharma@avagotech.com>
Signed-off-by: default avatarAnna Schumaker <Anna.Schumaker@Netapp.com>
parent 763f7e4e
...@@ -489,6 +489,7 @@ rpcrdma_marshal_req(struct rpc_rqst *rqst) ...@@ -489,6 +489,7 @@ rpcrdma_marshal_req(struct rpc_rqst *rqst)
} else if (rqst->rq_snd_buf.flags & XDRBUF_WRITE) { } else if (rqst->rq_snd_buf.flags & XDRBUF_WRITE) {
rtype = rpcrdma_readch; rtype = rpcrdma_readch;
} else { } else {
r_xprt->rx_stats.nomsg_call_count++;
headerp->rm_type = htonl(RDMA_NOMSG); headerp->rm_type = htonl(RDMA_NOMSG);
rtype = rpcrdma_areadch; rtype = rpcrdma_areadch;
rpclen = 0; rpclen = 0;
......
...@@ -659,7 +659,7 @@ static void xprt_rdma_print_stats(struct rpc_xprt *xprt, struct seq_file *seq) ...@@ -659,7 +659,7 @@ static void xprt_rdma_print_stats(struct rpc_xprt *xprt, struct seq_file *seq)
xprt->stat.bad_xids, xprt->stat.bad_xids,
xprt->stat.req_u, xprt->stat.req_u,
xprt->stat.bklog_u); xprt->stat.bklog_u);
seq_printf(seq, "%lu %lu %lu %llu %llu %llu %llu %lu %lu %lu\n", seq_printf(seq, "%lu %lu %lu %llu %llu %llu %llu %lu %lu %lu %lu\n",
r_xprt->rx_stats.read_chunk_count, r_xprt->rx_stats.read_chunk_count,
r_xprt->rx_stats.write_chunk_count, r_xprt->rx_stats.write_chunk_count,
r_xprt->rx_stats.reply_chunk_count, r_xprt->rx_stats.reply_chunk_count,
...@@ -669,7 +669,8 @@ static void xprt_rdma_print_stats(struct rpc_xprt *xprt, struct seq_file *seq) ...@@ -669,7 +669,8 @@ static void xprt_rdma_print_stats(struct rpc_xprt *xprt, struct seq_file *seq)
r_xprt->rx_stats.fixup_copy_count, r_xprt->rx_stats.fixup_copy_count,
r_xprt->rx_stats.hardway_register_count, r_xprt->rx_stats.hardway_register_count,
r_xprt->rx_stats.failed_marshal_count, r_xprt->rx_stats.failed_marshal_count,
r_xprt->rx_stats.bad_reply_count); r_xprt->rx_stats.bad_reply_count,
r_xprt->rx_stats.nomsg_call_count);
} }
static int static int
......
...@@ -340,6 +340,7 @@ struct rpcrdma_stats { ...@@ -340,6 +340,7 @@ struct rpcrdma_stats {
unsigned long hardway_register_count; unsigned long hardway_register_count;
unsigned long failed_marshal_count; unsigned long failed_marshal_count;
unsigned long bad_reply_count; unsigned long bad_reply_count;
unsigned long nomsg_call_count;
}; };
/* /*
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment