From: Dave Chinner <david@fromorbit.com>
To: "Darrick J. Wong" <darrick.wong@oracle.com>
Cc: Miklos Szeredi <miklos@szeredi.hu>,
linux-xfs <linux-xfs@vger.kernel.org>,
linux-fsdevel@vger.kernel.org, fstests <fstests@vger.kernel.org>
Subject: Re: generic/127 failure on xfs with hacked fsx
Date: Thu, 15 Mar 2018 08:27:31 +1100 [thread overview]
Message-ID: <20180314212731.GD7000@dastard> (raw)
In-Reply-To: <20180314211851.GW4865@magnolia>
On Wed, Mar 14, 2018 at 02:18:51PM -0700, Darrick J. Wong wrote:
> On Wed, Mar 14, 2018 at 04:50:02PM +0100, Miklos Szeredi wrote:
> > While testing overlayfs I found something which appears to be an
> > unwanted side effect of xfs_release().
> >
> > Apply the attached patch to xfstests and run generic/127.
> >
> > Result is:
> >
> > +ltp/fsx -q -l 262144 -o 65536 -S 191110531 -N 100000 fsx_std_mmap
> > +Mapped Read: non-zero data past EOF (0x8aa1) page offset 0xaa2 is 0xa091
> > +LOG DUMP (70315 total operations):
> >
> > Thanks,
> > Miklos
>
> Hmm.... so I applied this and ran generic/127 on xfs:
>
> FSTYP -- xfs (debug)
> PLATFORM -- Linux/x86_64 magnolia-mtr00 4.16.0-rc5-xfsx
> MKFS_OPTIONS -- -f -m reflink=1,rmapbt=1, -i sparse=1, /dev/pmem4
> MOUNT_OPTIONS -- -o usrquota,grpquota,prjquota, /dev/sda /opt
>
> generic/127 53s
> Ran: generic/127
> Passed all 1 tests
>
> Reran a couple of times without seeing any problems, so what am
> I missing? Are you running xfstests atop overlayfs atop xfs?
>
> /me confused. :/
I'm betting it's timing related. the close can run a filemap_flush()
call, so there's every chance the page is under IO during the
mapread operation after the openclose() call. On pmem, it won't
be under IO because the writeback IO is a synchronous memcpy....
CHeers,
Dave.
--
Dave Chinner
david@fromorbit.com
next prev parent reply other threads:[~2018-03-14 21:27 UTC|newest]
Thread overview: 4+ messages / expand[flat|nested] mbox.gz Atom feed top
2018-03-14 15:50 generic/127 failure on xfs with hacked fsx Miklos Szeredi
2018-03-14 21:18 ` Darrick J. Wong
2018-03-14 21:27 ` Dave Chinner [this message]
2018-03-15 7:58 ` Miklos Szeredi
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=20180314212731.GD7000@dastard \
--to=david@fromorbit.com \
--cc=darrick.wong@oracle.com \
--cc=fstests@vger.kernel.org \
--cc=linux-fsdevel@vger.kernel.org \
--cc=linux-xfs@vger.kernel.org \
--cc=miklos@szeredi.hu \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.