diff options
author | Jeff Layton <jlayton@redhat.com> | 2013-01-28 14:41:08 -0500 |
---|---|---|
committer | J. Bruce Fields <bfields@redhat.com> | 2013-02-04 09:16:19 -0500 |
commit | 6dc8889589678c228a675b9f01721318a31a8f5b (patch) | |
tree | 3cbfbef12dec62f99826c7f32a84fb6a86b8f4cf /fs/nfsd/nfscache.c | |
parent | 7b9e8522a65886d8ae168547a67c3617b6ba83f1 (diff) |
nfsd: remove unneeded spinlock in nfsd_cache_update
The locking rules for cache entries say that locking the cache_lock
isn't needed if you're just touching the current entry. Earlier
in this function we set rp->c_state to RC_UNUSED without any locking,
so I believe it's ok to do the same here.
Signed-off-by: Jeff Layton <jlayton@redhat.com>
Signed-off-by: J. Bruce Fields <bfields@redhat.com>
Diffstat (limited to 'fs/nfsd/nfscache.c')
-rw-r--r-- | fs/nfsd/nfscache.c | 2 |
1 files changed, 0 insertions, 2 deletions
diff --git a/fs/nfsd/nfscache.c b/fs/nfsd/nfscache.c index 5dd9ec2a177f..972c14a033ee 100644 --- a/fs/nfsd/nfscache.c +++ b/fs/nfsd/nfscache.c | |||
@@ -286,9 +286,7 @@ nfsd_cache_update(struct svc_rqst *rqstp, int cachetype, __be32 *statp) | |||
286 | cachv = &rp->c_replvec; | 286 | cachv = &rp->c_replvec; |
287 | cachv->iov_base = kmalloc(len << 2, GFP_KERNEL); | 287 | cachv->iov_base = kmalloc(len << 2, GFP_KERNEL); |
288 | if (!cachv->iov_base) { | 288 | if (!cachv->iov_base) { |
289 | spin_lock(&cache_lock); | ||
290 | rp->c_state = RC_UNUSED; | 289 | rp->c_state = RC_UNUSED; |
291 | spin_unlock(&cache_lock); | ||
292 | return; | 290 | return; |
293 | } | 291 | } |
294 | cachv->iov_len = len << 2; | 292 | cachv->iov_len = len << 2; |