Messages in this thread | | | Subject | Re: [RFC PATCH v2 10/19] RISC-V: KVM: Handle WFI exits for VCPU | From | Paolo Bonzini <> | Date | Mon, 5 Aug 2019 09:14:44 +0200 |
| |
On 05/08/19 09:12, Anup Patel wrote: > On Fri, Aug 2, 2019 at 2:33 PM Paolo Bonzini <pbonzini@redhat.com> 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
| |