Re: [PATCH v1 1/3] xfs/539: Skip noattr2 remount option on v5 filesystems

Nirjhar Roy (IBM)
2025-02-24T04:50:55Z

On 2/13/25 02:31, Dave Chinner wrote:
> On Wed, Feb 12, 2025 at 12:39:56PM +0000, Nirjhar Roy (IBM) wrote:
>> This test is to verify that repeated warnings are not printed
>> for default options (attr2, noikeep) and warnings are
>> printed for non default options (noattr2, ikeep). Remount
>> with noattr2 fails on a v5 filesystem, so skip the mount option.
> Why do we care if remount succeeds or fails? That's not what the
> test is exercising.
>
> i.e. We are testing to see if the appropriate deprecation warning
> for a deprecated mount option has been issued or not, and that
> should happen regardless of whether the mount option is valid or not
> for the given filesysetm format....
>
> Hence I don't see any reason for changing the test to exclude
> noattr2 testing on v5 filesystems...

Yes, this makes sense. The test indeed just checks for the dmesg
warnings, and they appear even if the remount fails. I wrote the patch
because xfs/539 has started failing in one of our fstests CI runs
because RHEL 10 has started disabling xfs v4 support i.e,
CONFIG_XFS_SUPPORT_V4=n. Do you think modifying this patch in such a way
that the test ignores the remount failures with noattr2 and continues
the test is an appropriate idea (since the test xfs/539 only intends to
check the dmesg warnings)? So something like:,

--- a/tests/xfs/539
+++ b/tests/xfs/539
@@ -61,7 +61,11 @@ for VAR in {attr2,noikeep}; do
done
for VAR in {noattr2,ikeep}; do
log_tag
-    _scratch_remount $VAR
+    _scratch_remount $VAR >> $seqres.full 2>&1
+    if [[ "$VAR" == "noattr2" && "$?" != "0" ]]; then
+        echo "remount will fail in v5 filesystem but the warning should
be printed" \
+            >> $seqres.full
+    fi
check_dmesg_for_since_tag "XFS: $VAR mount option is deprecated" || \
echo "Could not find deprecation warning for $VAR"

I also suggested something similar in one of my previous replies[1] in
this patch series. Can you please let me know your thoughts on this?

[1]
https://lore.kernel.org/all/90be3350-67e5-4dec-bc65-442762f5f856@gmail.com/

--NR

>
> -Dave.

--
Nirjhar Roy
Linux Kernel Developer
IBM, Bangalore