[PATCH 0/5] Generalize job enums across all hypervisors

Kristina Hanicova posted 5 patches 2 years, 1 month ago
Failed in applying to current master (apply log)
Test syntax-check passed
src/ch/ch_domain.c               |  24 +--
src/ch/ch_domain.h               |  18 +-
src/ch/ch_driver.c               |  20 +-
src/hypervisor/domain_job.c      |  32 +++
src/hypervisor/domain_job.h      |  64 ++++++
src/hypervisor/meson.build       |   1 +
src/libvirt_private.syms         |   5 +
src/libxl/libxl_domain.c         |  29 +--
src/libxl/libxl_domain.h         |  17 +-
src/libxl/libxl_driver.c         |  48 ++---
src/libxl/libxl_migration.c      |   6 +-
src/lxc/lxc_domain.c             |  25 +--
src/lxc/lxc_domain.h             |  19 +-
src/lxc/lxc_driver.c             |  46 ++---
src/qemu/MIGRATION.txt           |   6 +-
src/qemu/THREADS.txt             |  16 +-
src/qemu/qemu_backup.c           |  24 +--
src/qemu/qemu_block.c            |  20 +-
src/qemu/qemu_block.h            |  12 +-
src/qemu/qemu_blockjob.c         |  32 +--
src/qemu/qemu_checkpoint.c       |  18 +-
src/qemu/qemu_domain.c           |  26 +--
src/qemu/qemu_domain.h           |   4 +-
src/qemu/qemu_domainjob.c        | 240 ++++++++++------------
src/qemu/qemu_domainjob.h        |  86 ++------
src/qemu/qemu_driver.c           | 332 +++++++++++++++----------------
src/qemu/qemu_hotplug.c          |  50 ++---
src/qemu/qemu_hotplug.h          |  10 +-
src/qemu/qemu_migration.c        | 220 ++++++++++----------
src/qemu/qemu_migration.h        |   8 +-
src/qemu/qemu_migration_params.c |   4 +-
src/qemu/qemu_process.c          | 192 +++++++++---------
src/qemu/qemu_process.h          |  22 +-
src/qemu/qemu_saveimage.c        |   4 +-
src/qemu/qemu_saveimage.h        |   4 +-
src/qemu/qemu_snapshot.c         |  60 +++---
src/qemu/qemu_snapshot.h         |   2 +-
37 files changed, 849 insertions(+), 897 deletions(-)
[PATCH 0/5] Generalize job enums across all hypervisors
Posted by Kristina Hanicova 2 years, 1 month ago
Kristina Hanicova (5):
  move jobs enums QEMU_X into hypervisor as VIR_X
  qemu: move macros QEMU_X into hypervisor as VIR_X
  lxc: use virDomainJob enum instead of virLXCDomainJob
  libxl: use virDomainJob enum instead of libxlDomainJob
  ch: use virDomainJob enum instead of virCHDomainJob

 src/ch/ch_domain.c               |  24 +--
 src/ch/ch_domain.h               |  18 +-
 src/ch/ch_driver.c               |  20 +-
 src/hypervisor/domain_job.c      |  32 +++
 src/hypervisor/domain_job.h      |  64 ++++++
 src/hypervisor/meson.build       |   1 +
 src/libvirt_private.syms         |   5 +
 src/libxl/libxl_domain.c         |  29 +--
 src/libxl/libxl_domain.h         |  17 +-
 src/libxl/libxl_driver.c         |  48 ++---
 src/libxl/libxl_migration.c      |   6 +-
 src/lxc/lxc_domain.c             |  25 +--
 src/lxc/lxc_domain.h             |  19 +-
 src/lxc/lxc_driver.c             |  46 ++---
 src/qemu/MIGRATION.txt           |   6 +-
 src/qemu/THREADS.txt             |  16 +-
 src/qemu/qemu_backup.c           |  24 +--
 src/qemu/qemu_block.c            |  20 +-
 src/qemu/qemu_block.h            |  12 +-
 src/qemu/qemu_blockjob.c         |  32 +--
 src/qemu/qemu_checkpoint.c       |  18 +-
 src/qemu/qemu_domain.c           |  26 +--
 src/qemu/qemu_domain.h           |   4 +-
 src/qemu/qemu_domainjob.c        | 240 ++++++++++------------
 src/qemu/qemu_domainjob.h        |  86 ++------
 src/qemu/qemu_driver.c           | 332 +++++++++++++++----------------
 src/qemu/qemu_hotplug.c          |  50 ++---
 src/qemu/qemu_hotplug.h          |  10 +-
 src/qemu/qemu_migration.c        | 220 ++++++++++----------
 src/qemu/qemu_migration.h        |   8 +-
 src/qemu/qemu_migration_params.c |   4 +-
 src/qemu/qemu_process.c          | 192 +++++++++---------
 src/qemu/qemu_process.h          |  22 +-
 src/qemu/qemu_saveimage.c        |   4 +-
 src/qemu/qemu_saveimage.h        |   4 +-
 src/qemu/qemu_snapshot.c         |  60 +++---
 src/qemu/qemu_snapshot.h         |   2 +-
 37 files changed, 849 insertions(+), 897 deletions(-)

