[libvirt] [PATCH 03/16] Introduce migration cookies to QEMU driver
Eric Blake
eblake at redhat.com
Mon May 16 14:08:14 UTC 2011
On 05/16/2011 08:00 AM, Daniel P. Berrange wrote:
>>>> @@ -369,7 +648,7 @@ cleanup:
>>>> virDomainObjUnlock(vm);
>>>> if (event)
>>>> qemuDomainEventQueue(driver, event);
>>>> - qemuDriverUnlock(driver);
>>>> + qemuMigrationCookieFree(mig);
>>>
>>> Umm, did you really intend to drop the qemuDriverUnlock line?
>>
>> No, that's a merge rebase error
>
> Actually no it wasn't. This is a bug that's being fixed. The caller of
> this method, already unlocks the driver on completion, so we had a
> double unlock.
Then let's split that into a separate patch; bug fixes and new features
in the same patch are harder to deal with during bisection.
--
Eric Blake eblake at redhat.com +1-801-349-2682
Libvirt virtualization library http://libvirt.org
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 619 bytes
Desc: OpenPGP digital signature
URL: <http://listman.redhat.com/archives/libvir-list/attachments/20110516/790d4725/attachment-0001.sig>
More information about the libvir-list
mailing list