From: "Darrick J. Wong" <djwong@xxxxxxxxxx> Fix a few bugs in the single block atomic writes test, such as not requiring directio, using the page size for the ext4 max bsize, and making sure we check the max atomic write size. Cc: ritesh.list@xxxxxxxxx Signed-off-by: Catherine Hoang <catherine.hoang@xxxxxxxxxx> Reviewed-by: "Darrick J. Wong" <djwong@xxxxxxxxxx> Signed-off-by: "Darrick J. Wong" <djwong@xxxxxxxxxx> --- common/rc | 2 +- tests/generic/765 | 4 ++-- 2 files changed, 3 insertions(+), 3 deletions(-) diff --git a/common/rc b/common/rc index 657772e7..0ac90d3e 100644 --- a/common/rc +++ b/common/rc @@ -2989,7 +2989,7 @@ _require_xfs_io_command() fi if [ "$param" == "-A" ]; then opts+=" -d" - pwrite_opts+="-D -V 1 -b 4k" + pwrite_opts+="-V 1 -b 4k" fi testio=`$XFS_IO_PROG -f $opts -c \ "pwrite $pwrite_opts $param 0 4k" $testfile 2>&1` diff --git a/tests/generic/765 b/tests/generic/765 index 9bab3b8a..8695a306 100755 --- a/tests/generic/765 +++ b/tests/generic/765 @@ -28,7 +28,7 @@ get_supported_bsize() ;; "ext4") min_bsize=1024 - max_bsize=4096 + max_bsize=$(_get_page_size) ;; *) _notrun "$FSTYP does not support atomic writes" @@ -73,7 +73,7 @@ test_atomic_writes() # Check that atomic min/max = FS block size test $file_min_write -eq $bsize || \ echo "atomic write min $file_min_write, should be fs block size $bsize" - test $file_min_write -eq $bsize || \ + test $file_max_write -eq $bsize || \ echo "atomic write max $file_max_write, should be fs block size $bsize" test $file_max_segments -eq 1 || \ echo "atomic write max segments $file_max_segments, should be 1" -- 2.34.1