-- 
2.35.1
Re: [PATCH 0/5] Generalize job enums across all hypervisors
Posted by Michal Prívozník 2 years, 1 month ago
On 3/24/22 16:32, Kristina Hanicova wrote:
> 
> Kristina Hanicova (5):
>   move jobs enums QEMU_X into hypervisor as VIR_X
>   qemu: move macros QEMU_X into hypervisor as VIR_X
>   lxc: use virDomainJob enum instead of virLXCDomainJob
>   libxl: use virDomainJob enum instead of libxlDomainJob
>   ch: use virDomainJob enum instead of virCHDomainJob
> 
>  src/ch/ch_domain.c               |  24 +--
>  src/ch/ch_domain.h               |  18 +-
>  src/ch/ch_driver.c               |  20 +-
>  src/hypervisor/domain_job.c      |  32 +++
>  src/hypervisor/domain_job.h      |  64 ++++++
>  src/hypervisor/meson.build       |   1 +
>  src/libvirt_private.syms         |   5 +
>  src/libxl/libxl_domain.c         |  29 +--
>  src/libxl/libxl_domain.h         |  17 +-
>  src/libxl/libxl_driver.c         |  48 ++---
>  src/libxl/libxl_migration.c      |   6 +-
>  src/lxc/lxc_domain.c             |  25 +--
>  src/lxc/lxc_domain.h             |  19 +-
>  src/lxc/lxc_driver.c             |  46 ++---
>  src/qemu/MIGRATION.txt           |   6 +-
>  src/qemu/THREADS.txt             |  16 +-
>  src/qemu/qemu_backup.c           |  24 +--
>  src/qemu/qemu_block.c            |  20 +-
>  src/qemu/qemu_block.h            |  12 +-
>  src/qemu/qemu_blockjob.c         |  32 +--
>  src/qemu/qemu_checkpoint.c       |  18 +-
>  src/qemu/qemu_domain.c           |  26 +--
>  src/qemu/qemu_domain.h           |   4 +-
>  src/qemu/qemu_domainjob.c        | 240 ++++++++++------------
>  src/qemu/qemu_domainjob.h        |  86 ++------
>  src/qemu/qemu_driver.c           | 332 +++++++++++++++----------------
>  src/qemu/qemu_hotplug.c          |  50 ++---
>  src/qemu/qemu_hotplug.h          |  10 +-
>  src/qemu/qemu_migration.c        | 220 ++++++++++----------
>  src/qemu/qemu_migration.h        |   8 +-
>  src/qemu/qemu_migration_params.c |   4 +-
>  src/qemu/qemu_process.c          | 192 +++++++++---------
>  src/qemu/qemu_process.h          |  22 +-
>  src/qemu/qemu_saveimage.c        |   4 +-
>  src/qemu/qemu_saveimage.h        |   4 +-
>  src/qemu/qemu_snapshot.c         |  60 +++---
>  src/qemu/qemu_snapshot.h         |   2 +-
>  37 files changed, 849 insertions(+), 897 deletions(-)
> 

Reviewed-by: Michal Privoznik <mprivozn@redhat.com>

Michal