mbox series

[PATCHSET,v2,0/2] xfs: clean up zone terminology

Message ID 163466951226.2234337.10978241003370731405.stgit@magnolia (mailing list archive)
Headers show
Series xfs: clean up zone terminology | expand

Message

Darrick J. Wong Oct. 19, 2021, 6:51 p.m. UTC
Hi all,

Dave requested[1] that we stop using the old Irix "zone" terminology to
describe Linux slab caches.  Since we're using an ugly typedef to wrap
the new in the old, get rid of the typedef, and change the wording to
reflect the way Linux has been for a good 20+ years.  This enables
cleaning up of the bigger zone/cache mess in userspace.

[1] https://lore.kernel.org/linux-xfs/20210926004343.GC1756565@dread.disaster.area/

v2: rebase atop the final btree cursor series

If you're going to start using this mess, you probably ought to just
pull from my git trees, which are linked below.

This is an extraordinary way to destroy everything.  Enjoy!
Comments and questions are, as always, welcome.

--D

kernel git tree:
https://git.kernel.org/cgit/linux/kernel/git/djwong/xfs-linux.git/log/?h=slab-cache-cleanups-5.16
---
 fs/xfs/kmem.h                      |    4 -
 fs/xfs/libxfs/xfs_alloc.c          |    6 -
 fs/xfs/libxfs/xfs_alloc_btree.c    |    2 
 fs/xfs/libxfs/xfs_attr_leaf.c      |    2 
 fs/xfs/libxfs/xfs_bmap.c           |    6 -
 fs/xfs/libxfs/xfs_bmap.h           |    2 
 fs/xfs/libxfs/xfs_bmap_btree.c     |    2 
 fs/xfs/libxfs/xfs_btree.h          |    4 -
 fs/xfs/libxfs/xfs_da_btree.c       |    6 -
 fs/xfs/libxfs/xfs_da_btree.h       |    3 
 fs/xfs/libxfs/xfs_ialloc_btree.c   |    2 
 fs/xfs/libxfs/xfs_inode_fork.c     |    8 +
 fs/xfs/libxfs/xfs_inode_fork.h     |    2 
 fs/xfs/libxfs/xfs_refcount_btree.c |    2 
 fs/xfs/libxfs/xfs_rmap_btree.c     |    2 
 fs/xfs/xfs_attr_inactive.c         |    2 
 fs/xfs/xfs_bmap_item.c             |   12 +-
 fs/xfs/xfs_bmap_item.h             |    6 -
 fs/xfs/xfs_buf.c                   |   14 +-
 fs/xfs/xfs_buf_item.c              |    8 +
 fs/xfs/xfs_buf_item.h              |    2 
 fs/xfs/xfs_dquot.c                 |   26 ++--
 fs/xfs/xfs_extfree_item.c          |   18 +--
 fs/xfs/xfs_extfree_item.h          |    6 -
 fs/xfs/xfs_icache.c                |   10 +-
 fs/xfs/xfs_icreate_item.c          |    6 -
 fs/xfs/xfs_icreate_item.h          |    2 
 fs/xfs/xfs_inode.c                 |    2 
 fs/xfs/xfs_inode.h                 |    2 
 fs/xfs/xfs_inode_item.c            |    6 -
 fs/xfs/xfs_inode_item.h            |    2 
 fs/xfs/xfs_log.c                   |    6 -
 fs/xfs/xfs_log_priv.h              |    2 
 fs/xfs/xfs_mru_cache.c             |    2 
 fs/xfs/xfs_qm.h                    |    2 
 fs/xfs/xfs_refcount_item.c         |   12 +-
 fs/xfs/xfs_refcount_item.h         |    6 -
 fs/xfs/xfs_rmap_item.c             |   12 +-
 fs/xfs/xfs_rmap_item.h             |    6 -
 fs/xfs/xfs_super.c                 |  218 ++++++++++++++++++------------------
 fs/xfs/xfs_trans.c                 |    8 +
 fs/xfs/xfs_trans.h                 |    2 
 fs/xfs/xfs_trans_dquot.c           |    4 -
 43 files changed, 226 insertions(+), 231 deletions(-)

Comments

