target-i386: Consolidate calls of object_property_parse() in x86_cpu_parse_featurestr
Signed-off-by: Eduardo Habkost <ehabkost@redhat.com> Reviewed-by: Igor Mammedov <imammedo@redhat.com> Signed-off-by: Igor Mammedov <imammedo@redhat.com> Reviewed-by: Eduardo Habkost <ehabkost@redhat.com> Signed-off-by: Eduardo Habkost <ehabkost@redhat.com>
This commit is contained in:
parent
a57d0163e7
commit
f6750e959a
@ -1966,43 +1966,59 @@ static void x86_cpu_parse_featurestr(CPUState *cs, char *features,
|
||||
char *featurestr; /* Single 'key=value" string being parsed */
|
||||
Error *local_err = NULL;
|
||||
|
||||
featurestr = features ? strtok(features, ",") : NULL;
|
||||
if (!features) {
|
||||
return;
|
||||
}
|
||||
|
||||
while (featurestr) {
|
||||
char *val;
|
||||
for (featurestr = strtok(features, ",");
|
||||
featurestr && !local_err;
|
||||
featurestr = strtok(NULL, ",")) {
|
||||
const char *name;
|
||||
const char *val = NULL;
|
||||
char *eq = NULL;
|
||||
|
||||
/* Compatibility syntax: */
|
||||
if (featurestr[0] == '+') {
|
||||
add_flagname_to_bitmaps(featurestr + 1, plus_features, &local_err);
|
||||
continue;
|
||||
} else if (featurestr[0] == '-') {
|
||||
add_flagname_to_bitmaps(featurestr + 1, minus_features, &local_err);
|
||||
} else if ((val = strchr(featurestr, '='))) {
|
||||
*val = 0; val++;
|
||||
feat2prop(featurestr);
|
||||
if (!strcmp(featurestr, "tsc-freq")) {
|
||||
int64_t tsc_freq;
|
||||
char *err;
|
||||
char num[32];
|
||||
continue;
|
||||
}
|
||||
|
||||
tsc_freq = qemu_strtosz_suffix_unit(val, &err,
|
||||
QEMU_STRTOSZ_DEFSUFFIX_B, 1000);
|
||||
if (tsc_freq < 0 || *err) {
|
||||
error_setg(errp, "bad numerical value %s", val);
|
||||
return;
|
||||
}
|
||||
snprintf(num, sizeof(num), "%" PRId64, tsc_freq);
|
||||
object_property_parse(OBJECT(cpu), num, "tsc-frequency",
|
||||
&local_err);
|
||||
} else {
|
||||
object_property_parse(OBJECT(cpu), val, featurestr, &local_err);
|
||||
}
|
||||
eq = strchr(featurestr, '=');
|
||||
if (eq) {
|
||||
*eq++ = 0;
|
||||
val = eq;
|
||||
} else {
|
||||
feat2prop(featurestr);
|
||||
object_property_parse(OBJECT(cpu), "on", featurestr, &local_err);
|
||||
val = "on";
|
||||
}
|
||||
if (local_err) {
|
||||
error_propagate(errp, local_err);
|
||||
return;
|
||||
|
||||
feat2prop(featurestr);
|
||||
name = featurestr;
|
||||
|
||||
/* Special case: */
|
||||
if (!strcmp(name, "tsc-freq")) {
|
||||
int64_t tsc_freq;
|
||||
char *err;
|
||||
char num[32];
|
||||
|
||||
tsc_freq = qemu_strtosz_suffix_unit(val, &err,
|
||||
QEMU_STRTOSZ_DEFSUFFIX_B, 1000);
|
||||
if (tsc_freq < 0 || *err) {
|
||||
error_setg(errp, "bad numerical value %s", val);
|
||||
return;
|
||||
}
|
||||
snprintf(num, sizeof(num), "%" PRId64, tsc_freq);
|
||||
val = num;
|
||||
name = "tsc-frequency";
|
||||
}
|
||||
featurestr = strtok(NULL, ",");
|
||||
|
||||
object_property_parse(OBJECT(cpu), val, name, &local_err);
|
||||
}
|
||||
|
||||
if (local_err) {
|
||||
error_propagate(errp, local_err);
|
||||
}
|
||||
}
|
||||
|
||||
|
Loading…
Reference in New Issue
Block a user