generic/322: fix bad xfs_io sync_range command

Add the missing range arguments to the sync_range command in this test:
according to Josef Bacik, the sync_range command is required to make the test
reproduce the critical situation reliably.

[Eryu: fix dumping xfs_io output to $seqres.full, don't check
xfs_io's exit status]

Signed-off-by: Andreas Gruenbacher <agruenba@redhat.com>
Suggested-by: Josef Bacik <josef@toxicpanda.com>
Signed-off-by: Eryu Guan <guaneryu@gmail.com>
This commit is contained in:
Andreas Gruenbacher
2019-08-19 10:57:36 +02:00
committed by Eryu Guan
parent 3b8ea0d760
commit 8bbd45ad5b
+2 -2
View File
@@ -45,7 +45,7 @@ _rename_test()
echo "fsync rename test"
_mount_flakey
$XFS_IO_PROG -f -c "pwrite 0 1M" -c "fsync" $SCRATCH_MNT/foo \
> $seqres.full 2>&1 || _fail "xfs_io failed"
>> $seqres.full 2>&1
mv $SCRATCH_MNT/foo $SCRATCH_MNT/bar
$XFS_IO_PROG -c "fsync" $SCRATCH_MNT/bar
md5sum $SCRATCH_MNT/bar | _filter_scratch
@@ -65,7 +65,7 @@ _write_after_fsync_rename_test()
echo "fsync rename test"
_mount_flakey
$XFS_IO_PROG -f -c "pwrite 0 1M" -c "fsync" -c "pwrite 2M 1M" \
-c "sync_range -b" $SCRATCH_MNT/foo > $seqres.full 2>&1 || _fail "xfs_io failed"
-c "sync_range -b 2M 1M" $SCRATCH_MNT/foo >> $seqres.full 2>&1
mv $SCRATCH_MNT/foo $SCRATCH_MNT/bar
$XFS_IO_PROG -c "fsync" $SCRATCH_MNT/bar
md5sum $SCRATCH_MNT/bar | _filter_scratch