mbox series

[0/7] xfs: make btree cursor private unions anonymous

Message ID 20200305014537.11236-1-david@fromorbit.com (mailing list archive)
Headers show
Series xfs: make btree cursor private unions anonymous | expand

Message

Dave Chinner March 5, 2020, 1:45 a.m. UTC 
This is a "make things less verbose" cleanup from looking at the
changes Darrick is making to add a staging/fake cursor to the union
for bulk btree loading.

The process is to create a @defines of the new name to the existing
union name, then replace all users of each union via a script. Then
the union is made anonymous and the members renamed to match the new
code. Then the #defines get removed.

We do this for the bc_private union, then we name the ag and btree
structures and make them use anonymous unions internally via the
same process.

That means we go from doubly nested private stuff like this:

cur->bc_private.a.priv.abt.active

To the much cleaner, less verbose and more readable:

cur->bc_ag.abt.active

Simples, yes?

This series can be found at:

https://git.kernel.org/pub/scm/linux/kernel/git/dgc/linux-xfs.git/h?xfs-btree-cursor-cleanup

Note: the code changes are all scripted, I have not done any
followup to do things like aggregate split lines back into single
lines as that is out of scope of the structure definition cleanup
I'm trying to acheive here. That can be done in future as we modify
the code that now has lines that can be merged....

Signed-off-by: Dave Chinner <dchinner@redhat.com>


Dave Chinner (7):
  xfs: introduce new private btree cursor names
  xfs: convert btree cursor ag private member name
  xfs: convert btree cursor btree private member name
  xfs: rename btree cursur private btree member flags
  xfs: make btree cursor private union anonymous
  xfs: make the btree cursor union members named structure
  xfs: make the btree ag cursor private union anonymous

 fs/xfs/libxfs/xfs_alloc.c          |  16 ++---
 fs/xfs/libxfs/xfs_alloc_btree.c    |  24 +++----
 fs/xfs/libxfs/xfs_bmap.c           |  46 ++++++------
 fs/xfs/libxfs/xfs_bmap_btree.c     |  50 ++++++-------
 fs/xfs/libxfs/xfs_btree.c          |  62 ++++++++--------
 fs/xfs/libxfs/xfs_btree.h          |  51 ++++++-------
 fs/xfs/libxfs/xfs_ialloc.c         |   2 +-
 fs/xfs/libxfs/xfs_ialloc_btree.c   |  20 +++---
 fs/xfs/libxfs/xfs_refcount.c       | 110 ++++++++++++++---------------
 fs/xfs/libxfs/xfs_refcount_btree.c |  28 ++++----
 fs/xfs/libxfs/xfs_rmap.c           | 110 ++++++++++++++---------------
 fs/xfs/libxfs/xfs_rmap_btree.c     |  28 ++++----
 fs/xfs/scrub/agheader_repair.c     |   2 +-
 fs/xfs/scrub/alloc.c               |   2 +-
 fs/xfs/scrub/bmap.c                |   4 +-
 fs/xfs/scrub/ialloc.c              |   8 +--
 fs/xfs/scrub/refcount.c            |   2 +-
 fs/xfs/scrub/rmap.c                |   2 +-
 fs/xfs/scrub/trace.c               |   4 +-
 fs/xfs/scrub/trace.h               |   4 +-
 fs/xfs/xfs_fsmap.c                 |   4 +-
 21 files changed, 291 insertions(+), 288 deletions(-)