diff mbox series

xfs/288: use -d option of xfs_db write command for v5 XFS

Message ID 20180729142053.4240-1-zlang@redhat.com (mailing list archive)
State New, archived
Headers show
Series xfs/288: use -d option of xfs_db write command for v5 XFS | expand

Commit Message

Zorro Lang July 29, 2018, 2:20 p.m. UTC
The commit b3cf8b72334fd35ef961869506e5a72ab398bc82 update xfs/288
to support v5 filesystem testing. That commit thought xfs_db write
command can work well with -d option on V5 XFS. But the truth is
the case doesn't use that option.

So turn to use _scratch_xfs_set_metadata_field, it will help to use
proper options for xfs_db write command.

Signed-off-by: Zorro Lang <zlang@redhat.com>
---

Hi,

Missing '-d' option is a case issue. But from this issue, we found another
bug of xfsprogs. On current upstream, this case will test pass on v5 XFS,
even there's not '-d' option for write command.

That's a bug, we'll fix it later. For now, I'd like to fix this case
issue at first.

Thanks,
Zorro

 tests/xfs/288 | 5 ++---
 1 file changed, 2 insertions(+), 3 deletions(-)

Comments

Darrick J. Wong July 29, 2018, 3:56 p.m. UTC | #1
On Sun, Jul 29, 2018 at 10:20:53PM +0800, Zorro Lang wrote:
> The commit b3cf8b72334fd35ef961869506e5a72ab398bc82 update xfs/288
> to support v5 filesystem testing. That commit thought xfs_db write
> command can work well with -d option on V5 XFS. But the truth is
> the case doesn't use that option.
> 
> So turn to use _scratch_xfs_set_metadata_field, it will help to use
> proper options for xfs_db write command.
> 
> Signed-off-by: Zorro Lang <zlang@redhat.com>

Looks like a reasonable replacement,
Reviewed-by: Darrick J. Wong <darrick.wong@oracle.com>

--D

> ---
> 
> Hi,
> 
> Missing '-d' option is a case issue. But from this issue, we found another
> bug of xfsprogs. On current upstream, this case will test pass on v5 XFS,
> even there's not '-d' option for write command.
> 
> That's a bug, we'll fix it later. For now, I'd like to fix this case
> issue at first.
> 
> Thanks,
> Zorro
> 
>  tests/xfs/288 | 5 ++---
>  1 file changed, 2 insertions(+), 3 deletions(-)
> 
> diff --git a/tests/xfs/288 b/tests/xfs/288
> index bccdb6fc..f4165b6c 100755
> --- a/tests/xfs/288
> +++ b/tests/xfs/288
> @@ -56,9 +56,8 @@ $SETFATTR_PROG -n "user.testattr${seq}" \
>  
>  _scratch_unmount
>  # manually corrupt the XFS, by set the header count of attr to 0
> -_scratch_xfs_db -x -c "inode $inum" \
> -		   -c "ablock 0" \
> -		   -c "write hdr.count 0" >> $seqres.full
> +_scratch_xfs_set_metadata_field "hdr.count" "0" \
> +				"inode $inum" "ablock 0" >> $seqres.full
>  
>  # make sure xfs_repair can find above corruption. If it can't, that
>  # means we need to fix this bug on current xfs_repair
> -- 
> 2.14.4
> 
> --
> 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
--
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
diff mbox series

Patch

diff --git a/tests/xfs/288 b/tests/xfs/288
index bccdb6fc..f4165b6c 100755
--- a/tests/xfs/288
+++ b/tests/xfs/288
@@ -56,9 +56,8 @@  $SETFATTR_PROG -n "user.testattr${seq}" \
 
 _scratch_unmount
 # manually corrupt the XFS, by set the header count of attr to 0
-_scratch_xfs_db -x -c "inode $inum" \
-		   -c "ablock 0" \
-		   -c "write hdr.count 0" >> $seqres.full
+_scratch_xfs_set_metadata_field "hdr.count" "0" \
+				"inode $inum" "ablock 0" >> $seqres.full
 
 # make sure xfs_repair can find above corruption. If it can't, that
 # means we need to fix this bug on current xfs_repair