On 05/07/2012 11:32 PM, Eric Blake wrote:
On 05/06/2012 08:55 AM, Guannan Ren wrote:
src/qemu/qemu_hostdev.c:
refactor qemuPrepareHostdevUSBDevices function, make it focus on
adding usb device to activeUsbHostdevs after check. After that,
the usb hotplug function qemuDomainAttachHostDevice also could use
it.
expand qemuPrepareHostUSBDevices to perform the usb search,
rollback on failure.

src/qemu/qemu_hotplug.c:
If there are multiple usb devices available with same vendorID and productID,
but with different value of "bus, device", we give an error to let user
use<address>  to specify the desired one.
---
  src/qemu/qemu_hostdev.c |  120 ++++++++++++++++++++++++++++-------------------
  src/qemu/qemu_hostdev.h |    3 +-
  src/qemu/qemu_hotplug.c |   69 +++++++++++++++++++--------
  3 files changed, 122 insertions(+), 70 deletions(-)
ACK; this fixes my concerns.


   Thanks for the review.
   Pushed.

   Guannan Ren

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

Reply via email to