[PATCH 0/9] Grab modify job for changing domain XML

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

 



This is an alternative proposal to:

https://www.redhat.com/archives/libvir-list/2019-May/msg00830.html

The problem I'm trying to fix is described here:

https://www.redhat.com/archives/libvir-list/2019-May/msg00810.html

Michal Prívozník (9):
  virDomainObjListAddLocked: Drop useless @cleanup label
  virDomainObjListAddObjLocked: Don't expect vm->def to be set
  virDomainObjListAddLocked: Set vm->def only in success path
  virDomainObjIsActive: Allow vm->def to be NULL
  virDomainObjListAdd: Leave def assigning as an exercise for caller
  qemu: Allow vm->def == NULL in job control APIs
  qemu: Grab modify job for changing domain XML
  lxc: Grab modify job for changing domain XML
  libxl: Grab modify job for changing domain XML

 src/bhyve/bhyve_driver.c    | 10 +++++---
 src/conf/domain_conf.h      |  2 +-
 src/conf/virdomainobjlist.c | 48 ++++++++++++++----------------------
 src/conf/virdomainobjlist.h |  3 +--
 src/libxl/libxl_domain.c    |  3 ++-
 src/libxl/libxl_driver.c    | 48 ++++++++++++++++++++++++------------
 src/libxl/libxl_migration.c | 14 +++++------
 src/lxc/lxc_domain.c        |  3 ++-
 src/lxc/lxc_driver.c        | 23 +++++++++++------
 src/openvz/openvz_conf.c    | 12 ++++-----
 src/openvz/openvz_driver.c  | 17 +++++++------
 src/qemu/qemu_domain.c      | 30 ++++++++++++++---------
 src/qemu/qemu_driver.c      | 49 ++++++++++++++++++++++++++-----------
 src/qemu/qemu_migration.c   | 13 +++++++---
 src/test/test_driver.c      | 21 +++++++++-------
 src/vmware/vmware_conf.c    |  4 +--
 src/vmware/vmware_driver.c  |  9 +++----
 src/vz/vz_sdk.c             |  4 ++-
 18 files changed, 183 insertions(+), 130 deletions(-)

-- 
2.21.0

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




[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