x86: Avoid dynamic memory allocation during AP startup.

Signed-off-by: Keir Fraser <keir.fraser@citrix.com>
xen-unstable changeset:   18670:3ba06e8098cb3a5583dc5d38fc4f7138348bf897
xen-unstable date:        Tue Oct 21 11:39:22 2008 +0100

diff --git a/xen/arch/x86/smpboot.c b/xen/arch/x86/smpboot.c
index 2c77bb3afc56f9d560215dd3cbf09bb2db1e2de6..eb786fe8d54d8f3be03aad3225d7aff5d576dd6c 100644 (file)
--- a/xen/arch/x86/smpboot.c
+++ b/xen/arch/x86/smpboot.c
@@ -468,13 +468,6 @@ static void construct_percpu_idt(unsigned int cpu)
 {
        unsigned char idt_load[10];
 
-       /* If IDT table exists since last hotplug, reuse it */
-       if (!idt_tables[cpu]) {
-               idt_tables[cpu] = xmalloc_array(idt_entry_t, IDT_ENTRIES);
-               memcpy(idt_tables[cpu], idt_table,
-                               IDT_ENTRIES*sizeof(idt_entry_t));
-       }
-
        *(unsigned short *)(&idt_load[0]) = (IDT_ENTRIES*sizeof(idt_entry_t))-1;
        *(unsigned long  *)(&idt_load[2]) = (unsigned long)idt_tables[cpu];
        __asm__ __volatile__ ( "lidt %0" : "=m" (idt_load) );
@@ -859,6 +852,12 @@ static int __devinit do_boot_cpu(int apicid, int cpu)
 
        smpboot_setup_warm_reset_vector(start_eip);
 
+       if (!idt_tables[cpu]) {
+               idt_tables[cpu] = xmalloc_array(idt_entry_t, IDT_ENTRIES);
+               memcpy(idt_tables[cpu], idt_table,
+                      IDT_ENTRIES*sizeof(idt_entry_t));
+       }
+
        /*
         * Starting actual IPI sequence...
         */