From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from mgamail.intel.com (mgamail.intel.com [192.198.163.15]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by smtp.subspace.kernel.org (Postfix) with ESMTPS id 7B3B93E3D9A for ; Tue, 31 Mar 2026 12:43:29 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=192.198.163.15 ARC-Seal:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1774961011; cv=none; b=poiEaWuzvy4Az5k5+CphOV87KDIcseJR7iJP2shhGC/yOMdorQ71FALF+Iv5hgMxP9f5EKu0PCSl75WWneX64eN23XRjrAxke8kQUcqCVpTpjiUJweJA1B0p0n4H9q8CDR4qgN5r/7dRUbTKtDGNUiKM68i+7Doi8ebkO23e390= ARC-Message-Signature:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1774961011; c=relaxed/simple; bh=1UKoH75/zyVBSrt1yCLDfnFmKoFS6rxFjn9DluZ+Wjs=; h=From:To:Cc:Subject:Date:Message-ID:In-Reply-To:References: MIME-Version:Content-Type; b=Qq425ko0/JsCOCEOREdJpLyUBd9EmKamZY03HeGVWRqCUer6viok40GJRKJIkV6JAIVBo7NCMdFmEcoLv0XA48UdF+1gCNoJdwFn3dbFXWiqhJZWKFAO2Aqi8Yy5nthAidI3qNiaOOsh1F88bBkE64qTRA4lWhG3VyiMjkBR6v8= ARC-Authentication-Results:i=1; smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=intel.com; spf=pass smtp.mailfrom=intel.com; dkim=pass (2048-bit key) header.d=intel.com header.i=@intel.com header.b=j7PBuMLU; arc=none smtp.client-ip=192.198.163.15 Authentication-Results: smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=intel.com Authentication-Results: smtp.subspace.kernel.org; spf=pass smtp.mailfrom=intel.com Authentication-Results: smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=intel.com header.i=@intel.com header.b="j7PBuMLU" DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=intel.com; i=@intel.com; q=dns/txt; s=Intel; t=1774961009; x=1806497009; h=from:to:cc:subject:date:message-id:in-reply-to: references:mime-version:content-transfer-encoding; bh=1UKoH75/zyVBSrt1yCLDfnFmKoFS6rxFjn9DluZ+Wjs=; b=j7PBuMLU+HwyUlrHHgX4RCGH5LkVJI0QiDTCuMvgiVAXunTlK7sqMnng R3ekVmHF9XH6448Fa9XCTWmXymnDvsZoivwoj5IKswbZO3cnznC9px4/p /vfj4jTvIcDXX39xYOQg3ATQQSivfQEyttxfnklNrg2SCLeAK0ueF2Y7I bIkv4tlwb6ykXCDC5RosJ13rlFcDtv8iT57WVhnDwKRB11BQlET3ipt9S 5pFHclJSr8o/Vq1M/G1A1mpL9t0iiitFIizjpCLeIiI2I/iI3T/omCn0m 2qHCczBIJaJyDmP/nQrnIoW9FB0bpXQeUeE5i4uYjdbTjCzK9fhaqG7n1 w==; X-CSE-ConnectionGUID: FvMZezDhS1yXmKL3a7vy0Q== X-CSE-MsgGUID: +y/8lCsxSK2XWmaUFkTNWg== X-IronPort-AV: E=McAfee;i="6800,10657,11745"; a="76084484" X-IronPort-AV: E=Sophos;i="6.23,151,1770624000"; d="scan'208";a="76084484" Received: from fmviesa006.fm.intel.com ([10.60.135.146]) by fmvoesa109.fm.intel.com with ESMTP/TLS/ECDHE-RSA-AES256-GCM-SHA384; 31 Mar 2026 05:43:26 -0700 X-CSE-ConnectionGUID: tQSSMBygTOqaRcSQtGIcvw== X-CSE-MsgGUID: Dve5BFJcTH+mh/1B5LBoxg== X-ExtLoop1: 1 X-IronPort-AV: E=Sophos;i="6.23,151,1770624000"; d="scan'208";a="221492108" Received: from 984fee019967.jf.intel.com ([10.23.153.244]) by fmviesa006-auth.fm.intel.com with ESMTP/TLS/ECDHE-RSA-AES256-GCM-SHA384; 31 Mar 2026 05:43:26 -0700 From: Chao Gao To: linux-kernel@vger.kernel.org, linux-coco@lists.linux.dev, kvm@vger.kernel.org Cc: binbin.wu@linux.intel.com, dan.j.williams@intel.com, dave.hansen@linux.intel.com, ira.weiny@intel.com, kai.huang@intel.com, kas@kernel.org, nik.borisov@suse.com, paulmck@kernel.org, pbonzini@redhat.com, reinette.chatre@intel.com, rick.p.edgecombe@intel.com, sagis@google.com, seanjc@google.com, tony.lindgren@linux.intel.com, vannapurve@google.com, vishal.l.verma@intel.com, yilun.xu@linux.intel.com, xiaoyao.li@intel.com, yan.y.zhao@intel.com, Chao Gao , Thomas Gleixner , Ingo Molnar , Borislav Petkov , x86@kernel.org, "H. Peter Anvin" Subject: [PATCH v7 07/22] coco/tdx-host: Implement firmware upload sysfs ABI for TDX module updates Date: Tue, 31 Mar 2026 05:41:20 -0700 Message-ID: <20260331124214.117808-8-chao.gao@intel.com> X-Mailer: git-send-email 2.47.3 In-Reply-To: <20260331124214.117808-1-chao.gao@intel.com> References: <20260331124214.117808-1-chao.gao@intel.com> Precedence: bulk X-Mailing-List: linux-coco@lists.linux.dev List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit Linux kernel supports two primary firmware update mechanisms: - request_firmware() - firmware upload (or fw_upload) The former is used by microcode updates, SEV firmware updates, etc. The latter is used by CXL and FPGA firmware updates. One key difference between them is: request_firmware() loads a named file from the filesystem where the filename is kernel-controlled, while fw_upload accepts firmware data directly from userspace. Use fw_upload for TDX module updates as loading a named file isn't suitable for TDX (see below for more reasons). Specifically, register TDX faux device with fw_upload framework to expose sysfs interfaces and implement operations to process data blobs supplied by userspace. Implementation notes: 1. P-SEAMLDR processes the entire update at once rather than chunk-by-chunk, so .write() is called only once per update; so the offset should be always 0. 2. An update completes synchronously within .write(), meaning .poll_complete() is only called after the update succeeds and so always returns success Why fw_upload instead of request_firmware()? ============================================ The explicit file selection capabilities of fw_upload is preferred over the implicit file selection of request_firmware() for the following reasons: a. Intel distributes all versions of the TDX module, allowing admins to load any version rather than always defaulting to the latest. This flexibility is necessary because future extensions may require reverting to a previous version to clear fatal errors. b. Some module version series are platform-specific. For example, the 1.5.x series is for certain platform generations, while the 2.0.x series is intended for others. c. The update policy for TDX module updates is non-linear at times. The latest TDX module may not be compatible. For example, TDX module 1.5.x may be updated to 1.5.y but not to 1.5.y+1. This policy is documented separately in a file released along with each TDX module release. So, the default policy of "request_firmware()" of "always load latest", is not suitable for TDX. Userspace needs to deploy a more sophisticated policy check (e.g., latest may not be compatible), and there is potential operator choice to consider. Just have userspace pick rather than add kernel mechanism to change the default policy of request_firmware(). Signed-off-by: Chao Gao Reviewed-by: Tony Lindgren Reviewed-by: Kai Huang Reviewed-by: Kiryl Shutsemau (Meta) --- v6: - Remove unused "-ENOSPC" error mapping—this series never returns it v5: - remove a tail comment [Yan] - remove is_vmalloc_addr() check [Dave] - use devm_add_action_or_reset() for deinit [Yilun] - remove global tdx_fwl [Yilun] - clarify request_firmware() doesn't take filename from userspace [Rick] v4: - make tdx_fwl static [Kai] - don't support update canceling [Yilun] - explain why seamldr_init() doesn't return an error [Kai] - bail out if TDX module updates are not supported [Kai] - name the firmware "tdx_module" instead of "seamldr_upload" [Cedric] v3: - clear "cancel_request" in the "prepare" phase [Binbin] - Don't fail the whole tdx-host device if seamldr_init() met an error [Yilun] - Add kdoc for seamldr_install_module() and verify that the input buffer is vmalloc'd. [Yilun] --- arch/x86/include/asm/seamldr.h | 1 + arch/x86/virt/vmx/tdx/seamldr.c | 15 ++++ drivers/virt/coco/tdx-host/Kconfig | 2 + drivers/virt/coco/tdx-host/tdx-host.c | 99 ++++++++++++++++++++++++++- 4 files changed, 115 insertions(+), 2 deletions(-) diff --git a/arch/x86/include/asm/seamldr.h b/arch/x86/include/asm/seamldr.h index c67e5bc910a9..ac6f80f7208b 100644 --- a/arch/x86/include/asm/seamldr.h +++ b/arch/x86/include/asm/seamldr.h @@ -32,5 +32,6 @@ struct seamldr_info { static_assert(sizeof(struct seamldr_info) == 256); int seamldr_get_info(struct seamldr_info *seamldr_info); +int seamldr_install_module(const u8 *data, u32 size); #endif /* _ASM_X86_SEAMLDR_H */ diff --git a/arch/x86/virt/vmx/tdx/seamldr.c b/arch/x86/virt/vmx/tdx/seamldr.c index 8410df3a0bf4..e93a5d90a3ee 100644 --- a/arch/x86/virt/vmx/tdx/seamldr.c +++ b/arch/x86/virt/vmx/tdx/seamldr.c @@ -6,6 +6,7 @@ */ #define pr_fmt(fmt) "seamldr: " fmt +#include #include #include @@ -40,3 +41,17 @@ int seamldr_get_info(struct seamldr_info *seamldr_info) return seamldr_call(P_SEAMLDR_INFO, &args); } EXPORT_SYMBOL_FOR_MODULES(seamldr_get_info, "tdx-host"); + +/** + * seamldr_install_module - Install a new TDX module. + * @data: Pointer to the TDX module update blob. + * @size: Size of the TDX module update blob. + * + * Returns 0 on success, negative error code on failure. + */ +int seamldr_install_module(const u8 *data, u32 size) +{ + /* TODO: Update TDX module here */ + return 0; +} +EXPORT_SYMBOL_FOR_MODULES(seamldr_install_module, "tdx-host"); diff --git a/drivers/virt/coco/tdx-host/Kconfig b/drivers/virt/coco/tdx-host/Kconfig index d35d85ef91c0..ca600a39d97b 100644 --- a/drivers/virt/coco/tdx-host/Kconfig +++ b/drivers/virt/coco/tdx-host/Kconfig @@ -1,6 +1,8 @@ config TDX_HOST_SERVICES tristate "TDX Host Services Driver" depends on INTEL_TDX_HOST + select FW_LOADER + select FW_UPLOAD default m help Enable access to TDX host services like module update and diff --git a/drivers/virt/coco/tdx-host/tdx-host.c b/drivers/virt/coco/tdx-host/tdx-host.c index 5a672126f372..746a5eef004d 100644 --- a/drivers/virt/coco/tdx-host/tdx-host.c +++ b/drivers/virt/coco/tdx-host/tdx-host.c @@ -6,6 +6,7 @@ */ #include +#include #include #include #include @@ -92,7 +93,7 @@ static struct attribute *seamldr_attrs[] = { NULL, }; -static bool seamldr_group_visible(struct kobject *kobj) +static bool can_expose_seamldr(void) { const struct tdx_sys_info *sysinfo = tdx_get_sysinfo(); @@ -102,6 +103,11 @@ static bool seamldr_group_visible(struct kobject *kobj) return tdx_supports_runtime_update(sysinfo); } +static bool seamldr_group_visible(struct kobject *kobj) +{ + return can_expose_seamldr(); +} + DEFINE_SIMPLE_SYSFS_GROUP_VISIBLE(seamldr); static const struct attribute_group seamldr_group = { @@ -116,6 +122,95 @@ static const struct attribute_group *tdx_host_groups[] = { NULL, }; +static enum fw_upload_err tdx_fw_prepare(struct fw_upload *fwl, + const u8 *data, u32 size) +{ + return FW_UPLOAD_ERR_NONE; +} + +static enum fw_upload_err tdx_fw_write(struct fw_upload *fwl, const u8 *data, + u32 offset, u32 size, u32 *written) +{ + int ret; + + /* + * tdx_fw_write() always processes all data on the first call with + * offset == 0. Since it never returns partial success (it either + * succeeds completely or fails), there is no subsequent call with + * non-zero offsets. + */ + WARN_ON_ONCE(offset); + ret = seamldr_install_module(data, size); + switch (ret) { + case 0: + *written = size; + return FW_UPLOAD_ERR_NONE; + case -EBUSY: + return FW_UPLOAD_ERR_BUSY; + case -EIO: + return FW_UPLOAD_ERR_HW_ERROR; + case -ENOMEM: + return FW_UPLOAD_ERR_RW_ERROR; + default: + return FW_UPLOAD_ERR_FW_INVALID; + } +} + +static enum fw_upload_err tdx_fw_poll_complete(struct fw_upload *fwl) +{ + /* + * TDX module updates are completed in the previous phase + * (tdx_fw_write()). If any error occurred, the previous phase + * would return an error code to abort the update process. In + * other words, reaching this point means the update succeeded. + */ + return FW_UPLOAD_ERR_NONE; +} + +/* + * TDX module updates cannot be cancelled. Provide a stub function since + * the firmware upload framework requires a .cancel operation. + */ +static void tdx_fw_cancel(struct fw_upload *fwl) +{ +} + +static const struct fw_upload_ops tdx_fw_ops = { + .prepare = tdx_fw_prepare, + .write = tdx_fw_write, + .poll_complete = tdx_fw_poll_complete, + .cancel = tdx_fw_cancel, +}; + +static void seamldr_deinit(void *tdx_fwl) +{ + firmware_upload_unregister(tdx_fwl); +} + +static int seamldr_init(struct device *dev) +{ + struct fw_upload *tdx_fwl; + + if (!can_expose_seamldr()) + return 0; + + tdx_fwl = firmware_upload_register(THIS_MODULE, dev, "tdx_module", + &tdx_fw_ops, NULL); + if (IS_ERR(tdx_fwl)) + return PTR_ERR(tdx_fwl); + + return devm_add_action_or_reset(dev, seamldr_deinit, tdx_fwl); +} + +static int tdx_host_probe(struct faux_device *fdev) +{ + return seamldr_init(&fdev->dev); +} + +static const struct faux_device_ops tdx_host_ops = { + .probe = tdx_host_probe, +}; + static struct faux_device *fdev; static int __init tdx_host_init(void) @@ -123,7 +218,7 @@ static int __init tdx_host_init(void) if (!x86_match_cpu(tdx_host_ids) || !tdx_get_sysinfo()) return -ENODEV; - fdev = faux_device_create_with_groups(KBUILD_MODNAME, NULL, NULL, tdx_host_groups); + fdev = faux_device_create_with_groups(KBUILD_MODNAME, NULL, &tdx_host_ops, tdx_host_groups); if (!fdev) return -ENODEV; -- 2.47.3