Re: [libvirt] [PATCH]: Fix non-live migration failure

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

 



Daniel Veillard wrote:
> On Fri, Feb 27, 2009 at 01:12:40PM +0100, Chris Lalancette wrote:
>> There is a logic error in the Qemu driver when doing a non-live migrate.
>> During a non-live migrate, on the source host during the Perform step, we
>> pause the domain; however, if there was ever a failure, we were forgetting
>> to unpause the domain, meaning that the domain was paused forever.  Add a
>> flag to tell us when we should unpause the domain after a failure.
>>
>> Also, as pointed out by DV, we were unnecessarily using a snprintf to a
>> buffer to execute qemu monitor commands, when we could get away with
>> just sending a static string.  Fix that as well.
> 
>   Oh right I had missed the leak :-)
> Looks fine to me now,

Great, thanks for looking again.  Now committed.

-- 
Chris Lalancette

--
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]