svcrdma: Split the svcrdma_wc_send() tracepoint
authorChuck Lever <chuck.lever@oracle.com>
Mon, 4 Oct 2021 14:16:14 +0000 (10:16 -0400)
committerJ. Bruce Fields <bfields@redhat.com>
Mon, 4 Oct 2021 19:40:15 +0000 (15:40 -0400)
commiteef2d8d47c33aba5f430fc5f91a17e360f99a591
treee3ec009e525fa33e9f057b21ab891ad4fdac6e20
parent8dcc5721da7888685dde82be765018c8a379298c
svcrdma: Split the svcrdma_wc_send() tracepoint

There are currently three separate purposes being served by a single
tracepoint here. They need to be split up.

svcrdma_wc_send:
 - status is always zero, so there's no value in recording it.
 - vendor_err is meaningless unless status is not zero, so
   there's no value in recording it.
 - This tracepoint is needed only when developing modifications,
   so it should be left disabled most of the time.

svcrdma_wc_send_flush:
 - As above, needed only rarely, and not an error.

svcrdma_wc_send_err:
 - This tracepoint can be left persistently enabled because
   completion errors are run-time problems (except for FLUSHED_ERR).
 - Tracepoint name now ends in _err to reflect its purpose.

Signed-off-by: Chuck Lever <chuck.lever@oracle.com>
Signed-off-by: J. Bruce Fields <bfields@redhat.com>
include/trace/events/rpcrdma.h
net/sunrpc/xprtrdma/svc_rdma_sendto.c