Re: [PATCH 01/11] xfs: disentagle EFI release from the extent count

[Brian Foster <bfoster@xxxxxxxxxx>]

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