On 5/23/2025 10:27 AM, Xiaoyao Li wrote:
On 5/23/2025 8:11 AM, Sean Christopherson wrote:
Move TDX hardware setup to tdx.c, as the code is obviously TDX specific,
co-locating the setup with tdx_bringup() makes it easier to see and
document the success_disable_tdx "error" path, and configuring the TDX
specific hooks in tdx.c reduces the number of globally visible TDX
symbols.
Signed-off-by: Sean Christopherson <seanjc@xxxxxxxxxx>
...
diff --git a/arch/x86/kvm/vmx/tdx.h b/arch/x86/kvm/vmx/tdx.h
index 51f98443e8a2..ca39a9391db1 100644
--- a/arch/x86/kvm/vmx/tdx.h
+++ b/arch/x86/kvm/vmx/tdx.h
@@ -8,6 +8,7 @@
#ifdef CONFIG_KVM_INTEL_TDX
#include "common.h"
+void tdx_hardware_setup(void);
we need define stub function for the case of !CONFIG_KVM_INTEL_TDX
sorry that I missed the discussion on v3[*].
Based on kvm-x86/next, no issue.
[*] https://lore.kernel.org/all/aC0MIUOTQbb9-a7k@xxxxxxxxxx/
with it fixed,
Reviewed-by: Xiaoyao Li <xiaoyao.li@xxxxxxxxx>
