diff options
| author | Nadav Amit | 2015-04-02 01:58:36 +0200 |
|---|---|---|
| committer | Paolo Bonzini | 2015-04-02 15:57:27 +0200 |
| commit | 9cb11fd7539b5b787d8fb3834004804a58dd16ae (patch) | |
| tree | cd20dde0af304c6401ec4002b2f4e828a7c66a80 /target-i386 | |
| parent | qga: fitering out -fstack-protector-strong (diff) | |
| download | qemu-9cb11fd7539b5b787d8fb3834004804a58dd16ae.tar.gz qemu-9cb11fd7539b5b787d8fb3834004804a58dd16ae.tar.xz qemu-9cb11fd7539b5b787d8fb3834004804a58dd16ae.zip | |
target-i386: clear bsp bit when designating bsp
Since the BSP bit is writable on real hardware, during reset all the CPUs which
were not chosen to be the BSP should have their BSP bit cleared. This fix is
required for KVM to work correctly when it changes the BSP bit.
An additional fix is required for QEMU tcg to allow software to change the BSP
bit.
Signed-off-by: Nadav Amit <namit@cs.technion.ac.il>
Message-Id: <1427932716-11800-1-git-send-email-namit@cs.technion.ac.il>
Signed-off-by: Paolo Bonzini <pbonzini@redhat.com>
Diffstat (limited to 'target-i386')
| -rw-r--r-- | target-i386/cpu.c | 4 |
1 files changed, 1 insertions, 3 deletions
diff --git a/target-i386/cpu.c b/target-i386/cpu.c index b2d1c95df4..03b33cf3bd 100644 --- a/target-i386/cpu.c +++ b/target-i386/cpu.c @@ -2714,9 +2714,7 @@ static void x86_cpu_reset(CPUState *s) #if !defined(CONFIG_USER_ONLY) /* We hard-wire the BSP to the first CPU. */ - if (s->cpu_index == 0) { - apic_designate_bsp(cpu->apic_state); - } + apic_designate_bsp(cpu->apic_state, s->cpu_index == 0); s->halted = !cpu_is_bsp(cpu); |