* [PATCH] xfs/187: fix ftype brokenness
@ 2018-04-11 16:08 Darrick J. Wong
2018-04-13 6:14 ` Eryu Guan
0 siblings, 1 reply; 3+ messages in thread
From: Darrick J. Wong @ 2018-04-11 16:08 UTC (permalink / raw)
To: Eryu Guan; +Cc: fstests
From: Darrick J. Wong <darrick.wong@oracle.com>
This test requires XFS_SB_VERSION_MOREBITSBIT to be zero. ftype (which
is now enabled by default) causes this to be set, so detect it in mkfs
and disable it.
Signed-off-by: Darrick J. Wong <darrick.wong@oracle.com>
---
tests/xfs/187 | 10 ++++++++--
1 file changed, 8 insertions(+), 2 deletions(-)
diff --git a/tests/xfs/187 b/tests/xfs/187
index 07ef3ae..e1a8ce9 100755
--- a/tests/xfs/187
+++ b/tests/xfs/187
@@ -70,8 +70,14 @@ export MOUNT_OPTIONS=""
# lazysb, attr2 and other feature bits are held in features2 and will require
# morebitsbit on So test with lazysb and without it to see if the morebitsbit is
# okay etc. If the mkfs defaults change, these need to change as well.
-export MKFS_NO_LAZY="-m crc=0 -l lazy-count=0 -i projid32bit=0"
-export MKFS_LAZY="-m crc=0 -l lazy-count=1 -i projid32bit=0"
+MKFS_NO_LAZY="-m crc=0 -l lazy-count=0 -i projid32bit=0"
+MKFS_LAZY="-m crc=0 -l lazy-count=1 -i projid32bit=0"
+
+# Detect ftype
+if _scratch_mkfs --help 2>&1 | grep -q "ftype="; then
+ MKFS_NO_LAZY="$MKFS_NO_LAZY -n ftype=0"
+ MKFS_LAZY="$MKFS_LAZY -n ftype=0"
+fi
# Make sure that when we think we are testing with morebits off
# that we really are.
^ permalink raw reply related [flat|nested] 3+ messages in thread
* Re: [PATCH] xfs/187: fix ftype brokenness
2018-04-11 16:08 [PATCH] xfs/187: fix ftype brokenness Darrick J. Wong
@ 2018-04-13 6:14 ` Eryu Guan
2018-04-13 23:37 ` Darrick J. Wong
0 siblings, 1 reply; 3+ messages in thread
From: Eryu Guan @ 2018-04-13 6:14 UTC (permalink / raw)
To: Darrick J. Wong; +Cc: fstests
On Wed, Apr 11, 2018 at 09:08:09AM -0700, Darrick J. Wong wrote:
> From: Darrick J. Wong <darrick.wong@oracle.com>
>
> This test requires XFS_SB_VERSION_MOREBITSBIT to be zero. ftype (which
> is now enabled by default) causes this to be set, so detect it in mkfs
> and disable it.
>
> Signed-off-by: Darrick J. Wong <darrick.wong@oracle.com>
> ---
> tests/xfs/187 | 10 ++++++++--
> 1 file changed, 8 insertions(+), 2 deletions(-)
>
> diff --git a/tests/xfs/187 b/tests/xfs/187
> index 07ef3ae..e1a8ce9 100755
> --- a/tests/xfs/187
> +++ b/tests/xfs/187
> @@ -70,8 +70,14 @@ export MOUNT_OPTIONS=""
> # lazysb, attr2 and other feature bits are held in features2 and will require
> # morebitsbit on So test with lazysb and without it to see if the morebitsbit is
> # okay etc. If the mkfs defaults change, these need to change as well.
> -export MKFS_NO_LAZY="-m crc=0 -l lazy-count=0 -i projid32bit=0"
> -export MKFS_LAZY="-m crc=0 -l lazy-count=1 -i projid32bit=0"
> +MKFS_NO_LAZY="-m crc=0 -l lazy-count=0 -i projid32bit=0"
> +MKFS_LAZY="-m crc=0 -l lazy-count=1 -i projid32bit=0"
> +
> +# Detect ftype
I'd put the reason of detecting ftype support in comments here. I can
fix it on commit.
Thanks,
Eryu
> +if _scratch_mkfs --help 2>&1 | grep -q "ftype="; then
> + MKFS_NO_LAZY="$MKFS_NO_LAZY -n ftype=0"
> + MKFS_LAZY="$MKFS_LAZY -n ftype=0"
> +fi
>
> # Make sure that when we think we are testing with morebits off
> # that we really are.
^ permalink raw reply [flat|nested] 3+ messages in thread
* Re: [PATCH] xfs/187: fix ftype brokenness
2018-04-13 6:14 ` Eryu Guan
@ 2018-04-13 23:37 ` Darrick J. Wong
0 siblings, 0 replies; 3+ messages in thread
From: Darrick J. Wong @ 2018-04-13 23:37 UTC (permalink / raw)
To: Eryu Guan; +Cc: fstests
On Fri, Apr 13, 2018 at 02:14:08PM +0800, Eryu Guan wrote:
> On Wed, Apr 11, 2018 at 09:08:09AM -0700, Darrick J. Wong wrote:
> > From: Darrick J. Wong <darrick.wong@oracle.com>
> >
> > This test requires XFS_SB_VERSION_MOREBITSBIT to be zero. ftype (which
> > is now enabled by default) causes this to be set, so detect it in mkfs
> > and disable it.
> >
> > Signed-off-by: Darrick J. Wong <darrick.wong@oracle.com>
> > ---
> > tests/xfs/187 | 10 ++++++++--
> > 1 file changed, 8 insertions(+), 2 deletions(-)
> >
> > diff --git a/tests/xfs/187 b/tests/xfs/187
> > index 07ef3ae..e1a8ce9 100755
> > --- a/tests/xfs/187
> > +++ b/tests/xfs/187
> > @@ -70,8 +70,14 @@ export MOUNT_OPTIONS=""
> > # lazysb, attr2 and other feature bits are held in features2 and will require
> > # morebitsbit on So test with lazysb and without it to see if the morebitsbit is
> > # okay etc. If the mkfs defaults change, these need to change as well.
> > -export MKFS_NO_LAZY="-m crc=0 -l lazy-count=0 -i projid32bit=0"
> > -export MKFS_LAZY="-m crc=0 -l lazy-count=1 -i projid32bit=0"
> > +MKFS_NO_LAZY="-m crc=0 -l lazy-count=0 -i projid32bit=0"
> > +MKFS_LAZY="-m crc=0 -l lazy-count=1 -i projid32bit=0"
> > +
> > +# Detect ftype
>
> I'd put the reason of detecting ftype support in comments here. I can
> fix it on commit.
Ok.
# ftype is also stored in features2, so we have to detect its presence in
# mkfs and disable it here too.
?
--D
> Thanks,
> Eryu
>
> > +if _scratch_mkfs --help 2>&1 | grep -q "ftype="; then
> > + MKFS_NO_LAZY="$MKFS_NO_LAZY -n ftype=0"
> > + MKFS_LAZY="$MKFS_LAZY -n ftype=0"
> > +fi
> >
> > # Make sure that when we think we are testing with morebits off
> > # that we really are.
> --
> To unsubscribe from this list: send the line "unsubscribe fstests" in
> the body of a message to majordomo@vger.kernel.org
> More majordomo info at http://vger.kernel.org/majordomo-info.html
^ permalink raw reply [flat|nested] 3+ messages in thread
end of thread, other threads:[~2018-04-13 23:37 UTC | newest]
Thread overview: 3+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2018-04-11 16:08 [PATCH] xfs/187: fix ftype brokenness Darrick J. Wong
2018-04-13 6:14 ` Eryu Guan
2018-04-13 23:37 ` Darrick J. Wong
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.