xfs: move (and rename) the deferred bmap-free tracepoints
authorDarrick J. Wong <darrick.wong@oracle.com>
Wed, 3 Aug 2016 02:31:07 +0000 (12:31 +1000)
committerDave Chinner <david@fromorbit.com>
Wed, 3 Aug 2016 02:31:07 +0000 (12:31 +1000)
Rename the deferred bmap-free to extent_free and make them only
trigger when we're really running deferred ops.

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>
fs/xfs/libxfs/xfs_alloc.c
fs/xfs/xfs_trans_extfree.c

index a0dc299..776ae2f 100644 (file)
@@ -2819,8 +2819,6 @@ xfs_free_extent(
 
        ASSERT(len != 0);
 
-       trace_xfs_bmap_free_deferred(mp, agno, 0, agbno, len);
-
        if (XFS_TEST_ERROR(false, mp,
                        XFS_ERRTAG_FREE_EXTENT,
                        XFS_RANDOM_FREE_EXTENT))
index 7d1e84c..459ddec 100644 (file)
@@ -21,6 +21,7 @@
 #include "xfs_format.h"
 #include "xfs_log_format.h"
 #include "xfs_trans_resv.h"
+#include "xfs_bit.h"
 #include "xfs_mount.h"
 #include "xfs_defer.h"
 #include "xfs_trans.h"
@@ -28,6 +29,7 @@
 #include "xfs_extfree_item.h"
 #include "xfs_alloc.h"
 #include "xfs_bmap.h"
+#include "xfs_trace.h"
 
 /*
  * This routine is called to allocate an "extent free done"
@@ -68,10 +70,15 @@ xfs_trans_free_extent(
        xfs_extlen_t            ext_len,
        struct xfs_owner_info   *oinfo)
 {
+       struct xfs_mount        *mp = tp->t_mountp;
        uint                    next_extent;
+       xfs_agnumber_t          agno = XFS_FSB_TO_AGNO(mp, start_block);
+       xfs_agblock_t           agbno = XFS_FSB_TO_AGBNO(mp, start_block);
        struct xfs_extent       *extp;
        int                     error;
 
+       trace_xfs_bmap_free_deferred(tp->t_mountp, agno, 0, agbno, ext_len);
+
        error = xfs_free_extent(tp, start_block, ext_len, oinfo);
 
        /*