From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from eggs.gnu.org ([208.118.235.92]:37193) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1TJP03-0004Dv-Tk for qemu-devel@nongnu.org; Wed, 03 Oct 2012 09:28:39 -0400 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1TJOzt-0006bI-7d for qemu-devel@nongnu.org; Wed, 03 Oct 2012 09:28:31 -0400 Received: from mx1.redhat.com ([209.132.183.28]:30387) by eggs.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1TJOzs-0006aM-TQ for qemu-devel@nongnu.org; Wed, 03 Oct 2012 09:28:21 -0400 From: Eduardo Habkost Date: Wed, 3 Oct 2012 10:29:10 -0300 Message-Id: <1349270954-4657-15-git-send-email-ehabkost@redhat.com> In-Reply-To: <1349270954-4657-1-git-send-email-ehabkost@redhat.com> References: <1349270954-4657-1-git-send-email-ehabkost@redhat.com> Subject: [Qemu-devel] [RFC 14/18] pc: create apic_id_for_cpu() function (v3) List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: qemu-devel@nongnu.org Cc: Igor Mammedov , =?UTF-8?q?Andreas=20F=C3=A4rber?= , Gleb Natapov , Paolo Bonzini Currently we need a way to calculate the Initial APIC ID using only the CPU index (without needing a CPU object), as the NUMA fw_cfg data is APIC-ID-based, and may include data for hotplug CPUs (that don't exist yet), up to max_cpus. Changes v2 -> v3: - Move the whole code to hw/pc.c, now only the PC code will need to handle topology-based APIC IDs Changes v1 -> v2: - make function return value 'unsigned int' (it's not specific for the 8-bit xAPIC ID) - move implementation to cpu.c Signed-off-by: Eduardo Habkost --- hw/pc.c | 16 +++++++++++++++- 1 file changed, 15 insertions(+), 1 deletion(-) diff --git a/hw/pc.c b/hw/pc.c index c64c218..4b30f38 100644 --- a/hw/pc.c +++ b/hw/pc.c @@ -568,6 +568,20 @@ static void pc_register_type(void) type_init(pc_register_type); +/* Calculates initial APIC ID for a specific CPU index + * + * Currently we need to be able to calculate the APIC ID from the CPU index + * alone, as the QEMU<->Seabios interfaces have no concept of "CPU index", + * and the NUMA tables need the APIC ID of all CPUs up to max_cpus. + */ +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; +} + int e820_add_entry(uint64_t address, uint64_t length, uint32_t type) { int index = le32_to_cpu(e820_table.count); @@ -935,7 +949,7 @@ void pc_cpus_init(PC *pc, const char *cpu_model) } for(i = 0; i < smp_cpus; i++) { - pc_new_cpu(pc, cpu_model, i); + pc_new_cpu(pc, cpu_model, apic_id_for_cpu(pc, i)); } } -- 1.7.11.4