On Sun, Aug 09, 2015 at 12:36:41AM -0700, Christoph Hellwig wrote: > On Thu, Aug 06, 2015 at 01:44:22PM -0400, Brian Foster wrote: > > - __xfs_efi_release(efip); > + xfs_efi_release(efip); > > Can you explain in the changelog why this is safe? > I thought it did. I'll try to elaborate. > > > -xfs_efi_release(xfs_efi_log_item_t *efip, > > - uint nextents) > > +xfs_efi_release(struct xfs_efi_log_item *efip) > > Can you use normal XFS function formatting here? e.g. > > xfs_efi_release( > struct xfs_efi_log_item *efip) > Ok. > > As a follow on we should be able to remove atomic_inc_return and > replace it with a local iterator in xfs_bmap_finish(). > I'm not sure what you mean here... Brian > _______________________________________________ > xfs mailing list > xfs@xxxxxxxxxxx > http://oss.sgi.com/mailman/listinfo/xfs _______________________________________________ xfs mailing list xfs@xxxxxxxxxxx http://oss.sgi.com/mailman/listinfo/xfs
