KVM: x86/mmu: Replace open coded usage of tdp_mmu_page with is_tdp_mmu_page()

Use is_tdp_mmu_page() instead of querying sp->tdp_mmu_page directly so
that all users benefit if KVM ever finds a way to optimize the logic.

No functional change intended.

Signed-off-by: Sean Christopherson <seanjc@google.com>
Message-Id: <20221012181702.3663607-10-seanjc@google.com>
Signed-off-by: Paolo Bonzini <pbonzini@redhat.com>

diff --git a/arch/x86/kvm/mmu/mmu.c b/arch/x86/kvm/mmu/mmu.c
index 7fb7a07a389e5b4b6eb4431629ca340d7135136e..27b233eff6e5b0b9c31b93fdca39b2bbef838f2a 100644 (file)
--- a/arch/x86/kvm/mmu/mmu.c
+++ b/arch/x86/kvm/mmu/mmu.c
@@ -1928,7 +1928,7 @@ static bool is_obsolete_sp(struct kvm *kvm, struct kvm_mmu_page *sp)
                return true;
 
        /* TDP MMU pages do not use the MMU generation. */
-       return !sp->tdp_mmu_page &&
+       return !is_tdp_mmu_page(sp) &&
               unlikely(sp->mmu_valid_gen != kvm->arch.mmu_valid_gen);
 }
 

diff --git a/arch/x86/kvm/mmu/tdp_mmu.c b/arch/x86/kvm/mmu/tdp_mmu.c
index 7e5952e95d3bf7c4fe1f73bbee1471c1c5581013..cc1fb9a65620152249c79cbf1770602a209b1e66 100644 (file)
--- a/arch/x86/kvm/mmu/tdp_mmu.c
+++ b/arch/x86/kvm/mmu/tdp_mmu.c
@@ -133,7 +133,7 @@ void kvm_tdp_mmu_put_root(struct kvm *kvm, struct kvm_mmu_page *root,
        if (!refcount_dec_and_test(&root->tdp_mmu_root_count))
                return;
 
-       WARN_ON(!root->tdp_mmu_page);
+       WARN_ON(!is_tdp_mmu_page(root));
 
        /*
         * The root now has refcount=0.  It is valid, but readers already