[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[PATCH 03/14] qdev: Add DeviceClass::hide()
From: |
Akihiko Odaki |
Subject: |
[PATCH 03/14] qdev: Add DeviceClass::hide() |
Date: |
Sat, 02 Dec 2023 17:00:26 +0900 |
DeviceClass::hide() is a better alternative to
DeviceListener::hide_device() that does not need listener registration
and is contained in specific devices that need the hiding capability.
Signed-off-by: Akihiko Odaki <akihiko.odaki@daynix.com>
---
include/hw/qdev-core.h | 33 +++++++++++++++++++++++----------
system/qdev-monitor.c | 11 +++++++++++
2 files changed, 34 insertions(+), 10 deletions(-)
diff --git a/include/hw/qdev-core.h b/include/hw/qdev-core.h
index 6befbca311..de221b6f02 100644
--- a/include/hw/qdev-core.h
+++ b/include/hw/qdev-core.h
@@ -56,16 +56,15 @@
* Hiding a device
* ---------------
*
- * To hide a device, a DeviceListener function hide_device() needs to
- * be registered. It can be used to defer adding a device and
- * therefore hide it from the guest. The handler registering to this
- * DeviceListener can save the QOpts passed to it for re-using it
- * later. It must return if it wants the device to be hidden or
- * visible. When the handler function decides the device shall be
- * visible it will be added with qdev_device_add() and realized as any
- * other device. Otherwise qdev_device_add() will return early without
- * adding the device. The guest will not see a "hidden" device until
- * it was marked visible and qdev_device_add called again.
+ * To hide a device, a DeviceClass function hide() needs to be registered. It
+ * can be used to defer adding a device and therefore hide it from the guest.
+ * The handler can save the QOpts passed to it for re-using it later. It must
+ * return if it wants the device to be hidden or visible. When the handler
+ * function decides the device shall be visible it will be added with
+ * qdev_device_add() and realized as any other device. Otherwise
+ * qdev_device_add() will return early without adding the device. The guest
+ * will not see a "hidden" device until it was marked visible and
+ * qdev_device_add called again.
*
*/
@@ -90,6 +89,8 @@ typedef enum DeviceCategory {
DEVICE_CATEGORY_MAX
} DeviceCategory;
+typedef bool (*DeviceHide)(DeviceClass *dc, const QDict *device_opts,
+ bool from_json, Error **errp);
typedef void (*DeviceRealize)(DeviceState *dev, Error **errp);
typedef void (*DeviceUnrealize)(DeviceState *dev);
typedef void (*DeviceReset)(DeviceState *dev);
@@ -151,6 +152,18 @@ struct DeviceClass {
bool hotpluggable;
/* callbacks */
+ /**
+ * @hide: informs qdev if a device should be visible or hidden.
+ *
+ * This callback is called upon init of the DeviceState.
+ * We can hide a failover device depending for example on the device
+ * opts.
+ *
+ * On errors, it returns false and errp is set. Device creation
+ * should fail in this case.
+ */
+ DeviceHide hide;
+
/**
* @reset: deprecated device reset method pointer
*
diff --git a/system/qdev-monitor.c b/system/qdev-monitor.c
index 71c00f62ee..639beabc5f 100644
--- a/system/qdev-monitor.c
+++ b/system/qdev-monitor.c
@@ -669,6 +669,17 @@ DeviceState *qdev_device_add_from_qdict(const QDict *opts,
return NULL;
}
+ if (dc->hide) {
+ if (dc->hide(dc, opts, from_json, errp)) {
+ if (bus && !qbus_is_hotpluggable(bus)) {
+ error_setg(errp, QERR_BUS_NO_HOTPLUG, bus->name);
+ }
+ return NULL;
+ } else if (*errp) {
+ return NULL;
+ }
+ }
+
if (phase_check(PHASE_MACHINE_READY) && bus && !qbus_is_hotpluggable(bus))
{
error_setg(errp, QERR_BUS_NO_HOTPLUG, bus->name);
return NULL;
--
2.43.0
- [PATCH 00/14] virtio-net: add support for SR-IOV emulation, Akihiko Odaki, 2023/12/02
- [PATCH 01/14] vfio: Avoid inspecting option QDict for rombar, Akihiko Odaki, 2023/12/02
- [PATCH 02/14] hw/qdev: Remove opts member, Akihiko Odaki, 2023/12/02
- [PATCH 03/14] qdev: Add DeviceClass::hide(),
Akihiko Odaki <=
- [PATCH 04/14] hw/pci: Add pci-failover, Akihiko Odaki, 2023/12/02
- [PATCH 05/14] virtio-net: Implement pci-failover, Akihiko Odaki, 2023/12/02
- [PATCH 06/14] qdev: Remove DeviceListener::hide_device(), Akihiko Odaki, 2023/12/02
- [PATCH 08/14] qdev: Add qdev_device_new_from_qdict(), Akihiko Odaki, 2023/12/02
- [PATCH 07/14] hw/pci: Add hide(), Akihiko Odaki, 2023/12/02
- [PATCH 09/14] hw/pci: Do not add ROM BAR for SR-IOV VF, Akihiko Odaki, 2023/12/02
- [PATCH 10/14] msix: Call pcie_sriov_vf_register_bar() for SR-IOV VF, Akihiko Odaki, 2023/12/02
- [PATCH 11/14] pcie_sriov: Release VFs failed to realize, Akihiko Odaki, 2023/12/02
- [PATCH 12/14] pcie_sriov: Allow to specify VF device options, Akihiko Odaki, 2023/12/02
- [PATCH 13/14] virtio-pci: add SR-IOV capability, Akihiko Odaki, 2023/12/02