From 04d0cf3f8909e194acea73badf9914727e486ecc Mon Sep 17 00:00:00 2001 From: Christoph Hellwig Date: Mon, 17 Feb 2025 07:31:35 +0100 Subject: [PATCH] generic/370: don't exclude XFS 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 Reviewed-by: Zorro Lang Signed-off-by: Zorro Lang --- tests/generic/370 | 10 ++-------- 1 file changed, 2 insertions(+), 8 deletions(-) diff --git a/tests/generic/370 b/tests/generic/370 index 8adffd2e9..cbc18644a 100755 --- a/tests/generic/370 +++ b/tests/generic/370 @@ -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 -- 2.49.0