[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[PULL 06/33] spapr_vscsi: do not allow device hotplug
From: |
David Gibson |
Subject: |
[PULL 06/33] spapr_vscsi: do not allow device hotplug |
Date: |
Tue, 8 Sep 2020 15:19:26 +1000 |
From: Daniel Henrique Barboza <danielhb413@gmail.com>
We do not implement hotplug in the vscsi bus, but we forgot to
tell qdev about it. The result is that users are able to hotplug
devices in the vscsi bus, the devices appear in qdev, but they
aren't usable by the guest OS unless the user reboots it first.
Setting qbus hotplug_handler to NULL will tell qdev-monitor, via
qbus_is_hotpluggable(), that we do not support hotplug operations
in spapr_vscsi.
Fixes: https://bugzilla.redhat.com/show_bug.cgi?id=1862059
Signed-off-by: Daniel Henrique Barboza <danielhb413@gmail.com>
Message-Id: <20200820190635.379657-1-danielhb413@gmail.com>
Signed-off-by: David Gibson <david@gibson.dropbear.id.au>
---
hw/scsi/spapr_vscsi.c | 3 +++
1 file changed, 3 insertions(+)
diff --git a/hw/scsi/spapr_vscsi.c b/hw/scsi/spapr_vscsi.c
index d17dc03c73..57f0a1336f 100644
--- a/hw/scsi/spapr_vscsi.c
+++ b/hw/scsi/spapr_vscsi.c
@@ -1219,6 +1219,9 @@ static void spapr_vscsi_realize(SpaprVioDevice *dev,
Error **errp)
scsi_bus_new(&s->bus, sizeof(s->bus), DEVICE(dev),
&vscsi_scsi_info, NULL);
+
+ /* ibmvscsi SCSI bus does not allow hotplug. */
+ qbus_set_hotplug_handler(BUS(&s->bus), NULL);
}
void spapr_vscsi_create(SpaprVioBus *bus)
--
2.26.2
- [PULL 00/33] ppc-for-5.2 queue 20200908, David Gibson, 2020/09/08
- [PULL 01/33] adb: Correct class size on TYPE_ADB_DEVICE, David Gibson, 2020/09/08
- [PULL 02/33] ppc/pnv: Fix TypeInfo of PnvLpcController abstract class, David Gibson, 2020/09/08
- [PULL 03/33] spapr: Remove unnecessary DRC type-checker macros, David Gibson, 2020/09/08
- [PULL 04/33] spapr/xive: Add a 'hv-prio' property to represent the KVM escalation priority, David Gibson, 2020/09/08
- [PULL 07/33] spapr/xive: Use the xics flag to check for XIVE-only IRQ backends, David Gibson, 2020/09/08
- [PULL 05/33] ppc/pnv: Add a HIOMAP erase command, David Gibson, 2020/09/08
- [PULL 08/33] spapr/xive: Modify kvm_cpu_is_enabled() interface, David Gibson, 2020/09/08
- [PULL 06/33] spapr_vscsi: do not allow device hotplug,
David Gibson <=
- [PULL 09/33] spapr/xive: Use kvmppc_xive_source_reset() in post_load, David Gibson, 2020/09/08
- [PULL 10/33] spapr/xive: Allocate IPIs independently from the other sources, David Gibson, 2020/09/08
- [PULL 11/33] spapr/xive: Allocate vCPU IPIs from the vCPU contexts, David Gibson, 2020/09/08
- [PULL 12/33] ppc/spapr_nvdimm: use g_autofree in spapr_nvdimm_validate_opts(), David Gibson, 2020/09/08
- [PULL 13/33] spapr, spapr_nvdimm: fold NVDIMM validation in the same place, David Gibson, 2020/09/08
- [PULL 14/33] ppc/spapr_nvdimm: do not enable support with 'nvdimm=off', David Gibson, 2020/09/08
- [PULL 15/33] target/arm: Move start-powered-off property to generic CPUState, David Gibson, 2020/09/08
- [PULL 18/33] ppc/e500: Use start-powered-off CPUState property, David Gibson, 2020/09/08
- [PULL 17/33] ppc/spapr: Use start-powered-off CPUState property, David Gibson, 2020/09/08
- [PULL 20/33] sparc/sun4m: Don't set cs->halted = 0 in main_cpu_reset(), David Gibson, 2020/09/08