[PATCH 0/5] jobs: qemu & hypervisor: move hypervisor-based variables and functions

Kristina Hanicova posted 5 patches 1 year, 8 months ago
Patches applied successfully (tree, apply log)
git fetch https://github.com/patchew-project/libvirt tags/patchew/cover.1660136124.git.khanicov@redhat.com
src/ch/ch_domain.c               |   2 +-
src/hypervisor/domain_job.c      |   4 +-
src/hypervisor/domain_job.h      |   8 +-
src/libxl/libxl_domain.c         |   2 +-
src/lxc/lxc_domain.c             |   2 +-
src/qemu/qemu_backup.c           |  21 +-
src/qemu/qemu_backup.h           |   3 +-
src/qemu/qemu_block.c            |  26 +-
src/qemu/qemu_block.h            |   6 +-
src/qemu/qemu_blockjob.c         |  25 +-
src/qemu/qemu_blockjob.h         |   3 +-
src/qemu/qemu_checkpoint.c       |  15 +-
src/qemu/qemu_domain.c           |  51 ++-
src/qemu/qemu_domain.h           |  21 +-
src/qemu/qemu_domainjob.c        |  45 +--
src/qemu/qemu_domainjob.h        |  15 +-
src/qemu/qemu_driver.c           | 581 +++++++++++++------------------
src/qemu/qemu_hotplug.c          | 217 +++++-------
src/qemu/qemu_hotplug.h          |  39 +--
src/qemu/qemu_migration.c        | 235 ++++++-------
src/qemu/qemu_migration.h        |  15 +-
src/qemu/qemu_migration_cookie.c |   5 +-
src/qemu/qemu_migration_params.c |  38 +-
src/qemu/qemu_migration_params.h |  15 +-
src/qemu/qemu_process.c          | 201 +++++------
src/qemu/qemu_process.h          |  12 +-
src/qemu/qemu_snapshot.c         |  24 +-
tests/qemuhotplugtest.c          |   6 +-
28 files changed, 699 insertions(+), 938 deletions(-)
[PATCH 0/5] jobs: qemu & hypervisor: move hypervisor-based variables and functions
Posted by Kristina Hanicova 1 year, 8 months ago
*** BLURB HERE ***

Kristina Hanicova (5):
  qemu: beginJob: move saveStatus into private job callbacks
  hypervisor: domain_job: add maxQueuedJobs
  qemu: remove unused driver and all its propagations
  qemu: refactor functions with removed driver if possible
  hypervisor: domain_job: add JobData private callbacks into
    virDomainJobObj

 src/ch/ch_domain.c               |   2 +-
 src/hypervisor/domain_job.c      |   4 +-
 src/hypervisor/domain_job.h      |   8 +-
 src/libxl/libxl_domain.c         |   2 +-
 src/lxc/lxc_domain.c             |   2 +-
 src/qemu/qemu_backup.c           |  21 +-
 src/qemu/qemu_backup.h           |   3 +-
 src/qemu/qemu_block.c            |  26 +-
 src/qemu/qemu_block.h            |   6 +-
 src/qemu/qemu_blockjob.c         |  25 +-
 src/qemu/qemu_blockjob.h         |   3 +-
 src/qemu/qemu_checkpoint.c       |  15 +-
 src/qemu/qemu_domain.c           |  51 ++-
 src/qemu/qemu_domain.h           |  21 +-
 src/qemu/qemu_domainjob.c        |  45 +--
 src/qemu/qemu_domainjob.h        |  15 +-
 src/qemu/qemu_driver.c           | 581 +++++++++++++------------------
 src/qemu/qemu_hotplug.c          | 217 +++++-------
 src/qemu/qemu_hotplug.h          |  39 +--
 src/qemu/qemu_migration.c        | 235 ++++++-------
 src/qemu/qemu_migration.h        |  15 +-
 src/qemu/qemu_migration_cookie.c |   5 +-
 src/qemu/qemu_migration_params.c |  38 +-
 src/qemu/qemu_migration_params.h |  15 +-
 src/qemu/qemu_process.c          | 201 +++++------
 src/qemu/qemu_process.h          |  12 +-
 src/qemu/qemu_snapshot.c         |  24 +-
 tests/qemuhotplugtest.c          |   6 +-
 28 files changed, 699 insertions(+), 938 deletions(-)

-- 
2.37.1
Re: [PATCH 0/5] jobs: qemu & hypervisor: move hypervisor-based variables and functions
Posted by Ján Tomko 1 year, 8 months ago
On a Wednesday in 2022, Kristina Hanicova wrote:
>*** BLURB HERE ***
>
>Kristina Hanicova (5):
>  qemu: beginJob: move saveStatus into private job callbacks
>  hypervisor: domain_job: add maxQueuedJobs
>  qemu: remove unused driver and all its propagations
>  qemu: refactor functions with removed driver if possible
>  hypervisor: domain_job: add JobData private callbacks into
>    virDomainJobObj
>
> src/ch/ch_domain.c               |   2 +-
> src/hypervisor/domain_job.c      |   4 +-
> src/hypervisor/domain_job.h      |   8 +-
> src/libxl/libxl_domain.c         |   2 +-
> src/lxc/lxc_domain.c             |   2 +-
> src/qemu/qemu_backup.c           |  21 +-
> src/qemu/qemu_backup.h           |   3 +-
> src/qemu/qemu_block.c            |  26 +-
> src/qemu/qemu_block.h            |   6 +-
> src/qemu/qemu_blockjob.c         |  25 +-
> src/qemu/qemu_blockjob.h         |   3 +-
> src/qemu/qemu_checkpoint.c       |  15 +-
> src/qemu/qemu_domain.c           |  51 ++-
> src/qemu/qemu_domain.h           |  21 +-
> src/qemu/qemu_domainjob.c        |  45 +--
> src/qemu/qemu_domainjob.h        |  15 +-
> src/qemu/qemu_driver.c           | 581 +++++++++++++------------------
> src/qemu/qemu_hotplug.c          | 217 +++++-------
> src/qemu/qemu_hotplug.h          |  39 +--
> src/qemu/qemu_migration.c        | 235 ++++++-------
> src/qemu/qemu_migration.h        |  15 +-
> src/qemu/qemu_migration_cookie.c |   5 +-
> src/qemu/qemu_migration_params.c |  38 +-
> src/qemu/qemu_migration_params.h |  15 +-
> src/qemu/qemu_process.c          | 201 +++++------
> src/qemu/qemu_process.h          |  12 +-
> src/qemu/qemu_snapshot.c         |  24 +-
> tests/qemuhotplugtest.c          |   6 +-
> 28 files changed, 699 insertions(+), 938 deletions(-)
>

Reviewed-by: Ján Tomko <jtomko@redhat.com>

Good job with the jobs, Kristina!

Jano