On Sun, Mar 14, 2021 at 07:36:32PM +0800, Eryu Guan wrote: > Hi Zorro, > > On Tue, Feb 02, 2021 at 02:22:53PM +0800, Zorro Lang wrote: > > This case print both fio stdout and stderr to .full file, that cause > > we miss some unexpected failures when there's a bug. For example: > > > > file:io_u.c:1803, func=io_u error, error=Block device required > > > > This's an regression issue we find on a downstream kernel, not in > > upstream. So release unexpected fio error output to find more issues. > > > > Signed-off-by: Zorro Lang <zlang@xxxxxxxxxx> > > --- > > tests/generic/095 | 5 ++++- > > 1 file changed, 4 insertions(+), 1 deletion(-) > > > > diff --git a/tests/generic/095 b/tests/generic/095 > > index 9afaa761..30fe77a5 100755 > > --- a/tests/generic/095 > > +++ b/tests/generic/095 > > @@ -98,8 +98,11 @@ _require_fio $fio_config > > _scratch_mkfs >>$seqres.full 2>&1 > > _scratch_mount > > > > +# There's a known EIO failure to report collisions between directio and buffered > > +# writes to userspace, refer to upstream linux 5a9d929d6e13. So ignore EIO error > > +# at here. > > +$FIO_PROG $fio_config --ignore_error=,EIO --output=$seqres.full > > I found that with this change, EIO is ignored and the return value of > fio command is 0 as expected, but the output breaks golden image and > fails the test. > > [root@fedoravm xfstests]# ./check -s xfs_4k_reflink generic/095 > SECTION -- xfs_4k_reflink > RECREATING -- xfs on /dev/mapper/testvg-lv1 > FSTYP -- xfs (debug) > PLATFORM -- Linux/x86_64 fedoravm 5.11.0 #5 SMP Sun Feb 28 21:50:24 CST 2021 > MKFS_OPTIONS -- -f -f -b size=4k -m reflink=1,rmapbt=1 /dev/mapper/testvg-lv2 > MOUNT_OPTIONS -- /dev/mapper/testvg-lv2 /mnt/scratch > > generic/095 4s ... - output mismatch (see /root/workspace/xfstests/results//xfs_4k_reflink/generic/095.out.bad) > --- tests/generic/095.out 2018-02-25 15:15:00.097388035 +0800 > +++ /root/workspace/xfstests/results//xfs_4k_reflink/generic/095.out.bad 2021-03-14 19:32:04.126884255 +0800 > @@ -1,2 +1,7 @@ > QA output created by 095 > +job9: No I/O performed by mmap, perhaps try --debug=io option for details? > +job9: No I/O performed by mmap, perhaps try --debug=io option for details? > +job9: No I/O performed by mmap, perhaps try --debug=io option for details? > +job9: No I/O performed by mmap, perhaps try --debug=io option for details? > +job9: No I/O performed by mmap, perhaps try --debug=io option for details? > Silence is golden > > Did you hit the same failure? If so I think we should filter out this > message. Hmm... I never hit that. I don't think this patch can cause this error, can you still reproduce it without this patch? The mmap operation from fio might get some unexpected random arguments, you might find a fio bug :-P Thanks, Zorro > > Thanks, > Eryu > > > echo "Silence is golden" > > -$FIO_PROG $fio_config >>$seqres.full 2>&1 > > > > # xfs generates WARNINGs on purpose when applications mix buffered/mmap IO with > > # direct IO on the same file. On the other hand, this fio job has been proven > > -- > > 2.29.2 >
