From 4a7eb69a5ab0027a0f347a0cdd37b0275fcfecad Mon Sep 17 00:00:00 2001 From: Tom Caputi Date: Thu, 18 Oct 2018 16:53:27 -0400 Subject: Fix ztest deadman panic with indirect vdev damage This patch fixes an issue where ztest's deadman thread would trigger a panic because reconstructing artifically damaged blocks would take too long to reconstruct. This patch simply limits how often ztest inflicts split-block damage and how many segments it can damage when it does. Reviewed-by: Brian Behlendorf Reviewed-by: Serapheim Dimitropoulos Reviewed-by: Matthew Ahrens Signed-off-by: Tom Caputi Closes #8010 --- module/zfs/vdev_indirect.c | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) (limited to 'module/zfs/vdev_indirect.c') diff --git a/module/zfs/vdev_indirect.c b/module/zfs/vdev_indirect.c index 097be6f0e..7863fe9d0 100644 --- a/module/zfs/vdev_indirect.c +++ b/module/zfs/vdev_indirect.c @@ -1614,7 +1614,7 @@ vdev_indirect_splits_damage(indirect_vsd_t *iv, zio_t *zio) * result in two or less unique copies per indirect_child_t. * Both may need to be checked in order to reconstruct the block. * Set iv->iv_attempts_max such that all unique combinations will - * enumerated, but limit the damage to at most 16 indirect splits. + * enumerated, but limit the damage to at most 12 indirect splits. */ iv->iv_attempts_max = 1; @@ -1632,7 +1632,7 @@ vdev_indirect_splits_damage(indirect_vsd_t *iv, zio_t *zio) } iv->iv_attempts_max *= 2; - if (iv->iv_attempts_max > (1ULL << 16)) { + if (iv->iv_attempts_max >= (1ULL << 12)) { iv->iv_attempts_max = UINT64_MAX; break; } @@ -1718,7 +1718,7 @@ vdev_indirect_reconstruct_io_done(zio_t *zio) /* * If nonzero, every 1/x blocks will be damaged, in order to validate * reconstruction when there are split segments with damaged copies. - * Known_good will TRUE when reconstruction is known to be possible. + * Known_good will be TRUE when reconstruction is known to be possible. */ if (zfs_reconstruct_indirect_damage_fraction != 0 && spa_get_random(zfs_reconstruct_indirect_damage_fraction) == 0) -- cgit v1.2.3