[libvirt] [PATCH] Move all the QEMU migration code to a new file

[Eric Blake]

On 01/28/2011 06:22 AM, Daniel P. Berrange wrote:
> The introduction of the v3 migration protocol, along with
> support for migration cookies, will significantly expand
> the size of the migration code. Move it all to a separate
> file to make it more manageable
> 
> The functions are not moved 100%. The API entry points
> remain in the main QEMU driver, but once the public
> virDomainPtr is resolved to the internal virDomainObjPtr,
> all following code is moved.
> 
> This will allow the new v3 API entry points to call into the
> same shared internal migration functions
> 
> * src/qemu/qemu_domain.c, src/qemu/qemu_domain.h: Add
>   qemuDomainFormatXML helper method
> * src/qemu/qemu_driver.c: Remove all migration code
> * src/qemu/qemu_migration.c, src/qemu/qemu_migration.h: Add
>   all migration code.
> ---
>  src/Makefile.am           |    3 +-
>  src/qemu/qemu_domain.c    |   39 ++
>  src/qemu/qemu_domain.h    |    4 +
>  src/qemu/qemu_driver.c    | 1296 ++-------------------------------------------
>  src/qemu/qemu_migration.c | 1295 ++++++++++++++++++++++++++++++++++++++++++++
>  src/qemu/qemu_migration.h |   63 +++
>  6 files changed, 1444 insertions(+), 1256 deletions(-)
>  create mode 100644 src/qemu/qemu_migration.c
>  create mode 100644 src/qemu/qemu_migration.h

The diffstat makes this look like a cleaner diff than the qemu_process
split, so it should be easier to review.

> -		qemu/qemu_process.c qemu/qemu_process.h	\
> +		qemu/qemu_process.c qemu/qemu_process.h		\
> +		qemu/qemu_migration.c qemu/qemu_migration.h	\

However, this means that you depend on the qemu_process split to happen
first, and that one had issues, so I'm not reviewing this one yet (but I
do like the idea of the patch).

-- 
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/20110129/8cc313ef/attachment-0001.sig>