arm/arm64: vgic: Remove unreachable irq_clear_pending
authorChristoffer Dall <christoffer.dall@linaro.org>
Mon, 24 Nov 2014 09:41:56 +0000 (10:41 +0100)
committerMarc Zyngier <marc.zyngier@arm.com>
Tue, 25 Nov 2014 13:57:28 +0000 (13:57 +0000)
When 'injecting' an edge-triggered interrupt with a falling edge we
shouldn't clear the pending state on the distributor.  In fact, we
don't, because the check in vgic_validate_injection would prevent us
from ever reaching this bit of code.

Remove the unreachable snippet.

Signed-off-by: Christoffer Dall <christoffer.dall@linaro.org>
Signed-off-by: Marc Zyngier <marc.zyngier@arm.com>
virt/kvm/arm/vgic.c

index 5acf2c93f6165702f2965ae3e4e7d7735994e73a..631a472e41cff1b13413c0a934eaba042ba5efe5 100644 (file)
@@ -1643,8 +1643,6 @@ static bool vgic_update_irq_pending(struct kvm *kvm, int cpuid,
                        vgic_dist_irq_clear_level(vcpu, irq_num);
                        if (!vgic_dist_irq_soft_pend(vcpu, irq_num))
                                vgic_dist_irq_clear_pending(vcpu, irq_num);
-               } else {
-                       vgic_dist_irq_clear_pending(vcpu, irq_num);
                }
 
                ret = false;