diff options
author | Raghavendra G <raghavendra@gluster.com> | 2011-08-05 14:51:27 +0530 |
---|---|---|
committer | Anand Avati <avati@gluster.com> | 2011-08-05 02:40:10 -0700 |
commit | dfc317a77faa57f1bb6933b35c362035f467bf94 (patch) | |
tree | f8d649fccc01135c7b6a3672ae2c6ea63446c6fe | |
parent | 3e53823bd3e23502aa716a76debec093bd5dc7d3 (diff) |
performance/io-cache: fix deadlock in ioc_fault_cbkv3.1.6qa4
Change-Id: I899abecd17025c299819b36c2d39adc52adb1bd6
BUG: 3340
Reviewed-on: http://review.gluster.com/177
Tested-by: Gluster Build System <jenkins@build.gluster.com>
Reviewed-by: Anand Avati <avati@gluster.com>
-rw-r--r-- | xlators/performance/io-cache/src/page.c | 23 |
1 files changed, 2 insertions, 21 deletions
diff --git a/xlators/performance/io-cache/src/page.c b/xlators/performance/io-cache/src/page.c index 728f0373635..a8d98a4d8de 100644 --- a/xlators/performance/io-cache/src/page.c +++ b/xlators/performance/io-cache/src/page.c @@ -78,7 +78,7 @@ out: } /* - * ioc_page_destroy - + * __ioc_page_destroy - * * @page: * @@ -121,25 +121,6 @@ __ioc_page_destroy (ioc_page_t *page) } -int64_t -ioc_page_destroy (ioc_page_t *page) -{ - int64_t ret = 0; - - if (page == NULL) { - goto out; - } - - ioc_inode_lock (page->inode); - { - ret = __ioc_page_destroy (page); - } - ioc_inode_unlock (page->inode); - -out: - return ret; -} - int32_t __ioc_inode_prune (ioc_inode_t *curr, uint64_t *size_pruned, uint64_t size_to_prune, uint32_t index) @@ -920,7 +901,7 @@ __ioc_page_error (ioc_page_t *page, int32_t op_ret, int32_t op_errno) } table = page->inode->table; - ret = ioc_page_destroy (page); + ret = __ioc_page_destroy (page); if (ret != -1) { table->cache_used -= ret; |