Re: [PATCH] XFS: random cleanups of xfs_swap_extents

["Josef 'Jeff' Sipek" <jeffpc@josefsipek.net>]

On Wed, Nov 12, 2008 at 05:09:58AM -0500, Christoph Hellwig wrote:
> On Fri, Nov 07, 2008 at 06:00:54PM -0500, Josef 'Jeff' Sipek wrote:
> > XFS: random cleanups of xfs_swap_extents
> > 
> > From: Josef 'Jeff' Sipek <jeffpc@josefsipek.net>
> > 
> > 1) remove lock_flags var since it's never modified and only obfuscates the
> > code
> > 
> > 2) calling kfree/vfree on a NULL is valid
> 
> Looks good to me, but I'd go even further and kill the stupid locked
> variable by having the exit path structured as:
> 
>  out_trans_cancel:
> 	xfs_trans_cancel(tp, 0);
>  out_unlock:
> 	xfs_iunlock(ip,  XFS_ILOCK_EXCL | XFS_IOLOCK_EXCL);
> 	xfs_iunlock(tip, XFS_ILOCK_EXCL | XFS_IOLOCK_EXCL);
>  out_free:
> 	kmem_free(tempifp);
>  out:
> 	return error;
> }
> 
> With the one case that does a trans_cancel with partially unlocked
> inodes handcoding most of it.

Ok, I'll do that.

Josef 'Jeff' Sipek.

-- 
Mankind invented the atomic bomb, but no mouse would ever construct a
mousetrap.
		- Albert Einstein