On Tue, 6 Aug 2019 at 00:27, Andreas Gruenbacher <agruenba@xxxxxxxxxx> wrote: > > The xfs_io sync_range command requires offset and length arguments. Those are > missing here, so the command fails with: > > bad argument count 1 to sync_range, expected at least 2 arguments > > This went unnoticed because xfs_io still exits with status 0 in such cases, > which looks like a separate bug. > > I'm assuming that the test did catch regressions as is and that the sync_range > command isn't needed. If this isn't the case, please fix the test. Copying Josef who seems to be the author of this test case. > Signed-off-by: Andreas Gruenbacher <agruenba@xxxxxxxxxx> > --- > tests/generic/322 | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/tests/generic/322 b/tests/generic/322 > index 2afd7127..22797c04 100755 > --- a/tests/generic/322 > +++ b/tests/generic/322 > @@ -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" > + $SCRATCH_MNT/foo > $seqres.full 2>&1 || _fail "xfs_io failed" > mv $SCRATCH_MNT/foo $SCRATCH_MNT/bar > $XFS_IO_PROG -c "fsync" $SCRATCH_MNT/bar > md5sum $SCRATCH_MNT/bar | _filter_scratch > -- > 2.20.1 Thanks, Andreas
