<div dir="ltr"><div><div>Thanks Peter,<br><br></div><div>It looks good but I did no deep introspection of the code.<br></div><div><br></div>Should I rebase my patch [1] on that? I can test your code as well then.<br><br>[1] <a href="https://github.com/paboldin/libvirt/commits/master">https://github.com/paboldin/libvirt/commits/master</a><br><br></div>Pavel<br></div><div class="gmail_extra"><br><div class="gmail_quote">On Tue, Apr 28, 2015 at 4:42 PM, Peter Krempa <span dir="ltr"><<a href="mailto:pkrempa@redhat.com" target="_blank">pkrempa@redhat.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div class="HOEnZb"><div class="h5">On Tue, Apr 28, 2015 at 11:24:37 +0200, Michal Privoznik wrote:<br>
> On 28.04.2015 11:06, Pavel Boldin wrote:<br>
> > Well, actually that seems to be quite a different bug in there.<br>
> ><br>
> > I will start a new thread.<br>
> ><br>
> > In short: migration seems to be broken by commit<br>
> > 1a92c719101e5bfa6fe2b78006ad04c7f075ea28. This is because introduced job<br>
> > _MODIFY waits while MIGRATION_OUT is finished to change `mirrorState'<br>
> > variable. This deadlocks the libvirt.<br>
><br>
> Yep, this is known bug. I've told Peter already like two weeks ago. He<br>
> promised to fix it. It would be nice if we can get the fix into the release.<br>
<br>
</div></div>There are already patches for the issue:<br>
<br>
<a href="http://www.redhat.com/archives/libvir-list/2015-April/msg00724.html" target="_blank">http://www.redhat.com/archives/libvir-list/2015-April/msg00724.html</a><br>
</blockquote></div><br></div>