KVM: x86: hyper-v: Remove unused inline function kvm_hv_free_pa_page()

[PATCH -next] KVM: x86: hyper-v: Remove unused inline function kvm_hv_free_pa_page()

Posted by Yue Haibing 1 year, 4 months ago

There is no caller in tree since introduction in commit b4f69df0f65e ("KVM:
x86: Make Hyper-V emulation optional")

Signed-off-by: Yue Haibing <yuehaibing@huawei.com>
---
 arch/x86/kvm/hyperv.h | 1 -
 1 file changed, 1 deletion(-)

diff --git a/arch/x86/kvm/hyperv.h b/arch/x86/kvm/hyperv.h
index 923e64903da9..913bfc96959c 100644
--- a/arch/x86/kvm/hyperv.h
+++ b/arch/x86/kvm/hyperv.h
@@ -286,7 +286,6 @@ static inline int kvm_hv_hypercall(struct kvm_vcpu *vcpu)
 	return HV_STATUS_ACCESS_DENIED;
 }
 static inline void kvm_hv_vcpu_purge_flush_tlb(struct kvm_vcpu *vcpu) {}
-static inline void kvm_hv_free_pa_page(struct kvm *kvm) {}
 static inline bool kvm_hv_synic_has_vector(struct kvm_vcpu *vcpu, int vector)
 {
 	return false;
-- 
2.34.1

Re: [PATCH -next] KVM: x86: hyper-v: Remove unused inline function kvm_hv_free_pa_page()

Posted by Paolo Bonzini 1 year, 4 months ago

Queued, thanks.

Paolo

Re: [PATCH -next] KVM: x86: hyper-v: Remove unused inline function kvm_hv_free_pa_page()

Posted by Vitaly Kuznetsov 1 year, 4 months ago

Yue Haibing <yuehaibing@huawei.com> writes:

> There is no caller in tree since introduction in commit b4f69df0f65e ("KVM:
> x86: Make Hyper-V emulation optional")

Yea, I tried some history digging but came out empty handed and I have
to admit I have no idea how and when I added this stub. Thanks for the
cleanup!

Reviewed-by: Vitaly Kuznetsov <vkuznets@redhat.com>

>
> Signed-off-by: Yue Haibing <yuehaibing@huawei.com>
> ---
>  arch/x86/kvm/hyperv.h | 1 -
>  1 file changed, 1 deletion(-)
>
> diff --git a/arch/x86/kvm/hyperv.h b/arch/x86/kvm/hyperv.h
> index 923e64903da9..913bfc96959c 100644
> --- a/arch/x86/kvm/hyperv.h
> +++ b/arch/x86/kvm/hyperv.h
> @@ -286,7 +286,6 @@ static inline int kvm_hv_hypercall(struct kvm_vcpu *vcpu)
>  	return HV_STATUS_ACCESS_DENIED;
>  }
>  static inline void kvm_hv_vcpu_purge_flush_tlb(struct kvm_vcpu *vcpu) {}
> -static inline void kvm_hv_free_pa_page(struct kvm *kvm) {}
>  static inline bool kvm_hv_synic_has_vector(struct kvm_vcpu *vcpu, int vector)
>  {
>  	return false;

-- 
Vitaly