diff options
author | Marcelo Tosatti <mtosatti@redhat.com> | 2008-04-11 13:53:26 -0400 |
---|---|---|
committer | Avi Kivity <avi@qumranet.com> | 2008-04-27 05:04:11 -0400 |
commit | 3d80840d96127401ba6aeadd813c3a15b84e70fe (patch) | |
tree | b3e083c651f09fee362d88f0d1fc140194f2431d /include | |
parent | 3564990af1b9f77a63692c1079e9c41af229f066 (diff) |
KVM: hlt emulation should take in-kernel APIC/PIT timers into account
Timers that fire between guest hlt and vcpu_block's add_wait_queue() are
ignored, possibly resulting in hangs.
Also make sure that atomic_inc and waitqueue_active tests happen in the
specified order, otherwise the following race is open:
CPU0 CPU1
if (waitqueue_active(wq))
add_wait_queue()
if (!atomic_read(pit_timer->pending))
schedule()
atomic_inc(pit_timer->pending)
Signed-off-by: Marcelo Tosatti <mtosatti@redhat.com>
Signed-off-by: Avi Kivity <avi@qumranet.com>
Diffstat (limited to 'include')
-rw-r--r-- | include/linux/kvm_host.h | 1 |
1 files changed, 1 insertions, 0 deletions
diff --git a/include/linux/kvm_host.h b/include/linux/kvm_host.h index bd0c2d2d840f..0bc400387cae 100644 --- a/include/linux/kvm_host.h +++ b/include/linux/kvm_host.h | |||
@@ -269,6 +269,7 @@ void kvm_arch_destroy_vm(struct kvm *kvm); | |||
269 | 269 | ||
270 | int kvm_cpu_get_interrupt(struct kvm_vcpu *v); | 270 | int kvm_cpu_get_interrupt(struct kvm_vcpu *v); |
271 | int kvm_cpu_has_interrupt(struct kvm_vcpu *v); | 271 | int kvm_cpu_has_interrupt(struct kvm_vcpu *v); |
272 | int kvm_cpu_has_pending_timer(struct kvm_vcpu *vcpu); | ||
272 | void kvm_vcpu_kick(struct kvm_vcpu *vcpu); | 273 | void kvm_vcpu_kick(struct kvm_vcpu *vcpu); |
273 | 274 | ||
274 | static inline void kvm_guest_enter(void) | 275 | static inline void kvm_guest_enter(void) |