From fea60f70c810e93dce85470d94432afb65e7234f Mon Sep 17 00:00:00 2001 From: "Darrick J. Wong" Date: Tue, 1 Oct 2024 18:18:14 -0700 Subject: [PATCH] xfs: move xfs_extent_free_defer_add to xfs_extfree_item.c Source kernel commit: 84a3c1576c5aade32170fae6c61d51bd2d16010f Move the code that adds the incore xfs_extent_free_item deferred work data to a transaction to live with the EFI log item code. This means that the allocator code no longer has to know about the inner workings of the EFI log items. As a consequence, we can get rid of the _{get,put}_group helpers. Signed-off-by: Darrick J. Wong Reviewed-by: Christoph Hellwig --- libxfs/defer_item.c | 28 +++++++++++++++------------- libxfs/defer_item.h | 7 +++++++ libxfs/xfs_alloc.c | 12 ++---------- libxfs/xfs_alloc.h | 3 --- 4 files changed, 24 insertions(+), 26 deletions(-) diff --git a/libxfs/defer_item.c b/libxfs/defer_item.c index dd88e75e..2df0ce4e 100644 --- a/libxfs/defer_item.c +++ b/libxfs/defer_item.c @@ -27,6 +27,7 @@ #include "defer_item.h" #include "xfs_ag.h" #include "xfs_exchmaps.h" +#include "defer_item.h" /* Dummy defer item ops, since we don't do logging. */ @@ -75,21 +76,22 @@ xfs_extent_free_create_done( return NULL; } -/* Take an active ref to the AG containing the space we're freeing. */ +/* Add this deferred EFI to the transaction. */ void -xfs_extent_free_get_group( - struct xfs_mount *mp, - struct xfs_extent_free_item *xefi) +xfs_extent_free_defer_add( + struct xfs_trans *tp, + struct xfs_extent_free_item *xefi, + struct xfs_defer_pending **dfpp) { - xefi->xefi_pag = xfs_perag_intent_get(mp, xefi->xefi_startblock); -} + struct xfs_mount *mp = tp->t_mountp; -/* Release an active AG ref after some freeing work. */ -static inline void -xfs_extent_free_put_group( - struct xfs_extent_free_item *xefi) -{ - xfs_perag_intent_put(xefi->xefi_pag); + xefi->xefi_pag = xfs_perag_intent_get(mp, xefi->xefi_startblock); + if (xefi->xefi_agresv == XFS_AG_RESV_AGFL) + *dfpp = xfs_defer_add(tp, &xefi->xefi_list, + &xfs_agfl_free_defer_type); + else + *dfpp = xfs_defer_add(tp, &xefi->xefi_list, + &xfs_extent_free_defer_type); } /* Cancel a free extent. */ @@ -99,7 +101,7 @@ xfs_extent_free_cancel_item( { struct xfs_extent_free_item *xefi = xefi_entry(item); - xfs_extent_free_put_group(xefi); + xfs_perag_intent_put(xefi->xefi_pag); kmem_cache_free(xfs_extfree_item_cache, xefi); } diff --git a/libxfs/defer_item.h b/libxfs/defer_item.h index df2b8d68..03f3f150 100644 --- a/libxfs/defer_item.h +++ b/libxfs/defer_item.h @@ -23,4 +23,11 @@ struct xfs_exchmaps_intent; void xfs_exchmaps_defer_add(struct xfs_trans *tp, struct xfs_exchmaps_intent *xmi); +struct xfs_extent_free_item; +struct xfs_defer_pending; + +void xfs_extent_free_defer_add(struct xfs_trans *tp, + struct xfs_extent_free_item *xefi, + struct xfs_defer_pending **dfpp); + #endif /* __LIBXFS_DEFER_ITEM_H_ */ diff --git a/libxfs/xfs_alloc.c b/libxfs/xfs_alloc.c index 60ac7382..063ac197 100644 --- a/libxfs/xfs_alloc.c +++ b/libxfs/xfs_alloc.c @@ -23,6 +23,7 @@ #include "xfs_ag_resv.h" #include "xfs_bmap.h" #include "xfs_health.h" +#include "defer_item.h" struct kmem_cache *xfs_extfree_item_cache; @@ -2548,16 +2549,7 @@ xfs_defer_extent_free( xefi->xefi_owner = XFS_RMAP_OWN_NULL; } - trace_xfs_extent_free_defer(mp, xefi); - - xfs_extent_free_get_group(mp, xefi); - - if (xefi->xefi_agresv == XFS_AG_RESV_AGFL) - *dfpp = xfs_defer_add(tp, &xefi->xefi_list, - &xfs_agfl_free_defer_type); - else - *dfpp = xfs_defer_add(tp, &xefi->xefi_list, - &xfs_extent_free_defer_type); + xfs_extent_free_defer_add(tp, xefi, dfpp); return 0; } diff --git a/libxfs/xfs_alloc.h b/libxfs/xfs_alloc.h index 7f51b3cb..fae17082 100644 --- a/libxfs/xfs_alloc.h +++ b/libxfs/xfs_alloc.h @@ -256,9 +256,6 @@ struct xfs_extent_free_item { enum xfs_ag_resv_type xefi_agresv; }; -void xfs_extent_free_get_group(struct xfs_mount *mp, - struct xfs_extent_free_item *xefi); - #define XFS_EFI_SKIP_DISCARD (1U << 0) /* don't issue discard */ #define XFS_EFI_ATTR_FORK (1U << 1) /* freeing attr fork block */ #define XFS_EFI_BMBT_BLOCK (1U << 2) /* freeing bmap btree block */ -- 2.47.2