motorhead/qemu
1
0
Fork 0
mirror of https://github.com/Motorhead1991/qemu.git synced 2025-08-16 14:41:55 -06:00
Code Issues Projects Releases Packages Wiki Activity Actions

qdev: Legacy properties are now read-only

Browse source 
Reviewed-by: Igor Mammedov <imammedo@redhat.com>
Reviewed-by: Eric Blake <eblake@redhat.com>
Signed-off-by: Paolo Bonzini <pbonzini@redhat.com>
Signed-off-by: Andreas Färber <afaerber@suse.de>
This commit is contained in:
Paolo Bonzini 2014-02-08 11:01:47 +01:00 • committed by Andreas Färber
parent 9e4d9620c4
commit 03ff777048
3 changed files with 3 additions and 38 deletions
Download patch file Download diff file Expand all files Collapse all files

8
hw/core/qdev-properties.c
View file

@ -936,15 +936,7 @@ void error_set_from_qdev_prop_error(Error **errp, int ret, DeviceState *dev,
void qdev_prop_parse(DeviceState *dev, const char *name, const char *value, void qdev_prop_parse(DeviceState *dev, const char *name, const char *value,
Error **errp) Error **errp)
{ {
char *legacy_name;
legacy_name = g_strdup_printf("legacy-%s", name);
if (object_property_get_type(OBJECT(dev), legacy_name, NULL)) {
object_property_parse(OBJECT(dev), value, legacy_name, errp);
} else {
object_property_parse(OBJECT(dev), value, name, errp); object_property_parse(OBJECT(dev), value, name, errp);
}
g_free(legacy_name);
} }
void qdev_prop_set_bit(DeviceState *dev, const char *name, bool value) void qdev_prop_set_bit(DeviceState *dev, const char *name, bool value)

30
hw/core/qdev.c
View file

@ -578,31 +578,6 @@ static void qdev_get_legacy_property(Object *obj, Visitor *v, void *opaque,
visit_type_str(v, &ptr, name, errp); visit_type_str(v, &ptr, name, errp);
} }
static void qdev_set_legacy_property(Object *obj, Visitor *v, void *opaque,
const char *name, Error **errp)
{
DeviceState *dev = DEVICE(obj);
Property *prop = opaque;
Error *local_err = NULL;
char *ptr = NULL;
int ret;
if (dev->realized) {
qdev_prop_set_after_realize(dev, name, errp);
return;
}
visit_type_str(v, &ptr, name, &local_err);
if (local_err) {
error_propagate(errp, local_err);
return;
}
ret = prop->info->parse(dev, prop, ptr);
error_set_from_qdev_prop_error(errp, ret, dev, prop, ptr);
g_free(ptr);
}
/** /**
* @qdev_add_legacy_property - adds a legacy property * @qdev_add_legacy_property - adds a legacy property
* *
@ -618,8 +593,7 @@ void qdev_property_add_legacy(DeviceState *dev, Property *prop,
gchar *name, *type; gchar *name, *type;
/* Register pointer properties as legacy properties */ /* Register pointer properties as legacy properties */
if (!prop->info->print && !prop->info->parse && if (!prop->info->print && prop->info->get) {
(prop->info->set || prop->info->get)) {
return; return;
} }
@ -629,7 +603,7 @@ void qdev_property_add_legacy(DeviceState *dev, Property *prop,
object_property_add(OBJECT(dev), name, type, object_property_add(OBJECT(dev), name, type,
prop->info->print ? qdev_get_legacy_property : prop->info->get, prop->info->print ? qdev_get_legacy_property : prop->info->get,
prop->info->parse ? qdev_set_legacy_property : prop->info->set, NULL,
NULL, NULL,
prop, errp); prop, errp);

1
include/hw/qdev-core.h
View file

@ -209,7 +209,6 @@ struct PropertyInfo {
const char *name; const char *name;
const char *legacy_name; const char *legacy_name;
const char **enum_table; const char **enum_table;
int (*parse)(DeviceState *dev, Property *prop, const char *str);
int (*print)(DeviceState *dev, Property *prop, char *dest, size_t len); int (*print)(DeviceState *dev, Property *prop, char *dest, size_t len);
ObjectPropertyAccessor *get; ObjectPropertyAccessor *get;
ObjectPropertyAccessor *set; ObjectPropertyAccessor *set;