[libvirt] [PATCH v2 00/10] Cleanups in QEMU driver wrt virConnectPtr

Daniel P. Berrangé posted 10 patches 6 years, 2 months ago
Failed in applying to current master (apply log)
There is a newer version of this series
src/conf/domain_conf.c       |  90 +++++++-
src/conf/domain_conf.h       |  14 +-
src/driver.c                 | 172 +++++++++++++-
src/driver.h                 |   7 +
src/libvirt_private.syms     |   6 +
src/network/bridge_driver.c  |  76 +------
src/qemu/qemu_conf.c         |   3 +-
src/qemu/qemu_conf.h         |   3 +-
src/qemu/qemu_domain.c       | 111 +++++----
src/qemu/qemu_domain.h       |  15 +-
src/qemu/qemu_driver.c       | 278 ++++++++++-------------
src/qemu/qemu_hotplug.c      |  66 +++---
src/qemu/qemu_hotplug.h      |  15 +-
src/qemu/qemu_migration.c    | 526 +++++++++++++++++++++++--------------------
src/qemu/qemu_migration.h    | 192 +++++++---------
src/qemu/qemu_monitor.c      |  10 +-
src/qemu/qemu_monitor.h      |  11 +-
src/qemu/qemu_monitor_json.c |   3 +-
src/qemu/qemu_monitor_json.h |   3 +-
src/qemu/qemu_monitor_text.c |   9 +-
src/qemu/qemu_monitor_text.h |   3 +-
src/qemu/qemu_process.c      | 134 +++++------
src/qemu/qemu_process.h      |  12 +-
tests/Makefile.am            |   7 +-
tests/qemuhotplugtest.c      |   4 +-
tests/qemumonitorjsontest.c  |   2 +-
tests/qemuxml2argvtest.c     |   7 +-
27 files changed, 907 insertions(+), 872 deletions(-)
[libvirt] [PATCH v2 00/10] Cleanups in QEMU driver wrt virConnectPtr
Posted by Daniel P. Berrangé 6 years, 2 months ago
There are many places we can stop passing around virConnectPtr now
and directly open secondary drivers where required instead.

v2:

  - Now with working tests !

Daniel P. Berrangé (10):
  driver: allow override of connection for secondary drivers
  conf: reimplement virDomainNetResolveActualType in terms of public API
  qemu: stop passing virConnectPtr into qemuMonitorStartCPUs
  conf: stop passing virConnectPtr into virDomainDiskTranslateSourcePool
  qemu: don't pass virConnectPtr around for secrets
  qemu: stop passing in virConnectPtr for looking up networks
  qemu: remove virConnectPtr from some more startup code paths
  qemu: remove virConnectPtr in some migration methods
  qemu: don't export migration job APIs
  qemu: rename migration APIs to include Src or Dst in their name

 src/conf/domain_conf.c       |  90 +++++++-
 src/conf/domain_conf.h       |  14 +-
 src/driver.c                 | 172 +++++++++++++-
 src/driver.h                 |   7 +
 src/libvirt_private.syms     |   6 +
 src/network/bridge_driver.c  |  76 +------
 src/qemu/qemu_conf.c         |   3 +-
 src/qemu/qemu_conf.h         |   3 +-
 src/qemu/qemu_domain.c       | 111 +++++----
 src/qemu/qemu_domain.h       |  15 +-
 src/qemu/qemu_driver.c       | 278 ++++++++++-------------
 src/qemu/qemu_hotplug.c      |  66 +++---
 src/qemu/qemu_hotplug.h      |  15 +-
 src/qemu/qemu_migration.c    | 526 +++++++++++++++++++++++--------------------
 src/qemu/qemu_migration.h    | 192 +++++++---------
 src/qemu/qemu_monitor.c      |  10 +-
 src/qemu/qemu_monitor.h      |  11 +-
 src/qemu/qemu_monitor_json.c |   3 +-
 src/qemu/qemu_monitor_json.h |   3 +-
 src/qemu/qemu_monitor_text.c |   9 +-
 src/qemu/qemu_monitor_text.h |   3 +-
 src/qemu/qemu_process.c      | 134 +++++------
 src/qemu/qemu_process.h      |  12 +-
 tests/Makefile.am            |   7 +-
 tests/qemuhotplugtest.c      |   4 +-
 tests/qemumonitorjsontest.c  |   2 +-
 tests/qemuxml2argvtest.c     |   7 +-
 27 files changed, 907 insertions(+), 872 deletions(-)

-- 
2.14.3

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