RE: [PATCH v2] KVM: arm/arm64: BUG FIX: Do not inject spurious interrupts

2015-10-12 Thread Pavel Fedin
Hello! > It is really up to whoever adds support for LPIs to make sure it's done > correctly. So I think this is for Andre to work out in his ITS series. > > This patch fixes an issue with the current code in the correct way as > far as I can tell. Ok. An alternate way is to introduce a

Re: [PATCH v2] KVM: arm/arm64: BUG FIX: Do not inject spurious interrupts

2015-10-10 Thread Christoffer Dall
On Fri, Oct 09, 2015 at 05:41:11PM +0300, Pavel Fedin wrote: > Hello! > > > I reworked the commit message and applied this patch. > > During testing i discovered a problem with this patch and vITS series by > Andre. > The problem is that compute_pending_for_cpu() does not know anything about

RE: [PATCH v2] KVM: arm/arm64: BUG FIX: Do not inject spurious interrupts

2015-10-09 Thread Pavel Fedin
Hello! > I reworked the commit message and applied this patch. During testing i discovered a problem with this patch and vITS series by Andre. The problem is that compute_pending_for_cpu() does not know anything about LPIs. Therefore, we can reset this bit even if some LPIs (and only LPIs)

RE: [PATCH v2] KVM: arm/arm64: BUG FIX: Do not inject spurious interrupts

2015-09-30 Thread Pavel Fedin
Hello! > I reworked the commit message and applied this patch. Thank you very much. Kind regards, Pavel Fedin Expert Engineer Samsung Electronics Research center Russia -- To unsubscribe from this list: send the line "unsubscribe kvm" in the body of a message to majord...@vger.kernel.org

Re: [PATCH v2] KVM: arm/arm64: BUG FIX: Do not inject spurious interrupts

2015-09-29 Thread Christoffer Dall
On Fri, Sep 25, 2015 at 05:00:29PM +0300, Pavel Fedin wrote: > Commit 71760950bf3dc796e5e53ea3300dec724a09f593 > ("arm/arm64: KVM: add a common vgic_queue_irq_to_lr fn") introduced > vgic_queue_irq_to_lr() function with additional vgic_dist_irq_is_pending() > check before setting LR_STATE_PENDING

[PATCH v2] KVM: arm/arm64: BUG FIX: Do not inject spurious interrupts

2015-09-25 Thread Pavel Fedin
Commit 71760950bf3dc796e5e53ea3300dec724a09f593 ("arm/arm64: KVM: add a common vgic_queue_irq_to_lr fn") introduced vgic_queue_irq_to_lr() function with additional vgic_dist_irq_is_pending() check before setting LR_STATE_PENDING bit. In some cases it started causing the following situation if the