From: Eduardo Habkost <ehabkost@redhat.com>
Date: Fri, 19 Dec 2014 01:28:45 +0000 (-0200)
Subject: target-i386: Set APIC ID using cpu_index on CONFIG_USER
X-Git-Tag: v2.3.0-rc0~47^2~2
X-Git-Url: http://git.ipfire.org/?a=commitdiff_plain;h=9c235e83f1c3437be6ca45755909efb745c10deb;p=thirdparty%2Fqemu.git

target-i386: Set APIC ID using cpu_index on CONFIG_USER

The PC CPU initialization code already sets apic-id based on the CPU
topology, and CONFIG_USER doesn't need the topology-based APIC ID
calculation code.

Make CONFIG_USER set apic-id before realizing the CPU (just like PC
already does), so we can simplify x86_cpu_initfn later. As there is no
CPU topology configuration in CONFIG_USER, just use cpu_index as the
APIC ID.

Reviewed-by: Paolo Bonzini <pbonzini@redhat.com>
Signed-off-by: Eduardo Habkost <ehabkost@redhat.com>
---

diff --git a/target-i386/cpu.c b/target-i386/cpu.c
index e8cd7b38bad..3e0c39c8bd9 100644
--- a/target-i386/cpu.c
+++ b/target-i386/cpu.c
@@ -2145,6 +2145,12 @@ CPUX86State *cpu_x86_init_user(const char *cpu_model)
         goto error;
     }
 
+    object_property_set_int(OBJECT(cpu), CPU(cpu)->cpu_index, "apic-id",
+                            &error);
+    if (error) {
+        goto error;
+    }
+
     object_property_set_bool(OBJECT(cpu), true, "realized", &error);
     if (error) {
         goto error;