Re: [PATCH 05/17] conf: extend xmlopt with job config & add job object into domain object

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On a Wednesday in 2022, Kristina Hanicova wrote:
This patch adds the generalized job object into the domain object
so that it can be used by all drivers without the need to extract
it from the private data.

Because of this, the job object needs to be created and set
during the creation of the domain object. This patch also extends
xmlopt with possible job config containing virDomainJobObj
callbacks, its private data callbacks and one variable
(maxQueuedJobs).

This patch includes:
* addition of virDomainJobObj into virDomainObj (used in the
 following patches)
* extending xmlopt with job config structure
* new function for freeing the virDomainJobObj

Signed-off-by: Kristina Hanicova <khanicov@xxxxxxxxxx>
---
src/bhyve/bhyve_domain.c      |  2 +-
src/ch/ch_conf.c              |  2 +-
src/conf/domain_conf.c        | 13 ++++++++++++-
src/conf/domain_conf.h        | 16 +++++++++++++++-
src/conf/virconftypes.h       |  2 ++
src/conf/virdomainjob.c       | 11 +++++++++++
src/conf/virdomainjob.h       |  5 ++++-
src/hyperv/hyperv_driver.c    |  2 +-
src/libvirt_private.syms      |  1 +
src/libxl/libxl_conf.c        |  2 +-
src/lxc/lxc_conf.c            |  2 +-
src/openvz/openvz_conf.c      |  2 +-
src/qemu/qemu_conf.c          |  3 ++-
src/qemu/qemu_process.c       |  2 +-
src/security/virt-aa-helper.c |  2 +-
src/test/test_driver.c        |  2 +-
src/vbox/vbox_common.c        |  2 +-
src/vmware/vmware_driver.c    |  2 +-
src/vmx/vmx.c                 |  2 +-
src/vz/vz_driver.c            |  2 +-
tests/bhyveargv2xmltest.c     |  2 +-
tests/testutils.c             |  2 +-
22 files changed, 62 insertions(+), 19 deletions(-)


Reviewed-by: Ján Tomko <jtomko@xxxxxxxxxx>

Jano

Attachment: signature.asc
Description: PGP signature


[Index of Archives]     [Virt Tools]     [Libvirt Users]     [Lib OS Info]     [Fedora Users]     [Fedora Desktop]     [Fedora SELinux]     [Big List of Linux Books]     [Yosemite News]     [KDE Users]     [Fedora Tools]

  Powered by Linux