SUNRPC: Replace connect dprintk call sites with a tracepoint
authorChuck Lever <chuck.lever@oracle.com>
Wed, 8 Jul 2020 20:09:47 +0000 (16:09 -0400)
committerAnna Schumaker <Anna.Schumaker@Netapp.com>
Mon, 21 Sep 2020 14:21:09 +0000 (10:21 -0400)
This trace event can be used to audit transport connections from the
client.

Signed-off-by: Chuck Lever <chuck.lever@oracle.com>
Signed-off-by: Anna Schumaker <Anna.Schumaker@Netapp.com>
include/trace/events/sunrpc.h
net/sunrpc/clnt.c
net/sunrpc/xprt.c

index 90d7b99..0aa15cc 100644 (file)
@@ -983,6 +983,7 @@ DECLARE_EVENT_CLASS(rpc_xprt_lifetime_class,
                        TP_ARGS(xprt))
 
 DEFINE_RPC_XPRT_LIFETIME_EVENT(create);
+DEFINE_RPC_XPRT_LIFETIME_EVENT(connect);
 DEFINE_RPC_XPRT_LIFETIME_EVENT(disconnect_auto);
 DEFINE_RPC_XPRT_LIFETIME_EVENT(disconnect_done);
 DEFINE_RPC_XPRT_LIFETIME_EVENT(disconnect_force);
index 7d97d18..48d8681 100644 (file)
@@ -2055,10 +2055,6 @@ call_connect(struct rpc_task *task)
                return;
        }
 
-       dprintk("RPC: %5u call_connect xprt %p %s connected\n",
-                       task->tk_pid, xprt,
-                       (xprt_connected(xprt) ? "is" : "is not"));
-
        task->tk_action = call_connect_status;
        if (task->tk_status < 0)
                return;
index 9da7c6e..ceb190d 100644 (file)
@@ -834,8 +834,7 @@ void xprt_connect(struct rpc_task *task)
 {
        struct rpc_xprt *xprt = task->tk_rqstp->rq_xprt;
 
-       dprintk("RPC: %5u xprt_connect xprt %p %s connected\n", task->tk_pid,
-                       xprt, (xprt_connected(xprt) ? "is" : "is not"));
+       trace_xprt_connect(xprt);
 
        if (!xprt_bound(xprt)) {
                task->tk_status = -EAGAIN;