From e93011acb1e3ca499412d4ee5672d27cd1c8ea4c Mon Sep 17 00:00:00 2001 From: "Darrick J. Wong" Date: Tue, 9 Jan 2024 09:40:24 -0800 Subject: [PATCH] xfs: add a ci_entry helper Add a helper to translate from the item list head to the refcount_intent_item structure and use it so shorten assignments and avoid the need for extra local variables. Inspired-by: Christoph Hellwig Signed-off-by: Darrick J. Wong --- libxfs/defer_item.c | 19 +++++++++---------- 1 file changed, 9 insertions(+), 10 deletions(-) diff --git a/libxfs/defer_item.c b/libxfs/defer_item.c index 9c62a8215..a07103839 100644 --- a/libxfs/defer_item.c +++ b/libxfs/defer_item.c @@ -484,6 +484,11 @@ const struct xfs_defer_op_type xfs_rtrmap_update_defer_type = { /* Reference Counting */ +static inline struct xfs_refcount_intent *ci_entry(const struct list_head *e) +{ + return list_entry(e, struct xfs_refcount_intent, ri_list); +} + /* Sort refcount intents by AG. */ static int xfs_refcount_update_diff_items( @@ -491,11 +496,8 @@ xfs_refcount_update_diff_items( const struct list_head *a, const struct list_head *b) { - const struct xfs_refcount_intent *ra; - const struct xfs_refcount_intent *rb; - - ra = container_of(a, struct xfs_refcount_intent, ri_list); - rb = container_of(b, struct xfs_refcount_intent, ri_list); + struct xfs_refcount_intent *ra = ci_entry(a); + struct xfs_refcount_intent *rb = ci_entry(b); return ra->ri_pag->pag_agno - rb->ri_pag->pag_agno; } @@ -550,10 +552,9 @@ xfs_refcount_update_finish_item( struct list_head *item, struct xfs_btree_cur **state) { - struct xfs_refcount_intent *ri; + struct xfs_refcount_intent *ri = ci_entry(item); int error; - ri = container_of(item, struct xfs_refcount_intent, ri_list); error = xfs_refcount_finish_one(tp, ri, state); /* Did we run out of reservation? Requeue what we didn't finish. */ @@ -580,9 +581,7 @@ STATIC void xfs_refcount_update_cancel_item( struct list_head *item) { - struct xfs_refcount_intent *ri; - - ri = container_of(item, struct xfs_refcount_intent, ri_list); + struct xfs_refcount_intent *ri = ci_entry(item); xfs_refcount_update_put_group(ri); kmem_cache_free(xfs_refcount_intent_cache, ri); -- 2.50.1