Re: [PATCH] xfs/490: fix failure due to deprecated ikeep mount option

["Darrick J. Wong" <djwong@kernel.org>]

On Wed, Dec 17, 2025 at 12:08:19PM +0100, Anthony Iliopoulos wrote:
> The ikeep mount option has been deprecated as of kernel commit
> b9a176e54162 ("xfs: remove deprecated mount options"), and after commit
> 3e7ec343f066 ("xfs: loudly complain about defunct mount options") there
> is a warning emitted when testing v5 filesystems.
> 
> Fix it by explicitly requiring and formatting scratch to nocrc/v4, as
> the test cannot otherwise be triggered on v5 without the ikeep option.
> 
> Signed-off-by: Anthony Iliopoulos <ailiop@suse.com>

<nod> sorry I missed this one, I only test with XFS_DEBUG=y
Reviewed-by: "Darrick J. Wong" <djwong@kernel.org>

--D

> ---
>  tests/xfs/490 | 12 ++----------
>  1 file changed, 2 insertions(+), 10 deletions(-)
> 
> diff --git a/tests/xfs/490 b/tests/xfs/490
> index a3b0747580ea..366acbfa3ef9 100755
> --- a/tests/xfs/490
> +++ b/tests/xfs/490
> @@ -20,6 +20,7 @@ _begin_fstest auto quick
>  # Modify as appropriate.
>  _require_scratch_nocheck
>  _require_xfs_mkfs_finobt
> +_require_xfs_nocrc
>  
>  # Skip the verifier "xfs_check_agi_freecount()" which verify the number of free
>  # inodes in the AGI is correct, when XFS_DEBUG is enabled
> @@ -35,16 +36,7 @@ filter_dmesg()
>  # If enable free inode B+tree, this case will fail on xfs_dialloc_ag_update_inobt,
>  # that's not what we want to test. Due to finobt feature is not necessary for this
>  # test, so disable it directly.
> -_scratch_mkfs_xfs -m finobt=0 | _filter_mkfs 2>$tmp.mkfs >> $seqres.full
> -
> -# On V5 filesystem, this case can't trigger bug because it doesn't read inodes
> -# we are allocating from disk - it simply overwrites them with new inode
> -# information. So use ikeep mount option to stop that.
> -source $tmp.mkfs
> -mount_opt=""
> -if [ $_fs_has_crcs -eq 1 ]; then
> -	mount_opt="-o ikeep"
> -fi
> +_scratch_mkfs_xfs -m crc=0,finobt=0 | _filter_mkfs 2>$tmp.mkfs >> $seqres.full
>  
>  blksz=$(_scratch_xfs_get_sb_field blocksize)
>  agcount=$(_scratch_xfs_get_sb_field agcount)
> -- 
> 2.52.0
> 
>