@@ -77,16 +77,23 @@ static void bus_add_child(BusState *bus, DeviceState *child)
kid->child = child;
object_ref(OBJECT(kid->child));
- QTAILQ_INSERT_HEAD(&bus->children, kid, sibling);
+ if(child->hidden)
+ {
+ QTAILQ_INSERT_HEAD(&bus->hidden_children, kid, sibling);
+ }
+ else
+ {
+ QTAILQ_INSERT_HEAD(&bus->children, kid, sibling);
- /* This transfers ownership of kid->child to the property. */
- snprintf(name, sizeof(name), "child[%d]", kid->index);
- object_property_add_link(OBJECT(bus), name,
+ /* This transfers ownership of kid->child to the property. */
+ snprintf(name, sizeof(name), "child[%d]", kid->index);
+ object_property_add_link(OBJECT(bus), name,
object_get_typename(OBJECT(child)),
(Object **)&kid->child,
NULL, /* read-only property */
0, /* return ownership on prop deletion */
NULL);
+ }
}
void qdev_set_parent_bus(DeviceState *dev, BusState *bus)
@@ -104,12 +111,38 @@ void qdev_set_parent_bus(DeviceState *dev, BusState *bus)
}
dev->parent_bus = bus;
object_ref(OBJECT(bus));
+
bus_add_child(bus, dev);
+
if (replugging) {
object_unref(OBJECT(dev));
}
}
+void qdev_unhide(const char *dev_id, BusState *bus, Error **errp)
+{
+ BusChild *kid;
+ DeviceState *dev = NULL;
+
+ QTAILQ_FOREACH(kid, &bus->hidden_children, sibling) {
+ if (!strcmp(kid->child->id,dev_id)) {
+ dev = kid->child;
+ break;
+ }
+ }
+
+ if (dev && dev->hidden)
+ {
+ dev->hidden = false;
+ QTAILQ_REMOVE(&bus->hidden_children, kid, sibling);
+ qdev_set_parent_bus(dev, dev->parent_bus);
+ object_property_set_bool(OBJECT(dev), true, "realized", errp);
+ if (!errp)
+ hotplug_handler_plug(bus->hotplug_handler, dev,
+ errp);
+ }
+}
+
/* Create a new device. This only initializes the device state
structure and allows properties to be set. The device still needs
to be realized. See qdev-core.h. */
@@ -208,6 +241,13 @@ void device_listener_unregister(DeviceListener *listener)
QTAILQ_REMOVE(&device_listeners, listener, link);
}
+bool qdev_should_hide_device(const char *dev_id, BusState *bus)
+{
+ bool res;
+ DEVICE_LISTENER_CALL(should_be_hidden, Forward, dev_id, bus, &res);
+ return res;
+}
+
void qdev_set_legacy_instance_id(DeviceState *dev, int alias_id,
int required_for_version)
{
@@ -70,6 +70,7 @@ static Property pci_props[] = {
QEMU_PCIE_LNKSTA_DLLLA_BITNR, true),
DEFINE_PROP_BIT("x-pcie-extcap-init", PCIDevice, cap_present,
QEMU_PCIE_EXTCAP_INIT_BITNR, true),
+ DEFINE_PROP_STRING("standby", PCIDevice, standby_id_str),
DEFINE_PROP_END_OF_LIST()
};
@@ -351,6 +351,8 @@ struct PCIDevice {
MSIVectorUseNotifier msix_vector_use_notifier;
MSIVectorReleaseNotifier msix_vector_release_notifier;
MSIVectorPollNotifier msix_vector_poll_notifier;
+
+ char *standby_id_str;
};
void pci_register_bar(PCIDevice *pci_dev, int region_num,
@@ -143,6 +143,7 @@ struct DeviceState {
char *canonical_path;
bool realized;
bool pending_deleted_event;
+ bool hidden;
QemuOpts *opts;
int hotplugged;
BusState *parent_bus;
@@ -156,6 +157,11 @@ struct DeviceState {
struct DeviceListener {
void (*realize)(DeviceListener *listener, DeviceState *dev);
void (*unrealize)(DeviceListener *listener, DeviceState *dev);
+ /* This callback is called just upon init of the DeviceState
+ * and can be used by a standby device for informing qdev if this
+ * device should be hidden by cross checking the ids
+ */
+ void (*should_be_hidden)(DeviceListener *listener, const char *dev_id, BusState *bus, bool *res);
QTAILQ_ENTRY(DeviceListener) link;
};
@@ -206,6 +212,7 @@ struct BusState {
int max_index;
bool realized;
QTAILQ_HEAD(ChildrenHead, BusChild) children;
+ QTAILQ_HEAD(HiddenChildrenHead, BusChild) hidden_children;
QLIST_ENTRY(BusState) sibling;
};
@@ -360,7 +367,7 @@ int qdev_walk_children(DeviceState *dev,
void qdev_reset_all(DeviceState *dev);
void qdev_reset_all_fn(void *opaque);
-
+void qdev_unhide(const char *dev_id, BusState *bus, Error **errp);
/**
* @qbus_reset_all:
* @bus: Bus to be reset.
@@ -434,4 +441,6 @@ static inline bool qbus_is_hotpluggable(BusState *bus)
void device_listener_register(DeviceListener *listener);
void device_listener_unregister(DeviceListener *listener);
+bool qdev_should_hide_device(const char *dev_id, BusState *bus);
+
#endif
@@ -554,6 +554,49 @@ void qdev_set_id(DeviceState *dev, const char *id)
}
}
+struct DeviceBusTuple
+{
+ DeviceState *dev;
+ BusState *bus;
+} DeviceBusTuple;
+
+static int has_standby_device(void *opaque, const char *name, const char *value,
+ Error **errp)
+{
+ if (strcmp(name, "standby") == 0)
+ {
+ struct DeviceBusTuple *tuple = (struct DeviceBusTuple *)opaque;
+ const char *dev_id = (tuple->dev)->id;
+ BusState *bus = tuple->bus;
+
+ if (qdev_should_hide_device(dev_id, bus))
+ {
+ return 1;
+ }
+ else
+ {
+ error_setg(errp, "An error occurred: Please note that the primary device should be"
+ " must be placed after the standby device in the command line");
+ return -1;
+ }
+ }
+ return 0;
+}
+
+static bool should_hide_device(DeviceState *dev, QemuOpts *opts,BusState *bus, Error **err)
+{
+ struct DeviceBusTuple tuple;
+ tuple.dev = dev;
+ tuple.bus = bus;
+
+ if (//!qemu_opt_get(opts, "vfio-pci") ||
+ qemu_opt_foreach(opts, has_standby_device, &tuple, err) == 0)
+ {
+ return false;
+ }
+ return true;
+}
+
DeviceState *qdev_device_add(QemuOpts *opts, Error **errp)
{
DeviceClass *dc;
@@ -607,6 +650,13 @@ DeviceState *qdev_device_add(QemuOpts *opts, Error **errp)
/* create device */
dev = DEVICE(object_new(driver));
+ qdev_set_id(dev, qemu_opts_id(opts));
+
+ dev->hidden = should_hide_device(dev, opts, bus, &err);
+ if (err)
+ {
+ goto err_del_dev;
+ }
if (bus) {
qdev_set_parent_bus(dev, bus);
} else if (qdev_hotplug && !qdev_get_machine_hotplug_handler(dev)) {
@@ -616,15 +666,17 @@ DeviceState *qdev_device_add(QemuOpts *opts, Error **errp)
goto err_del_dev;
}
- qdev_set_id(dev, qemu_opts_id(opts));
-
/* set properties */
if (qemu_opt_foreach(opts, set_property, dev, &err)) {
goto err_del_dev;
}
dev->opts = opts;
- object_property_set_bool(OBJECT(dev), true, "realized", &err);
+ if (!dev->hidden)
+ {
+ object_property_set_bool(OBJECT(dev), true, "realized", &err);
+ }
+
if (err != NULL) {
dev->opts = NULL;
goto err_del_dev;