On Fri, May 10, 2019 at 03:18:25PM -0500, Eric Sandeen wrote: > Pull functions out of libxfs/*.c into trans_buf.c, if they roughly match > the kernel's xfs_trans_buf.c file. > > Signed-off-by: Eric Sandeen <sandeen@xxxxxxxxxx> So I have no problems with this, but I'm not sure what the eventual goal is? Just sharing code, or is there some functionality that requires a more complete transaction subsystem in userspace? I'm asking because if the goal is eventual unification with the kernel code, then we probably should name the files the same as the kernel code so we don't have to rename them again when we do the unification. That will make history searching a bit easier - less file names to follow across and git blame works a whole lot better... > +int > +xfs_trans_read_buf_map( > + xfs_mount_t *mp, > + xfs_trans_t *tp, > + struct xfs_buftarg *btp, > + struct xfs_buf_map *map, > + int nmaps, > + uint flags, > + xfs_buf_t **bpp, > + const struct xfs_buf_ops *ops) Hmmmm. Will there be any follow-up to de-typedef these new files? /me would love to just have a flag day that de-typedefs all of the userspace code. Cheers, Dave. -- Dave Chinner david@xxxxxxxxxxxxx
