xfs
[Top] [All Lists]

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

To: Brian Foster <bfoster@xxxxxxxxxx>
Subject: Re: [PATCH v3 3/4] xfs: push down inactive transaction mgmt for ifree
From: Dave Chinner <david@xxxxxxxxxxxxx>
Date: Mon, 30 Sep 2013 10:34:53 +1000
Cc: xfs@xxxxxxxxxxx
Delivered-to: xfs@xxxxxxxxxxx
In-reply-to: <1379689572-5543-4-git-send-email-bfoster@xxxxxxxxxx>
References: <1379689572-5543-1-git-send-email-bfoster@xxxxxxxxxx> <1379689572-5543-4-git-send-email-bfoster@xxxxxxxxxx>
User-agent: Mutt/1.5.21 (2010-09-15)
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@xxxxxxxxxx>

Looks OK to me.

Reviewed-by: Dave Chinner <dchinner@xxxxxxxxxx>
-- 
Dave Chinner
david@xxxxxxxxxxxxx

<Prev in Thread] Current Thread [Next in Thread>