Hi,
Could you help me review the v2 patch. Thanks a lot!
Thanks,
Xiao Yang.
On 2017/08/04 15:08, Xiao Yang wrote:
> 1) This test can check if setting types causes error regardless of
> supporting crc, so we can update existed comments about it. We
> also add new comments about known issues triggered in this test.
>
> 2) When finobt is disabled, xfs_db fails to get current address of
> free_root, as below:
> xfs_db -c "agi" -c "addr free_root" -c "daddr" /dev/sda11
> Metadata CRC error detected at xfs_inobt block 0x0/0x1000
> ...
> Running related tests without finobt makes no sense, so we add
> check for finobt.
>
> Signed-off-by: Xiao Yang <yangx.jy@xxxxxxxxxxxxxx>
> ---
> tests/xfs/424 | 28 ++++++++++++++++++++--------
> 1 file changed, 20 insertions(+), 8 deletions(-)
>
> diff --git a/tests/xfs/424 b/tests/xfs/424
> index 0a1eef9..41180d8 100755
> --- a/tests/xfs/424
> +++ b/tests/xfs/424
> @@ -1,11 +1,19 @@
> #! /bin/bash
> # FS QA Test 424
> #
> -# xfs_db should take type size into account when setting type.
> -# If type size isn't updated whenever type is set, a false crc
> -# error can occur due to the stale size. This test checks for
> -# that false crc error.
> +# This case checks if setting type causes error.
> #
> +# On crc filesystems, xfs_db doesn't take sector size into account
> +# when setting type, and this can result in an errant crc.
> +# This issue has been fixed in xfsprogs-dev:
> +# '55f224b ("xfs_db: update buffer size when new type is set")'
> +#
> +# On crc filesystems, when setting the type to "inode" the verifier
> +# validates multiple inodes in the current fs block, so setting the
> +# buffer size to that of just one inode is not sufficient and it'll
> +# emit spurious verifier errors for all but the first.
> +# This issue has been fixed in xfsprogs-dev:
> +# '533d1d2 ("xfs_db: properly set inode type")'
> #-----------------------------------------------------------------------
> # Copyright (c) 2017 Red Hat, Inc. All Rights Reserved.
> #
> @@ -77,7 +85,9 @@ while [ $sec_sz -le 4096 ]; do
> done
>
> for SECTOR_SIZE in $sector_sizes; do
> - $MKFS_XFS_PROG -f -s size=$SECTOR_SIZE $SCRATCH_DEV > /dev/null
> + finobt_enabled=0
> + $MKFS_XFS_PROG -f -s size=$SECTOR_SIZE $SCRATCH_DEV | \
> + grep -q 'finobt=1' && finobt_enabled=1
>
> for TYPE in agf agi agfl sb; do
> DADDR=`_scratch_xfs_db -c "$TYPE" -c "daddr" | filter_dbval`
> @@ -96,9 +106,11 @@ for SECTOR_SIZE in $sector_sizes; do
> DADDR=`_scratch_xfs_db -c "agi" -c "addr root" -c "daddr" |
> filter_dbval`
> _scratch_xfs_db -c "daddr $DADDR" -c "type inobt"
> - DADDR=`_scratch_xfs_db -c "agi" -c "addr free_root" -c "daddr" |
> - filter_dbval`
> - _scratch_xfs_db -c "daddr $DADDR" -c "type finobt"
> + if [ $finobt_enabled -eq 1 ]; then
> + DADDR=`_scratch_xfs_db -c "agi" -c "addr free_root" -c "daddr" |
> + filter_dbval`
> + _scratch_xfs_db -c "daddr $DADDR" -c "type finobt"
> + fi
>
> _scratch_xfs_db -c "daddr $DADDR" -c "type text"
> _scratch_xfs_db -c "daddr $DADDR" -c "type data"
--
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
