Re: [PATCH v3 3/4] xfs: push down inactive transaction mgmt for ifree

Dave Chinner
2013-09-30T00:34:58Z

On Fri, Sep 20, 2013 at 11:06:11AM -0400, Brian Foster wrote:
> Push the inode free work performed during xfs_inactive() down into
> a new xfs_inactive_ifree() helper. This clears xfs_inactive() from
> all inode locking and transaction management more directly
> associated with freeing the inode xattrs, extents and the inode
> itself.
>
> Signed-off-by: Brian Foster <bfoster@redhat.com>

Looks OK to me.

Reviewed-by: Dave Chinner <dchinner@redhat.com>
--
Dave Chinner
david@fromorbit.com