]> www.infradead.org Git - users/hch/xfsprogs.git/commitdiff
xfs: create specialized classes for refcount tracepoints
authorDarrick J. Wong <djwong@kernel.org>
Wed, 3 Jul 2024 21:21:41 +0000 (14:21 -0700)
committerDarrick J. Wong <djwong@kernel.org>
Wed, 31 Jul 2024 01:46:44 +0000 (18:46 -0700)
The only user of the "ag" tracepoint event classes is the refcount
btree, so rename them to make that obvious and make them take the btree
cursor to simplify the arguments.  This will save us a lot of trouble
later on.

Signed-off-by: Darrick J. Wong <djwong@kernel.org>
libxfs/xfs_refcount.c

index c78d42728fc02933cd2d88f3cb9892ea9f30e875..4143aca5f7c94a457579dc77e0ceb79ce90bb6c9 100644 (file)
@@ -50,7 +50,7 @@ xfs_refcount_lookup_le(
        xfs_agblock_t           bno,
        int                     *stat)
 {
-       trace_xfs_refcount_lookup(cur->bc_mp, cur->bc_ag.pag->pag_agno,
+       trace_xfs_refcount_lookup(cur,
                        xfs_refcount_encode_startblock(bno, domain),
                        XFS_LOOKUP_LE);
        cur->bc_rec.rc.rc_startblock = bno;
@@ -70,7 +70,7 @@ xfs_refcount_lookup_ge(
        xfs_agblock_t           bno,
        int                     *stat)
 {
-       trace_xfs_refcount_lookup(cur->bc_mp, cur->bc_ag.pag->pag_agno,
+       trace_xfs_refcount_lookup(cur,
                        xfs_refcount_encode_startblock(bno, domain),
                        XFS_LOOKUP_GE);
        cur->bc_rec.rc.rc_startblock = bno;
@@ -90,7 +90,7 @@ xfs_refcount_lookup_eq(
        xfs_agblock_t           bno,
        int                     *stat)
 {
-       trace_xfs_refcount_lookup(cur->bc_mp, cur->bc_ag.pag->pag_agno,
+       trace_xfs_refcount_lookup(cur,
                        xfs_refcount_encode_startblock(bno, domain),
                        XFS_LOOKUP_LE);
        cur->bc_rec.rc.rc_startblock = bno;
@@ -1261,11 +1261,9 @@ xfs_refcount_adjust(
        int                     error;
 
        if (adj == XFS_REFCOUNT_ADJUST_INCREASE)
-               trace_xfs_refcount_increase(cur->bc_mp,
-                               cur->bc_ag.pag->pag_agno, *agbno, *aglen);
+               trace_xfs_refcount_increase(cur, *agbno, *aglen);
        else
-               trace_xfs_refcount_decrease(cur->bc_mp,
-                               cur->bc_ag.pag->pag_agno, *agbno, *aglen);
+               trace_xfs_refcount_decrease(cur, *agbno, *aglen);
 
        /*
         * Ensure that no rcextents cross the boundary of the adjustment range.
@@ -1525,8 +1523,7 @@ xfs_refcount_find_shared(
        int                             have;
        int                             error;
 
-       trace_xfs_refcount_find_shared(cur->bc_mp, cur->bc_ag.pag->pag_agno,
-                       agbno, aglen);
+       trace_xfs_refcount_find_shared(cur, agbno, aglen);
 
        /* By default, skip the whole range */
        *fbno = NULLAGBLOCK;
@@ -1613,8 +1610,7 @@ xfs_refcount_find_shared(
        }
 
 done:
-       trace_xfs_refcount_find_shared_result(cur->bc_mp,
-                       cur->bc_ag.pag->pag_agno, *fbno, *flen);
+       trace_xfs_refcount_find_shared_result(cur, *fbno, *flen);
 
 out_error:
        if (error)
@@ -1832,8 +1828,7 @@ __xfs_refcount_cow_alloc(
        xfs_agblock_t           agbno,
        xfs_extlen_t            aglen)
 {
-       trace_xfs_refcount_cow_increase(rcur->bc_mp, rcur->bc_ag.pag->pag_agno,
-                       agbno, aglen);
+       trace_xfs_refcount_cow_increase(rcur, agbno, aglen);
 
        /* Add refcount btree reservation */
        return xfs_refcount_adjust_cow(rcur, agbno, aglen,
@@ -1849,8 +1844,7 @@ __xfs_refcount_cow_free(
        xfs_agblock_t           agbno,
        xfs_extlen_t            aglen)
 {
-       trace_xfs_refcount_cow_decrease(rcur->bc_mp, rcur->bc_ag.pag->pag_agno,
-                       agbno, aglen);
+       trace_xfs_refcount_cow_decrease(rcur, agbno, aglen);
 
        /* Remove refcount btree reservation */
        return xfs_refcount_adjust_cow(rcur, agbno, aglen,