Re: [PATCH] Don't generate cookies with v2 migration protocol.

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

 



On 05/18/2011 11:08 AM, Daniel P. Berrange wrote:
> The v2 migration protocol had a limit on cookie length that was
> too small to be useful for QEMU. Avoid generating cookies with
> v2 protocol, so that old libvirtd can still reliably migrate a
> guest to new libvirtd uses v2 protocol.
> 
> * src/qemu/qemu_driver.c: Avoid migration cookies with v2
>   migration
> ---
>  src/qemu/qemu_driver.c |   27 ++++++++++++++++++++-------
>  1 files changed, 20 insertions(+), 7 deletions(-)

Is it any better to try generating the cookie, and only discard it if it
was too long, than to blindly skip the cookie?  Or are we pretty sure
that the cookie will always be longer than the old limit (256, if I'm
reading REMOTE_MIGRATE_COOKIE_MAX correctly in remote_protocol.x)?

If the latter, then ACK.

-- 
Eric Blake   eblake@xxxxxxxxxx    +1-801-349-2682
Libvirt virtualization library http://libvirt.org

Attachment: signature.asc
Description: OpenPGP digital signature

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