On Tue, Aug 30, 2022 at 12:44:33PM +0800, Murphy Zhou wrote:
> Since this xfsprogs commit
> 1b580a773 mkfs: don't let internal logs bump the root dir inode chunk to AG 1
> this operation is not allowed.
>
> Signed-off-by: Murphy Zhou <jencce.kernel@xxxxxxxxx>
> ---
> tests/xfs/144 | 10 +---------
> 1 file changed, 1 insertion(+), 9 deletions(-)
>
> diff --git a/tests/xfs/144 b/tests/xfs/144
> index 706aff61..3f80d0ee 100755
> --- a/tests/xfs/144
> +++ b/tests/xfs/144
> @@ -17,9 +17,6 @@ _begin_fstest auto mkfs
> _supported_fs xfs
> _require_test
>
> -# The last testcase creates a (sparse) fs image with a 2GB log, so we need
> -# 3GB to avoid failing the mkfs due to ENOSPC.
> -_require_fs_space $TEST_DIR $((3 * 1048576))
> echo Silence is golden
>
> testfile=$TEST_DIR/a
> @@ -36,7 +33,7 @@ test_format() {
> }
>
> # First we try various small filesystems and stripe sizes.
> -for M in `seq 298 302` `seq 490 520`; do
> +for M in `seq 1024 1030` ; do
Can `seq 1024 1030` replace `seq 298 302` `seq 490 520`? I don't know how
Darrick choose these numbers, better to ask the original authoer of this
case. Others looks reasonable for me.
Thanks,
Zorro
> for S in `seq 32 4 64`; do
> test_format "M=$M S=$S" -dsu=${S}k,sw=1,size=${M}m -N
> done
> @@ -45,11 +42,6 @@ done
> # log end rounded beyond EOAG due to stripe unit
> test_format "log end beyond eoag" -d agcount=3200,size=6366g -d su=256k,sw=4 -N
>
> -# Log so large it pushes the root dir into AG 1. We can't use -N for the mkfs
> -# because this check only occurs after the root directory has been allocated,
> -# which mkfs -N doesn't do.
> -test_format "log pushes rootdir into AG 1" -d agcount=3200,size=6366g -lagnum=0
> -
> # success, all done
> status=0
> exit
> --
> 2.31.1
>
