summaryrefslogtreecommitdiffstats
path: root/xlators/performance
diff options
context:
space:
mode:
authorRaghavendra Bhat <raghavendra@redhat.com>2013-01-24 16:05:36 +0530
committerAnand Avati <avati@redhat.com>2013-02-03 14:09:49 -0800
commit4840090c8bf9f08d6da8a4101e2095c6295cf039 (patch)
treea29a8be9af9adbe9720b6d4c5d56b5b44917a1d5 /xlators/performance
parent9708bd1f922ca38973cba65b4890c67c2edc6e2b (diff)
performance/write-behind: do not try to take LOCK in forget
LOCK attempt in wb_forget is unnecessary Change-Id: Ibdedc23d0c829c34aedd6fc5bc0e0a584b832514 BUG: 903566 Signed-off-by: Raghavendra Bhat <raghavendra@redhat.com> Reviewed-on: http://review.gluster.org/4423 Reviewed-by: Anand Avati <avati@redhat.com> Tested-by: Gluster Build System <jenkins@build.gluster.com>
Diffstat (limited to 'xlators/performance')
-rw-r--r--xlators/performance/write-behind/src/write-behind.c10
1 files changed, 3 insertions, 7 deletions
diff --git a/xlators/performance/write-behind/src/write-behind.c b/xlators/performance/write-behind/src/write-behind.c
index 0663dbc7..b94b18a4 100644
--- a/xlators/performance/write-behind/src/write-behind.c
+++ b/xlators/performance/write-behind/src/write-behind.c
@@ -1652,13 +1652,9 @@ wb_forget (xlator_t *this, inode_t *inode)
if (!wb_inode)
return 0;
- LOCK (&wb_inode->lock);
- {
- GF_ASSERT (list_empty (&wb_inode->todo));
- GF_ASSERT (list_empty (&wb_inode->liability));
- GF_ASSERT (list_empty (&wb_inode->temptation));
- }
- UNLOCK (&wb_inode->lock);
+ GF_ASSERT (list_empty (&wb_inode->todo));
+ GF_ASSERT (list_empty (&wb_inode->liability));
+ GF_ASSERT (list_empty (&wb_inode->temptation));
GF_FREE (wb_inode);