On 12/19/2017 11:35 AM, Chuck Lever wrote:
> The kernel is counting RDMA segments, not chunks. Correct the units
> labels displayed by "mountstats -x", and fix up the internal names
> of the counters.
>
> Fixes: 6742ba53ae79 ("mountstats: Add option to pretty-print ...")
> Signed-off-by: Chuck Lever <chuck.lever@xxxxxxxxxx>
Committed...
steved.
> ---
> tools/mountstats/mountstats.py | 12 ++++++------
> 1 file changed, 6 insertions(+), 6 deletions(-)
>
> diff --git a/tools/mountstats/mountstats.py b/tools/mountstats/mountstats.py
> index 440aa72..c7fb8bb 100755
> --- a/tools/mountstats/mountstats.py
> +++ b/tools/mountstats/mountstats.py
> @@ -120,9 +120,9 @@ XprtRdmaCounters = [
> 'badxids',
> 'inflightsends',
> 'backlogutil',
> - 'read_chunks',
> - 'write_chunks',
> - 'reply_chunks',
> + 'read_segments',
> + 'write_segments',
> + 'reply_segments',
> 'total_rdma_req',
> 'total_rdma_rep',
> 'pullup',
> @@ -718,9 +718,9 @@ class DeviceData:
> if self.__rpc_data['rpcsends'] != 0:
> print('\tAvg backlog length: %d' % \
> (float(self.__rpc_data['backlogutil']) / self.__rpc_data['rpcsends']))
> - print('\tRead chunks: %d' % self.__rpc_data['read_chunks'])
> - print('\tWrite chunks: %d' % self.__rpc_data['write_chunks'])
> - print('\tReply chunks: %d' % self.__rpc_data['reply_chunks'])
> + print('\tRead segments: %d' % self.__rpc_data['read_segments'])
> + print('\tWrite segments: %d' % self.__rpc_data['write_segments'])
> + print('\tReply segments: %d' % self.__rpc_data['reply_segments'])
> print('\tRegistered: %d bytes' % self.__rpc_data['total_rdma_req'])
> print('\tRDMA received: %d bytes' % self.__rpc_data['total_rdma_rep'])
> print('\tTotal pull-up: %d bytes' % self.__rpc_data['pullup'])
>
--
To unsubscribe from this list: send the line "unsubscribe linux-nfs" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
