]> granicus.if.org Git - spl/commit
Debug cv_destroy() with mutex held
authorBrian Behlendorf <behlendorf1@llnl.gov>
Fri, 7 Sep 2012 18:05:46 +0000 (11:05 -0700)
committerBrian Behlendorf <behlendorf1@llnl.gov>
Mon, 10 Sep 2012 17:23:26 +0000 (10:23 -0700)
commit3c60f5054cf567ddc4a87e72d16d0a425721e422
tree80da7afe2ad1acacdfd78becd114e701bf25cdcd
parent95331f4437d57f0d0a719d38355159b90a52f40d
Debug cv_destroy() with mutex held

There still appears to be a race in the condition variables where
->cv_mutex is set after we are woken from the cv_destroy wait queue.
This might be possible when cv_destroy() is called immediately after
cv_broadcast().  We had some troubles with this previously but
there may still be a small race, see commit d599e4f.

The following patch closes one small race and improves the ASSERTs
such that they log the offending value.

Signed-off-by: Brian Behlendorf <behlendorf1@llnl.gov>
zfsonlinux/zfs#943
module/spl/spl-condvar.c