On 2015/12/9 0:59, Marc Zyngier wrote:
>> > +  }
>> > +
>> > +  /* If all overflow bits are cleared, kick the vcpu to clear interrupt
>> > +   * pending status.
>> > +   */
>> > +  if (val == 0)
>> > +          kvm_vcpu_kick(vcpu);
> Do we really need to do so? This will be dropped on the next entry
> anyway, so i don't see the need to kick the vcpu again. Or am I missing
> something?
> 
I thought maybe it could not set the interrupt low immediately when all
the overflow bits are cleared for some reason, so add a kick to force it
to sync the interrupt. But as you said, I'll remove this.

Thanks,
-- 
Shannon

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

Reply via email to