[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[Qemu-devel] [RFC 18/18] pc: generate APIC IDs according to CPU topology
From: |
Eduardo Habkost |
Subject: |
[Qemu-devel] [RFC 18/18] pc: generate APIC IDs according to CPU topology (v3) |
Date: |
Wed, 3 Oct 2012 10:29:14 -0300 |
This keeps compatibility on machine-types pc-1.2 and older, and prints a
warning in case the requested configuration won't get the correct
topology.
At first I was going to use object_property_add() on PC instance_init
function, but this wouldn't allow us to use global properties to set
it[1]. So I had to use a static DEFINE_PROP_BIT for the
contiguous_apic_ids property.
[1] See my qdev question at:
http://article.gmane.org/gmane.comp.emulators.qemu/173625
Changes v2 -> v3:
- Now all code is inside hw/pc.c
- Use a real "PC" class and a "contiguous_apic_ids" property
Changes v1 -> v2:
- Move code to cpu.c
- keep using cpu_index on *-user
- Use SMP.contiguous_apic_ids global property
- Prints warning in case the compatibility mode will expose incorrect
topology
Signed-off-by: Eduardo Habkost <address@hidden>
---
hw/pc.c | 45 ++++++++++++++++++++++++++++++++++++++++-----
hw/pc_piix.c | 4 ++++
2 files changed, 44 insertions(+), 5 deletions(-)
diff --git a/hw/pc.c b/hw/pc.c
index 4c9b2f6..c33768b 100644
--- a/hw/pc.c
+++ b/hw/pc.c
@@ -52,6 +52,8 @@
#include "arch_init.h"
#include "bitmap.h"
#include "vga-pci.h"
+#include "topology.h"
+#include "cpus.h"
/* debug PC/ISA interrupts */
//#define DEBUG_IRQ
@@ -552,13 +554,34 @@ static void bochs_bios_write(void *opaque, uint32_t addr,
uint32_t val)
typedef struct PC {
DeviceState parent_obj;
+
+ /* Compatibility mode that force APIC IDs to be contiguous
+ *
+ * Setting it to true may break some CPU topologies (e.g. if core or thread
+ * count is not a power of 2).
+ */
+#define PC_FLAG_CONTIGUOUS_APIC_IDS 0
+
+ uint32_t flags;
} PC;
+static Property pc_properties[] = {
+ DEFINE_PROP_BIT("contiguous_apic_ids", PC, flags,
+ PC_FLAG_CONTIGUOUS_APIC_IDS, false),
+ DEFINE_PROP_END_OF_LIST()
+};
+
+static void pc_class_init(ObjectClass *klass, void *data)
+{
+ DeviceClass *dc = DEVICE_CLASS(klass);
+ dc->props = pc_properties;
+}
+
static const TypeInfo pc_type_info = {
.name = TYPE_PC_MACHINE,
.parent = TYPE_DEVICE,
.instance_size = sizeof(PC),
- .class_size = sizeof(DeviceClass),
+ .class_init = pc_class_init,
};
static void pc_register_type(void)
@@ -576,10 +599,22 @@ type_init(pc_register_type);
*/
static uint32_t apic_id_for_cpu(PC *pc, int cpu_index)
{
- /* right now APIC ID == CPU index. this will eventually change to use
- * the CPU topology configuration properly
- */
- return cpu_index;
+ unsigned int correct_id;
+ static bool warned;
+ bool contig;
+
+ correct_id = topo_apicid_for_cpu(smp_cores, smp_threads, cpu_index);
+ contig = object_property_get_bool(OBJECT(pc), "contiguous_apic_ids", NULL);
+ if (contig) {
+ if (cpu_index != correct_id && !warned) {
+ fprintf(stderr, "warning: CPU topology in compat mode, "
+ "results won't match the requested topology\n");
+ warned = true;
+ }
+ return cpu_index;
+ } else {
+ return correct_id;
+ }
}
int e820_add_entry(uint64_t address, uint64_t length, uint32_t type)
diff --git a/hw/pc_piix.c b/hw/pc_piix.c
index f861907..8a4a5f6 100644
--- a/hw/pc_piix.c
+++ b/hw/pc_piix.c
@@ -374,6 +374,10 @@ static QEMUMachine pc_machine_v1_3 = {
.driver = "ivshmem",\
.property = "use64",\
.value = "0",\
+ },{\
+ .driver = "PC",\
+ .property = "contiguous_apic_ids",\
+ .value = "true",\
}
static QEMUMachine pc_machine_v1_2 = {
--
1.7.11.4
- Re: [Qemu-devel] [RFC 02/18] pc: create PC object on pc_init1(), (continued)
[Qemu-devel] [RFC 16/18] tests: support target-specific unit tests, Eduardo Habkost, 2012/10/03
[Qemu-devel] [RFC 11/18] target-i386: cpu_x86_init: allow APIC ID to be set by caller, Eduardo Habkost, 2012/10/03
[Qemu-devel] [RFC 07/18] kvm: create kvm_arch_vcpu_id() function, Eduardo Habkost, 2012/10/03
[Qemu-devel] [RFC 13/18] pc: set explicit APIC ID for CPUs, Eduardo Habkost, 2012/10/03
[Qemu-devel] [RFC 15/18] pc: set fw_cfg data based on APIC ID calculation (v2), Eduardo Habkost, 2012/10/03
[Qemu-devel] [RFC 14/18] pc: create apic_id_for_cpu() function (v3), Eduardo Habkost, 2012/10/03
[Qemu-devel] [RFC 18/18] pc: generate APIC IDs according to CPU topology (v3),
Eduardo Habkost <=
[Qemu-devel] [RFC 08/18] target-i386: kvm: set vcpu_id to APIC ID instead of CPU index (v2), Eduardo Habkost, 2012/10/03
[Qemu-devel] [RFC 17/18] target-i386: topology & APIC ID utility functions (v2), Eduardo Habkost, 2012/10/03
[Qemu-devel] [RFC 03/18] pc: add PC object argument to some init functions, Eduardo Habkost, 2012/10/03
[Qemu-devel] [RFC 10/18] target-i386: cpu: add apic_id argument to cpu_x86_register(), Eduardo Habkost, 2012/10/03