On 02/08/19 09:47, Anup Patel wrote:
> + if (!kvm_riscv_vcpu_has_interrupt(vcpu)) {
This can be kvm_arch_vcpu_runnable instead, since kvm_vcpu_block will
check it anyway before sleeping.
Paolo
> + kvm_vcpu_block(vcpu);
> + kvm_clear_request(KVM_REQ_UNHALT, vcpu);
> + }
On Fri, Aug 2, 2019 at 2:33 PM Paolo Bonzini <[email protected]> wrote:
>
> On 02/08/19 09:47, Anup Patel wrote:
> > + if (!kvm_riscv_vcpu_has_interrupt(vcpu)) {
>
> This can be kvm_arch_vcpu_runnable instead, since kvm_vcpu_block will
> check it anyway before sleeping.
I think we can skip this check here because kvm_vcpu_block() is
checking it anyway. Agree ??
Regards,
Anup
On 05/08/19 09:12, Anup Patel wrote:
> On Fri, Aug 2, 2019 at 2:33 PM Paolo Bonzini <[email protected]> wrote:
>>
>> On 02/08/19 09:47, Anup Patel wrote:
>>> + if (!kvm_riscv_vcpu_has_interrupt(vcpu)) {
>>
>> This can be kvm_arch_vcpu_runnable instead, since kvm_vcpu_block will
>> check it anyway before sleeping.
>
> I think we can skip this check here because kvm_vcpu_block() is
> checking it anyway. Agree ??
Yes, but it's quite a bit faster to do this outside the call. There's a
bunch of setup before kvm_vcpu_block reaches that point, and it includes
mfences too once you add srcu_read_unlock/lock here.
Paolo
On Mon, Aug 5, 2019 at 12:44 PM Paolo Bonzini <[email protected]> wrote:
>
> On 05/08/19 09:12, Anup Patel wrote:
> > On Fri, Aug 2, 2019 at 2:33 PM Paolo Bonzini <[email protected]> wrote:
> >>
> >> On 02/08/19 09:47, Anup Patel wrote:
> >>> + if (!kvm_riscv_vcpu_has_interrupt(vcpu)) {
> >>
> >> This can be kvm_arch_vcpu_runnable instead, since kvm_vcpu_block will
> >> check it anyway before sleeping.
> >
> > I think we can skip this check here because kvm_vcpu_block() is
> > checking it anyway. Agree ??
>
> Yes, but it's quite a bit faster to do this outside the call. There's a
> bunch of setup before kvm_vcpu_block reaches that point, and it includes
> mfences too once you add srcu_read_unlock/lock here.
No problem, I will use kvm_arch_vcpu_runnable() here.
Regards,
Anup