Runtime TDX module update introduces a mechanism to update the module firmware while preserving and restoring TDX operations. As part of the restoration process, the host must re-execute all extensions initialization steps to restore extension SEAMCALL functionality. However, Linux runs the update in stop_machine() context, which prevents memory allocation. This introduces a hard restriction that the updated TDX environment must not consume more memory for the extensions. Consequently, the post-update initialization for the extensions is implemented as: 1. Detect if the extensions are supported and required. 2. Detect if the extensions require additional memory. If yes, fail the update. 3. Initialize the extensions via TDH.EXT.INIT. The memory allocation problem is greatly mitigated since Linux applies a policy that configures the same add-on features for boot and for update. This policy minimizes the chance of increased memory demand. So now the restriction only affects the compatibility rule for choosing the update image. Signed-off-by: Xu Yilun --- arch/x86/virt/vmx/tdx/tdx.c | 31 ++++++++++++++++++++- arch/x86/virt/vmx/tdx/tdx_global_metadata.c | 2 +- 2 files changed, 31 insertions(+), 2 deletions(-) diff --git a/arch/x86/virt/vmx/tdx/tdx.c b/arch/x86/virt/vmx/tdx/tdx.c index 900928de373a..4d2940f4538a 100644 --- a/arch/x86/virt/vmx/tdx/tdx.c +++ b/arch/x86/virt/vmx/tdx/tdx.c @@ -1168,7 +1168,7 @@ static __init int init_tdmrs(struct tdmr_info_list *tdmr_list) } /* Initialize TDX module extensions for extension SEAMCALLs */ -static __init int tdx_ext_init(void) +static int tdx_ext_init(void) { struct tdx_module_args args = {}; u64 r; @@ -1308,6 +1308,31 @@ static __init int init_tdx_module_extensions(void) return tdx_ext_init(); } +/* + * Mostly the same flow as init_tdx_module_extensions(), but rejects adding + * more memory. + */ +static int update_tdx_module_extensions(void) +{ + struct tdx_sys_info_ext sysinfo_ext; + int ret; + + if (!(tdx_sysinfo.features.tdx_features0 & TDX_FEATURES0_EXT)) + return 0; + + ret = get_tdx_sys_info_ext(&sysinfo_ext); + if (ret) + return ret; + + if (!sysinfo_ext.ext_required) + return 0; + + if (sysinfo_ext.memory_pool_required_pages) + return -EFAULT; + + return tdx_ext_init(); +} + static __init int init_tdx_module(void) { int ret; @@ -1498,6 +1523,10 @@ int tdx_module_run_update(void) */ WARN_ON_ONCE(ret); + ret = update_tdx_module_extensions(); + if (ret) + return ret; + tdx_module_state.initialized = true; return 0; } diff --git a/arch/x86/virt/vmx/tdx/tdx_global_metadata.c b/arch/x86/virt/vmx/tdx/tdx_global_metadata.c index 720cdaf76492..84364da89649 100644 --- a/arch/x86/virt/vmx/tdx/tdx_global_metadata.c +++ b/arch/x86/virt/vmx/tdx/tdx_global_metadata.c @@ -132,7 +132,7 @@ static __init int get_tdx_sys_info(struct tdx_sys_info *sysinfo) return ret; } -static __init int get_tdx_sys_info_ext(struct tdx_sys_info_ext *sysinfo_ext) +static int get_tdx_sys_info_ext(struct tdx_sys_info_ext *sysinfo_ext) { int ret; u64 val; -- 2.25.1