[PATCH 0/4] Clean up xfs_attr_sf_entry

Carlos Maiolino
2020-08-31T13:04:53Z

Hi,

this series has been suggested by Eric, and it's intended as a small clean up
for xfS_attr_sf_entry usage.

First patch changes the nameval array definition from 1 element to a
variable-size array. The array is already being used as a variable size array,
but by now, we need to subtract 1 from every time we use it.

Second patch just convert some macros to inline functions.

The remaining two patches are just 2 typedef cleanups that I think it's
appropriate since I'm already touching this code. I opted to leave the typedef
cleanups by last, since, if by any reason anybody think it's not worth it, both
patches can be simply discarded without needing to change any of the first 2.

All patches survived a few xfstests runs and the reproducer Eric shared on his
previous patch:

#touch file
#setfatt -n user.a file

Comments?

Cheers

Carlos Maiolino (4):
xfs: Use variable-size array for nameval in xfs_attr_sf_entry
xfs: Convert xfs_attr_sf macros to inline functions
xfs: remove typedef xfs_attr_sf_entry_t
xfs: Remove typedef xfs_attr_shortform_t

fs/xfs/libxfs/xfs_attr.c | 15 ++++++++++---
fs/xfs/libxfs/xfs_attr_leaf.c | 42 +++++++++++++++++------------------
fs/xfs/libxfs/xfs_attr_sf.h | 27 +++++++++++++---------
fs/xfs/libxfs/xfs_da_format.h | 6 ++---
fs/xfs/xfs_attr_list.c | 6 ++---
fs/xfs/xfs_ondisk.h | 12 +++++-----
6 files changed, 60 insertions(+), 48 deletions(-)

--
2.26.2