diff options
author | Radim Krčmář <rkrcmar@redhat.com> | 2014-11-27 20:03:12 +0100 |
---|---|---|
committer | Paolo Bonzini <pbonzini@redhat.com> | 2014-12-04 15:29:07 +0100 |
commit | fa834e919788dfa0965495f21760eb38fe480342 (patch) | |
tree | c5d672e0e098bfde84e7635616cb0dcea73e569f /arch/x86/kvm/lapic.c | |
parent | 085563fb043c29cc09e5ec5e21a7f6bb519d91e2 (diff) | |
download | op-kernel-dev-fa834e919788dfa0965495f21760eb38fe480342.zip op-kernel-dev-fa834e919788dfa0965495f21760eb38fe480342.tar.gz |
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/kvm/lapic.c')
-rw-r--r-- | arch/x86/kvm/lapic.c | 5 |
1 files changed, 4 insertions, 1 deletions
diff --git a/arch/x86/kvm/lapic.c b/arch/x86/kvm/lapic.c index 541e178..bd82054 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); |