summaryrefslogtreecommitdiffstats
path: root/fs/xfs/xfs_refcount_item.c
diff options
context:
space:
mode:
authorBrian Foster <bfoster@redhat.com>2018-07-24 22:43:13 +0200
committerDarrick J. Wong <darrick.wong@oracle.com>2018-07-26 19:15:15 +0200
commit91ef75b6572498face47746c253926e733a4da3b (patch)
treeced5fa1936991dd3a718d0ef4176f9d4e9fd3eca /fs/xfs/xfs_refcount_item.c
parentxfs: use internal dfops in attr code (diff)
downloadlinux-91ef75b6572498face47746c253926e733a4da3b.tar.xz
linux-91ef75b6572498face47746c253926e733a4da3b.zip
xfs: use internal dfops during [b|c]ui recovery
bmap and refcount intent processing associates a dfops from the caller with a local transaction to collect all deferred items for post-processing. Use the internal dfops in both of these functions and move the deferred items to the parent dfops before the transaction commits. Signed-off-by: Brian Foster <bfoster@redhat.com> Reviewed-by: Bill O'Donnell <billodo@redhat.com> Reviewed-by: Christoph Hellwig <hch@lst.de> Reviewed-by: Darrick J. Wong <darrick.wong@oracle.com> Signed-off-by: Darrick J. Wong <darrick.wong@oracle.com>
Diffstat (limited to 'fs/xfs/xfs_refcount_item.c')
-rw-r--r--fs/xfs/xfs_refcount_item.c30
1 files changed, 16 insertions, 14 deletions
diff --git a/fs/xfs/xfs_refcount_item.c b/fs/xfs/xfs_refcount_item.c
index 2064c689bc72..d3582a06626f 100644
--- a/fs/xfs/xfs_refcount_item.c
+++ b/fs/xfs/xfs_refcount_item.c
@@ -452,7 +452,12 @@ xfs_cui_recover(
mp->m_refc_maxlevels * 2, 0, XFS_TRANS_RESERVE, &tp);
if (error)
return error;
- tp->t_dfops = dfops;
+ /*
+ * Recovery stashes all deferred ops during intent processing and
+ * finishes them on completion. Transfer current dfops state to this
+ * transaction and transfer the result back before we return.
+ */
+ xfs_defer_move(tp->t_dfops, dfops);
cudp = xfs_trans_get_cud(tp, cuip);
for (i = 0; i < cuip->cui_format.cui_nextents; i++) {
@@ -474,8 +479,8 @@ xfs_cui_recover(
new_len = refc->pe_len;
} else
error = xfs_trans_log_finish_refcount_update(tp, cudp,
- dfops, type, refc->pe_startblock, refc->pe_len,
- &new_fsb, &new_len, &rcur);
+ tp->t_dfops, type, refc->pe_startblock,
+ refc->pe_len, &new_fsb, &new_len, &rcur);
if (error)
goto abort_error;
@@ -486,21 +491,23 @@ xfs_cui_recover(
switch (type) {
case XFS_REFCOUNT_INCREASE:
error = xfs_refcount_increase_extent(
- tp->t_mountp, dfops, &irec);
+ tp->t_mountp, tp->t_dfops,
+ &irec);
break;
case XFS_REFCOUNT_DECREASE:
error = xfs_refcount_decrease_extent(
- tp->t_mountp, dfops, &irec);
+ tp->t_mountp, tp->t_dfops,
+ &irec);
break;
case XFS_REFCOUNT_ALLOC_COW:
error = xfs_refcount_alloc_cow_extent(
- tp->t_mountp, dfops,
+ tp->t_mountp, tp->t_dfops,
irec.br_startblock,
irec.br_blockcount);
break;
case XFS_REFCOUNT_FREE_COW:
error = xfs_refcount_free_cow_extent(
- tp->t_mountp, dfops,
+ tp->t_mountp, tp->t_dfops,
irec.br_startblock,
irec.br_blockcount);
break;
@@ -515,18 +522,13 @@ xfs_cui_recover(
xfs_refcount_finish_one_cleanup(tp, rcur, error);
set_bit(XFS_CUI_RECOVERED, &cuip->cui_flags);
- /*
- * Recovery finishes all deferred ops once intent processing is
- * complete. Reset the trans reference because commit expects a finished
- * dfops or none at all.
- */
- tp->t_dfops = NULL;
+ xfs_defer_move(dfops, tp->t_dfops);
error = xfs_trans_commit(tp);
return error;
abort_error:
xfs_refcount_finish_one_cleanup(tp, rcur, error);
- tp->t_dfops = NULL;
+ xfs_defer_move(dfops, tp->t_dfops);
xfs_trans_cancel(tp);
return error;
}