[libvirt] [PATCH 0/7] Error path cleanup for hotplug and one function move/rename

John Ferlan jferlan at redhat.com
Fri Jul 15 11:50:20 UTC 2016


During review of the LUKS series:

http://www.redhat.com/archives/libvir-list/2016-July/msg00488.html

it was noted that the error paths should use bools rather than goto failxxx
type labels.  So this series does that for a few of the affect APIs. There's
also changes to qemuDomainAttachMemory since it seems it wasn't properly
handling 'props' at least w/r/t how qemuDomainAttachRNGDevice would clean
things up.

Finally - another part of the review noted that the qemu command line
comma escaping could be made more available since the storage driver
would potentially need it to build the qemuimg command line. So I moved
to virqemu and renamed.

John Ferlan (7):
  qemu: Alter error path cleanup for qemuDomainAttachHostSCSIDevice
  qemu: Alter error path cleanup for qemuDomainAttachVirtioDiskDevice
  qemu: Alter error path cleanup for qemuDomainAttachSCSIDisk
  qemu: Alter error path cleanup for qemuDomainAttachChrDevice
  qemu: Alter error path cleanup for qemuDomainAttachRNGDevice
  qemu: Alter error path cleanup for qemuDomainAttachMemory
  qemu: Move and rename qemuBufferEscapeComma

 src/libvirt_private.syms |   1 +
 src/qemu/qemu_command.c  |  33 +++-------
 src/qemu/qemu_hotplug.c  | 163 +++++++++++++++++++++++++++--------------------
 src/util/virqemu.c       |  15 +++++
 src/util/virqemu.h       |   2 +
 5 files changed, 122 insertions(+), 92 deletions(-)

-- 
2.5.5




More information about the libvir-list mailing list