From 4840090c8bf9f08d6da8a4101e2095c6295cf039 Mon Sep 17 00:00:00 2001 From: Raghavendra Bhat Date: Thu, 24 Jan 2013 16:05:36 +0530 Subject: 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 Reviewed-on: http://review.gluster.org/4423 Reviewed-by: Anand Avati Tested-by: Gluster Build System --- xlators/performance/write-behind/src/write-behind.c | 10 +++------- 1 file changed, 3 insertions(+), 7 deletions(-) (limited to 'xlators/performance') 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); -- cgit