nfsd4: remove some dprintk's
authorJ. Bruce Fields <bfields@redhat.com>
Thu, 14 Mar 2013 22:12:03 +0000 (18:12 -0400)
committerJ. Bruce Fields <bfields@redhat.com>
Wed, 3 Apr 2013 15:48:36 +0000 (11:48 -0400)
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>
fs/nfsd/nfs4state.c

index a0ab6ad7239d885d69675d551d98cc390d0a1196..84dfbdfd2d2c14526c90f04f4e87df8af407e6bd 100644 (file)
@@ -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)
@@ -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;
 }
 
@@ -2116,7 +2113,6 @@ out:
        }
        kfree(conn);
        spin_unlock(&nn->client_lock);
-       dprintk("%s: return %d\n", __func__, ntohl(status));
        return status;
 }
 
@@ -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;
 }
 
@@ -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;