diff options
| author | Raghavendra Bhat <raghavendra@redhat.com> | 2013-01-24 16:05:36 +0530 |
|---|---|---|
| committer | Anand Avati <avati@redhat.com> | 2013-02-03 14:09:49 -0800 |
| commit | 4840090c8bf9f08d6da8a4101e2095c6295cf039 (patch) | |
| tree | a29a8be9af9adbe9720b6d4c5d56b5b44917a1d5 | |
| parent | 9708bd1f922ca38973cba65b4890c67c2edc6e2b (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>
| -rw-r--r-- | xlators/performance/write-behind/src/write-behind.c | 10 |
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 0663dbc7d..b94b18a4a 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); |