diff options
author | Igor Mammedov <imammedo@redhat.com> | 2016-06-09 19:11:03 +0200 |
---|---|---|
committer | Eduardo Habkost <ehabkost@redhat.com> | 2016-07-07 15:25:06 -0300 |
commit | 6aff24c6a61c6fec31e555c7748ba6085b7b2c06 (patch) | |
tree | 9fa1ae089159deb7effd2af104fcef3d17af29a3 /target-i386 | |
parent | 09f71b054a95161950a03fafc9023637929bd404 (diff) | |
download | qemu-6aff24c6a61c6fec31e555c7748ba6085b7b2c06.zip qemu-6aff24c6a61c6fec31e555c7748ba6085b7b2c06.tar.gz qemu-6aff24c6a61c6fec31e555c7748ba6085b7b2c06.tar.bz2 |
pc: Parse CPU features only once
Considering that features are converted to global properties and
global properties are automatically applied to every new instance
of created CPU (at object_new() time), there is no point in
parsing cpu_model string every time a CPU created. So move
parsing outside CPU creation loop and do it only once.
Parsing also should be done before any CPU is created so that
features would affect the first CPU a well.
Signed-off-by: Igor Mammedov <imammedo@redhat.com>
Signed-off-by: Eduardo Habkost <ehabkost@redhat.com>
Diffstat (limited to 'target-i386')
-rw-r--r-- | target-i386/cpu.c | 44 | ||||
-rw-r--r-- | target-i386/cpu.h | 1 |
2 files changed, 0 insertions, 45 deletions
diff --git a/target-i386/cpu.c b/target-i386/cpu.c index 16977d5..80e1767 100644 --- a/target-i386/cpu.c +++ b/target-i386/cpu.c @@ -2209,50 +2209,6 @@ static void x86_cpu_load_def(X86CPU *cpu, X86CPUDefinition *def, Error **errp) } -X86CPU *cpu_x86_create(const char *cpu_model, Error **errp) -{ - X86CPU *cpu = NULL; - ObjectClass *oc; - CPUClass *cc; - gchar **model_pieces; - char *name, *features; - Error *error = NULL; - const char *typename; - - model_pieces = g_strsplit(cpu_model, ",", 2); - if (!model_pieces[0]) { - error_setg(&error, "Invalid/empty CPU model name"); - goto out; - } - name = model_pieces[0]; - features = model_pieces[1]; - - oc = x86_cpu_class_by_name(name); - if (oc == NULL) { - error_setg(&error, "Unable to find CPU definition: %s", name); - goto out; - } - cc = CPU_CLASS(oc); - typename = object_class_get_name(oc); - - cc->parse_features(typename, features, &error); - cpu = X86_CPU(object_new(typename)); - if (error) { - goto out; - } - -out: - if (error != NULL) { - error_propagate(errp, error); - if (cpu) { - object_unref(OBJECT(cpu)); - cpu = NULL; - } - } - g_strfreev(model_pieces); - return cpu; -} - X86CPU *cpu_x86_init(const char *cpu_model) { return X86_CPU(cpu_generic_init(TYPE_X86_CPU, cpu_model)); diff --git a/target-i386/cpu.h b/target-i386/cpu.h index 474b0b9..738958e 100644 --- a/target-i386/cpu.h +++ b/target-i386/cpu.h @@ -1234,7 +1234,6 @@ void x86_cpu_exec_enter(CPUState *cpu); void x86_cpu_exec_exit(CPUState *cpu); X86CPU *cpu_x86_init(const char *cpu_model); -X86CPU *cpu_x86_create(const char *cpu_model, Error **errp); void x86_cpu_list(FILE *f, fprintf_function cpu_fprintf); int cpu_x86_support_mca_broadcast(CPUX86State *env); |