[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[Qemu-devel] [PATCH 24/25] qdev: implement qdev_init on top of realize
From: |
Paolo Bonzini |
Subject: |
[Qemu-devel] [PATCH 24/25] qdev: implement qdev_init on top of realize |
Date: |
Tue, 3 Apr 2012 13:15:52 +0200 |
Switch the qdev_init function to simply set the "realized"
property to true. The old contents of qdev_init become the
realize and realize_children methods.
Signed-off-by: Paolo Bonzini <address@hidden>
---
hw/qdev.c | 36 ++++++++++++++++++++++++++++++------
1 file changed, 30 insertions(+), 6 deletions(-)
diff --git a/hw/qdev.c b/hw/qdev.c
index d9f811f..e674248 100644
--- a/hw/qdev.c
+++ b/hw/qdev.c
@@ -119,17 +119,17 @@ DeviceState *qdev_try_create(BusState *bus, const char
*type)
calling this function.
On failure, destroy the device and return negative value.
Return 0 on success. */
-int qdev_init(DeviceState *dev)
+static void device_realize(Object *obj, Error **errp)
{
+ DeviceState *dev = DEVICE(obj);
DeviceClass *dc = DEVICE_GET_CLASS(dev);
int rc;
- assert(!object_is_realized(OBJECT(dev)));
-
rc = dc->init(dev);
if (rc < 0) {
qdev_free(dev);
- return rc;
+ error_set(errp, QERR_INVALID_PARAMETER_COMBINATION);
+ return;
}
if (!OBJECT(dev)->parent) {
@@ -146,11 +146,28 @@ int qdev_init(DeviceState *dev)
dev->instance_id_alias,
dev->alias_required_for_version);
}
- OBJECT(dev)->state = OBJECT_STATE_REALIZED;
+}
+
+static void device_realize_children(Object *obj, Error **errp)
+{
+ DeviceState *dev = DEVICE(obj);
+ Error *err = NULL;
+
+ object_realize_children(OBJECT(dev), &err);
+ if (err) {
+ error_propagate(errp, err);
+ return;
+ }
if (dev->hotplugged) {
device_reset(dev);
}
- return 0;
+}
+
+int qdev_init(DeviceState *dev)
+{
+ Error *err = NULL;
+ object_property_set_bool(OBJECT(dev), true, "realized", &err);
+ return err ? -EINVAL : 0;
}
void qdev_set_legacy_instance_id(DeviceState *dev, int alias_id,
@@ -628,6 +645,12 @@ Object *qdev_get_machine(void)
return dev;
}
+static void device_class_init(ObjectClass *klass, void *data)
+{
+ klass->realize = device_realize;
+ klass->realize_children = device_realize_children;
+}
+
static TypeInfo device_type_info = {
.name = TYPE_DEVICE,
.parent = TYPE_OBJECT,
@@ -636,6 +659,7 @@ static TypeInfo device_type_info = {
.instance_finalize = device_finalize,
.abstract = true,
.class_size = sizeof(DeviceClass),
+ .class_init = device_class_init,
};
static void qdev_register_types(void)
--
1.7.9.3
- [Qemu-devel] [PATCH 17/25] qdev: remove PropertyInfo range checking, (continued)
- [Qemu-devel] [PATCH 17/25] qdev: remove PropertyInfo range checking, Paolo Bonzini, 2012/04/03
- [Qemu-devel] [PATCH 19/25] qom: push state up to Object, Paolo Bonzini, 2012/04/03
- [Qemu-devel] [PATCH 23/25] qom: add realized property, Paolo Bonzini, 2012/04/03
- Re: [Qemu-devel] [PATCH 23/25] qom: add realized property, Andreas Färber, 2012/04/03
- Re: [Qemu-devel] [PATCH 23/25] qom: add realized property, Paolo Bonzini, 2012/04/03
- Re: [Qemu-devel] [PATCH 23/25] qom: add realized property, Andreas Färber, 2012/04/05
- Re: [Qemu-devel] [PATCH 23/25] qom: add realized property, Paolo Bonzini, 2012/04/05
- Re: [Qemu-devel] [PATCH 23/25] qom: add realized property, Andreas Färber, 2012/04/05
- Re: [Qemu-devel] [PATCH 23/25] qom: add realized property, Paolo Bonzini, 2012/04/05
- Re: [Qemu-devel] [PATCH 23/25] qom: add realized property, Anthony Liguori, 2012/04/05
[Qemu-devel] [PATCH 24/25] qdev: implement qdev_init on top of realize,
Paolo Bonzini <=
[Qemu-devel] [PATCH 18/25] qdev: remove qdev_prop_exists, Paolo Bonzini, 2012/04/03
[Qemu-devel] [PATCH 20/25] qdev: generalize properties to Objects, Paolo Bonzini, 2012/04/03
[Qemu-devel] [PATCH 25/25] qdev: split part of device_finalize to device_unrealize, Paolo Bonzini, 2012/04/03
[Qemu-devel] [PATCH 21/25] qdev: move bulk of qdev-properties.c to qom/object.c, Paolo Bonzini, 2012/04/03
[Qemu-devel] [PATCH 16/25] qdev: switch property accessors to fixed-width visitor interfaces, Paolo Bonzini, 2012/04/03
[Qemu-devel] [PATCH 22/25] qom: push static properties to Object, Paolo Bonzini, 2012/04/03