diff options
author | Avi Kivity <avi@redhat.com> | 2010-12-02 10:52:50 -0500 |
---|---|---|
committer | Avi Kivity <avi@redhat.com> | 2011-01-12 04:30:18 -0500 |
commit | b7c4145ba2eb0717db0ddac1b5f7f48012189c53 (patch) | |
tree | 9b259970f56451ecb0f5d84ecd81d67386f63224 /arch/x86/include/asm | |
parent | 4cc703100b1fd97c71804e2fa5c9c377c5b37dc5 (diff) |
KVM: Don't spin on virt instruction faults during reboot
Since vmx blocks INIT signals, we disable virtualization extensions during
reboot. This leads to virtualization instructions faulting; we trap these
faults and spin while the reboot continues.
Unfortunately spinning on a non-preemptible kernel may block a task that
reboot depends on; this causes the reboot to hang.
Fix by skipping over the instruction and hoping for the best.
Signed-off-by: Avi Kivity <avi@redhat.com>
Diffstat (limited to 'arch/x86/include/asm')
-rw-r--r-- | arch/x86/include/asm/kvm_host.h | 8 |
1 files changed, 6 insertions, 2 deletions
diff --git a/arch/x86/include/asm/kvm_host.h b/arch/x86/include/asm/kvm_host.h index 56e45a2ed2de..d968cc501799 100644 --- a/arch/x86/include/asm/kvm_host.h +++ b/arch/x86/include/asm/kvm_host.h | |||
@@ -786,14 +786,18 @@ enum { | |||
786 | * reboot turns off virtualization while processes are running. | 786 | * reboot turns off virtualization while processes are running. |
787 | * Trap the fault and ignore the instruction if that happens. | 787 | * Trap the fault and ignore the instruction if that happens. |
788 | */ | 788 | */ |
789 | asmlinkage void kvm_handle_fault_on_reboot(void); | 789 | asmlinkage void kvm_spurious_fault(void); |
790 | extern bool kvm_rebooting; | ||
790 | 791 | ||
791 | #define __kvm_handle_fault_on_reboot(insn) \ | 792 | #define __kvm_handle_fault_on_reboot(insn) \ |
792 | "666: " insn "\n\t" \ | 793 | "666: " insn "\n\t" \ |
794 | "668: \n\t" \ | ||
793 | ".pushsection .fixup, \"ax\" \n" \ | 795 | ".pushsection .fixup, \"ax\" \n" \ |
794 | "667: \n\t" \ | 796 | "667: \n\t" \ |
797 | "cmpb $0, kvm_rebooting \n\t" \ | ||
798 | "jne 668b \n\t" \ | ||
795 | __ASM_SIZE(push) " $666b \n\t" \ | 799 | __ASM_SIZE(push) " $666b \n\t" \ |
796 | "jmp kvm_handle_fault_on_reboot \n\t" \ | 800 | "call kvm_spurious_fault \n\t" \ |
797 | ".popsection \n\t" \ | 801 | ".popsection \n\t" \ |
798 | ".pushsection __ex_table, \"a\" \n\t" \ | 802 | ".pushsection __ex_table, \"a\" \n\t" \ |
799 | _ASM_PTR " 666b, 667b \n\t" \ | 803 | _ASM_PTR " 666b, 667b \n\t" \ |