On Mon, May 23, 2022 at 10:36:42PM -0700, Darrick J. Wong wrote: > From: Darrick J. Wong <djwong@xxxxxxxxxx> > > The LARP patchset added an awkward coupling point between libxfs and > what would be libxlog, if the XFS log were actually its own library. > Move the code that enables logged xattr updates out of "lib"xlog and into > xfs_xattr.c so that it no longer has to know about xlog_* functions. > > While we're at it, give xfs_xattr.c its own header file. > > Signed-off-by: Darrick J. Wong <djwong@xxxxxxxxxx> > --- > fs/xfs/libxfs/xfs_attr.c | 6 +++--- > fs/xfs/xfs_log.c | 41 -------------------------------------- > fs/xfs/xfs_super.c | 1 + > fs/xfs/xfs_super.h | 1 - > fs/xfs/xfs_xattr.c | 49 ++++++++++++++++++++++++++++++++++++++++++++++ > fs/xfs/xfs_xattr.h | 14 +++++++++++++ > 6 files changed, 67 insertions(+), 45 deletions(-) > create mode 100644 fs/xfs/xfs_xattr.h fs/xfs/xfs_xattr.c was the only place I could really think that this kinda fitted, too. Reviewed-by: Dave Chinner <dchinner@xxxxxxxxxx> -- Dave Chinner david@xxxxxxxxxxxxx