diff options
author | Jeff Layton <jlayton@redhat.com> | 2013-01-28 14:41:12 -0500 |
---|---|---|
committer | J. Bruce Fields <bfields@redhat.com> | 2013-02-04 09:16:22 -0500 |
commit | 25e6b8b0e1a247747db5275b1b6b362f5acf2245 (patch) | |
tree | 89f9f8b7553581978555a81bce873cab807d3f01 /fs/nfsd | |
parent | f09841fdfad8dd95593725dfa70388f55b170947 (diff) |
nfsd: remove redundant test from nfsd_reply_cache_free
Entries can only get a c_type of RC_REPLBUFF iff they are
RC_DONE. Therefore the test for RC_DONE isn't necessary here.
Signed-off-by: Jeff Layton <jlayton@redhat.com>
Signed-off-by: J. Bruce Fields <bfields@redhat.com>
Diffstat (limited to 'fs/nfsd')
-rw-r--r-- | fs/nfsd/nfscache.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/fs/nfsd/nfscache.c b/fs/nfsd/nfscache.c index 363bc6188155..2cdc4be84553 100644 --- a/fs/nfsd/nfscache.c +++ b/fs/nfsd/nfscache.c | |||
@@ -65,7 +65,7 @@ nfsd_reply_cache_alloc(void) | |||
65 | static void | 65 | static void |
66 | nfsd_reply_cache_free_locked(struct svc_cacherep *rp) | 66 | nfsd_reply_cache_free_locked(struct svc_cacherep *rp) |
67 | { | 67 | { |
68 | if (rp->c_state == RC_DONE && rp->c_type == RC_REPLBUFF) | 68 | if (rp->c_type == RC_REPLBUFF) |
69 | kfree(rp->c_replvec.iov_base); | 69 | kfree(rp->c_replvec.iov_base); |
70 | list_del(&rp->c_lru); | 70 | list_del(&rp->c_lru); |
71 | kmem_cache_free(drc_slab, rp); | 71 | kmem_cache_free(drc_slab, rp); |