diff options
author | Darrick J. Wong <djwong@kernel.org> | 2022-07-14 11:15:28 -0700 |
---|---|---|
committer | Darrick J. Wong <djwong@kernel.org> | 2022-10-14 14:17:12 -0700 |
commit | 64bf9fdc147bec479190d00a2c4746673ed0290f (patch) | |
tree | d30d6e54a78366334452c2574edc12a70f66700c | |
parent | 8371e166a7df2f84096b91e98626eadf5cf02cf1 (diff) |
xfs: fix confusing xfs_extent_item variable namesrealtime-extfree-intents_2022-10-14
Change the name of all pointers to xfs_extent_item structures to "xefi"
to make the name consistent and because the current selections ("new"
and "free") mean other things in C.
Signed-off-by: Darrick J. Wong <djwong@kernel.org>
-rw-r--r-- | fs/xfs/libxfs/xfs_alloc.c | 38 | ||||
-rw-r--r-- | fs/xfs/xfs_extfree_item.c | 89 |
2 files changed, 64 insertions, 63 deletions
diff --git a/fs/xfs/libxfs/xfs_alloc.c b/fs/xfs/libxfs/xfs_alloc.c index 7567c296b53e..0b3b4ace3915 100644 --- a/fs/xfs/libxfs/xfs_alloc.c +++ b/fs/xfs/libxfs/xfs_alloc.c @@ -2540,19 +2540,19 @@ xfs_defer_agfl_block( struct xfs_owner_info *oinfo) { struct xfs_mount *mp = tp->t_mountp; - struct xfs_extent_free_item *new; /* new element */ + struct xfs_extent_free_item *xefi; ASSERT(xfs_extfree_item_cache != NULL); ASSERT(oinfo != NULL); - new = kmem_cache_zalloc(xfs_extfree_item_cache, + xefi = kmem_cache_zalloc(xfs_extfree_item_cache, GFP_KERNEL | __GFP_NOFAIL); - new->xefi_startblock = XFS_AGB_TO_FSB(mp, agno, agbno); - new->xefi_blockcount = 1; - new->xefi_owner = oinfo->oi_owner; + xefi->xefi_startblock = XFS_AGB_TO_FSB(mp, agno, agbno); + xefi->xefi_blockcount = 1; + xefi->xefi_owner = oinfo->oi_owner; - trace_xfs_agfl_free_defer(mp, new); - xfs_defer_add(tp, XFS_DEFER_OPS_TYPE_AGFL_FREE, &new->xefi_list); + trace_xfs_agfl_free_defer(mp, xefi); + xfs_defer_add(tp, XFS_DEFER_OPS_TYPE_AGFL_FREE, &xefi->xefi_list); } /* @@ -2567,7 +2567,7 @@ xfs_free_extent_later( const struct xfs_owner_info *oinfo, unsigned int flags) { - struct xfs_extent_free_item *new; /* new element */ + struct xfs_extent_free_item *xefi; #ifdef DEBUG struct xfs_mount *mp = tp->t_mountp; @@ -2594,27 +2594,27 @@ xfs_free_extent_later( #endif ASSERT(xfs_extfree_item_cache != NULL); - new = kmem_cache_zalloc(xfs_extfree_item_cache, + xefi = kmem_cache_zalloc(xfs_extfree_item_cache, GFP_KERNEL | __GFP_NOFAIL); - new->xefi_startblock = bno; - new->xefi_blockcount = (xfs_extlen_t)len; + xefi->xefi_startblock = bno; + xefi->xefi_blockcount = (xfs_extlen_t)len; if (flags & XFS_FREE_EXTENT_SKIP_DISCARD) - new->xefi_flags |= XFS_EFI_SKIP_DISCARD; + xefi->xefi_flags |= XFS_EFI_SKIP_DISCARD; if (flags & XFS_FREE_EXTENT_REALTIME) - new->xefi_flags |= XFS_EFI_REALTIME; + xefi->xefi_flags |= XFS_EFI_REALTIME; if (oinfo) { ASSERT(oinfo->oi_offset == 0); if (oinfo->oi_flags & XFS_OWNER_INFO_ATTR_FORK) - new->xefi_flags |= XFS_EFI_ATTR_FORK; + xefi->xefi_flags |= XFS_EFI_ATTR_FORK; if (oinfo->oi_flags & XFS_OWNER_INFO_BMBT_BLOCK) - new->xefi_flags |= XFS_EFI_BMBT_BLOCK; - new->xefi_owner = oinfo->oi_owner; + xefi->xefi_flags |= XFS_EFI_BMBT_BLOCK; + xefi->xefi_owner = oinfo->oi_owner; } else { - new->xefi_owner = XFS_RMAP_OWN_NULL; + xefi->xefi_owner = XFS_RMAP_OWN_NULL; } - trace_xfs_extent_free_defer(tp->t_mountp, new); - xfs_defer_add(tp, XFS_DEFER_OPS_TYPE_FREE, &new->xefi_list); + trace_xfs_extent_free_defer(tp->t_mountp, xefi); + xfs_defer_add(tp, XFS_DEFER_OPS_TYPE_FREE, &xefi->xefi_list); } #ifdef DEBUG diff --git a/fs/xfs/xfs_extfree_item.c b/fs/xfs/xfs_extfree_item.c index d259b68ec67e..bade81a5897d 100644 --- a/fs/xfs/xfs_extfree_item.c +++ b/fs/xfs/xfs_extfree_item.c @@ -370,7 +370,7 @@ static int xfs_trans_free_extent( struct xfs_trans *tp, struct xfs_efd_log_item *efdp, - struct xfs_extent_free_item *free) + struct xfs_extent_free_item *xefi) { struct xfs_owner_info oinfo = { }; struct xfs_mount *mp = tp->t_mountp; @@ -378,24 +378,24 @@ xfs_trans_free_extent( uint next_extent; int error; - oinfo.oi_owner = free->xefi_owner; - if (free->xefi_flags & XFS_EFI_ATTR_FORK) + oinfo.oi_owner = xefi->xefi_owner; + if (xefi->xefi_flags & XFS_EFI_ATTR_FORK) oinfo.oi_flags |= XFS_OWNER_INFO_ATTR_FORK; - if (free->xefi_flags & XFS_EFI_BMBT_BLOCK) + if (xefi->xefi_flags & XFS_EFI_BMBT_BLOCK) oinfo.oi_flags |= XFS_OWNER_INFO_BMBT_BLOCK; - trace_xfs_extent_free_deferred(mp, free); + trace_xfs_extent_free_deferred(mp, xefi); - if (free->xefi_flags & XFS_EFI_REALTIME) { - ASSERT(free->xefi_owner == XFS_RMAP_OWN_NULL || - free->xefi_owner == XFS_RMAP_OWN_UNKNOWN); + if (xefi->xefi_flags & XFS_EFI_REALTIME) { + ASSERT(xefi->xefi_owner == XFS_RMAP_OWN_NULL || + xefi->xefi_owner == XFS_RMAP_OWN_UNKNOWN); - error = xfs_rtfree_blocks(tp, free->xefi_startblock, - free->xefi_blockcount); + error = xfs_rtfree_blocks(tp, xefi->xefi_startblock, + xefi->xefi_blockcount); } else { - error = __xfs_free_extent(tp, free->xefi_startblock, - free->xefi_blockcount, &oinfo, XFS_AG_RESV_NONE, - free->xefi_flags & XFS_EFI_SKIP_DISCARD); + error = __xfs_free_extent(tp, xefi->xefi_startblock, + xefi->xefi_blockcount, &oinfo, XFS_AG_RESV_NONE, + xefi->xefi_flags & XFS_EFI_SKIP_DISCARD); } /* @@ -411,8 +411,8 @@ xfs_trans_free_extent( next_extent = efdp->efd_next_extent; ASSERT(next_extent < efdp->efd_format.efd_nextents); extp = &(efdp->efd_format.efd_extents[next_extent]); - extp->ext_start = free->xefi_startblock; - extp->ext_len = free->xefi_blockcount; + extp->ext_start = xefi->xefi_startblock; + extp->ext_len = xefi->xefi_blockcount; efdp->efd_next_extent++; return error; @@ -448,7 +448,7 @@ STATIC void xfs_extent_free_log_item( struct xfs_trans *tp, struct xfs_efi_log_item *efip, - struct xfs_extent_free_item *free) + struct xfs_extent_free_item *xefi) { uint next_extent; struct xfs_extent *extp; @@ -464,9 +464,9 @@ xfs_extent_free_log_item( next_extent = atomic_inc_return(&efip->efi_next_extent) - 1; ASSERT(next_extent < efip->efi_format.efi_nextents); extp = &efip->efi_format.efi_extents[next_extent]; - extp->ext_start = free->xefi_startblock; - extp->ext_len = free->xefi_blockcount; - if (free->xefi_flags & XFS_EFI_REALTIME) + extp->ext_start = xefi->xefi_startblock; + extp->ext_len = xefi->xefi_blockcount; + if (xefi->xefi_flags & XFS_EFI_REALTIME) extp->ext_len |= XFS_EFI_EXTLEN_REALTIME_EXT; } @@ -479,15 +479,15 @@ xfs_extent_free_create_intent( { struct xfs_mount *mp = tp->t_mountp; struct xfs_efi_log_item *efip = xfs_efi_init(mp, count); - struct xfs_extent_free_item *free; + struct xfs_extent_free_item *xefi; ASSERT(count > 0); xfs_trans_add_item(tp, &efip->efi_item); if (sort) list_sort(mp, items, xfs_extent_free_diff_items); - list_for_each_entry(free, items, xefi_list) - xfs_extent_free_log_item(tp, efip, free); + list_for_each_entry(xefi, items, xefi_list) + xfs_extent_free_log_item(tp, efip, xefi); return &efip->efi_item; } @@ -519,24 +519,24 @@ xfs_extent_free_finish_item( struct xfs_btree_cur **state) { struct xfs_mount *mp = tp->t_mountp; - struct xfs_extent_free_item *free; + struct xfs_extent_free_item *xefi; int error; - free = container_of(item, struct xfs_extent_free_item, xefi_list); + xefi = container_of(item, struct xfs_extent_free_item, xefi_list); /* * Lock the rt bitmap if we've any realtime extents to free and we * haven't locked the rt inodes yet. */ - if (*state == NULL && (free->xefi_flags & XFS_EFI_REALTIME)) { + if (*state == NULL && (xefi->xefi_flags & XFS_EFI_REALTIME)) { xfs_rtbitmap_lock(tp, mp); *state = (struct xfs_btree_cur *)1; } - error = xfs_trans_free_extent(tp, EFD_ITEM(done), free); + error = xfs_trans_free_extent(tp, EFD_ITEM(done), xefi); - xfs_extent_free_drop_intents(mp, free); - kmem_cache_free(xfs_extfree_item_cache, free); + xfs_extent_free_drop_intents(mp, xefi); + kmem_cache_free(xfs_extfree_item_cache, xefi); return error; } @@ -554,11 +554,12 @@ xfs_extent_free_cancel_item( struct xfs_mount *mp, struct list_head *item) { - struct xfs_extent_free_item *free; + struct xfs_extent_free_item *xefi; + + xefi = container_of(item, struct xfs_extent_free_item, xefi_list); - free = container_of(item, struct xfs_extent_free_item, xefi_list); - xfs_extent_free_drop_intents(mp, free); - kmem_cache_free(xfs_extfree_item_cache, free); + xfs_extent_free_drop_intents(mp, xefi); + kmem_cache_free(xfs_extfree_item_cache, xefi); } /* Add a deferred free extent. */ @@ -600,7 +601,7 @@ xfs_agfl_free_finish_item( struct xfs_owner_info oinfo = { }; struct xfs_mount *mp = tp->t_mountp; struct xfs_efd_log_item *efdp = EFD_ITEM(done); - struct xfs_extent_free_item *free; + struct xfs_extent_free_item *xefi; struct xfs_extent *extp; struct xfs_buf *agbp; int error; @@ -609,14 +610,14 @@ xfs_agfl_free_finish_item( uint next_extent; struct xfs_perag *pag; - free = container_of(item, struct xfs_extent_free_item, xefi_list); - ASSERT(free->xefi_blockcount == 1); - ASSERT(!(free->xefi_flags & XFS_EFI_REALTIME)); - agno = XFS_FSB_TO_AGNO(mp, free->xefi_startblock); - agbno = XFS_FSB_TO_AGBNO(mp, free->xefi_startblock); - oinfo.oi_owner = free->xefi_owner; + xefi = container_of(item, struct xfs_extent_free_item, xefi_list); + ASSERT(xefi->xefi_blockcount == 1); + ASSERT(!(xefi->xefi_flags & XFS_EFI_REALTIME)); + agno = XFS_FSB_TO_AGNO(mp, xefi->xefi_startblock); + agbno = XFS_FSB_TO_AGBNO(mp, xefi->xefi_startblock); + oinfo.oi_owner = xefi->xefi_owner; - trace_xfs_agfl_free_deferred(mp, free); + trace_xfs_agfl_free_deferred(mp, xefi); pag = xfs_perag_get(mp, agno); error = xfs_alloc_read_agf(pag, tp, 0, &agbp); @@ -637,12 +638,12 @@ xfs_agfl_free_finish_item( next_extent = efdp->efd_next_extent; ASSERT(next_extent < efdp->efd_format.efd_nextents); extp = &(efdp->efd_format.efd_extents[next_extent]); - extp->ext_start = free->xefi_startblock; - extp->ext_len = free->xefi_blockcount; + extp->ext_start = xefi->xefi_startblock; + extp->ext_len = xefi->xefi_blockcount; efdp->efd_next_extent++; - xfs_extent_free_drop_intents(mp, free); - kmem_cache_free(xfs_extfree_item_cache, free); + xfs_extent_free_drop_intents(mp, xefi); + kmem_cache_free(xfs_extfree_item_cache, xefi); return error; } |