Messages in this thread | | | Date | Thu, 11 Jan 2024 20:49:45 +0800 | From | Yuan Yao <> | Subject | Re: [PATCH 3/4] KVM: x86: Clean up directed yield API for "has pending interrupt" |
| |
On Tue, Jan 09, 2024 at 04:39:37PM -0800, Sean Christopherson wrote: > Directly return the boolean result of whether or not a vCPU has a pending > interrupt instead of effectively doing: > > if (true) > return true; > > return false; > > Signed-off-by: Sean Christopherson <seanjc@google.com>
Reviewed-by: Yuan Yao <yuan.yao@intel.com>
> --- > arch/x86/kvm/x86.c | 7 ++----- > 1 file changed, 2 insertions(+), 5 deletions(-) > > diff --git a/arch/x86/kvm/x86.c b/arch/x86/kvm/x86.c > index 77494f9c8d49..b7996a75d9a3 100644 > --- a/arch/x86/kvm/x86.c > +++ b/arch/x86/kvm/x86.c > @@ -13083,11 +13083,8 @@ int kvm_arch_vcpu_runnable(struct kvm_vcpu *vcpu) > > bool kvm_arch_dy_has_pending_interrupt(struct kvm_vcpu *vcpu) > { > - if (kvm_vcpu_apicv_active(vcpu) && > - static_call(kvm_x86_dy_apicv_has_pending_interrupt)(vcpu)) > - return true; > - > - return false; > + return kvm_vcpu_apicv_active(vcpu) && > + static_call(kvm_x86_dy_apicv_has_pending_interrupt)(vcpu); > } > > bool kvm_arch_vcpu_preempted_in_kernel(struct kvm_vcpu *vcpu) > -- > 2.43.0.472.g3155946c3a-goog > >
| |