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

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On 03.12.2014 17:39, Eric Blake wrote:
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.



Fixed and pushed now.

Michal

--
libvir-list mailing list
libvir-list@xxxxxxxxxx
https://www.redhat.com/mailman/listinfo/libvir-list




[Index of Archives]     [Virt Tools]     [Libvirt Users]     [Lib OS Info]     [Fedora Users]     [Fedora Desktop]     [Fedora SELinux]     [Big List of Linux Books]     [Yosemite News]     [KDE Users]     [Fedora Tools]