NFS: Remove extra dprintk()s from nfs4client.c

Signed-off-by: Anna Schumaker <Anna.Schumaker@Netapp.com>
Signed-off-by: Trond Myklebust <trond.myklebust@primarydata.com>
This commit is contained in:
Anna Schumaker 2017-04-07 14:15:18 -04:00 committed by Trond Myklebust
parent 1073d9b49a
commit 4fe6b366d9

View File

@ -359,11 +359,9 @@ struct nfs_client *nfs4_init_client(struct nfs_client *clp,
struct nfs_client *old; struct nfs_client *old;
int error; int error;
if (clp->cl_cons_state == NFS_CS_READY) { if (clp->cl_cons_state == NFS_CS_READY)
/* the client is initialised already */ /* the client is initialised already */
dprintk("<-- nfs4_init_client() = 0 [already %p]\n", clp);
return clp; return clp;
}
/* Check NFS protocol revision and initialize RPC op vector */ /* Check NFS protocol revision and initialize RPC op vector */
clp->rpc_ops = &nfs_v4_clientops; clp->rpc_ops = &nfs_v4_clientops;
@ -421,7 +419,6 @@ struct nfs_client *nfs4_init_client(struct nfs_client *clp,
error: error:
nfs_mark_client_ready(clp, error); nfs_mark_client_ready(clp, error);
nfs_put_client(clp); nfs_put_client(clp);
dprintk("<-- nfs4_init_client() = xerror %d\n", error);
return ERR_PTR(error); return ERR_PTR(error);
} }
@ -577,8 +574,6 @@ int nfs40_walk_client_list(struct nfs_client *new,
prev = NULL; prev = NULL;
*result = pos; *result = pos;
dprintk("NFS: <-- %s using nfs_client = %p ({%d})\n",
__func__, pos, atomic_read(&pos->cl_count));
goto out; goto out;
case -ERESTARTSYS: case -ERESTARTSYS:
case -ETIMEDOUT: case -ETIMEDOUT:
@ -599,7 +594,6 @@ out_unlock:
/* No match found. The server lost our clientid */ /* No match found. The server lost our clientid */
out: out:
nfs_put_client(prev); nfs_put_client(prev);
dprintk("NFS: <-- %s status = %d\n", __func__, status);
return status; return status;
} }
@ -909,7 +903,6 @@ struct nfs_client *nfs4_set_ds_client(struct nfs_server *mds_srv,
.net = mds_clp->cl_net, .net = mds_clp->cl_net,
.timeparms = &ds_timeout, .timeparms = &ds_timeout,
}; };
struct nfs_client *clp;
char buf[INET6_ADDRSTRLEN + 1]; char buf[INET6_ADDRSTRLEN + 1];
if (rpc_ntop(ds_addr, buf, sizeof(buf)) <= 0) if (rpc_ntop(ds_addr, buf, sizeof(buf)) <= 0)
@ -925,10 +918,7 @@ struct nfs_client *nfs4_set_ds_client(struct nfs_server *mds_srv,
* (section 13.1 RFC 5661). * (section 13.1 RFC 5661).
*/ */
nfs_init_timeout_values(&ds_timeout, ds_proto, ds_timeo, ds_retrans); nfs_init_timeout_values(&ds_timeout, ds_proto, ds_timeo, ds_retrans);
clp = nfs_get_client(&cl_init); return nfs_get_client(&cl_init);
dprintk("<-- %s %p\n", __func__, clp);
return clp;
} }
EXPORT_SYMBOL_GPL(nfs4_set_ds_client); EXPORT_SYMBOL_GPL(nfs4_set_ds_client);
@ -1082,8 +1072,6 @@ struct nfs_server *nfs4_create_server(struct nfs_mount_info *mount_info,
bool auth_probe; bool auth_probe;
int error; int error;
dprintk("--> nfs4_create_server()\n");
server = nfs_alloc_server(); server = nfs_alloc_server();
if (!server) if (!server)
return ERR_PTR(-ENOMEM); return ERR_PTR(-ENOMEM);
@ -1099,12 +1087,10 @@ struct nfs_server *nfs4_create_server(struct nfs_mount_info *mount_info,
if (error < 0) if (error < 0)
goto error; goto error;
dprintk("<-- nfs4_create_server() = %p\n", server);
return server; return server;
error: error:
nfs_free_server(server); nfs_free_server(server);
dprintk("<-- nfs4_create_server() = error %d\n", error);
return ERR_PTR(error); return ERR_PTR(error);
} }
@ -1119,8 +1105,6 @@ struct nfs_server *nfs4_create_referral_server(struct nfs_clone_mount *data,
bool auth_probe; bool auth_probe;
int error; int error;
dprintk("--> nfs4_create_referral_server()\n");
server = nfs_alloc_server(); server = nfs_alloc_server();
if (!server) if (!server)
return ERR_PTR(-ENOMEM); return ERR_PTR(-ENOMEM);
@ -1154,12 +1138,10 @@ struct nfs_server *nfs4_create_referral_server(struct nfs_clone_mount *data,
if (error < 0) if (error < 0)
goto error; goto error;
dprintk("<-- nfs_create_referral_server() = %p\n", server);
return server; return server;
error: error:
nfs_free_server(server); nfs_free_server(server);
dprintk("<-- nfs4_create_referral_server() = error %d\n", error);
return ERR_PTR(error); return ERR_PTR(error);
} }
@ -1219,31 +1201,16 @@ int nfs4_update_server(struct nfs_server *server, const char *hostname,
struct sockaddr *localaddr = (struct sockaddr *)&address; struct sockaddr *localaddr = (struct sockaddr *)&address;
int error; int error;
dprintk("--> %s: move FSID %llx:%llx to \"%s\")\n", __func__,
(unsigned long long)server->fsid.major,
(unsigned long long)server->fsid.minor,
hostname);
error = rpc_switch_client_transport(clnt, &xargs, clnt->cl_timeout); error = rpc_switch_client_transport(clnt, &xargs, clnt->cl_timeout);
if (error != 0) { if (error != 0)
dprintk("<-- %s(): rpc_switch_client_transport returned %d\n", return error;
__func__, error);
goto out;
}
error = rpc_localaddr(clnt, localaddr, sizeof(address)); error = rpc_localaddr(clnt, localaddr, sizeof(address));
if (error != 0) { if (error != 0)
dprintk("<-- %s(): rpc_localaddr returned %d\n", return error;
__func__, error);
goto out;
}
error = -EAFNOSUPPORT; if (rpc_ntop(localaddr, buf, sizeof(buf)) == 0)
if (rpc_ntop(localaddr, buf, sizeof(buf)) == 0) { return -EAFNOSUPPORT;
dprintk("<-- %s(): rpc_ntop returned %d\n",
__func__, error);
goto out;
}
nfs_server_remove_lists(server); nfs_server_remove_lists(server);
error = nfs4_set_client(server, hostname, sap, salen, buf, error = nfs4_set_client(server, hostname, sap, salen, buf,
@ -1252,21 +1219,12 @@ int nfs4_update_server(struct nfs_server *server, const char *hostname,
nfs_put_client(clp); nfs_put_client(clp);
if (error != 0) { if (error != 0) {
nfs_server_insert_lists(server); nfs_server_insert_lists(server);
dprintk("<-- %s(): nfs4_set_client returned %d\n", return error;
__func__, error);
goto out;
} }
if (server->nfs_client->cl_hostname == NULL) if (server->nfs_client->cl_hostname == NULL)
server->nfs_client->cl_hostname = kstrdup(hostname, GFP_KERNEL); server->nfs_client->cl_hostname = kstrdup(hostname, GFP_KERNEL);
nfs_server_insert_lists(server); nfs_server_insert_lists(server);
error = nfs_probe_destination(server); return nfs_probe_destination(server);
if (error < 0)
goto out;
dprintk("<-- %s() succeeded\n", __func__);
out:
return error;
} }