From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from mgamail.intel.com (mgamail.intel.com [192.198.163.9]) (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 2652530C147; Wed, 30 Sep 2026 05:41:05 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=192.198.163.9 ARC-Seal:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1790746866; cv=none; b=mw6/+qLXNzNPRhtYsSXf1MSBnDZOQDh6SZy82Uq8NxyORHf161d/5FVb0IoaOlZTxN2zK/YKGiOFFZpTYZkCfyO072mQC0uVTRyysUJWm9FqNgSlJ6vUbRiM5FdDyd+pOSTY+OKQhBzvfQZhC6yyKviCT+Ar6anbxdxetDEfipM= ARC-Message-Signature:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1790746866; c=relaxed/simple; bh=T9P6mcgxpWhKz75d2zCXZHo0i7BHFj5BIcLfYB+EXZE=; h=From:To:Cc:Subject:Date:Message-ID:In-Reply-To:References: MIME-Version; b=iv4rAafEVBEp2KBPDzZRsdLKXfig1KSP4DPm9egeF4X21zzUVg8p3VO0NViNxnBjHHfdMaxhJQefpUL9MDf9z4aBDLIopbCxYwIMYoEt+11eWSH3KZlv+5z+95h5vVMrfGXOqD/QTbD6xi1+RVFmGBVhA60Ylnuu7kyaoZwFbeQ= 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=IRjeSaVZ; arc=none smtp.client-ip=192.198.163.9 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="IRjeSaVZ" DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=intel.com; i=@intel.com; q=dns/txt; s=Intel; t=1790746865; x=1822282865; h=from:to:cc:subject:date:message-id:in-reply-to: references:mime-version:content-transfer-encoding; bh=T9P6mcgxpWhKz75d2zCXZHo0i7BHFj5BIcLfYB+EXZE=; b=IRjeSaVZX/qAPuZ6yXuTTxcj0x4TY2h1XMFjf8eLDiZ9wM2nvW5jzDEU oLoIHt1f5fTrMwT9aaPtasohUBPBKpLtFKZ9wMi/6JNQOjyUydrjRDBUu L686GlCXNv6JdvA+wocbZR/yQFYq/oCFskqUGQJc8qf8Xak5RD6vAN9kr aaGI2TS0F9FcoZhbjMrQtulvdt6WMUZx+vaaDaQdCWls1IQKI4ewUJgg7 cHvURSjjClHxnhTGtMTR4XTwBTEK1Ct0imROrvQn1MHRCNESpQ8BwvFSF lm5jjdcx4GFO8wj1WvTQ2CrK8wzrKo2J+1wrt96XBGud/5xS+kTd6/IeN Q==; X-CSE-ConnectionGUID: 6Fbea9tqQrC4KyXiY7t6EA== X-CSE-MsgGUID: 14eF4eLwSQmnVKFHr2xkqw== X-IronPort-AV: E=McAfee;i="6800,10657,11920"; a="102149077" X-IronPort-AV: E=Sophos;i="6.27,132,1787036400"; d="scan'208";a="102149077" Received: from orviesa004.jf.intel.com ([10.64.159.144]) by fmvoesa103.fm.intel.com with ESMTP/TLS/ECDHE-RSA-AES256-GCM-SHA384; 29 Sep 2026 22:40:59 -0700 X-CSE-ConnectionGUID: vdfn8dUtTOy84d3pfGUCJw== X-CSE-MsgGUID: C7cJUmLVR1GPTbLUo+tUHQ== X-ExtLoop1: 1 X-IronPort-AV: E=Sophos;i="6.27,132,1787036400"; d="scan'208";a="278936392" Received: from 984fee019967.jf.intel.com ([10.23.153.244]) by orviesa004-auth.jf.intel.com with ESMTP/TLS/ECDHE-RSA-AES256-GCM-SHA384; 29 Sep 2026 22:40:58 -0700 From: Chao Gao To: linux-kernel@vger.kernel.org, linux-coco@lists.linux.dev, kvm@vger.kernel.org Cc: yilun.xu@linux.intel.com, chao.gao@intel.com, binbin.wu@linux.intel.com, tony.lindgren@linux.intel.com, Thomas Gleixner , Ingo Molnar , Borislav Petkov , Dave Hansen , x86@kernel.org, "H. Peter Anvin" , Kiryl Shutsemau , Rick Edgecombe Subject: [PATCH v3 01/10] x86/virt/tdx: Add a helper to read a table of metadata fields Date: Tue, 29 Sep 2026 22:38:35 -0700 Message-ID: <20260930053901.22528-2-chao.gao@intel.com> X-Mailer: git-send-email 2.52.0 In-Reply-To: <20260930053901.22528-1-chao.gao@intel.com> References: <20260930053901.22528-1-chao.gao@intel.com> Precedence: bulk X-Mailing-List: linux-kernel@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 Content-Transfer-Encoding: 8bit The metadata field readers get_tdx_sys_info_() in tdx_global_metadata.c were generated by an out-of-tree script. That has not worked out: the JSON file they were generated from is neither stable nor authoritative enough [1]. The goal now is to maintain the readers by hand and to establish one standard way of adding a metadata field. Take get_tdx_sys_info_version() as an example: if (!ret && !(ret = read_sys_metadata_field(0x0800000100000003, &val))) sysinfo_version->minor_version = val; if (!ret && !(ret = read_sys_metadata_field(0x0800000100000004, &val))) sysinfo_version->major_version = val; if (!ret && !(ret = read_sys_metadata_field(0x0800000100000005, &val))) sysinfo_version->update_version = val; Two patterns stand out: the read-check-store sequence repeats once per field, and the error of each read is chained into the reads that follow. Neither is common in hand-written code. Prepare to eliminate both with a loop that reads each field, stores the value into its structure member, and returns on the first error. Add 'struct field_mapping' to describe one field as its ID plus the offset and size of the member that receives its value. Add TDX_SYSINFO_MAP() to build such an entry from a field ID, a structure type and a member name. Add __read_sys_metadata_table() helper to read every field in a table. Annotate that helper __maybe_unused as there is no caller right now. Following changes will convert the existing readers to use the new helper. AI was used under supervision to review code and workshop logs. It suggested adding read_sys_metadata_table() macro, which avoids repeating the table name when passing both the table and its size. Signed-off-by: Chao Gao Link: https://lore.kernel.org/kvm/1e7bcbad-eb26-44b7-97ca-88ab53467212@intel.com/ # [1] --- v3: - Pass the field ID to TDX_SYSINFO_MAP() instead of pasting the TDX_MD_FIELD_ID_ prefix onto a suffix inside the macro [Rick] - Call out that this patch only prepares to eliminate the two patterns. [Rick] - Define @offset in struct field_mapping as size_t [Binbin] --- arch/x86/virt/vmx/tdx/tdx.c | 39 +++++++++++++++++++++++++++++++++++++ 1 file changed, 39 insertions(+) diff --git a/arch/x86/virt/vmx/tdx/tdx.c b/arch/x86/virt/vmx/tdx/tdx.c index 96ced0494b68..67a80ed86bd3 100644 --- a/arch/x86/virt/vmx/tdx/tdx.c +++ b/arch/x86/virt/vmx/tdx/tdx.c @@ -395,6 +395,45 @@ static int read_sys_metadata_field(u64 field_id, u64 *data) return 0; } +/* + * Map a TDX global metadata field to a structure member. + * @field_id: The TDX global metadata field ID. + * @size: The size of the structure member. + * @offset: The member's offset within its containing structure. + */ +struct field_mapping { + u64 field_id; + size_t size; + size_t offset; +}; + +/* Read each metadata field listed in @mappings[] into @data. */ +static int __maybe_unused __read_sys_metadata_table(const struct field_mapping *mappings, + int num_mappings, void *data) +{ + int i, ret; + u64 val; + + for (i = 0; i < num_mappings; i++) { + ret = read_sys_metadata_field(mappings[i].field_id, &val); + if (ret) + return ret; + memcpy((char *)data + mappings[i].offset, &val, mappings[i].size); + } + + return 0; +} + +#define read_sys_metadata_table(_mappings, _data) \ + __read_sys_metadata_table(_mappings, ARRAY_SIZE(_mappings), _data) + +#define TDX_SYSINFO_MAP(_field, _type, _member) \ +{ \ + .field_id = _field, \ + .offset = offsetof(_type, _member), \ + .size = sizeof_field(_type, _member), \ +} + #include "tdx_global_metadata.c" static __init int check_features(struct tdx_sys_info *sysinfo) -- 2.52.0