]> git.ipfire.org Git - thirdparty/linux.git/commitdiff
KVM: TDX: Fix sparse warnings from using 0 for NULL
authorDave Hansen <dave.hansen@linux.intel.com>
Mon, 3 Nov 2025 23:44:39 +0000 (15:44 -0800)
committerSean Christopherson <seanjc@google.com>
Thu, 13 Nov 2025 15:10:40 +0000 (07:10 -0800)
Stop using 0 for NULL.

sparse moans:

... arch/x86/kvm/vmx/tdx.c:859:38: warning: Using plain integer as NULL pointer

for several TDX pointer initializations. While I love a good ptr=0
now and then, it's good to have quiet sparse builds.

Signed-off-by: Dave Hansen <dave.hansen@linux.intel.com>
Fixes: a50f673f25e0 ("KVM: TDX: Do TDX specific vcpu initialization")
Fixes: 8d032b683c29 ("KVM: TDX: create/destroy VM structure")
Reviewed-by: Rick Edgecombe <rick.p.edgecombe@intel.com>
Cc: Xiaoyao Li <xiaoyao.li@intel.com>
Cc: Sean Christopherson <seanjc@google.com>
Cc: Paolo Bonzini <pbonzini@redhat.com>
Cc: Thomas Gleixner <tglx@linutronix.de>
Cc: Ingo Molnar <mingo@redhat.com>
Cc: Borislav Petkov <bp@alien8.de>
Cc: x86@kernel.org
Cc: "H. Peter Anvin" <hpa@zytor.com>
Cc: "Kirill A. Shutemov" <kas@kernel.org>
Cc: Rick Edgecombe <rick.p.edgecombe@intel.com>
Cc: kvm@vger.kernel.org
Cc: linux-kernel@vger.kernel.org
Reviewed-by: Xiaoyao Li <xiaoyao.li@intel.com>
Acked-by: Kiryl Shutsemau <kas@kernel.org>
Link: https://patch.msgid.link/20251103234439.DC8227E4@davehans-spike.ostc.intel.com
Signed-off-by: Sean Christopherson <seanjc@google.com>
arch/x86/kvm/vmx/tdx.c

index 3525e0e9d0738d9fa5627369e6ddb6f8f61d391d..0ffca14c10718edca98798378a7611e8a84c3c50 100644 (file)
@@ -903,7 +903,7 @@ void tdx_vcpu_free(struct kvm_vcpu *vcpu)
        }
        if (tdx->vp.tdvpr_page) {
                tdx_reclaim_control_page(tdx->vp.tdvpr_page);
-               tdx->vp.tdvpr_page = 0;
+               tdx->vp.tdvpr_page = NULL;
                tdx->vp.tdvpr_pa = 0;
        }
 
@@ -2581,7 +2581,7 @@ free_tdcs:
 free_tdr:
        if (tdr_page)
                __free_page(tdr_page);
-       kvm_tdx->td.tdr_page = 0;
+       kvm_tdx->td.tdr_page = NULL;
 
 free_hkid:
        tdx_hkid_free(kvm_tdx);
@@ -3000,7 +3000,7 @@ free_tdcx:
 free_tdvpr:
        if (tdx->vp.tdvpr_page)
                __free_page(tdx->vp.tdvpr_page);
-       tdx->vp.tdvpr_page = 0;
+       tdx->vp.tdvpr_page = NULL;
        tdx->vp.tdvpr_pa = 0;
 
        return ret;