Re: [PATCH v2 0/9] Various hotplug cleanups

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

 



On Tue, Jul 19, 2016 at 10:30:43AM -0400, John Ferlan wrote:
v1: http://www.redhat.com/archives/libvir-list/2016-June/msg02207.html

Changes since v1...
- Use the bz noted by Cole from the v1 series
- Merged in recent hotplug cleanup changes
- Patches 6 - 9 are new

Resolves: https://bugzilla.redhat.com/show_bug.cgi?id=1336225

and probably: https://bugzilla.redhat.com/show_bug.cgi?id=1289391

Patches 1-5 address the USB hotplug cleanup

Patch 6 addresses the SCSI hotplug cleanup

Patch 7-9 address some SCSI host device cleanups found while looking at
  the other cleanups...

John Ferlan (9):
 qemu: Reorder qemuDomainAttachUSBMassStorageDevice failure path
 qemu: Move drive alias processing to qemu_alias
 qemu: Use qemuAssignDeviceDiskDriveAlias
 qemu: Make QEMU_DRIVE_HOST_PREFIX more private
 qemu: Add attempt to call qemuMonitorDriveDel for USB failure path
 qemu: Add attempt to call qemuMonitorDriveDel for AttachSCSI failure
   path
 qemu: Introduce qemuAssignSCSIHostDeviceDriveAlias
 qemu: Use qemuAssignSCSIHostDeviceDriveAlias
 qemu: Use the hostdev alias in qemuDomainAttachHostSCSIDevice error
   path

src/qemu/qemu_alias.c        | 52 +++++++++++++++++++++++++
src/qemu/qemu_alias.h        |  6 +++
src/qemu/qemu_command.c      | 35 ++++++++---------
src/qemu/qemu_command.h      |  1 -
src/qemu/qemu_domain.c       |  3 +-
src/qemu/qemu_driver.c       |  3 +-
src/qemu/qemu_hotplug.c      | 92 +++++++++++++++++++++++++++++---------------
src/qemu/qemu_migration.c    | 11 ++----
src/qemu/qemu_monitor_json.c | 10 ++---
src/qemu/qemu_monitor_text.c | 18 +++++----
src/qemu/qemu_process.c      |  3 +-
11 files changed, 156 insertions(+), 78 deletions(-)


ACK series.

Jan

--
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]