* [PATCH] KVM: x86: Take advantage of kvm_arch_dy_has_pending_interrupt()
@ 2021-04-21 3:25 lihaiwei.kernel
2021-04-22 9:21 ` Wanpeng Li
2021-04-22 10:35 ` Paolo Bonzini
0 siblings, 2 replies; 3+ messages in thread
From: lihaiwei.kernel @ 2021-04-21 3:25 UTC (permalink / raw)
To: linux-kernel, kvm
Cc: pbonzini, seanjc, vkuznets, wanpengli, jmattson, joro, Haiwei Li
From: Haiwei Li <lihaiwei@tencent.com>
`kvm_arch_dy_runnable` checks the pending_interrupt as the code in
`kvm_arch_dy_has_pending_interrupt`. So take advantage of it.
Signed-off-by: Haiwei Li <lihaiwei@tencent.com>
---
arch/x86/kvm/x86.c | 21 +++++++++------------
1 file changed, 9 insertions(+), 12 deletions(-)
diff --git a/arch/x86/kvm/x86.c b/arch/x86/kvm/x86.c
index d696a9f..08bd616 100644
--- a/arch/x86/kvm/x86.c
+++ b/arch/x86/kvm/x86.c
@@ -11125,28 +11125,25 @@ int kvm_arch_vcpu_runnable(struct kvm_vcpu *vcpu)
return kvm_vcpu_running(vcpu) || kvm_vcpu_has_events(vcpu);
}
-bool kvm_arch_dy_runnable(struct kvm_vcpu *vcpu)
+bool kvm_arch_dy_has_pending_interrupt(struct kvm_vcpu *vcpu)
{
- if (READ_ONCE(vcpu->arch.pv.pv_unhalted))
- return true;
-
- if (kvm_test_request(KVM_REQ_NMI, vcpu) ||
- kvm_test_request(KVM_REQ_SMI, vcpu) ||
- kvm_test_request(KVM_REQ_EVENT, vcpu))
- return true;
-
if (vcpu->arch.apicv_active && static_call(kvm_x86_dy_apicv_has_pending_interrupt)(vcpu))
return true;
return false;
}
-bool kvm_arch_dy_has_pending_interrupt(struct kvm_vcpu *vcpu)
+bool kvm_arch_dy_runnable(struct kvm_vcpu *vcpu)
{
- if (vcpu->arch.apicv_active && static_call(kvm_x86_dy_apicv_has_pending_interrupt)(vcpu))
+ if (READ_ONCE(vcpu->arch.pv.pv_unhalted))
return true;
- return false;
+ if (kvm_test_request(KVM_REQ_NMI, vcpu) ||
+ kvm_test_request(KVM_REQ_SMI, vcpu) ||
+ kvm_test_request(KVM_REQ_EVENT, vcpu))
+ return true;
+
+ return kvm_arch_dy_has_pending_interrupt(vcpu);
}
bool kvm_arch_vcpu_in_kernel(struct kvm_vcpu *vcpu)
--
1.8.3.1
^ permalink raw reply related [flat|nested] 3+ messages in thread
* Re: [PATCH] KVM: x86: Take advantage of kvm_arch_dy_has_pending_interrupt()
2021-04-21 3:25 [PATCH] KVM: x86: Take advantage of kvm_arch_dy_has_pending_interrupt() lihaiwei.kernel
@ 2021-04-22 9:21 ` Wanpeng Li
2021-04-22 10:35 ` Paolo Bonzini
1 sibling, 0 replies; 3+ messages in thread
From: Wanpeng Li @ 2021-04-22 9:21 UTC (permalink / raw)
To: Haiwei Li
Cc: LKML, kvm, Paolo Bonzini, Sean Christopherson, Vitaly Kuznetsov,
Wanpeng Li, Jim Mattson, Joerg Roedel, Haiwei Li
On Wed, 21 Apr 2021 at 11:26, <lihaiwei.kernel@gmail.com> wrote:
>
> From: Haiwei Li <lihaiwei@tencent.com>
>
> `kvm_arch_dy_runnable` checks the pending_interrupt as the code in
> `kvm_arch_dy_has_pending_interrupt`. So take advantage of it.
>
> Signed-off-by: Haiwei Li <lihaiwei@tencent.com>
Reviewed-by: Wanpeng Li <wanpengli@tencent.com>
^ permalink raw reply [flat|nested] 3+ messages in thread
* Re: [PATCH] KVM: x86: Take advantage of kvm_arch_dy_has_pending_interrupt()
2021-04-21 3:25 [PATCH] KVM: x86: Take advantage of kvm_arch_dy_has_pending_interrupt() lihaiwei.kernel
2021-04-22 9:21 ` Wanpeng Li
@ 2021-04-22 10:35 ` Paolo Bonzini
1 sibling, 0 replies; 3+ messages in thread
From: Paolo Bonzini @ 2021-04-22 10:35 UTC (permalink / raw)
To: lihaiwei.kernel, linux-kernel, kvm
Cc: seanjc, vkuznets, wanpengli, jmattson, joro, Haiwei Li
On 21/04/21 05:25, lihaiwei.kernel@gmail.com wrote:
> From: Haiwei Li <lihaiwei@tencent.com>
>
> `kvm_arch_dy_runnable` checks the pending_interrupt as the code in
> `kvm_arch_dy_has_pending_interrupt`. So take advantage of it.
>
> Signed-off-by: Haiwei Li <lihaiwei@tencent.com>
> ---
> arch/x86/kvm/x86.c | 21 +++++++++------------
> 1 file changed, 9 insertions(+), 12 deletions(-)
>
> diff --git a/arch/x86/kvm/x86.c b/arch/x86/kvm/x86.c
> index d696a9f..08bd616 100644
> --- a/arch/x86/kvm/x86.c
> +++ b/arch/x86/kvm/x86.c
> @@ -11125,28 +11125,25 @@ int kvm_arch_vcpu_runnable(struct kvm_vcpu *vcpu)
> return kvm_vcpu_running(vcpu) || kvm_vcpu_has_events(vcpu);
> }
>
> -bool kvm_arch_dy_runnable(struct kvm_vcpu *vcpu)
> +bool kvm_arch_dy_has_pending_interrupt(struct kvm_vcpu *vcpu)
> {
> - if (READ_ONCE(vcpu->arch.pv.pv_unhalted))
> - return true;
> -
> - if (kvm_test_request(KVM_REQ_NMI, vcpu) ||
> - kvm_test_request(KVM_REQ_SMI, vcpu) ||
> - kvm_test_request(KVM_REQ_EVENT, vcpu))
> - return true;
> -
> if (vcpu->arch.apicv_active && static_call(kvm_x86_dy_apicv_has_pending_interrupt)(vcpu))
> return true;
>
> return false;
> }
>
> -bool kvm_arch_dy_has_pending_interrupt(struct kvm_vcpu *vcpu)
> +bool kvm_arch_dy_runnable(struct kvm_vcpu *vcpu)
> {
> - if (vcpu->arch.apicv_active && static_call(kvm_x86_dy_apicv_has_pending_interrupt)(vcpu))
> + if (READ_ONCE(vcpu->arch.pv.pv_unhalted))
> return true;
>
> - return false;
> + if (kvm_test_request(KVM_REQ_NMI, vcpu) ||
> + kvm_test_request(KVM_REQ_SMI, vcpu) ||
> + kvm_test_request(KVM_REQ_EVENT, vcpu))
> + return true;
> +
> + return kvm_arch_dy_has_pending_interrupt(vcpu);
> }
>
> bool kvm_arch_vcpu_in_kernel(struct kvm_vcpu *vcpu)
>
Looks good, but I'd like to take a look at the other patches for
directed yield first. Thanks!
Paolo
^ permalink raw reply [flat|nested] 3+ messages in thread
end of thread, other threads:[~2021-04-22 10:36 UTC | newest]
Thread overview: 3+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2021-04-21 3:25 [PATCH] KVM: x86: Take advantage of kvm_arch_dy_has_pending_interrupt() lihaiwei.kernel
2021-04-22 9:21 ` Wanpeng Li
2021-04-22 10:35 ` Paolo Bonzini
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).