[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[Qemu-ppc] [PATCH for-3.2 v3 10/14] qdev-props: call object_apply_global
From: |
Marc-André Lureau |
Subject: |
[Qemu-ppc] [PATCH for-3.2 v3 10/14] qdev-props: call object_apply_global_props() |
Date: |
Wed, 7 Nov 2018 16:36:48 +0400 |
It's now possible to use the common function.
Teach object_apply_global_props() to warn if Error argument is NULL.
Signed-off-by: Marc-André Lureau <address@hidden>
---
hw/core/qdev-properties.c | 24 ++----------------------
qom/object.c | 6 +++++-
2 files changed, 7 insertions(+), 23 deletions(-)
diff --git a/hw/core/qdev-properties.c b/hw/core/qdev-properties.c
index 8728cbab9f..239535a4cb 100644
--- a/hw/core/qdev-properties.c
+++ b/hw/core/qdev-properties.c
@@ -1223,28 +1223,8 @@ int qdev_prop_check_globals(void)
void qdev_prop_set_globals(DeviceState *dev)
{
- int i;
-
- for (i = 0; i < global_props()->len; i++) {
- GlobalProperty *prop;
- Error *err = NULL;
-
- prop = g_array_index(global_props(), GlobalProperty *, i);
- if (object_dynamic_cast(OBJECT(dev), prop->driver) == NULL) {
- continue;
- }
- prop->used = true;
- object_property_parse(OBJECT(dev), prop->value, prop->property, &err);
- if (err != NULL) {
- error_prepend(&err, "can't apply global %s.%s=%s: ",
- prop->driver, prop->property, prop->value);
- if (!dev->hotplugged) {
- error_propagate(&error_fatal, err);
- } else {
- warn_report_err(err);
- }
- }
- }
+ object_apply_global_props(OBJECT(dev), global_props(),
+ dev->hotplugged ? NULL : &error_fatal);
}
/* --- 64bit unsigned int 'size' type --- */
diff --git a/qom/object.c b/qom/object.c
index 9acdf9e16d..b1a7f70550 100644
--- a/qom/object.c
+++ b/qom/object.c
@@ -392,7 +392,11 @@ void object_apply_global_props(Object *obj, GArray *props,
Error **errp)
if (err != NULL) {
error_prepend(&err, "can't apply global %s.%s=%s: ",
p->driver, p->property, p->value);
- error_propagate(errp, err);
+ if (errp) {
+ error_propagate(errp, err);
+ } else {
+ warn_report_err(err);
+ }
}
}
}
--
2.19.1.708.g4ede3d42df
- Re: [Qemu-ppc] [PATCH for-3.2 v3 06/14] qdev: do not mix compat props with global props, (continued)
- [Qemu-ppc] [PATCH for-3.2 v3 07/14] qdev: all globals are now user-provided, Marc-André Lureau, 2018/11/07
- [Qemu-ppc] [PATCH for-3.2 v3 08/14] qdev-props: convert global_props to GArray, Marc-André Lureau, 2018/11/07
- [Qemu-ppc] [PATCH for-3.2 v3 09/14] qdev-props: remove errp from GlobalProperty, Marc-André Lureau, 2018/11/07
- [Qemu-ppc] [PATCH for-3.2 v3 11/14] qom: teach interfaces to implement post-init, Marc-André Lureau, 2018/11/07
- [Qemu-ppc] [PATCH for-3.2 v3 13/14] hw/i386: add pc-i440fx-3.2 & pc-q35-3.2, Marc-André Lureau, 2018/11/07
- [Qemu-ppc] [PATCH for-3.2 v3 10/14] qdev-props: call object_apply_global_props(),
Marc-André Lureau <=
- [Qemu-ppc] [PATCH for-3.2 v3 12/14] machine: add compat-props interface, Marc-André Lureau, 2018/11/07
- [Qemu-ppc] [PATCH for-3.2 v3 14/14] hostmem: use object id for memory region name with >= 3.1, Marc-André Lureau, 2018/11/07
- Re: [Qemu-ppc] [Qemu-devel] [PATCH for-3.2 v3 00/14] Generalize machine compatibility properties, Igor Mammedov, 2018/11/26