aboutsummaryrefslogtreecommitdiffstats
path: root/module/zfs/dbuf.c
diff options
context:
space:
mode:
authorIsaac Huang <[email protected]>2015-02-26 22:46:45 -0700
committerBrian Behlendorf <[email protected]>2015-03-02 11:50:06 -0800
commitd14cfd83dae0b1a261667acd416dba17a98d15fa (patch)
treeb5052cb4d9fcd75ef873fbe767222774ae183b43 /module/zfs/dbuf.c
parent87a63dd702370c00322e8d1d84102075c775324b (diff)
Fix deadlock between zpool export and zfs list
Pool reference count is NOT checked in spa_export_common() if the pool has been imported readonly=on, i.e. spa->spa_sync_on is FALSE. Then zpool export and zfs list may deadlock: 1. Pool A is imported readonly. 2. zpool export A and zfs list are run concurrently. 3. zfs command gets reference on the spa, which holds a dbuf on on the MOS meta dnode. 4. zpool command grabs spa_namespace_lock, and tries to evict dbufs of the MOS meta dnode. The dbuf held by zfs command can't be evicted as its reference count is not 0. 5. zpool command blocks in dnode_special_close() waiting for the MOS meta dnode reference count to drop to 0, with spa_namespace_lock held. 6. zfs command tries to get the spa_namespace_lock with a reference on the spa held, which holds a dbuf on the MOS meta dnode. 7. Now zpool command and zfs command deadlock each other. Also any further zfs/zpool command will block on spa_namespace_lock forever. The fix is to always check pool reference count in spa_export_common(), no matter whether the pool was imported readonly or not. Signed-off-by: Isaac Huang <[email protected]> Signed-off-by: Brian Behlendorf <[email protected]> Closes #2034
Diffstat (limited to 'module/zfs/dbuf.c')
0 files changed, 0 insertions, 0 deletions