* [PATCH] ext4/021: add -F flag for xfs_io
@ 2016-09-02 13:55 Xiao Yang
2016-09-05 5:33 ` Eryu Guan
0 siblings, 1 reply; 3+ messages in thread
From: Xiao Yang @ 2016-09-02 13:55 UTC (permalink / raw)
To: fstests; +Cc: Xiao Yang
xfs_io needs -F flag to write data in non-XFS filesystem on xfsprogs 3.1.1.
The -F flag has been removed from usage() & man pages since xfsprogs 3.1.8,
but is still accepted. Please see the following commit d1b8818
("xfs_io: deprecate the "-F" foreign flag")
Signed-off-by: Xiao Yang <yangx.jy@cn.fujitsu.com>
---
tests/ext4/021 | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/tests/ext4/021 b/tests/ext4/021
index d758818..f82d7fb 100755
--- a/tests/ext4/021
+++ b/tests/ext4/021
@@ -80,7 +80,7 @@ fi
# disk, the s_squence offset to the beginning of journal superblock is 24
# we do this to let jbd2 start to run with a initial big transaction id,
# which will reduce the time taken to trigger this bug.
-xfs_io -c "pwrite -S 0x81 $((offset+24)) 1" \
+xfs_io -F -c "pwrite -S 0x81 $((offset+24)) 1" \
-c "pwrite -S 0xd1 $((offset+25)) 1" \
-c "pwrite -S 0xa4 $((offset+26)) 1" \
-c "pwrite -S 0x80 $((offset+27)) 1" $SCRATCH_DEV >> $seqres.full 2>&1
--
1.8.3.1
^ permalink raw reply related [flat|nested] 3+ messages in thread
* Re: [PATCH] ext4/021: add -F flag for xfs_io
2016-09-02 13:55 [PATCH] ext4/021: add -F flag for xfs_io Xiao Yang
@ 2016-09-05 5:33 ` Eryu Guan
2016-09-05 5:56 ` [PATCH v2] ext4/021: use $XFS_IO_PROG instead of xfs_io Xiao Yang
0 siblings, 1 reply; 3+ messages in thread
From: Eryu Guan @ 2016-09-05 5:33 UTC (permalink / raw)
To: Xiao Yang; +Cc: fstests
On Fri, Sep 02, 2016 at 09:55:35PM +0800, Xiao Yang wrote:
> xfs_io needs -F flag to write data in non-XFS filesystem on xfsprogs 3.1.1.
> The -F flag has been removed from usage() & man pages since xfsprogs 3.1.8,
> but is still accepted. Please see the following commit d1b8818
> ("xfs_io: deprecate the "-F" foreign flag")
>
> Signed-off-by: Xiao Yang <yangx.jy@cn.fujitsu.com>
> ---
> tests/ext4/021 | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/tests/ext4/021 b/tests/ext4/021
> index d758818..f82d7fb 100755
> --- a/tests/ext4/021
> +++ b/tests/ext4/021
> @@ -80,7 +80,7 @@ fi
> # disk, the s_squence offset to the beginning of journal superblock is 24
> # we do this to let jbd2 start to run with a initial big transaction id,
> # which will reduce the time taken to trigger this bug.
> -xfs_io -c "pwrite -S 0x81 $((offset+24)) 1" \
> +xfs_io -F -c "pwrite -S 0x81 $((offset+24)) 1" \
Use $XFS_IO_PROG and "-F" is added when needed. See init_rc().
# Figure out if we need to add -F ("foreign", deprecated) option to xfs_io
xfs_io -c stat $TEST_DIR 2>&1 | grep -q "is not on an XFS filesystem" && \
export XFS_IO_PROG="$XFS_IO_PROG -F"
Thanks,
Eryu
> -c "pwrite -S 0xd1 $((offset+25)) 1" \
> -c "pwrite -S 0xa4 $((offset+26)) 1" \
> -c "pwrite -S 0x80 $((offset+27)) 1" $SCRATCH_DEV >> $seqres.full 2>&1
> --
> 1.8.3.1
>
>
>
> --
> 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
* [PATCH v2] ext4/021: use $XFS_IO_PROG instead of xfs_io
2016-09-05 5:33 ` Eryu Guan
@ 2016-09-05 5:56 ` Xiao Yang
0 siblings, 0 replies; 3+ messages in thread
From: Xiao Yang @ 2016-09-05 5:56 UTC (permalink / raw)
To: eguan; +Cc: fstests, Xiao Yang
We should use $XFS_IO_PROG to add "-F" option when we need it.
Signed-off-by: Xiao Yang <yangx.jy@cn.fujitsu.com>
---
tests/ext4/021 | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/tests/ext4/021 b/tests/ext4/021
index d758818..d5fad7c 100755
--- a/tests/ext4/021
+++ b/tests/ext4/021
@@ -80,7 +80,7 @@ fi
# disk, the s_squence offset to the beginning of journal superblock is 24
# we do this to let jbd2 start to run with a initial big transaction id,
# which will reduce the time taken to trigger this bug.
-xfs_io -c "pwrite -S 0x81 $((offset+24)) 1" \
+$XFS_IO_PROG -c "pwrite -S 0x81 $((offset+24)) 1" \
-c "pwrite -S 0xd1 $((offset+25)) 1" \
-c "pwrite -S 0xa4 $((offset+26)) 1" \
-c "pwrite -S 0x80 $((offset+27)) 1" $SCRATCH_DEV >> $seqres.full 2>&1
--
1.8.3.1
^ permalink raw reply related [flat|nested] 3+ messages in thread
end of thread, other threads:[~2016-09-05 5:57 UTC | newest]
Thread overview: 3+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2016-09-02 13:55 [PATCH] ext4/021: add -F flag for xfs_io Xiao Yang
2016-09-05 5:33 ` Eryu Guan
2016-09-05 5:56 ` [PATCH v2] ext4/021: use $XFS_IO_PROG instead of xfs_io Xiao Yang
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox