Skip to content

Commit 40b9ad1

Browse files
amotintonyhutter
authored andcommitted
ZTS: Remove TXG_TIMEOUT from dedup_quota test (#17150)
It seems `fio` in `ddt_dedup_vdev_limit` overwhelms the system with the amount of dirty data caused by DDT updates within one TXG due to tiny 1KB records used, while I see no reason for this test to extend the TXGs beyond default. Signed-off-by: Alexander Motin <[email protected]> Sponsored by: iXsystems, Inc. Reviewed-by: Allan Jude <[email protected]> Reviewed-by: @ImAwsumm Reviewed-by: Tony Hutter <[email protected]> (cherry picked from commit 21850f5)
1 parent 602fecc commit 40b9ad1

File tree

1 file changed

+0
-4
lines changed

1 file changed

+0
-4
lines changed

tests/zfs-tests/tests/functional/dedup/dedup_quota.ksh

Lines changed: 0 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -50,8 +50,6 @@ VDEV_GENERAL="$TEST_BASE_DIR/vdevfile.general.$$"
5050
VDEV_DEDUP="$TEST_BASE_DIR/vdevfile.dedup.$$"
5151
POOL="dedup_pool"
5252

53-
save_tunable TXG_TIMEOUT
54-
5553
# we set the dedup log txg interval to 1, to get a log flush every txg,
5654
# effectively disabling the log. without this it's hard to predict when and
5755
# where things appear on-disk
@@ -64,7 +62,6 @@ function cleanup
6462
destroy_pool $POOL
6563
fi
6664
log_must rm -fd $VDEV_GENERAL $VDEV_DEDUP $MOUNTDIR
67-
log_must restore_tunable TXG_TIMEOUT
6865
log_must restore_tunable DEDUP_LOG_TXG_MAX
6966
}
7067

@@ -81,7 +78,6 @@ function do_setup
8178
# Use 'xattr=sa' to prevent selinux xattrs influencing our accounting
8279
log_must zpool create -o ashift=12 -f -O xattr=sa -m $MOUNTDIR $POOL $VDEV_GENERAL
8380
log_must zfs set compression=off dedup=on $POOL
84-
log_must set_tunable32 TXG_TIMEOUT 600
8581
}
8682

8783
function dedup_table_size

0 commit comments

Comments
 (0)