diff options
author | Marc Zyngier <marc.zyngier@arm.com> | 2016-06-02 04:24:06 -0400 |
---|---|---|
committer | Christoffer Dall <christoffer.dall@linaro.org> | 2016-06-02 05:52:21 -0400 |
commit | 05fb05a6ca25e02ad8c31bc440b3c4996864f379 (patch) | |
tree | 88be4e84a1c257e7e999d7bd344c511c66e7973e | |
parent | c58513284029229842844929ddeaca44d013c128 (diff) |
KVM: arm/arm64: vgic-new: Removel harmful BUG_ON
When changing the active bit from an MMIO trap, we decide to
explode if the intid is that of a private interrupt.
This flawed logic comes from the fact that we were assuming that
kvm_vcpu_kick() as called by kvm_arm_halt_vcpu() would not return before
the called vcpu responded, but this is not the case, so we need to
perform this wait even for private interrupts.
Dropping the BUG_ON seems like the right thing to do.
[ Commit message tweaked by Christoffer ]
Signed-off-by: Marc Zyngier <marc.zyngier@arm.com>
Signed-off-by: Christoffer Dall <christoffer.dall@linaro.org>
-rw-r--r-- | virt/kvm/arm/vgic/vgic-mmio.c | 4 |
1 files changed, 1 insertions, 3 deletions
diff --git a/virt/kvm/arm/vgic/vgic-mmio.c b/virt/kvm/arm/vgic/vgic-mmio.c index 059595ec3da0..9f6fab74dce7 100644 --- a/virt/kvm/arm/vgic/vgic-mmio.c +++ b/virt/kvm/arm/vgic/vgic-mmio.c | |||
@@ -191,10 +191,8 @@ static void vgic_mmio_change_active(struct kvm_vcpu *vcpu, struct vgic_irq *irq, | |||
191 | * other thread sync back the IRQ. | 191 | * other thread sync back the IRQ. |
192 | */ | 192 | */ |
193 | while (irq->vcpu && /* IRQ may have state in an LR somewhere */ | 193 | while (irq->vcpu && /* IRQ may have state in an LR somewhere */ |
194 | irq->vcpu->cpu != -1) { /* VCPU thread is running */ | 194 | irq->vcpu->cpu != -1) /* VCPU thread is running */ |
195 | BUG_ON(irq->intid < VGIC_NR_PRIVATE_IRQS); | ||
196 | cond_resched_lock(&irq->irq_lock); | 195 | cond_resched_lock(&irq->irq_lock); |
197 | } | ||
198 | 196 | ||
199 | irq->active = new_active_state; | 197 | irq->active = new_active_state; |
200 | if (new_active_state) | 198 | if (new_active_state) |