@@ -501,10 +501,9 @@ static inline struct vmcs *alloc_vmcs(bool shadow)
u64 construct_eptp(struct kvm_vcpu *vcpu, unsigned long root_hpa);
-static inline void __vmx_flush_tlb(struct kvm_vcpu *vcpu, int vpid,
- bool invalidate_gpa)
+static inline void __vmx_flush_tlb(struct kvm_vcpu *vcpu, int vpid)
{
- if (enable_ept && (invalidate_gpa || !enable_vpid)) {
+ if (enable_ept) {
if (!VALID_PAGE(vcpu->arch.mmu->root_hpa))
return;
ept_sync_context(construct_eptp(vcpu,
@@ -516,7 +515,7 @@ static inline void __vmx_flush_tlb(struct kvm_vcpu *vcpu, int vpid,
static inline void vmx_flush_tlb(struct kvm_vcpu *vcpu)
{
- __vmx_flush_tlb(vcpu, to_vmx(vcpu)->vpid, true);
+ __vmx_flush_tlb(vcpu, to_vmx(vcpu)->vpid);
}
static inline void decache_tsc_multiplier(struct vcpu_vmx *vmx)
Drop @invalidate_gpa from __vmx_flush_tlb() now that it's sole caller unconditionally passes %true, i.e. "(invalidate_gpa || !enable_vpid)" will always evaluate true. No functional change intended. Signed-off-by: Sean Christopherson <sean.j.christopherson@intel.com> --- arch/x86/kvm/vmx/vmx.h | 7 +++---- 1 file changed, 3 insertions(+), 4 deletions(-)