OSDN Git Service

sunrpc: Change rpc_print_iostats to rpc_clnt_show_stats and handle rpc_clnt clones
authorDave Wysochanski <dwysocha@redhat.com>
Tue, 31 Jul 2018 14:10:51 +0000 (10:10 -0400)
committerAnna Schumaker <Anna.Schumaker@Netapp.com>
Tue, 31 Jul 2018 16:53:35 +0000 (12:53 -0400)
commit016583d7030cec9b69e0d55269a5967f4ee871d2
treef85c91d451a92ff7c5fcc9de2888a0a1c39bf871
parent189e1955677d6a704fed3dfbc33893836a48a700
sunrpc: Change rpc_print_iostats to rpc_clnt_show_stats and handle rpc_clnt clones

The existing rpc_print_iostats has a few shortcomings.  First, the naming
is not consistent with other functions in the kernel that display stats.
Second, it is really displaying stats for an rpc_clnt structure as it
displays both xprt stats and per-op stats.  Third, it does not handle
rpc_clnt clones, which is important for the one in-kernel tree caller
of this function, the NFS client's nfs_show_stats function.

Fix all of the above by renaming the rpc_print_iostats to
rpc_clnt_show_stats and looping through any rpc_clnt clones via
cl_parent.

Once this interface is fixed, this addresses a problem with NFSv4.
Before this patch, the /proc/self/mountstats always showed incorrect
counts for NFSv4 lease and session related opcodes such as SEQUENCE,
RENEW, SETCLIENTID, CREATE_SESSION, etc.  These counts were always 0
even though many ops would go over the wire.  The reason for this is
there are multiple rpc_clnt structures allocated for any given NFSv4
mount, and inside nfs_show_stats() we callled into rpc_print_iostats()
which only handled one of them, nfs_server->client.  Fix these counts
by calling sunrpc's new rpc_clnt_show_stats() function, which handles
cloned rpc_clnt structs and prints the stats together.

Note that one side-effect of the above is that multiple mounts from
the same NFS server will show identical counts in the above ops due
to the fact the one rpc_clnt (representing the NFSv4 client state)
is shared across mounts.

Signed-off-by: Dave Wysochanski <dwysocha@redhat.com>
Signed-off-by: Anna Schumaker <Anna.Schumaker@Netapp.com>
fs/nfs/super.c
include/linux/sunrpc/metrics.h
net/sunrpc/stats.c