[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[Qemu-commits] [COMMIT cd57c0f] Don't register cpu reset handler for cpu
From: |
Anthony Liguori |
Subject: |
[Qemu-commits] [COMMIT cd57c0f] Don't register cpu reset handler for cpu with APIC. |
Date: |
Mon, 22 Jun 2009 15:51:10 -0000 |
From: Gleb Natapov <address@hidden>
APIC reset handler already resets cpu, no need to reset it twice.
Also register cpu_reset handler directly to make it impossible to
add additional code to main_cpu_reset() by mistake.
Signed-off-by: Gleb Natapov <address@hidden>
Signed-off-by: Anthony Liguori <address@hidden>
diff --git a/hw/pc.c b/hw/pc.c
index 19021ea..86e5cfe 100644
--- a/hw/pc.c
+++ b/hw/pc.c
@@ -751,12 +751,6 @@ static void load_linux(target_phys_addr_t option_rom,
generate_bootsect(option_rom, gpr, seg, 0);
}
-static void main_cpu_reset(void *opaque)
-{
- CPUState *env = opaque;
- cpu_reset(env);
-}
-
static const int ide_iobase[2] = { 0x1f0, 0x170 };
static const int ide_iobase2[2] = { 0x3f6, 0x376 };
static const int ide_irq[2] = { 14, 15 };
@@ -884,9 +878,11 @@ static void pc_init1(ram_addr_t ram_size,
}
if ((env->cpuid_features & CPUID_APIC) || smp_cpus > 1) {
env->cpuid_apic_id = env->cpu_index;
+ /* APIC reset callback resets cpu */
apic_init(env);
+ } else {
+ qemu_register_reset((QEMUResetHandler*)cpu_reset, 0, env);
}
- qemu_register_reset(main_cpu_reset, 0, env);
}
vmport_init();
[Prev in Thread] |
Current Thread |
[Next in Thread] |
- [Qemu-commits] [COMMIT cd57c0f] Don't register cpu reset handler for cpu with APIC.,
Anthony Liguori <=