From mboxrd@z Thu Jan 1 00:00:00 1970 From: Jan Kiszka Subject: [PATCH] qemu-kvm: Reindent pc_new_cpu Date: Fri, 18 Sep 2009 19:28:36 +0200 Message-ID: <4AB3C344.9000809@siemens.com> Mime-Version: 1.0 Content-Type: text/plain; charset=ISO-8859-15 Content-Transfer-Encoding: 7bit Cc: kvm-devel , Avi Kivity To: Marcelo Tosatti Return-path: Received: from goliath.siemens.de ([192.35.17.28]:19835 "EHLO goliath.siemens.de" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1756986AbZIRR2s (ORCPT ); Fri, 18 Sep 2009 13:28:48 -0400 Sender: kvm-owner@vger.kernel.org List-ID: ...for better match with upstream. Signed-off-by: Jan Kiszka --- PS: regs_modified = 1 is another sign that the initialization order in qemu-kvm needs a rework. hw/pc.c | 28 +++++++++++++++------------- 1 files changed, 15 insertions(+), 13 deletions(-) diff --git a/hw/pc.c b/hw/pc.c index 5f892c7..d407c47 100644 --- a/hw/pc.c +++ b/hw/pc.c @@ -1097,19 +1097,21 @@ int cpu_is_bsp(CPUState *env) CPUState *pc_new_cpu(const char *cpu_model) { - CPUState *env = cpu_init(cpu_model); - if (!env) { - fprintf(stderr, "Unable to find x86 CPU definition\n"); - exit(1); - } - env->kvm_cpu_state.regs_modified = 1; - if ((env->cpuid_features & CPUID_APIC) || smp_cpus > 1) { - env->cpuid_apic_id = env->cpu_index; - /* APIC reset callback resets cpu */ - apic_init(env); - } else { - qemu_register_reset((QEMUResetHandler*)cpu_reset, env); - } + CPUState *env; + + env = cpu_init(cpu_model); + if (!env) { + fprintf(stderr, "Unable to find x86 CPU definition\n"); + exit(1); + } + env->kvm_cpu_state.regs_modified = 1; + if ((env->cpuid_features & CPUID_APIC) || smp_cpus > 1) { + env->cpuid_apic_id = env->cpu_index; + /* APIC reset callback resets cpu */ + apic_init(env); + } else { + qemu_register_reset((QEMUResetHandler*)cpu_reset, env); + } /* kvm needs this to run after the apic is initialized. Otherwise, * it can access invalid state and crash.