xfs: finish dfops on every insert range shift iteration
authorBrian Foster <bfoster@redhat.com>
Tue, 18 Aug 2020 15:05:58 +0000 (08:05 -0700)
committerDarrick J. Wong <darrick.wong@oracle.com>
Wed, 26 Aug 2020 21:13:21 +0000 (14:13 -0700)
commit9c516e0e4554e8f26ab73d46cbc789d7d8db664d
treed7b26532bc2090709268c3e0a09bf6876065925a
parent9123e3a74ec7b934a4a099e98af6a61c2f80bbf5
xfs: finish dfops on every insert range shift iteration

The recent change to make insert range an atomic operation used the
incorrect transaction rolling mechanism. The explicit transaction
roll does not finish deferred operations. This means that intents
for rmapbt updates caused by extent shifts are not logged until the
final transaction commits. Thus if a crash occurs during an insert
range, log recovery might leave the rmapbt in an inconsistent state.
This was discovered by repeated runs of generic/455.

Update insert range to finish dfops on every shift iteration. This
is similar to collapse range and ensures that intents are logged
with the transactions that make associated changes.

Fixes: dd87f87d87fa ("xfs: rework insert range into an atomic operation")
Signed-off-by: Brian Foster <bfoster@redhat.com>
Reviewed-by: Darrick J. Wong <darrick.wong@oracle.com>
Signed-off-by: Darrick J. Wong <darrick.wong@oracle.com>
fs/xfs/xfs_bmap_util.c