@@ -774,7 +774,8 @@ static void machine_class_finalize(ObjectClass *klass, void *data)
static void machine_register_compat_for_subclass(ObjectClass *oc, void *opaque)
{
GlobalProperty *p = opaque;
- register_compat_prop(object_class_get_name(oc), p->property, p->value);
+ register_compat_prop(object_class_get_name(oc), p->property,
+ p->value, false);
}
void machine_register_compat_props(MachineState *machine)
@@ -804,7 +805,8 @@ void machine_register_compat_props(MachineState *machine)
object_class_foreach(machine_register_compat_for_subclass,
p->driver, false, p);
} else {
- register_compat_prop(p->driver, p->property, p->value);
+ register_compat_prop(p->driver, p->property,
+ p->value, false);
}
}
}
@@ -1043,7 +1043,8 @@ void qdev_prop_set_ptr(DeviceState *dev, const char *name, void *value)
static GList *global_props;
GList *global_prop_list_add(GList *list, const char *driver,
- const char *property, const char *value)
+ const char *property, const char *value,
+ bool user_provided)
{
GList *l;
GlobalProperty *p;
@@ -1055,6 +1056,7 @@ GList *global_prop_list_add(GList *list, const char *driver,
if (value) {
/* Modify existing value */
p->value = value;
+ p->user_provided = user_provided;
} else {
/* Delete this entry entirely */
list = g_list_remove_link(list, l);
@@ -1078,21 +1080,18 @@ GList *global_prop_list_add(GList *list, const char *driver,
p->driver = driver;
p->property = property;
p->value = value;
+ p->user_provided = user_provided;
return g_list_append(list, p);
}
void register_compat_prop(const char *driver,
const char *property,
- const char *value)
+ const char *value,
+ bool user_provided)
{
- global_props = global_prop_list_add(global_props, driver,
- property, value);
-}
-
-void qdev_prop_register_global(GlobalProperty *prop)
-{
- global_props = g_list_append(global_props, prop);
+ global_props = global_prop_list_add(global_props, driver, property,
+ value, user_provided);
}
void qdev_prop_register_global_list(GlobalProperty *props)
@@ -1100,7 +1099,8 @@ void qdev_prop_register_global_list(GlobalProperty *props)
int i;
for (i = 0; props[i].driver != NULL; i++) {
- qdev_prop_register_global(props+i);
+ register_compat_prop(props[i].driver, props[i].property,
+ props[i].value, props[i].user_provided);
}
}
@@ -194,7 +194,6 @@ void qdev_prop_set_enum(DeviceState *dev, const char *name, int value);
/* FIXME: Remove opaque pointer properties. */
void qdev_prop_set_ptr(DeviceState *dev, const char *name, void *value);
-void qdev_prop_register_global(GlobalProperty *prop);
void qdev_prop_register_global_list(GlobalProperty *props);
int qdev_prop_check_globals(void);
void qdev_prop_set_globals(DeviceState *dev);
@@ -212,9 +211,10 @@ void error_set_from_qdev_prop_error(Error **errp, int ret, DeviceState *dev,
* - if value == NULL, delete the entry
*/
GList *global_prop_list_add(GList *list, const char *driver,
- const char *property, const char *value);
+ const char *property, const char *value,
+ bool user_provided);
void register_compat_prop(const char *driver, const char *property,
- const char *value);
+ const char *value, bool user_provided);
/**
* qdev_property_add_static:
@@ -342,14 +342,10 @@ static void cpu_common_parse_features(const char *typename, char *features,
while (featurestr) {
val = strchr(featurestr, '=');
if (val) {
- GlobalProperty *prop = g_new0(typeof(*prop), 1);
*val = 0;
val++;
- prop->driver = typename;
- prop->property = g_strdup(featurestr);
- prop->value = g_strdup(val);
- prop->errp = &error_fatal;
- qdev_prop_register_global(prop);
+ register_compat_prop(typename, g_strdup(featurestr),
+ g_strdup(val), false);
} else {
error_setg(errp, "Expected key=value format, found %s.",
featurestr);
@@ -2039,7 +2039,6 @@ static void x86_cpu_parse_featurestr(const char *typename, char *features,
const char *val = NULL;
char *eq = NULL;
char num[32];
- GlobalProperty *prop;
/* Compatibility syntax: */
if (featurestr[0] == '+') {
@@ -2091,12 +2090,8 @@ static void x86_cpu_parse_featurestr(const char *typename, char *features,
name = "tsc-frequency";
}
- prop = g_new0(typeof(*prop), 1);
- prop->driver = typename;
- prop->property = g_strdup(name);
- prop->value = g_strdup(val);
- prop->errp = &error_fatal;
- qdev_prop_register_global(prop);
+ register_compat_prop(typename, g_strdup(name),
+ g_strdup(val), false);
}
if (ambiguous) {
@@ -891,13 +891,8 @@ static void configure_rtc(QemuOpts *opts)
value = qemu_opt_get(opts, "driftfix");
if (value) {
if (!strcmp(value, "slew")) {
- static GlobalProperty slew_lost_ticks = {
- .driver = "mc146818rtc",
- .property = "lost_tick_policy",
- .value = "slew",
- };
-
- qdev_prop_register_global(&slew_lost_ticks);
+ register_compat_prop("mc146818rtc", "lost_tick_policy",
+ "slew", false);
} else if (!strcmp(value, "none")) {
/* discard is default */
} else {
@@ -2945,15 +2940,9 @@ static void set_memory_options(uint64_t *ram_slots, ram_addr_t *maxram_size,
static int global_init_func(void *opaque, QemuOpts *opts, Error **errp)
{
- GlobalProperty *g;
-
- g = g_malloc0(sizeof(*g));
- g->driver = qemu_opt_get(opts, "driver");
- g->property = qemu_opt_get(opts, "property");
- g->value = qemu_opt_get(opts, "value");
- g->user_provided = true;
- g->errp = &error_fatal;
- qdev_prop_register_global(g);
+ register_compat_prop(qemu_opt_get(opts, "driver"),
+ qemu_opt_get(opts, "property"),
+ qemu_opt_get(opts, "value"), true);
return 0;
}
@@ -3689,13 +3678,8 @@ int main(int argc, char **argv, char **envp)
win2k_install_hack = 1;
break;
case QEMU_OPTION_rtc_td_hack: {
- static GlobalProperty slew_lost_ticks = {
- .driver = "mc146818rtc",
- .property = "lost_tick_policy",
- .value = "slew",
- };
-
- qdev_prop_register_global(&slew_lost_ticks);
+ register_compat_prop("mc146818rtc", "lost_tick_policy",
+ "slew", false);
break;
}
case QEMU_OPTION_acpitable:
@@ -3746,15 +3730,10 @@ int main(int argc, char **argv, char **envp)
break;
}
case QEMU_OPTION_no_kvm_pit_reinjection: {
- static GlobalProperty kvm_pit_lost_tick_policy = {
- .driver = "kvm-pit",
- .property = "lost_tick_policy",
- .value = "discard",
- };
-
error_report("warning: deprecated, replaced by "
"-global kvm-pit.lost_tick_policy=discard");
- qdev_prop_register_global(&kvm_pit_lost_tick_policy);
+ register_compat_prop("kvm-pit", "lost_tick_policy",
+ "discard", false);
break;
}
case QEMU_OPTION_accel: {
It works very similar to register_compat_prop() but does not need a malloc, however mostly all the callers of it are doing malloc on their own. Then it makes little sense to keep it. Replacing all the callers with register_compat_prop(), meanwhile adding one more parameter "user_provided" for it to make it even more powerful (there is only one usage of it to be true). This greatly reduces tens of LOCs and should have no functional change. Signed-off-by: Peter Xu <peterx@redhat.com> --- hw/core/machine.c | 6 ++++-- hw/core/qdev-properties.c | 20 ++++++++++---------- include/hw/qdev-properties.h | 6 +++--- qom/cpu.c | 8 ++------ target/i386/cpu.c | 9 ++------- vl.c | 39 +++++++++------------------------------ 6 files changed, 30 insertions(+), 58 deletions(-)