[libvirt] [PATCH] qemu: Fix virsh freeze when blockcopy storage file is removed

Eric Blake eblake at redhat.com
Wed Dec 3 16:39:10 UTC 2014


On 12/03/2014 05:56 AM, Erik Skultety wrote:
> If someone removes blockcopy storage file when still in mirroring phase

That's fishy behavior on the part of the user.  But since the patch is
simple enough, it's still worth protecting ourselves from stupid users.

> and then requesting blockjob abort using pivot, virsh cmd freezes. This
> is not an issue with older qemu versions which did not support
> asynchronous jobs (which we prefer by default).
> As we have reached the mirroring phase successfully, polling monitor for
> blockjob info always returns 1 and the loop never ends.
> This fix introduces a check for qemuDomainBlockPivot return code, possibly
> skipping the asynchronous waiting completely, if an error occured and

s/occured/occurred/

> asynchronous waiting was the preferred method.
> 
> Resolves: https://bugzilla.redhat.com/show_bug.cgi?id=1139567
> ---
>  src/qemu/qemu_driver.c | 2 ++
>  1 file changed, 2 insertions(+)

ACK.

-- 
Eric Blake   eblake redhat com    +1-919-301-3266
Libvirt virtualization library http://libvirt.org

-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 539 bytes
Desc: OpenPGP digital signature
URL: <http://listman.redhat.com/archives/libvir-list/attachments/20141203/6083ad7d/attachment-0001.sig>


More information about the libvir-list mailing list