[libvirt] [PATCH v3] Fix up basic migration.

[Eric Blake]

On 05/25/2010 08:33 AM, Chris Lalancette wrote:
> Basic live migration was broken by the commit that added
> non-shared block support in two ways:
> 
> 1)  It added a virCheckFlags() to doNativeMigrate().  Besides
> the fact that typical usage of virCheckFlags() is in driver
> entry points, and doNativeMigrate() is not an entry point,
> it was missing important flags like VIR_MIGRATE_LIVE.  Move
> the virCheckFlags to the top-level qemuDomainMigratePrepare2
> and friends.
> 
> 2)  It also added a memory leak in qemuMonitorTextMigrate()
> by not freeing the memory used by virBufferContentAndReset().
> This is fixed by storing the pointer in a temporary variable
> and freeing it at the end.
> 
> With this patch in place, normal live migration works again.

ACK.

> 
> v3: Instead of the churn for virCheckFlagsUI and UL, instead
> always promote flags to an unsigned long and always use %lx
> for the fprintf.

Yeah, that is much nicer.  And it is easier to extend to unsigned long
long if we ever need it in the future.

-- 
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/20100525/e3a8c8a6/attachment-0001.sig>