[libvirt] [PATCHv3 0/7] Finish the conversion to virConfGetValue* functions

Fabiano Fidêncio fabiano at fidencio.org
Sun May 27 22:28:19 UTC 2018


This patchset finishes the conversion to virConfGetValue* functions,
started by Daniel Berrange a few months ago.

Please, mind that although we could make virConfGetValue* functions more
generic in order to support numbers and booleans as strings, that
doesn't seem the safest path to take. The side-effect of this is that we
will have to live with some specific code doing that as part of vmx and
xen_common.

Once this patchset gets merged,
https://wiki.libvirt.org/page/BiteSizedTasks#Finish_conversion_to_virConfGetValue.2A_functions
can be removed.

- Changes since v1:
  All the "values" from virConfGetValueString() are freed

- Changes since v2:
  All comments from Ján Tomko have been addressed;
  A few leaks were (possibly) found and they're addressed in the last
  patch of the series;

Fabiano Fidêncio (7):
  xen_xm: Split the per-disk logic from xenParseXMDisk()
  xen_vm: convert to typesafe virConf accessors
  vmx: convert to typesafe virConf accessors
  xen_common: Split per-PCI logic from xenParsePCI()
  xen_common: Split per-Vfi logic from xenParseVif()
  xen_common: convert to typesafe virConf acessors
  xen_common: Fix a few memory leaks

 src/vmx/vmx.c              | 194 +++++--------
 src/xenconfig/xen_common.c | 701 +++++++++++++++++++++++----------------------
 src/xenconfig/xen_xm.c     | 306 ++++++++++----------
 3 files changed, 595 insertions(+), 606 deletions(-)

-- 
2.14.3




More information about the libvir-list mailing list