Re: [PATCH] xfstests 073: unmount source image in cleanup

[Christoph Hellwig <hch@xxxxxxxxxxxxx>]

On Fri, Jan 15, 2010 at 06:41:37PM -0600, Eric Sandeen wrote:
> Test 073 is currently failing w/ xfs_repair problems, and when
> it does, the whole series stops because the test dir still has
> a loopback mounted filesystem on it, and it can't unmount.
> 
> Fix that.
> 
> Signed-off-by: Eric Sandeen <sandeen@xxxxxxxxxxx>
> ---
> 
> diff --git a/073 b/073
> index 0ef9287..756c1c8 100755
> --- a/073
> +++ b/073
> @@ -44,6 +44,7 @@ _cleanup()
>       umount $SCRATCH_MNT 2>/dev/null
>       umount -d $imgs.loop 2>/dev/null
>       [ -d $imgs.loop ] && rmdir $imgs.loop
> +     umount -d $imgs.source_dir 2>/dev/null
>       [ -d $imgs.source_dir ] && rm -rf $imgs.source_dir
>       rm -f $imgs.* $tmp.* /var/tmp/xfs_copy.log.*

Yes, we should always unmount everything in the cleanup handler.

Reviewed-by: Christoph Hellwig <hch@xxxxxx>