[Top][All Lists]

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

[Qemu-devel] [PATCH v3 3/5] target-i386: Set APIC ID using cpu_index on

From: Eduardo Habkost
Subject: [Qemu-devel] [PATCH v3 3/5] target-i386: Set APIC ID using cpu_index on legacy cpu_x86_init()
Date: Thu, 26 Feb 2015 18:42:30 -0300

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 cpu_x86_init() (used only by 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.

Signed-off-by: Eduardo Habkost <address@hidden>
Changes v1 -> v2: none
Changes v2 -> v3:
 * Use "goto out" instead of "goto error" after patch changing error
   handling was removed from the series
 target-i386/cpu.c | 6 ++++++
 1 file changed, 6 insertions(+)

diff --git a/target-i386/cpu.c b/target-i386/cpu.c
index 079ce60..c4a96b6 100644
--- a/target-i386/cpu.c
+++ b/target-i386/cpu.c
@@ -2159,6 +2159,12 @@ X86CPU *cpu_x86_init(const char *cpu_model)
         goto out;
+    object_property_set_int(OBJECT(cpu), CPU(cpu)->cpu_index, "apic-id",
+                            &error);
+    if (error) {
+        goto out;
+    }
     object_property_set_bool(OBJECT(cpu), true, "realized", &error);

reply via email to

[Prev in Thread] Current Thread [Next in Thread]