[PATCH 1/5] xfs: clean up xfs_inactive

Rich Johnston rjohnston at sgi.com
Thu Jul 26 10:30:56 CDT 2012


On 07/04/2012 10:13 AM, Christoph Hellwig wrote:
> The code to reserve log space and join the inode to the transaction is
> common for all cases, so don't duplicate it.  Also remove the trivial
> xfs_inactive_symlink_local helper which can simply be opencode now.
>
> Signed-off-by: Christoph Hellwig <hch at lst.de>
>
> ---
>  fs/xfs/xfs_vnodeops.c |  171 ++++++++++++--------------------------------------
>  1 file changed, 43 insertions(+), 128 deletions(-)
>
As stated this patch cleans up duplicate code and removed local helper 
function.  Looks good.

Reviewed-by:	Rich Johnston <rjohnston at sgi.com>



More information about the xfs mailing list