From: Anthony Iliopoulos <ailiop@suse.com>
Date: Thu, 1 Feb 2024 16:17:31 +0000 (+0100)
Subject: generic/68[12]: use the dir blocksize for xfs filesystems
X-Git-Tag: xfs-zoned-2024-06-07~149
X-Git-Url: https://www.infradead.org/git/?a=commitdiff_plain;h=cc23dcb3ebe6fe6ab8307da183e660717eea38fc;p=users%2Fhch%2Fxfstests-dev.git

generic/68[12]: use the dir blocksize for xfs filesystems

The tests are using the filesystem block size for calculating the number
of dirents required to fill a 2-block directory. For v4 xfs filesystems
formatted with fs blocksize of 512 bytes this is failing, as the tests
do not take into account that the directory block size is not always
equal to the filesystem block size. As such, the tests never go over
quota, and even if they did there is no hard block limit being set (due
to 512 / 1024 = 0 calculation in setquota).

Use the directory blocksize instead of the filesystem blocksize, when
the fstype under test is xfs.

Signed-off-by: Anthony Iliopoulos <ailiop@suse.com>
Reviewed-by: Andrey Albershteyn <aalbersh@redhat.com>
Signed-off-by: Zorro Lang <zlang@kernel.org>
---

diff --git a/common/rc b/common/rc
index b53a1cbb5..50dde313b 100644
--- a/common/rc
+++ b/common/rc
@@ -4756,6 +4756,17 @@ _get_block_size()
 	stat -f -c %S $1
 }
 
+# Obtain the directory block size of an fs.
+_get_dir_block_size()
+{
+	case "$FSTYP" in
+	xfs)
+		_xfs_get_dir_blocksize $1 ;;
+	*)
+		_get_block_size $1 ;;
+	esac
+}
+
 # Require that the fundamental allocation unit of a file is the same as the
 # filesystem block size.  The sole parameter must be the root dir of a
 # filesystem.
diff --git a/tests/generic/681 b/tests/generic/681
index 090da795c..f41647741 100755
--- a/tests/generic/681
+++ b/tests/generic/681
@@ -30,7 +30,7 @@ _scratch_mkfs > "$seqres.full" 2>&1
 _qmount_option usrquota
 _qmount
 
-blocksize=$(_get_block_size $SCRATCH_MNT)
+blocksize=$(_get_dir_block_size $SCRATCH_MNT)
 scratchdir=$SCRATCH_MNT/dir
 scratchfile=$SCRATCH_MNT/file
 mkdir $scratchdir
diff --git a/tests/generic/682 b/tests/generic/682
index b4cd0cd9d..a51d18338 100755
--- a/tests/generic/682
+++ b/tests/generic/682
@@ -30,7 +30,7 @@ _scratch_mkfs > "$seqres.full" 2>&1
 _qmount_option usrquota
 _qmount
 
-blocksize=$(_get_block_size $SCRATCH_MNT)
+blocksize=$(_get_dir_block_size $SCRATCH_MNT)
 scratchdir=$SCRATCH_MNT/dir
 scratchfile=$SCRATCH_MNT/file
 stagedir=$SCRATCH_MNT/staging