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


--
To unsubscribe from this list: send the line "unsubscribe kvm" in
the body of a message to [email protected]
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Reply via email to