summaryrefslogtreecommitdiff
path: root/fs/xfs/xfs_trans_rmap.c
diff options
context:
space:
mode:
authorDarrick J. Wong <darrick.wong@oracle.com>2016-08-03 12:28:43 +1000
committerDave Chinner <david@fromorbit.com>2016-08-03 12:28:43 +1000
commit722e251770306ee325151b28e40b5d7e5497d687 (patch)
treee08729ba5724bf31c9b6df0331bbb0359ab5ecc1 /fs/xfs/xfs_trans_rmap.c
parentc1d22ae89cf6086d6a457b3b9241fcb36ebddd14 (diff)
downloadlinux-722e251770306ee325151b28e40b5d7e5497d687.tar.gz
xfs: remove the extents array from the rmap update done log item
Nothing ever uses the extent array in the rmap update done redo item, so remove it before it is fixed in the on-disk log format. Signed-off-by: Darrick J. Wong <darrick.wong@oracle.com> Reviewed-by: Dave Chinner <dchinner@redhat.com> Signed-off-by: Dave Chinner <david@fromorbit.com>
Diffstat (limited to 'fs/xfs/xfs_trans_rmap.c')
-rw-r--r--fs/xfs/xfs_trans_rmap.c33
1 files changed, 3 insertions, 30 deletions
diff --git a/fs/xfs/xfs_trans_rmap.c b/fs/xfs/xfs_trans_rmap.c
index 83414764e1ab..35650d6b2606 100644
--- a/fs/xfs/xfs_trans_rmap.c
+++ b/fs/xfs/xfs_trans_rmap.c
@@ -129,29 +129,14 @@ xfs_trans_log_start_rmap_update(
xfs_trans_set_rmap_flags(rmap, type, whichfork, state);
}
-/*
- * This routine is called to allocate an "rmap update done"
- * log item that will hold nextents worth of extents. The
- * caller must use all nextents extents, because we are not
- * flexible about this at all.
- */
struct xfs_rud_log_item *
xfs_trans_get_rud(
struct xfs_trans *tp,
- struct xfs_rui_log_item *ruip,
- uint nextents)
+ struct xfs_rui_log_item *ruip)
{
struct xfs_rud_log_item *rudp;
- ASSERT(tp != NULL);
- ASSERT(nextents > 0);
-
- rudp = xfs_rud_init(tp->t_mountp, ruip, nextents);
- ASSERT(rudp != NULL);
-
- /*
- * Get a log_item_desc to point at the new item.
- */
+ rudp = xfs_rud_init(tp->t_mountp, ruip);
xfs_trans_add_item(tp, &rudp->rud_item);
return rudp;
}
@@ -174,8 +159,6 @@ xfs_trans_log_finish_rmap_update(
xfs_exntst_t state,
struct xfs_btree_cur **pcur)
{
- uint next_extent;
- struct xfs_map_extent *rmap;
int error;
error = xfs_rmap_finish_one(tp, type, owner, whichfork, startoff,
@@ -191,16 +174,6 @@ xfs_trans_log_finish_rmap_update(
tp->t_flags |= XFS_TRANS_DIRTY;
rudp->rud_item.li_desc->lid_flags |= XFS_LID_DIRTY;
- next_extent = rudp->rud_next_extent;
- ASSERT(next_extent < rudp->rud_format.rud_nextents);
- rmap = &(rudp->rud_format.rud_extents[next_extent]);
- rmap->me_owner = owner;
- rmap->me_startblock = startblock;
- rmap->me_startoff = startoff;
- rmap->me_len = blockcount;
- xfs_trans_set_rmap_flags(rmap, type, whichfork, state);
- rudp->rud_next_extent++;
-
return error;
}
@@ -255,7 +228,7 @@ xfs_rmap_update_create_done(
void *intent,
unsigned int count)
{
- return xfs_trans_get_rud(tp, intent, count);
+ return xfs_trans_get_rud(tp, intent);
}
/* Process a deferred rmap update. */