]> git-server-git.apps.pok.os.sepia.ceph.com Git - xfstests-dev.git/commit
xfs/018: remove inline xattr recovery tests
authorDarrick J. Wong <djwong@kernel.org>
Fri, 6 Feb 2026 21:54:00 +0000 (13:54 -0800)
committerZorro Lang <zlang@kernel.org>
Sun, 8 Feb 2026 17:30:04 +0000 (01:30 +0800)
commit98dd6070d7eb5b037501345a1da5503daaab1697
tree8678e43e266e8aeaa24beac1e47257c5caefb50c
parent163dbbfaacf0b6d1c62e9363546356e198644f45
xfs/018: remove inline xattr recovery tests

In Linux 7.0 we've changed the extended attribute update code to try to
take a shortcut for performance reasons.  Before walking through the
attr intent state machine (slow), the update will check to see if the
attr structure is in short format and will stay in that format after the
change.  If so, then the incore inode can be updated and logged, and
the update is complete (fast) in a single transaction.

(Obviously, for complex attr structures or large changes we still walk
through the intent machinery.)

However, xfs/018 tests the behavior of the "larp" error injector, which
only triggers from inside the attr intent state machine.  Therefore, the
short format tests don't actually trip the injector.  It makes no sense
to add a new larp injection callsite for the shortcut because either the
single transaction gets written to disk or it doesn't.

The golden output no longer matches because the attr update doesn't
return EIO and shut down the filesystem due to the larp injection.

Signed-off-by: Darrick J. Wong <djwong@kernel.org>
Reviewed-by: Christoph Hellwig <hch@lst.de>
Reviewed-by: Carlos Maiolino <cmaiolino@redhat.com>
Reviewed-by: Zorro Lang <zlang@redhat.com>
Signed-off-by: Zorro Lang <zlang@kernel.org>
tests/xfs/018
tests/xfs/018.out