summaryrefslogtreecommitdiffstats
path: root/arch/x86
diff options
context:
space:
mode:
authorRadim Krčmář <rkrcmar@redhat.com>2014-11-27 20:03:12 +0100
committerPaolo Bonzini <pbonzini@redhat.com>2014-12-04 15:29:07 +0100
commitfa834e919788dfa0965495f21760eb38fe480342 (patch)
treec5d672e0e098bfde84e7635616cb0dcea73e569f /arch/x86
parent085563fb043c29cc09e5ec5e21a7f6bb519d91e2 (diff)
downloadlinux-fa834e919788dfa0965495f21760eb38fe480342.tar.gz
linux-fa834e919788dfa0965495f21760eb38fe480342.tar.bz2
linux-fa834e919788dfa0965495f21760eb38fe480342.zip
KVM: x86: fix APIC physical destination wrapping
x2apic allows destinations > 0xff and we don't want them delivered to lower APICs. They are correctly handled by doing nothing. Signed-off-by: Radim Krčmář <rkrcmar@redhat.com> Signed-off-by: Paolo Bonzini <pbonzini@redhat.com>
Diffstat (limited to 'arch/x86')
-rw-r--r--arch/x86/kvm/lapic.c5
1 files changed, 4 insertions, 1 deletions
diff --git a/arch/x86/kvm/lapic.c b/arch/x86/kvm/lapic.c
index 541e17865e63..bd82054664fd 100644
--- a/arch/x86/kvm/lapic.c
+++ b/arch/x86/kvm/lapic.c
@@ -693,7 +693,10 @@ bool kvm_irq_delivery_to_apic_fast(struct kvm *kvm, struct kvm_lapic *src,
ret = true;
if (irq->dest_mode == 0) { /* physical mode */
- dst = &map->phys_map[irq->dest_id & 0xff];
+ if (irq->dest_id >= ARRAY_SIZE(map->phys_map))
+ goto out;
+
+ dst = &map->phys_map[irq->dest_id];
} else {
u32 mda = irq->dest_id << (32 - map->ldr_bits);