Skip to content

Commit

Permalink
nfsd4: remove some dprintk's
Browse files Browse the repository at this point in the history
E.g. printk's that just report the return value from an op are
uninteresting as we already do that in the main proc_compound loop.

Signed-off-by: J. Bruce Fields <bfields@redhat.com>
  • Loading branch information
J. Bruce Fields committed Apr 3, 2013
1 parent 0eb6f20 commit 1ca5079
Showing 1 changed file with 1 addition and 8 deletions.
9 changes: 1 addition & 8 deletions fs/nfsd/nfs4state.c
Original file line number Diff line number Diff line change
Expand Up @@ -1843,15 +1843,13 @@ nfsd4_create_session(struct svc_rqst *rqstp,
/* cache solo and embedded create sessions under the state lock */
nfsd4_cache_create_session(cr_ses, cs_slot, status);
nfs4_unlock_state();
out:
dprintk("%s returns %d\n", __func__, ntohl(status));
return status;
out_free_conn:
nfs4_unlock_state();
free_conn(conn);
out_free_session:
__free_session(new);
goto out;
return status;
}

static __be32 nfsd4_map_bcts_dir(u32 *dir)
Expand Down Expand Up @@ -1963,7 +1961,6 @@ nfsd4_destroy_session(struct svc_rqst *r,
spin_unlock(&nn->client_lock);
status = nfs_ok;
out:
dprintk("%s returns %d\n", __func__, ntohl(status));
return status;
}

Expand Down Expand Up @@ -2116,7 +2113,6 @@ nfsd4_sequence(struct svc_rqst *rqstp,
}
kfree(conn);
spin_unlock(&nn->client_lock);
dprintk("%s: return %d\n", __func__, ntohl(status));
return status;
}

Expand Down Expand Up @@ -2155,7 +2151,6 @@ nfsd4_destroy_clientid(struct svc_rqst *rqstp, struct nfsd4_compound_state *csta
expire_client(clp);
out:
nfs4_unlock_state();
dprintk("%s return %d\n", __func__, ntohl(status));
return status;
}

Expand Down Expand Up @@ -2532,8 +2527,6 @@ nfs4_share_conflict(struct svc_fh *current_fh, unsigned int deny_type)
struct nfs4_ol_stateid *stp;
__be32 ret;

dprintk("NFSD: nfs4_share_conflict\n");

fp = find_file(ino);
if (!fp)
return nfs_ok;
Expand Down

0 comments on commit 1ca5079

Please sign in to comment.