* Re: seek sanity check failures
[not found] ` <CAH2r5muncmiTKBuaYEie6L7PeXmQ-MjKsBy9WrET4v85f36=qA@mail.gmail.com>
@ 2019-09-16 8:47 ` Murphy Zhou
0 siblings, 0 replies; only message in thread
From: Murphy Zhou @ 2019-09-16 8:47 UTC (permalink / raw)
To: Steve French; +Cc: Murphy Zhou, CIFS, amir73il, fstests
On Sat, Sep 14, 2019 at 11:11:49PM -0500, Steve French wrote:
> Test 285 and 490 broke when xfstests were updated on the buildbot, and
> are temporarily deleted (commented out in the buildbot script). We
> had a difficult choice - update xfstests and add 10 new tests but
> disable 3 tests which broke with updated tests, or not add the 10
> tests by keeping running last year's old xfstest test suite.
>
> Ronnie and I did some investigation of why 285 and 490 failed and any
> additional data would help - since we both got distracted with other
> features and fixes
In Apr 2019, Amir updated seek_sanity_check:
(xfstests commit 34081087748)
pwrite a file, then double the filesize by ftruncate, then SEEK_HOLE.
If SEEK_HOLE returns EOF, and fstype is in this list:
+ btrfs|ext4|xfs|ceph|cifs|f2fs|gfs2|nfs*|ocfs2|tmpfs)
It's the default behavior and it's NOT allowed. Test prints error and exit.
If cifs is removed from above list, all seek sanity check tests pass on cifs.
cifs SEEK_DATA SEEK_HOLE support were added in May 2019 with Linux
commit dece44e381a
A xfstests patch removing cifs from the list can address these failures,
but is the "default behavior" of SEEK_HOLE on cifs expected?
If it's not expected and need to work on, we should keep current status.
Thanks!
M
>
> On Sat, Sep 14, 2019 at 10:14 PM Murphy Zhou <jencce.kernel@gmail.com> wrote:
> >
> > Hi,
> >
> > On latest Linus tree, several seek tests fails:
> >
> > generic/285 generic/490 generic/539 fails on v3.11 v3.02 v3.0
> > generic/285 generic/490 fails on v2.0 v2.1
> > generic/285 generic/448 generic/490 fails on v1.0
> >
> >
> > Are they expected or being worked on?
> >
> > Thanks!
> >
> >
> > FSTYP -- cifs
> > PLATFORM -- Linux/x86_64 hp-dl380pg8-14 5.3.0-rc8+ #1 SMP Thu Sep 12 07:31:21 EDT 2019
> > MKFS_OPTIONS -- //hp-dl380pg8-14.rhts.eng.pek2.redhat.com/scratch
> > MOUNT_OPTIONS -- -o vers=3.11,mfsymlinks -o username=root,password=redhat -o context=system_u:object_r:nfs_t:s0 //hp-dl380pg8-14.rhts.eng.pek2.redhat.com/scratch /mnt/testarea/scratch
> >
> > generic/285 [failed, exit status 1]- output mismatch (see /var/lib/xfstests/results//generic/285.out.bad)
> > --- tests/generic/285.out 2019-09-12 08:04:14.120990746 -0400
> > +++ /var/lib/xfstests/results//generic/285.out.bad 2019-09-12 08:36:49.151230335 -0400
> > @@ -1 +1,3 @@
> > QA output created by 285
> > +seek sanity check failed!
> > +(see /var/lib/xfstests/results//generic/285.full for details)
> > ...
>
>
>
> --
> Thanks,
>
> Steve
^ permalink raw reply [flat|nested] only message in thread
only message in thread, other threads:[~2019-09-16 8:47 UTC | newest]
Thread overview: (only message) (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
[not found] <20190915020810.jy5pxjtk7buenmqk@XZHOUW.usersys.redhat.com>
[not found] ` <CAH2r5muncmiTKBuaYEie6L7PeXmQ-MjKsBy9WrET4v85f36=qA@mail.gmail.com>
2019-09-16 8:47 ` seek sanity check failures Murphy Zhou
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).