mirrors/qemu
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

qom: use object_new_with_class when possible

Browse Source 
A small optimization/code simplification, that also makes it clear that
we won't look for a type in a not-loaded-yet module---the module will
have been loaded by a call to module_object_class_by_name(), if present.

Reviewed-by: Richard Henderson <richard.henderson@linaro.org>
Reviewed-by: Philippe Mathieu-Daudé <philmd@linaro.org>
Signed-off-by: Paolo Bonzini <pbonzini@redhat.com>
This commit is contained in:
Paolo Bonzini 2024-10-29 10:45:51 +01:00
parent 845b54efaf
commit b801e3cb2a
3 changed files with 5 additions and 4 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

5
hw/core/qdev.c
View File

@ -167,10 +167,11 @@ DeviceState *qdev_new(const char *name)
DeviceState *qdev_try_new(const char *name)
{
if (!module_object_class_by_name(name)) {
ObjectClass *oc = module_object_class_by_name(name);
if (!oc) {
return NULL;
}
return DEVICE(object_new(name));
return DEVICE(object_new_with_class(oc));
}
static QTAILQ_HEAD(, DeviceListener) device_listeners

2
qom/object_interfaces.c
View File

@ -108,7 +108,7 @@ Object *user_creatable_add_type(const char *type, const char *id,
}
assert(qdict);
obj = object_new(type);
obj = object_new_with_class(klass);
object_set_properties_from_qdict(obj, qdict, v, &local_err);
if (local_err) {
goto out;

2
qom/qom-qmp-cmds.c
View File

@ -141,7 +141,7 @@ ObjectPropertyInfoList *qmp_device_list_properties(const char *typename,
return NULL;
}
obj = object_new(typename);
obj = object_new_with_class(klass);
object_property_iter_init(&iter, obj);
while ((prop = object_property_iter_next(&iter))) {