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>
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(-)