mbox series

[PATCHSET,v9r2d1.1,00/13] xfs: remove parent pointer hashing

Message ID 167786347827.1543331.2803518928321606576.stgit@magnolia (mailing list archive)
Headers show
Series xfs: remove parent pointer hashing | expand

Message

Darrick J. Wong March 3, 2023, 5:11 p.m. UTC
Hi all,

Dave Chinner pointed out (a bit too subtly) that hashing the dirent name
to try to squash it into the parent pointer xattr name is unnecessary
because we could simply make the xattr matching predicate compare names.
Do that instead and drop the hashing.

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.
kernel git tree:
https://git.kernel.org/cgit/linux/kernel/git/djwong/xfs-linux.git/log/?h=pptrs-vlookup

xfsprogs git tree:
https://git.kernel.org/cgit/linux/kernel/git/djwong/xfsprogs-dev.git/log/?h=pptrs-vlookup
---
 fs/xfs/Kconfig                 |    1 
 fs/xfs/libxfs/xfs_attr.c       |   39 +++--
 fs/xfs/libxfs/xfs_attr_leaf.c  |   41 +++++
 fs/xfs/libxfs/xfs_da_btree.h   |    6 +
 fs/xfs/libxfs/xfs_da_format.h  |   34 ++---
 fs/xfs/libxfs/xfs_log_format.h |   30 +++-
 fs/xfs/libxfs/xfs_parent.c     |  302 +++++++++++++---------------------------
 fs/xfs/libxfs/xfs_parent.h     |   16 --
 fs/xfs/libxfs/xfs_trans_resv.c |    1 
 fs/xfs/scrub/dir.c             |   38 +----
 fs/xfs/scrub/parent.c          |   61 +-------
 fs/xfs/scrub/parent_repair.c   |   34 +----
 fs/xfs/xfs_attr_item.c         |  217 ++++++++++++++++++++---------
 fs/xfs/xfs_attr_item.h         |    3 
 fs/xfs/xfs_linux.h             |    1 
 fs/xfs/xfs_mount.c             |   13 --
 fs/xfs/xfs_mount.h             |    3 
 fs/xfs/xfs_ondisk.h            |    3 
 fs/xfs/xfs_sha512.h            |   42 ------
 fs/xfs/xfs_super.c             |    3 
 fs/xfs/xfs_xattr.c             |    5 +
 21 files changed, 383 insertions(+), 510 deletions(-)
 delete mode 100644 fs/xfs/xfs_sha512.h