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

Nikolay Shirokovskiy posted 3 patches 1 week ago
Test syntax-check passed
Patches applied successfully (tree, apply log)
git fetch https://github.com/patchew-project/libvirt tags/patchew/1549612380-898556-1-git-send-email-nshirokovskiy@virtuozzo.com
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(-)

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

Posted by Nikolay Shirokovskiy 1 week ago
Patches 1 and 2 are already Reviewed-by: John. Patch 3 needs Peter comments.

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

-- 
1.8.3.1

--
libvir-list mailing list
libvir-list@redhat.com
https://www.redhat.com/mailman/listinfo/libvir-list

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

Posted by John Ferlan 1 week ago

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

--
libvir-list mailing list
libvir-list@redhat.com
https://www.redhat.com/mailman/listinfo/libvir-list