[libvirt] [PATCH v2 0/8] Simplify and fix qemuMigrationFinish

Jiri Denemark jdenemar at redhat.com
Fri Sep 18 13:30:48 UTC 2015


On Fri, Sep 11, 2015 at 15:26:02 +0200, Jiri Denemark wrote:
> While hacking qemuMigrationFinish I found it pretty hard to follow and
> revealed few bugs (patches 3 to 5, and 8) in the code.
> 
> Version 2:
> - rebased and review comments addressed
> - new patch "qemu: Fix some corner cases in persistent migration"
> 
> Jiri Denemark (8):
>   qemu: Split qemuMigrationFinish
>   qemu: Simplify qemuMigrationFinish
>   qemu: Don't fail migration on save status failure
>   qemu: Kill domain when migration finish fails
>   qemu: Don't report false errors in migration protocol v2
>   qemuDomainEventQueue: Check if event is non-NULL
>   qemu: Queue events in migration Finish phase ASAP
>   qemu: Fix some corner cases in persistent migration
> 
>  src/qemu/qemu_blockjob.c  |   6 +-
>  src/qemu/qemu_cgroup.c    |   3 +-
>  src/qemu/qemu_domain.c    |   6 +-
>  src/qemu/qemu_driver.c    |  87 +++++++-----------
>  src/qemu/qemu_hotplug.c   |  26 +++---
>  src/qemu/qemu_migration.c | 223 ++++++++++++++++++++++++----------------------
>  src/qemu/qemu_process.c   |  72 +++++----------
>  7 files changed, 187 insertions(+), 236 deletions(-)

Pushed with the suggested changes.

Jirka




More information about the libvir-list mailing list