[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH 25/30] usb-bot: drop not needed "allow_hotplug =
From: |
Gerd Hoffmann |
Subject: |
Re: [Qemu-devel] [PATCH 25/30] usb-bot: drop not needed "allow_hotplug = 0" |
Date: |
Thu, 25 Sep 2014 09:59:31 +0200 |
Hi,
> But there is room for improvement here,
> it it's possible to error out even earlier if usb-bot would be marked
> as not hotpluggable device and add to qdev_device_add() check if
> device is hotpluggable even before it's created.
usb-bot not being hot-pluggable is a consequence of the fact that we can
only hotplug single devices in qemu today. usb-bot without scsi devices
hooked up is useless. Plugging in usb-bot first and plugging in disks
later doesn't work (well, could sort-of work when you plug them fast,
but you are racing with the guest initializing the device then). With
some way to first compose the whole thing (usb-bot + disks), then
plug-in as whole usb-bot hotplugging would work though.
cheers,
Gerd
- Re: [Qemu-devel] [PATCH 23/30] scsi: convert virtio-scsi HBA to hotplug handler API, (continued)
- [Qemu-devel] [PATCH 24/30] scsi: cleanup not used anymore SCSIBusInfo{hotplug, hot_unplug} fields, Igor Mammedov, 2014/09/24
- [Qemu-devel] [PATCH 25/30] usb-bot: drop not needed "allow_hotplug = 0", Igor Mammedov, 2014/09/24
- Re: [Qemu-devel] [PATCH 25/30] usb-bot: drop not needed "allow_hotplug = 0", Paolo Bonzini, 2014/09/24
- Re: [Qemu-devel] [PATCH 25/30] usb-bot: drop not needed "allow_hotplug = 0", Igor Mammedov, 2014/09/24
- Re: [Qemu-devel] [PATCH 25/30] usb-bot: drop not needed "allow_hotplug = 0", Paolo Bonzini, 2014/09/24
- Re: [Qemu-devel] [PATCH 25/30] usb-bot: drop not needed "allow_hotplug = 0", Gerd Hoffmann, 2014/09/25
- Re: [Qemu-devel] [PATCH 25/30] usb-bot: drop not needed "allow_hotplug = 0", Igor Mammedov, 2014/09/25
- Re: [Qemu-devel] [PATCH 25/30] usb-bot: drop not needed "allow_hotplug = 0", Gerd Hoffmann, 2014/09/25
- Re: [Qemu-devel] [PATCH 25/30] usb-bot: drop not needed "allow_hotplug = 0",
Gerd Hoffmann <=
[Qemu-devel] [PATCH 26/30] usb-storage: make its storage SCSI bus hotpluggable explicitly, Igor Mammedov, 2014/09/24
[Qemu-devel] [PATCH 30/30] qdev: HotplugHandler: add support for unplugging BUS-less devices, Igor Mammedov, 2014/09/24
[Qemu-devel] [PATCH 16/30] virtio-serial: convert to hotplug-handler API, Igor Mammedov, 2014/09/24