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@xxxxxxxxxxxxxx>
> >
> > 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
|