Re: [libvirt] [PATCH v2 0/3] qemu: don't duplicate suspended events and state changes

On 2/8/19 2:52 AM, Nikolay Shirokovskiy wrote: > Patches 1 and 2 are already Reviewed-by: John. Patch 3 needs Peter comments. > Right - feel free to add my : Reviewed-by: John Ferlan <jferlan(a)redhat.com&gt; to the first 2 patches for sure. To help push this along, Peter is again CC'd and of importance is the v1 review of patch3: https://www.redhat.com/archives/libvir-list/2018-October/msg00831.html where I noted a specific commit and case to be considered. John > Diff from v1: > ============ > - minor rebase changes > - minor changes according to review > > [1] PATCH v1 : https://www.redhat.com/archives/libvir-list/2018-October/msg00591.html > > Nikolay Shirokovskiy (3): > qemu: Pass stop reason from qemuProcessStopCPUs to stop handler > qemu: Map suspended state reason to suspended event detail > qemu: Don't duplicate suspend events and state changes > > src/qemu/qemu_domain.c | 34 ++++++++++++++++++++++++++++++++++ > src/qemu/qemu_domain.h | 7 +++++++ > src/qemu/qemu_driver.c | 26 +++----------------------- > src/qemu/qemu_migration.c | 42 ++++++------------------------------------ > src/qemu/qemu_migration.h | 4 ---- > src/qemu/qemu_process.c | 38 +++++++++++++++++++++++++------------- > 6 files changed, 75 insertions(+), 76 deletions(-) >