Hello!

>  Shouldn't we also have 'break' here? If vgic_queue_irq() returns false, this 
> means we have no
> more
> LRs to use, therefore it makes no sense to keep iterating.

 No, don't listen to me. :) Because of piggyback, we indeed have to recheck all 
the interrupts.

 P.S. I still sometimes lose LPIs, and this is not related to spurious 
injection fix, because i
tried to omit resetting irq_pending_on_cpu bit, and still lost some LPIs. Will 
try to compare with
v2, because with v2 i don't remember this problem.

Kind regards,
Pavel Fedin
Expert Engineer
Samsung Electronics Research center Russia


_______________________________________________
kvmarm mailing list
[email protected]
https://lists.cs.columbia.edu/mailman/listinfo/kvmarm

Reply via email to