Carlos Maiolino Nov. 1, 2021, 12:40 p.m. UTC | #1
On Tue, Oct 19, 2021 at 11:51:52AM -0700, Darrick J. Wong wrote:
> Hi all,
> 
> Dave requested[1] that we stop using the old Irix "zone" terminology to
> describe Linux slab caches.  Since we're using an ugly typedef to wrap
> the new in the old, get rid of the typedef, and change the wording to
> reflect the way Linux has been for a good 20+ years.  This enables
> cleaning up of the bigger zone/cache mess in userspace.
> 
> [1] https://lore.kernel.org/linux-xfs/20210926004343.GC1756565@dread.disaster.area/
> 
> v2: rebase atop the final btree cursor series
> 
> If you're going to start using this mess, you probably ought to just
> pull from my git trees, which are linked below.
> 
> This is an extraordinary way to destroy everything.  Enjoy!
> Comments and questions are, as always, welcome.

LGTM. For the series:

Reviewed-by: Carlos Maiolino <cmaiolino@redhat.com>

> 
> --D
> 
> kernel git tree:
> https://git.kernel.org/cgit/linux/kernel/git/djwong/xfs-linux.git/log/?h=slab-cache-cleanups-5.16
> ---
>  fs/xfs/kmem.h                      |    4 -
>  fs/xfs/libxfs/xfs_alloc.c          |    6 -
>  fs/xfs/libxfs/xfs_alloc_btree.c    |    2 
>  fs/xfs/libxfs/xfs_attr_leaf.c      |    2 
>  fs/xfs/libxfs/xfs_bmap.c           |    6 -
>  fs/xfs/libxfs/xfs_bmap.h           |    2 
>  fs/xfs/libxfs/xfs_bmap_btree.c     |    2 
>  fs/xfs/libxfs/xfs_btree.h          |    4 -
>  fs/xfs/libxfs/xfs_da_btree.c       |    6 -
>  fs/xfs/libxfs/xfs_da_btree.h       |    3 
>  fs/xfs/libxfs/xfs_ialloc_btree.c   |    2 
>  fs/xfs/libxfs/xfs_inode_fork.c     |    8 +
>  fs/xfs/libxfs/xfs_inode_fork.h     |    2 
>  fs/xfs/libxfs/xfs_refcount_btree.c |    2 
>  fs/xfs/libxfs/xfs_rmap_btree.c     |    2 
>  fs/xfs/xfs_attr_inactive.c         |    2 
>  fs/xfs/xfs_bmap_item.c             |   12 +-
>  fs/xfs/xfs_bmap_item.h             |    6 -
>  fs/xfs/xfs_buf.c                   |   14 +-
>  fs/xfs/xfs_buf_item.c              |    8 +
>  fs/xfs/xfs_buf_item.h              |    2 
>  fs/xfs/xfs_dquot.c                 |   26 ++--
>  fs/xfs/xfs_extfree_item.c          |   18 +--
>  fs/xfs/xfs_extfree_item.h          |    6 -
>  fs/xfs/xfs_icache.c                |   10 +-
>  fs/xfs/xfs_icreate_item.c          |    6 -
>  fs/xfs/xfs_icreate_item.h          |    2 
>  fs/xfs/xfs_inode.c                 |    2 
>  fs/xfs/xfs_inode.h                 |    2 
>  fs/xfs/xfs_inode_item.c            |    6 -
>  fs/xfs/xfs_inode_item.h            |    2 
>  fs/xfs/xfs_log.c                   |    6 -
>  fs/xfs/xfs_log_priv.h              |    2 
>  fs/xfs/xfs_mru_cache.c             |    2 
>  fs/xfs/xfs_qm.h                    |    2 
>  fs/xfs/xfs_refcount_item.c         |   12 +-
>  fs/xfs/xfs_refcount_item.h         |    6 -
>  fs/xfs/xfs_rmap_item.c             |   12 +-
>  fs/xfs/xfs_rmap_item.h             |    6 -
>  fs/xfs/xfs_super.c                 |  218 ++++++++++++++++++------------------
>  fs/xfs/xfs_trans.c                 |    8 +
>  fs/xfs/xfs_trans.h                 |    2 
>  fs/xfs/xfs_trans_dquot.c           |    4 -
>  43 files changed, 226 insertions(+), 231 deletions(-)
>