On Fri, Feb 17, 2017 at 02:46:24PM +0800, Eryu Guan wrote: > On Wed, Feb 08, 2017 at 12:11:18PM +0800, Xiong Zhou wrote: > > Mount TEST_DEV as non-DAX, SCRATCH_DEV as DAX, then > > do some IO between them. In this case we use mmap > > and dio/buffered IO read/write test programme. > > > > Signed-off-by: Ross Zwisler <ross.zwisler@xxxxxxxxxxxxxxx> > > Signed-off-by: Xiong Zhou <xzhou@xxxxxxxxxx> > > --- > > tests/generic/407 | 152 ++++++++++++++++++++++++++++++++++++++++++++++++++ > > tests/generic/407.out | 2 + > > tests/generic/group | 1 + > > 3 files changed, 155 insertions(+) > > create mode 100755 tests/generic/407 > > create mode 100644 tests/generic/407.out > > > > diff --git a/tests/generic/407 b/tests/generic/407 > > new file mode 100755 > > index 0000000..b5b2ce4 > > --- /dev/null > > +++ b/tests/generic/407 > > @@ -0,0 +1,152 @@ > > +#! /bin/bash > > +# FS QA Test 407 > > +# > > +# mmap direct/buffered io between DAX and non-DAX mountpoints. > [snip] > > + > > +# make fs 2Mb aligned for PMD fault testing > > +if [[ $FSTYP =~ ext ]] ; then > > + _scratch_mkfs "-E stride=512,stripe_width=1" \ > > + > /dev/null 2>&1 > > +elif [[ $FSTYP =~ xfs ]] ; then > > + _scratch_mkfs "-d su=2m,sw=1" > /dev/null 2>&1 > > +fi > > If FSTYP isn't extN nor xfs, then scratch device is not mkfs'ed, I think > you're missing a default "else" here, which calls _scratch_mkfs without > any options. > > If you're OK with this update, I can fix it at commit time. Okay, Thank you! > > Thanks, > Eryu -- To unsubscribe from this list: send the line "unsubscribe fstests" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html