CPUClass::class_by_name is called after the CPU model name and options were already split, there's no need to split the string again.
Fixes: fb8d6c24b095 ("target-arm: Add CPU property to disable AArch64") Signed-off-by: Eduardo Habkost <ehabk...@redhat.com> --- Cc: Greg Bellows <greg.bell...@linaro.org> Cc: Peter Maydell <peter.mayd...@linaro.org> Cc: qemu-...@nongnu.org --- target/arm/cpu.c | 11 +++-------- 1 file changed, 3 insertions(+), 8 deletions(-) diff --git a/target/arm/cpu.c b/target/arm/cpu.c index 4155782197..6848d9c94d 100644 --- a/target/arm/cpu.c +++ b/target/arm/cpu.c @@ -1212,22 +1212,17 @@ static ObjectClass *arm_cpu_class_by_name(const char *cpu_model) { ObjectClass *oc; char *typename; - char **cpuname; - const char *cpunamestr; - cpuname = g_strsplit(cpu_model, ",", 1); - cpunamestr = cpuname[0]; #ifdef CONFIG_USER_ONLY /* For backwards compatibility usermode emulation allows "-cpu any", * which has the same semantics as "-cpu max". */ - if (!strcmp(cpunamestr, "any")) { - cpunamestr = "max"; + if (!strcmp(cpu_model, "any")) { + cpu_model = "max"; } #endif - typename = g_strdup_printf(ARM_CPU_TYPE_NAME("%s"), cpunamestr); + typename = g_strdup_printf(ARM_CPU_TYPE_NAME("%s"), cpu_model); oc = object_class_by_name(typename); - g_strfreev(cpuname); g_free(typename); if (!oc || !object_class_dynamic_cast(oc, TYPE_ARM_CPU) || object_class_is_abstract(oc)) { -- 2.18.0.rc1.1.g3f1ff2140