On Mon, Feb 17, 2025 at 07:31:35AM +0100, Christoph Hellwig wrote: > 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@xxxxxx> > --- > tests/generic/370 | 8 -------- > 1 file changed, 8 deletions(-) > > diff --git a/tests/generic/370 b/tests/generic/370 > index 8adffd2e9650..8101ef42724e 100755 > --- a/tests/generic/370 > +++ b/tests/generic/370 > @@ -22,14 +22,6 @@ _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@xxxxxxxxxxxxxx/ > -# https://lore.kernel.org/fstests/dca49a16a7aacdab831b8895bdecbbb52c0e609c.1733928765.git.fdmanana@xxxxxxxx/ > -# > -_exclude_fs xfs Needs a _fixed_by_xfs_commit XXXXXXXX "xfs: flush inodegc before swapon" --D > - > _require_scratch_swapfile > _require_scratch_reflink > _require_cp_reflink > -- > 2.45.2 > >
