SUNRPC: Remove more dprintks in rpcb_clnt.c
authorChuck Lever <chuck.lever@oracle.com>
Wed, 8 Jul 2020 20:10:35 +0000 (16:10 -0400)
committerAnna Schumaker <Anna.Schumaker@Netapp.com>
Mon, 21 Sep 2020 14:21:09 +0000 (10:21 -0400)
Clean up: These are superfluous now that rpc_create() and friends
have tracepoints to report errors.

Signed-off-by: Chuck Lever <chuck.lever@oracle.com>
Signed-off-by: Anna Schumaker <Anna.Schumaker@Netapp.com>
net/sunrpc/rpcb_clnt.c

index 6df12a13edc63370f204b46f9566bc6a3b7f8b74..af2882c62a3b156d8b0ae947a67da62c7d5f9a39 100644 (file)
@@ -218,10 +218,6 @@ static void rpcb_set_local(struct net *net, struct rpc_clnt *clnt,
        sn->rpcb_is_af_local = is_af_local ? 1 : 0;
        smp_wmb();
        sn->rpcb_users = 1;
-       dprintk("RPC:       created new rpcb local clients (rpcb_local_clnt: "
-               "%p, rpcb_local_clnt4: %p) for net %x%s\n",
-               sn->rpcb_local_clnt, sn->rpcb_local_clnt4,
-               net->ns.inum, (net == &init_net) ? " (init_net)" : "");
 }
 
 /*
@@ -263,19 +259,13 @@ static int rpcb_create_local_unix(struct net *net)
         */
        clnt = rpc_create(&args);
        if (IS_ERR(clnt)) {
-               dprintk("RPC:       failed to create AF_LOCAL rpcbind "
-                               "client (errno %ld).\n", PTR_ERR(clnt));
                result = PTR_ERR(clnt);
                goto out;
        }
 
        clnt4 = rpc_bind_new_program(clnt, &rpcb_program, RPCBVERS_4);
-       if (IS_ERR(clnt4)) {
-               dprintk("RPC:       failed to bind second program to "
-                               "rpcbind v4 client (errno %ld).\n",
-                               PTR_ERR(clnt4));
+       if (IS_ERR(clnt4))
                clnt4 = NULL;
-       }
 
        rpcb_set_local(net, clnt, clnt4, true);
 
@@ -311,8 +301,6 @@ static int rpcb_create_local_net(struct net *net)
 
        clnt = rpc_create(&args);
        if (IS_ERR(clnt)) {
-               dprintk("RPC:       failed to create local rpcbind "
-                               "client (errno %ld).\n", PTR_ERR(clnt));
                result = PTR_ERR(clnt);
                goto out;
        }
@@ -323,12 +311,8 @@ static int rpcb_create_local_net(struct net *net)
         * v4 upcalls.
         */
        clnt4 = rpc_bind_new_program(clnt, &rpcb_program, RPCBVERS_4);
-       if (IS_ERR(clnt4)) {
-               dprintk("RPC:       failed to bind second program to "
-                               "rpcbind v4 client (errno %ld).\n",
-                               PTR_ERR(clnt4));
+       if (IS_ERR(clnt4))
                clnt4 = NULL;
-       }
 
        rpcb_set_local(net, clnt, clnt4, false);
 
@@ -405,11 +389,8 @@ static int rpcb_register_call(struct sunrpc_net *sn, struct rpc_clnt *clnt, stru
        msg->rpc_resp = &result;
 
        error = rpc_call_sync(clnt, msg, flags);
-       if (error < 0) {
-               dprintk("RPC:       failed to contact local rpcbind "
-                               "server (errno %d).\n", -error);
+       if (error < 0)
                return error;
-       }
 
        if (!result)
                return -EACCES;