xfs: move pre/post-bmap tracing into xfs_iext_update_extent

Source kernel commit: ca5d8e5b7b9030005e38e7c43e08c0cd4eb2a78f

xfs_iext_update_extent already has basically all the information needed
to centralize the bmap pre/post tracing.  We just need to pass inode +
bmap state instead of the inode fork pointer to get all trace annotations.

In addition to covering all the existing trace points this gives us
tracing coverage for the extent shifting operations for free.

Signed-off-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>
Signed-off-by: Eric Sandeen <sandeen@sandeen.net>

diff --git a/libxfs/xfs_bmap.c b/libxfs/xfs_bmap.c
index 57728f9c739998a632762a9fb3af9a2f2cfafc64..cb3dc85943bf88baa9e111c5a6a8e4f67b8a9700 100644 (file)
--- a/libxfs/xfs_bmap.c
+++ b/libxfs/xfs_bmap.c
@@ -1660,10 +1660,8 @@ xfs_bmap_add_extent_delay_real(
                 * The left and right neighbors are both contiguous with new.
                 */
                bma->idx--;
-               trace_xfs_bmap_pre_update(bma->ip, bma->idx, state, _THIS_IP_);
                LEFT.br_blockcount += PREV.br_blockcount + RIGHT.br_blockcount;
-               xfs_iext_update_extent(ifp, bma->idx, &LEFT);
-               trace_xfs_bmap_post_update(bma->ip, bma->idx, state, _THIS_IP_);
+               xfs_iext_update_extent(bma->ip, state, bma->idx, &LEFT);
 
                xfs_iext_remove(bma->ip, bma->idx + 1, 2, state);
                (*nextents)--;
@@ -1697,10 +1695,8 @@ xfs_bmap_add_extent_delay_real(
                bma->idx--;
 
                old = LEFT;
-               trace_xfs_bmap_pre_update(bma->ip, bma->idx, state, _THIS_IP_);
                LEFT.br_blockcount += PREV.br_blockcount;
-               xfs_iext_update_extent(ifp, bma->idx, &LEFT);
-               trace_xfs_bmap_post_update(bma->ip, bma->idx, state, _THIS_IP_);
+               xfs_iext_update_extent(bma->ip, state, bma->idx, &LEFT);
 
                xfs_iext_remove(bma->ip, bma->idx + 1, 1, state);
                if (bma->cur == NULL)
@@ -1722,11 +1718,9 @@ xfs_bmap_add_extent_delay_real(
                 * Filling in all of a previously delayed allocation extent.
                 * The right neighbor is contiguous, the left is not.
                 */
-               trace_xfs_bmap_pre_update(bma->ip, bma->idx, state, _THIS_IP_);
                PREV.br_startblock = new->br_startblock;
                PREV.br_blockcount += RIGHT.br_blockcount;
-               xfs_iext_update_extent(ifp, bma->idx, &PREV);
-               trace_xfs_bmap_post_update(bma->ip, bma->idx, state, _THIS_IP_);
+               xfs_iext_update_extent(bma->ip, state, bma->idx, &PREV);
 
                xfs_iext_remove(bma->ip, bma->idx + 1, 1, state);
                if (bma->cur == NULL)
@@ -1749,11 +1743,9 @@ xfs_bmap_add_extent_delay_real(
                 * Neither the left nor right neighbors are contiguous with
                 * the new one.
                 */
-               trace_xfs_bmap_pre_update(bma->ip, bma->idx, state, _THIS_IP_);
                PREV.br_startblock = new->br_startblock;
                PREV.br_state = new->br_state;
-               xfs_iext_update_extent(ifp, bma->idx, &PREV);
-               trace_xfs_bmap_post_update(bma->ip, bma->idx, state, _THIS_IP_);
+               xfs_iext_update_extent(bma->ip, state, bma->idx, &PREV);
 
                (*nextents)++;
                if (bma->cur == NULL)
@@ -1781,17 +1773,13 @@ xfs_bmap_add_extent_delay_real(
                da_new = XFS_FILBLKS_MIN(xfs_bmap_worst_indlen(bma->ip, temp),
                                startblockval(PREV.br_startblock));
 
-               trace_xfs_bmap_pre_update(bma->ip, bma->idx - 1, state, _THIS_IP_);
                LEFT.br_blockcount += new->br_blockcount;
-               xfs_iext_update_extent(ifp, bma->idx - 1, &LEFT);
-               trace_xfs_bmap_post_update(bma->ip, bma->idx - 1, state, _THIS_IP_);
+               xfs_iext_update_extent(bma->ip, state, bma->idx - 1, &LEFT);
 
-               trace_xfs_bmap_pre_update(bma->ip, bma->idx, state, _THIS_IP_);
                PREV.br_blockcount = temp = PREV.br_blockcount - new->br_blockcount;
                PREV.br_startoff += new->br_blockcount;
                PREV.br_startblock = nullstartblock(da_new);
-               xfs_iext_update_extent(ifp, bma->idx, &PREV);
-               trace_xfs_bmap_post_update(bma->ip, bma->idx, state, _THIS_IP_);
+               xfs_iext_update_extent(bma->ip, state, bma->idx, &PREV);
 
                if (bma->cur == NULL)
                        rval = XFS_ILOG_DEXT;
@@ -1844,13 +1832,10 @@ xfs_bmap_add_extent_delay_real(
                        startblockval(PREV.br_startblock) -
                        (bma->cur ? bma->cur->bc_private.b.allocated : 0));
 
-               trace_xfs_bmap_pre_update(bma->ip, bma->idx + 1, state, _THIS_IP_);
                PREV.br_startoff = new_endoff;
                PREV.br_blockcount = temp;
                PREV.br_startblock = nullstartblock(da_new);
-               xfs_iext_update_extent(ifp, bma->idx + 1, &PREV);
-               trace_xfs_bmap_post_update(bma->ip, bma->idx + 1, state, _THIS_IP_);
-
+               xfs_iext_update_extent(bma->ip, state, bma->idx + 1, &PREV);
                break;
 
        case BMAP_RIGHT_FILLING | BMAP_RIGHT_CONTIG:
@@ -1859,12 +1844,10 @@ xfs_bmap_add_extent_delay_real(
                 * The right neighbor is contiguous with the new allocation.
                 */
                old = RIGHT;
-               trace_xfs_bmap_pre_update(bma->ip, bma->idx + 1, state, _THIS_IP_);
                RIGHT.br_startoff = new->br_startoff;
                RIGHT.br_startblock = new->br_startblock;
                RIGHT.br_blockcount += new->br_blockcount;
-               xfs_iext_update_extent(ifp, bma->idx + 1, &RIGHT);
-               trace_xfs_bmap_post_update(bma->ip, bma->idx + 1, state, _THIS_IP_);
+               xfs_iext_update_extent(bma->ip, state, bma->idx + 1, &RIGHT);
 
                if (bma->cur == NULL)
                        rval = XFS_ILOG_DEXT;
@@ -1883,11 +1866,9 @@ xfs_bmap_add_extent_delay_real(
                da_new = XFS_FILBLKS_MIN(xfs_bmap_worst_indlen(bma->ip, temp),
                        startblockval(PREV.br_startblock));
 
-               trace_xfs_bmap_pre_update(bma->ip, bma->idx, state, _THIS_IP_);
                PREV.br_blockcount = temp;
                PREV.br_startblock = nullstartblock(da_new);
-               xfs_iext_update_extent(ifp, bma->idx, &PREV);
-               trace_xfs_bmap_post_update(bma->ip, bma->idx, state, _THIS_IP_);
+               xfs_iext_update_extent(bma->ip, state, bma->idx, &PREV);
 
                bma->idx++;
                break;
@@ -1927,11 +1908,9 @@ xfs_bmap_add_extent_delay_real(
                        startblockval(PREV.br_startblock) -
                        (bma->cur ? bma->cur->bc_private.b.allocated : 0));
 
-               trace_xfs_bmap_pre_update(bma->ip, bma->idx, state, _THIS_IP_);
                PREV.br_startblock = nullstartblock(da_new);
                PREV.br_blockcount = temp;
-               xfs_iext_update_extent(ifp, bma->idx, &PREV);
-               trace_xfs_bmap_post_update(bma->ip, bma->idx, state, _THIS_IP_);
+               xfs_iext_update_extent(bma->ip, state, bma->idx, &PREV);
 
                bma->idx++;
                break;
@@ -1972,13 +1951,11 @@ xfs_bmap_add_extent_delay_real(
                                        RIGHT.br_blockcount));
 
                /* truncate PREV */
-               trace_xfs_bmap_pre_update(bma->ip, bma->idx, 0, _THIS_IP_);
                PREV.br_blockcount = new->br_startoff - PREV.br_startoff;
                PREV.br_startblock =
                        nullstartblock(xfs_bmap_worst_indlen(bma->ip,
                                        PREV.br_blockcount));
-               xfs_iext_update_extent(ifp, bma->idx, &PREV);
-               trace_xfs_bmap_post_update(bma->ip, bma->idx, state, _THIS_IP_);
+               xfs_iext_update_extent(bma->ip, state, bma->idx, &PREV);
 
                /* insert LEFT (r[0]) and RIGHT (r[1]) at the same time */
                xfs_iext_insert(bma->ip, bma->idx + 1, 2, &LEFT, state);
@@ -2183,10 +2160,8 @@ xfs_bmap_add_extent_unwritten_real(
                 */
                --*idx;
 
-               trace_xfs_bmap_pre_update(ip, *idx, state, _THIS_IP_);
                LEFT.br_blockcount += PREV.br_blockcount + RIGHT.br_blockcount;
-               xfs_iext_update_extent(ifp, *idx, &LEFT);
-               trace_xfs_bmap_post_update(ip, *idx, state, _THIS_IP_);
+               xfs_iext_update_extent(ip, state, *idx, &LEFT);
 
                xfs_iext_remove(ip, *idx + 1, 2, state);
                XFS_IFORK_NEXT_SET(ip, whichfork,
@@ -2224,10 +2199,8 @@ xfs_bmap_add_extent_unwritten_real(
                 */
                --*idx;
 
-               trace_xfs_bmap_pre_update(ip, *idx, state, _THIS_IP_);
                LEFT.br_blockcount += PREV.br_blockcount;
-               xfs_iext_update_extent(ifp, *idx, &LEFT);
-               trace_xfs_bmap_post_update(ip, *idx, state, _THIS_IP_);
+               xfs_iext_update_extent(ip, state, *idx, &LEFT);
 
                xfs_iext_remove(ip, *idx + 1, 1, state);
                XFS_IFORK_NEXT_SET(ip, whichfork,
@@ -2257,11 +2230,9 @@ xfs_bmap_add_extent_unwritten_real(
                 * Setting all of a previous oldext extent to newext.
                 * The right neighbor is contiguous, the left is not.
                 */
-               trace_xfs_bmap_pre_update(ip, *idx, state, _THIS_IP_);
                PREV.br_blockcount += RIGHT.br_blockcount;
                PREV.br_state = new->br_state;
-               xfs_iext_update_extent(ifp, *idx, &PREV);
-               trace_xfs_bmap_post_update(ip, *idx, state, _THIS_IP_);
+               xfs_iext_update_extent(ip, state, *idx, &PREV);
 
                xfs_iext_remove(ip, *idx + 1, 1, state);
                XFS_IFORK_NEXT_SET(ip, whichfork,
@@ -2292,10 +2263,8 @@ xfs_bmap_add_extent_unwritten_real(
                 * Neither the left nor right neighbors are contiguous with
                 * the new one.
                 */
-               trace_xfs_bmap_pre_update(ip, *idx, state, _THIS_IP_);
                PREV.br_state = new->br_state;
-               xfs_iext_update_extent(ifp, *idx, &PREV);
-               trace_xfs_bmap_post_update(ip, *idx, state, _THIS_IP_);
+               xfs_iext_update_extent(ip, state, *idx, &PREV);
 
                if (cur == NULL)
                        rval = XFS_ILOG_DEXT;
@@ -2316,18 +2285,14 @@ xfs_bmap_add_extent_unwritten_real(
                 * Setting the first part of a previous oldext extent to newext.
                 * The left neighbor is contiguous.
                 */
-               trace_xfs_bmap_pre_update(ip, *idx - 1, state, _THIS_IP_);
                LEFT.br_blockcount += new->br_blockcount;
-               xfs_iext_update_extent(ifp, *idx - 1, &LEFT);
-               trace_xfs_bmap_post_update(ip, *idx - 1, state, _THIS_IP_);
+               xfs_iext_update_extent(ip, state, *idx - 1, &LEFT);
 
                old = PREV;
-               trace_xfs_bmap_pre_update(ip, *idx, state, _THIS_IP_);
                PREV.br_startoff += new->br_blockcount;
                PREV.br_startblock += new->br_blockcount;
                PREV.br_blockcount -= new->br_blockcount;
-               xfs_iext_update_extent(ifp, *idx, &PREV);
-               trace_xfs_bmap_post_update(ip, *idx, state, _THIS_IP_);
+               xfs_iext_update_extent(ip, state, *idx, &PREV);
 
                --*idx;
 
@@ -2357,12 +2322,10 @@ xfs_bmap_add_extent_unwritten_real(
                 * The left neighbor is not contiguous.
                 */
                old = PREV;
-               trace_xfs_bmap_pre_update(ip, *idx, state, _THIS_IP_);
                PREV.br_startoff += new->br_blockcount;
                PREV.br_startblock += new->br_blockcount;
                PREV.br_blockcount -= new->br_blockcount;
-               xfs_iext_update_extent(ifp, *idx, &PREV);
-               trace_xfs_bmap_post_update(ip, *idx, state, _THIS_IP_);
+               xfs_iext_update_extent(ip, state, *idx, &PREV);
 
                xfs_iext_insert(ip, *idx, 1, new, state);
                XFS_IFORK_NEXT_SET(ip, whichfork,
@@ -2391,19 +2354,15 @@ xfs_bmap_add_extent_unwritten_real(
                 * The right neighbor is contiguous with the new allocation.
                 */
                old = PREV;
-               trace_xfs_bmap_pre_update(ip, *idx, state, _THIS_IP_);
                PREV.br_blockcount -= new->br_blockcount;
-               xfs_iext_update_extent(ifp, *idx, &PREV);
-               trace_xfs_bmap_post_update(ip, *idx, state, _THIS_IP_);
+               xfs_iext_update_extent(ip, state, *idx, &PREV);
 
                ++*idx;
 
-               trace_xfs_bmap_pre_update(ip, *idx, state, _THIS_IP_);
                RIGHT.br_startoff = new->br_startoff;
                RIGHT.br_startblock = new->br_startblock;
                RIGHT.br_blockcount += new->br_blockcount;
-               xfs_iext_update_extent(ifp, *idx, &RIGHT);
-               trace_xfs_bmap_post_update(ip, *idx, state, _THIS_IP_);
+               xfs_iext_update_extent(ip, state, *idx, &RIGHT);
 
                if (cur == NULL)
                        rval = XFS_ILOG_DEXT;
@@ -2431,10 +2390,8 @@ xfs_bmap_add_extent_unwritten_real(
                 * The right neighbor is not contiguous.
                 */
                old = PREV;
-               trace_xfs_bmap_pre_update(ip, *idx, state, _THIS_IP_);
                PREV.br_blockcount -= new->br_blockcount;
-               xfs_iext_update_extent(ifp, *idx, &PREV);
-               trace_xfs_bmap_post_update(ip, *idx, state, _THIS_IP_);
+               xfs_iext_update_extent(ip, state, *idx, &PREV);
 
                ++*idx;
                xfs_iext_insert(ip, *idx, 1, new, state);
@@ -2469,10 +2426,8 @@ xfs_bmap_add_extent_unwritten_real(
                 * One extent becomes three extents.
                 */
                old = PREV;
-               trace_xfs_bmap_pre_update(ip, *idx, state, _THIS_IP_);
                PREV.br_blockcount = new->br_startoff - PREV.br_startoff;
-               xfs_iext_update_extent(ifp, *idx, &PREV);
-               trace_xfs_bmap_post_update(ip, *idx, state, _THIS_IP_);
+               xfs_iext_update_extent(ip, state, *idx, &PREV);
 
                r[0] = *new;
                r[1].br_startoff = new_endoff;
@@ -2639,7 +2594,6 @@ xfs_bmap_add_extent_hole_delay(
                temp = left.br_blockcount + new->br_blockcount +
                        right.br_blockcount;
 
-               trace_xfs_bmap_pre_update(ip, *idx, state, _THIS_IP_);
                oldlen = startblockval(left.br_startblock) +
                        startblockval(new->br_startblock) +
                        startblockval(right.br_startblock);
@@ -2647,8 +2601,7 @@ xfs_bmap_add_extent_hole_delay(
                                         oldlen);
                left.br_startblock = nullstartblock(newlen);
                left.br_blockcount = temp;
-               xfs_iext_update_extent(ifp, *idx, &left);
-               trace_xfs_bmap_post_update(ip, *idx, state, _THIS_IP_);
+               xfs_iext_update_extent(ip, state, *idx, &left);
 
                xfs_iext_remove(ip, *idx + 1, 1, state);
                break;
@@ -2662,15 +2615,13 @@ xfs_bmap_add_extent_hole_delay(
                --*idx;
                temp = left.br_blockcount + new->br_blockcount;
 
-               trace_xfs_bmap_pre_update(ip, *idx, state, _THIS_IP_);
                oldlen = startblockval(left.br_startblock) +
                        startblockval(new->br_startblock);
                newlen = XFS_FILBLKS_MIN(xfs_bmap_worst_indlen(ip, temp),
                                         oldlen);
                left.br_blockcount = temp;
                left.br_startblock = nullstartblock(newlen);
-               xfs_iext_update_extent(ifp, *idx, &left);
-               trace_xfs_bmap_post_update(ip, *idx, state, _THIS_IP_);
+               xfs_iext_update_extent(ip, state, *idx, &left);
                break;
 
        case BMAP_RIGHT_CONTIG:
@@ -2679,7 +2630,6 @@ xfs_bmap_add_extent_hole_delay(
                 * on the right.
                 * Merge the new allocation with the right neighbor.
                 */
-               trace_xfs_bmap_pre_update(ip, *idx, state, _THIS_IP_);
                temp = new->br_blockcount + right.br_blockcount;
                oldlen = startblockval(new->br_startblock) +
                        startblockval(right.br_startblock);
@@ -2688,8 +2638,7 @@ xfs_bmap_add_extent_hole_delay(
                right.br_startoff = new->br_startoff;
                right.br_startblock = nullstartblock(newlen);
                right.br_blockcount = temp;
-               xfs_iext_update_extent(ifp, *idx, &right);
-               trace_xfs_bmap_post_update(ip, *idx, state, _THIS_IP_);
+               xfs_iext_update_extent(ip, state, *idx, &right);
                break;
 
        case 0:
@@ -2799,10 +2748,8 @@ xfs_bmap_add_extent_hole_real(
                 * Merge all three into a single extent record.
                 */
                --*idx;
-               trace_xfs_bmap_pre_update(ip, *idx, state, _THIS_IP_);
                left.br_blockcount += new->br_blockcount + right.br_blockcount;
-               xfs_iext_update_extent(ifp, *idx, &left);
-               trace_xfs_bmap_post_update(ip, *idx, state, _THIS_IP_);
+               xfs_iext_update_extent(ip, state, *idx, &left);
 
                xfs_iext_remove(ip, *idx + 1, 1, state);
 
@@ -2838,10 +2785,9 @@ xfs_bmap_add_extent_hole_real(
                 */
                --*idx;
                old = left;
-               trace_xfs_bmap_pre_update(ip, *idx, state, _THIS_IP_);
+
                left.br_blockcount += new->br_blockcount;
-               xfs_iext_update_extent(ifp, *idx, &left);
-               trace_xfs_bmap_post_update(ip, *idx, state, _THIS_IP_);
+               xfs_iext_update_extent(ip, state, *idx, &left);
 
                if (cur == NULL) {
                        rval = xfs_ilog_fext(whichfork);
@@ -2864,12 +2810,11 @@ xfs_bmap_add_extent_hole_real(
                 * Merge the new allocation with the right neighbor.
                 */
                old = right;
-               trace_xfs_bmap_pre_update(ip, *idx, state, _THIS_IP_);
+
                right.br_startoff = new->br_startoff;
                right.br_startblock = new->br_startblock;
                right.br_blockcount += new->br_blockcount;
-               xfs_iext_update_extent(ifp, *idx, &right);
-               trace_xfs_bmap_post_update(ip, *idx, state, _THIS_IP_);
+               xfs_iext_update_extent(ip, state, *idx, &right);
 
                if (cur == NULL) {
                        rval = xfs_ilog_fext(whichfork);
@@ -4771,26 +4716,22 @@ xfs_bmap_del_extent_delay(
                /*
                 * Deleting the first part of the extent.
                 */
-               trace_xfs_bmap_pre_update(ip, *idx, state, _THIS_IP_);
                got->br_startoff = del_endoff;
                got->br_blockcount -= del->br_blockcount;
                da_new = XFS_FILBLKS_MIN(xfs_bmap_worst_indlen(ip,
                                got->br_blockcount), da_old);
                got->br_startblock = nullstartblock((int)da_new);
-               xfs_iext_update_extent(ifp, *idx, got);
-               trace_xfs_bmap_post_update(ip, *idx, state, _THIS_IP_);
+               xfs_iext_update_extent(ip, state, *idx, got);
                break;
        case BMAP_RIGHT_FILLING:
                /*
                 * Deleting the last part of the extent.
                 */
-               trace_xfs_bmap_pre_update(ip, *idx, state, _THIS_IP_);
                got->br_blockcount = got->br_blockcount - del->br_blockcount;
                da_new = XFS_FILBLKS_MIN(xfs_bmap_worst_indlen(ip,
                                got->br_blockcount), da_old);
                got->br_startblock = nullstartblock((int)da_new);
-               xfs_iext_update_extent(ifp, *idx, got);
-               trace_xfs_bmap_post_update(ip, *idx, state, _THIS_IP_);
+               xfs_iext_update_extent(ip, state, *idx, got);
                break;
        case 0:
                /*
@@ -4802,8 +4743,6 @@ xfs_bmap_del_extent_delay(
                 * Warn if either of the new indlen reservations is zero as this
                 * can lead to delalloc problems.
                 */
-               trace_xfs_bmap_pre_update(ip, *idx, state, _THIS_IP_);
-
                got->br_blockcount = del->br_startoff - got->br_startoff;
                got_indlen = xfs_bmap_worst_indlen(ip, got->br_blockcount);
 
@@ -4815,8 +4754,7 @@ xfs_bmap_del_extent_delay(
                                                       del->br_blockcount);
 
                got->br_startblock = nullstartblock((int)got_indlen);
-               xfs_iext_update_extent(ifp, *idx, got);
-               trace_xfs_bmap_post_update(ip, *idx, 0, _THIS_IP_);
+               xfs_iext_update_extent(ip, state, *idx, got);
 
                new.br_startoff = del_endoff;
                new.br_state = got->br_state;
@@ -4881,30 +4819,24 @@ xfs_bmap_del_extent_cow(
                /*
                 * Deleting the first part of the extent.
                 */
-               trace_xfs_bmap_pre_update(ip, *idx, state, _THIS_IP_);
                got->br_startoff = del_endoff;
                got->br_blockcount -= del->br_blockcount;
                got->br_startblock = del->br_startblock + del->br_blockcount;
-               xfs_iext_update_extent(ifp, *idx, got);
-               trace_xfs_bmap_post_update(ip, *idx, state, _THIS_IP_);
+               xfs_iext_update_extent(ip, state, *idx, got);
                break;
        case BMAP_RIGHT_FILLING:
                /*
                 * Deleting the last part of the extent.
                 */
-               trace_xfs_bmap_pre_update(ip, *idx, state, _THIS_IP_);
                got->br_blockcount -= del->br_blockcount;
-               xfs_iext_update_extent(ifp, *idx, got);
-               trace_xfs_bmap_post_update(ip, *idx, state, _THIS_IP_);
+               xfs_iext_update_extent(ip, state, *idx, got);
                break;
        case 0:
                /*
                 * Deleting the middle of the extent.
                 */
-               trace_xfs_bmap_pre_update(ip, *idx, state, _THIS_IP_);
                got->br_blockcount = del->br_startoff - got->br_startoff;
-               xfs_iext_update_extent(ifp, *idx, got);
-               trace_xfs_bmap_post_update(ip, *idx, state, _THIS_IP_);
+               xfs_iext_update_extent(ip, state, *idx, got);
 
                new.br_startoff = del_endoff;
                new.br_blockcount = got_endoff - del_endoff;
@@ -5021,7 +4953,6 @@ xfs_bmap_del_extent_real(
                /*
                 * Matches the whole extent.  Delete the entry.
                 */
-               trace_xfs_bmap_pre_update(ip, *idx, state, _THIS_IP_);
                xfs_iext_remove(ip, *idx, 1, state);
                --*idx;
 
@@ -5040,12 +4971,10 @@ xfs_bmap_del_extent_real(
                /*
                 * Deleting the first part of the extent.
                 */
-               trace_xfs_bmap_pre_update(ip, *idx, state, _THIS_IP_);
                got.br_startoff = del_endoff;
                got.br_startblock = del_endblock;
                got.br_blockcount -= del->br_blockcount;
-               xfs_iext_update_extent(ifp, *idx, &got);
-               trace_xfs_bmap_post_update(ip, *idx, state, _THIS_IP_);
+               xfs_iext_update_extent(ip, state, *idx, &got);
                if (!cur) {
                        flags |= xfs_ilog_fext(whichfork);
                        break;
@@ -5058,10 +4987,8 @@ xfs_bmap_del_extent_real(
                /*
                 * Deleting the last part of the extent.
                 */
-               trace_xfs_bmap_pre_update(ip, *idx, state, _THIS_IP_);
                got.br_blockcount -= del->br_blockcount;
-               xfs_iext_update_extent(ifp, *idx, &got);
-               trace_xfs_bmap_post_update(ip, *idx, state, _THIS_IP_);
+               xfs_iext_update_extent(ip, state, *idx, &got);
                if (!cur) {
                        flags |= xfs_ilog_fext(whichfork);
                        break;
@@ -5074,11 +5001,10 @@ xfs_bmap_del_extent_real(
                /*
                 * Deleting the middle of the extent.
                 */
-               trace_xfs_bmap_pre_update(ip, *idx, state, _THIS_IP_);
-
                old = got;
+
                got.br_blockcount = del->br_startoff - got.br_startoff;
-               xfs_iext_update_extent(ifp, *idx, &got);
+               xfs_iext_update_extent(ip, state, *idx, &got);
 
                new.br_startoff = del_endoff;
                new.br_blockcount = got_endoff - del_endoff;
@@ -5122,7 +5048,7 @@ xfs_bmap_del_extent_real(
                                 * Reset the extent record back
                                 * to the original value.
                                 */
-                               xfs_iext_update_extent(ifp, *idx, &old);
+                               xfs_iext_update_extent(ip, state, *idx, &old);
                                flags = 0;
                                error = -ENOSPC;
                                goto done;
@@ -5132,7 +5058,6 @@ xfs_bmap_del_extent_real(
                        flags |= xfs_ilog_fext(whichfork);
                XFS_IFORK_NEXT_SET(ip, whichfork,
                        XFS_IFORK_NEXTENTS(ip, whichfork) + 1);
-               trace_xfs_bmap_post_update(ip, *idx, state, _THIS_IP_);
                xfs_iext_insert(ip, *idx + 1, 1, &new, state);
                ++*idx;
                break;
@@ -5607,7 +5532,6 @@ xfs_bmse_merge(
        int                             *logflags,      /* output */
        struct xfs_defer_ops            *dfops)
 {
-       struct xfs_ifork                *ifp = XFS_IFORK_PTR(ip, whichfork);
        struct xfs_bmbt_irec            new;
        xfs_filblks_t                   blockcount;
        int                             error, i;
@@ -5656,7 +5580,8 @@ xfs_bmse_merge(
                return error;
 
 done:
-       xfs_iext_update_extent(ifp, current_ext - 1, &new);
+       xfs_iext_update_extent(ip, xfs_bmap_fork_to_state(whichfork),
+                       current_ext - 1, &new);
        xfs_iext_remove(ip, current_ext, 1, 0);
 
        /* update reverse mapping. rmap functions merge the rmaps for us */
@@ -5774,7 +5699,8 @@ update_current_ext:
                *logflags |= XFS_ILOG_DEXT;
        }
 
-       xfs_iext_update_extent(ifp, *current_ext, &new);
+       xfs_iext_update_extent(ip, xfs_bmap_fork_to_state(whichfork),
+                       *current_ext, &new);
 
        if (direction == SHIFT_LEFT)
                (*current_ext)++;
@@ -6021,7 +5947,8 @@ xfs_bmap_split_extent_at(
        }
 
        got.br_blockcount = gotblkcnt;
-       xfs_iext_update_extent(ifp, current_ext, &got);
+       xfs_iext_update_extent(ip, xfs_bmap_fork_to_state(whichfork),
+                       current_ext, &got);
 
        logflags = XFS_ILOG_CORE;
        if (cur) {

diff --git a/libxfs/xfs_inode_fork.c b/libxfs/xfs_inode_fork.c
index 0c695cb5b3513bf28ee39f0f5b452eb9a5c5ab94..f4fa3250281f30ff60981fcaf3c7075fc8ed997b 100644 (file)
--- a/libxfs/xfs_inode_fork.c
+++ b/libxfs/xfs_inode_fork.c
@@ -2022,12 +2022,17 @@ xfs_iext_get_extent(
 
 void
 xfs_iext_update_extent(
-       struct xfs_ifork        *ifp,
+       struct xfs_inode        *ip,
+       int                     state,
        xfs_extnum_t            idx,
        struct xfs_bmbt_irec    *gotp)
 {
+       struct xfs_ifork        *ifp = xfs_iext_state_to_fork(ip, state);
+
        ASSERT(idx >= 0);
        ASSERT(idx < xfs_iext_count(ifp));
 
+       trace_xfs_bmap_pre_update(ip, idx, state, _RET_IP_);
        xfs_bmbt_set_all(xfs_iext_get_ext(ifp, idx), gotp);
+       trace_xfs_bmap_post_update(ip, idx, state, _RET_IP_);
 }

diff --git a/libxfs/xfs_inode_fork.h b/libxfs/xfs_inode_fork.h
index 11af705219f6d7dacace149dd8d6b00f2a794ef3..6750f0462d2129d9aa4f96a71c6c7da47e3dacba 100644 (file)
--- a/libxfs/xfs_inode_fork.h
+++ b/libxfs/xfs_inode_fork.h
@@ -187,8 +187,8 @@ bool                xfs_iext_lookup_extent(struct xfs_inode *ip,
                        xfs_extnum_t *idxp, struct xfs_bmbt_irec *gotp);
 bool           xfs_iext_get_extent(struct xfs_ifork *ifp, xfs_extnum_t idx,
                        struct xfs_bmbt_irec *gotp);
-void           xfs_iext_update_extent(struct xfs_ifork *ifp, xfs_extnum_t idx,
-                       struct xfs_bmbt_irec *gotp);
+void           xfs_iext_update_extent(struct xfs_inode *ip, int state,
+                       xfs_extnum_t idx, struct xfs_bmbt_irec *gotp);
 
 extern struct kmem_zone        *xfs_ifork_zone;