[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH] vmdk: Fix backing file handling
From: |
Christoph Hellwig |
Subject: |
Re: [Qemu-devel] [PATCH] vmdk: Fix backing file handling |
Date: |
Fri, 17 Jul 2009 12:34:38 +0200 |
User-agent: |
Mutt/1.3.28i |
On Fri, Jul 17, 2009 at 08:20:41AM +0200, Kevin Wolf wrote:
> Instead of storing the backing file in its own BlockDriverState, VMDK uses the
> BlockDriverState of the raw image file it opened. This is wrong and breaks
> functions that access the backing file or protocols. This fix replaces all
> occurrences of s->hd->backing_* with bs->backing_*.
>
> This fixes qemu-iotests failure in 020 (Commit changes to backing file).
Wow, that's an interesting one. Looks good to me.
Reviewed-by: Christoph Hellwig <address@hidden>
Btw, the vmdk seems to assume the backing_hd always is a vmdk image,
too. I'm not sure if it is a good assumption. While the backing_hd is
found following the parentFileNameHint field in the image there's
nothing preventing a user / admin from having a different kind of image
in that place.