aboutsummaryrefslogtreecommitdiffstats
path: root/lib
diff options
context:
space:
mode:
authorBrian Behlendorf <[email protected]>2014-10-30 11:11:00 -0700
committerBrian Behlendorf <[email protected]>2015-02-11 13:58:40 -0800
commit340dfbe193193be6a3d301d8111de232cd537ddd (patch)
treee337be14caa39304b2afa7cceea1454e7d503814 /lib
parent5f15fa22167ff665d0db0159551eb90759683984 (diff)
Change VERIFY to ASSERT in mutex_destroy()
There have been multiple reports of 'zdb' tripping the VERIFY in mutex_destroy() because pthread_mutex_destroy() returns EBUSY. Exactly how this can happen still needs to be explained, but this doesn't strictly need to be fatal for non-debug builds. Therefore, this patch converts the VERIFY to an ASSERT until the root cause is determined and resolved. Signed-off-by: Brian Behlendorf <[email protected]> Issue #2027
Diffstat (limited to 'lib')
-rw-r--r--lib/libzpool/kernel.c2
1 files changed, 1 insertions, 1 deletions
diff --git a/lib/libzpool/kernel.c b/lib/libzpool/kernel.c
index 325cb3cea..9136560df 100644
--- a/lib/libzpool/kernel.c
+++ b/lib/libzpool/kernel.c
@@ -289,7 +289,7 @@ mutex_destroy(kmutex_t *mp)
{
ASSERT3U(mp->m_magic, ==, MTX_MAGIC);
ASSERT3P(mp->m_owner, ==, MTX_INIT);
- VERIFY3S(pthread_mutex_destroy(&(mp)->m_lock), ==, 0);
+ ASSERT0(pthread_mutex_destroy(&(mp)->m_lock));
mp->m_owner = MTX_DEST;
mp->m_magic = 0;
}