]> www.infradead.org Git - users/hch/xfstests-dev.git/commitdiff
generic/370: don't exclude XFS
authorChristoph Hellwig <hch@lst.de>
Mon, 17 Feb 2025 06:31:35 +0000 (07:31 +0100)
committerZorro Lang <zlang@kernel.org>
Fri, 21 Feb 2025 06:38:05 +0000 (14:38 +0800)
Skipping tests just because a file system is broken is always a bad idea
as it hides bugs.

(Fortunately despite hiding the bug this is now getting fixed in XFS as
 well).

Signed-off-by: Christoph Hellwig <hch@lst.de>
Reviewed-by: Zorro Lang <zlang@redhat.com>
Signed-off-by: Zorro Lang <zlang@kernel.org>
tests/generic/370

index 8adffd2e9650a2781fe987eec1472a0c544e1005..cbc18644ab37b34ecaa3b8fb5d5d2d8b98c14e2a 100755 (executable)
@@ -21,14 +21,8 @@ _cleanup()
 
 [ "$FSTYP" = "btrfs" ] && _fixed_by_kernel_commit 03018e5d8508 \
     "btrfs: fix swap file activation failure due to extents that used to be shared"
-
-# Skip XFS for now because this exposes an issue that is hard to fix.
-# See the following threads for details about it:
-#
-# https://lore.kernel.org/linux-xfs/CAL3q7H7cURmnkJfUUx44HM3q=xKmqHb80eRdisErD_x8rU4+0Q@mail.gmail.com/
-# https://lore.kernel.org/fstests/dca49a16a7aacdab831b8895bdecbbb52c0e609c.1733928765.git.fdmanana@suse.com/
-#
-_exclude_fs xfs
+[ "$FSTYP" = "xfs" ] && _fixed_by_kernel_commit XXXXXXXXXXXX \
+       "xfs: flush inodegc before swapon"
 
 _require_scratch_swapfile
 _require_scratch_reflink