From: Xu Yilun <yilun.xu@linux.intel.com>
To: x86@kernel.org, dave.hansen@linux.intel.com, kas@kernel.org,
linux-kernel@vger.kernel.org
Cc: chao.gao@intel.com, rick.p.edgecombe@intel.com,
dan.j.williams@intel.com, baolu.lu@linux.intel.com,
yilun.xu@linux.intel.com, yilun.xu@intel.com,
zhenzhong.duan@intel.com, kvm@vger.kernel.org,
adrian.hunter@intel.com
Subject: [PATCH 3/6] x86/virt/tdx: Refactor metadata reading with a clearer for loop
Date: Tue, 2 Dec 2025 13:08:41 +0800 [thread overview]
Message-ID: <20251202050844.2520762-4-yilun.xu@linux.intel.com> (raw)
In-Reply-To: <20251202050844.2520762-1-yilun.xu@linux.intel.com>
Replace confusing "ret = ret ?: " code pattern with a normal for loop.
The existing code pattern is compact and friendly to auto-generation.
However, hiding the stop-on-failure logic within the sequential
execution pattern reduces readability. Revive the field mapping table
which lists the field_ids to read and where to store each readout value.
Iterate the table with a normal for loop.
For now this metadata reading process doesn't work well with array typed
fields. Use dedicated method to read these 2 array typed fields. Will
improve in later patches.
Cc: Kai Huang <kai.huang@intel.com>
Signed-off-by: Xu Yilun <yilun.xu@linux.intel.com>
---
arch/x86/virt/vmx/tdx/tdx_global_metadata.c | 172 ++++++++++----------
1 file changed, 86 insertions(+), 86 deletions(-)
diff --git a/arch/x86/virt/vmx/tdx/tdx_global_metadata.c b/arch/x86/virt/vmx/tdx/tdx_global_metadata.c
index 0dfb3a9995fe..3db87c4accd6 100644
--- a/arch/x86/virt/vmx/tdx/tdx_global_metadata.c
+++ b/arch/x86/virt/vmx/tdx/tdx_global_metadata.c
@@ -7,8 +7,50 @@
* Include this file to other C file instead.
*/
-static int read_sys_metadata_field(u64 field_id, u64 *data)
+struct field_mapping {
+ u64 field_id;
+ int offset;
+ int size;
+};
+
+#define TD_SYSINFO_MAP(_field_id, _member) \
+ { .field_id = _field_id, \
+ .offset = offsetof(struct tdx_sys_info, _member), \
+ .size = sizeof_field(struct tdx_sys_info, _member) }
+
+/* Map TD_SYSINFO fields into 'struct tdx_sys_info': */
+static const struct field_mapping mappings[] = {
+ TD_SYSINFO_MAP(0x0A00000300000008, features.tdx_features0),
+
+ TD_SYSINFO_MAP(0x9100000100000008, tdmr.max_tdmrs),
+ TD_SYSINFO_MAP(0x9100000100000009, tdmr.max_reserved_per_tdmr),
+ TD_SYSINFO_MAP(0x9100000100000010, tdmr.pamt_4k_entry_size),
+ TD_SYSINFO_MAP(0x9100000100000011, tdmr.pamt_2m_entry_size),
+ TD_SYSINFO_MAP(0x9100000100000012, tdmr.pamt_1g_entry_size),
+
+ TD_SYSINFO_MAP(0x9800000100000000, td_ctrl.tdr_base_size),
+ TD_SYSINFO_MAP(0x9800000100000100, td_ctrl.tdcs_base_size),
+ TD_SYSINFO_MAP(0x9800000100000200, td_ctrl.tdvps_base_size),
+
+ TD_SYSINFO_MAP(0x1900000300000000, td_conf.attributes_fixed0),
+ TD_SYSINFO_MAP(0x1900000300000001, td_conf.attributes_fixed1),
+ TD_SYSINFO_MAP(0x1900000300000002, td_conf.xfam_fixed0),
+ TD_SYSINFO_MAP(0x1900000300000003, td_conf.xfam_fixed1),
+ TD_SYSINFO_MAP(0x9900000100000004, td_conf.num_cpuid_config),
+ TD_SYSINFO_MAP(0x9900000100000008, td_conf.max_vcpus_per_td),
+};
+
+/* Populate the following fields in special manner, separate them out. */
+static const struct field_mapping cpuid_config_leaves =
+ TD_SYSINFO_MAP(0x9900000300000400, td_conf.cpuid_config_leaves[0]);
+
+static const struct field_mapping cpuid_config_values =
+ TD_SYSINFO_MAP(0x9900000300000500, td_conf.cpuid_config_values[0][0]);
+
+static int read_sys_metadata_field(u64 field_id, int offset, int size,
+ struct tdx_sys_info *ts)
{
+ void *field = ((void *)ts) + offset;
struct tdx_module_args args = {};
int ret;
@@ -22,97 +64,55 @@ static int read_sys_metadata_field(u64 field_id, u64 *data)
if (ret)
return ret;
- *data = args.r8;
+ memcpy(field, &args.r8, size);
return 0;
}
-static int get_tdx_sys_info_features(struct tdx_sys_info_features *sysinfo_features)
-{
- int ret = 0;
- u64 val;
-
- if (!ret && !(ret = read_sys_metadata_field(0x0A00000300000008, &val)))
- sysinfo_features->tdx_features0 = val;
-
- return ret;
-}
-
-static int get_tdx_sys_info_tdmr(struct tdx_sys_info_tdmr *sysinfo_tdmr)
-{
- int ret = 0;
- u64 val;
-
- if (!ret && !(ret = read_sys_metadata_field(0x9100000100000008, &val)))
- sysinfo_tdmr->max_tdmrs = val;
- if (!ret && !(ret = read_sys_metadata_field(0x9100000100000009, &val)))
- sysinfo_tdmr->max_reserved_per_tdmr = val;
- if (!ret && !(ret = read_sys_metadata_field(0x9100000100000010, &val)))
- sysinfo_tdmr->pamt_4k_entry_size = val;
- if (!ret && !(ret = read_sys_metadata_field(0x9100000100000011, &val)))
- sysinfo_tdmr->pamt_2m_entry_size = val;
- if (!ret && !(ret = read_sys_metadata_field(0x9100000100000012, &val)))
- sysinfo_tdmr->pamt_1g_entry_size = val;
-
- return ret;
-}
-
-static int get_tdx_sys_info_td_ctrl(struct tdx_sys_info_td_ctrl *sysinfo_td_ctrl)
-{
- int ret = 0;
- u64 val;
-
- if (!ret && !(ret = read_sys_metadata_field(0x9800000100000000, &val)))
- sysinfo_td_ctrl->tdr_base_size = val;
- if (!ret && !(ret = read_sys_metadata_field(0x9800000100000100, &val)))
- sysinfo_td_ctrl->tdcs_base_size = val;
- if (!ret && !(ret = read_sys_metadata_field(0x9800000100000200, &val)))
- sysinfo_td_ctrl->tdvps_base_size = val;
-
- return ret;
-}
-
-static int get_tdx_sys_info_td_conf(struct tdx_sys_info_td_conf *sysinfo_td_conf)
-{
- int ret = 0;
- u64 val;
- int i, j;
-
- if (!ret && !(ret = read_sys_metadata_field(0x1900000300000000, &val)))
- sysinfo_td_conf->attributes_fixed0 = val;
- if (!ret && !(ret = read_sys_metadata_field(0x1900000300000001, &val)))
- sysinfo_td_conf->attributes_fixed1 = val;
- if (!ret && !(ret = read_sys_metadata_field(0x1900000300000002, &val)))
- sysinfo_td_conf->xfam_fixed0 = val;
- if (!ret && !(ret = read_sys_metadata_field(0x1900000300000003, &val)))
- sysinfo_td_conf->xfam_fixed1 = val;
- if (!ret && !(ret = read_sys_metadata_field(0x9900000100000004, &val)))
- sysinfo_td_conf->num_cpuid_config = val;
- if (!ret && !(ret = read_sys_metadata_field(0x9900000100000008, &val)))
- sysinfo_td_conf->max_vcpus_per_td = val;
- if (sysinfo_td_conf->num_cpuid_config > ARRAY_SIZE(sysinfo_td_conf->cpuid_config_leaves))
- return -EINVAL;
- for (i = 0; i < sysinfo_td_conf->num_cpuid_config; i++)
- if (!ret && !(ret = read_sys_metadata_field(0x9900000300000400 + i, &val)))
- sysinfo_td_conf->cpuid_config_leaves[i] = val;
- if (sysinfo_td_conf->num_cpuid_config > ARRAY_SIZE(sysinfo_td_conf->cpuid_config_values))
- return -EINVAL;
- for (i = 0; i < sysinfo_td_conf->num_cpuid_config; i++)
- for (j = 0; j < 2; j++)
- if (!ret && !(ret = read_sys_metadata_field(0x9900000300000500 + i * 2 + j, &val)))
- sysinfo_td_conf->cpuid_config_values[i][j] = val;
-
- return ret;
-}
-
static int get_tdx_sys_info(struct tdx_sys_info *sysinfo)
{
- int ret = 0;
+ struct tdx_sys_info_td_conf *td_conf = &sysinfo->td_conf;
+ int ret, i;
+
+ /* Populate 'tdx_sys_info' fields using the mapping structure above: */
+ for (i = 0; i < ARRAY_SIZE(mappings); i++) {
+ ret = read_sys_metadata_field(mappings[i].field_id,
+ mappings[i].offset,
+ mappings[i].size,
+ sysinfo);
+ if (ret)
+ return ret;
+ }
+
+ if (td_conf->num_cpuid_config > ARRAY_SIZE(td_conf->cpuid_config_leaves) ||
+ td_conf->num_cpuid_config > ARRAY_SIZE(td_conf->cpuid_config_values))
+ return -EINVAL;
- ret = ret ?: get_tdx_sys_info_features(&sysinfo->features);
- ret = ret ?: get_tdx_sys_info_tdmr(&sysinfo->tdmr);
- ret = ret ?: get_tdx_sys_info_td_ctrl(&sysinfo->td_ctrl);
- ret = ret ?: get_tdx_sys_info_td_conf(&sysinfo->td_conf);
+ /*
+ * Populate 2 special fields, td_conf.cpuid_config_leaves[] and
+ * td_conf.cpuid_config_values[][]
+ */
+ for (i = 0; i < td_conf->num_cpuid_config; i++) {
+ ret = read_sys_metadata_field(cpuid_config_leaves.field_id + i,
+ cpuid_config_leaves.offset +
+ cpuid_config_leaves.size * i,
+ cpuid_config_leaves.size,
+ sysinfo);
+ if (ret)
+ return ret;
+ }
+
+ for (i = 0;
+ i < td_conf->num_cpuid_config * ARRAY_SIZE(td_conf->cpuid_config_values[0]);
+ i++) {
+ ret = read_sys_metadata_field(cpuid_config_values.field_id + i,
+ cpuid_config_values.offset +
+ cpuid_config_values.size * i,
+ cpuid_config_values.size,
+ sysinfo);
+ if (ret)
+ return ret;
+ }
- return ret;
+ return 0;
}
--
2.25.1
next prev parent reply other threads:[~2025-12-02 5:24 UTC|newest]
Thread overview: 9+ messages / expand[flat|nested] mbox.gz Atom feed top
2025-12-02 5:08 [PATCH 0/6] TDX: Stop metadata auto-generation, improve readability Xu Yilun
2025-12-02 5:08 ` [PATCH 1/6] x86/virt/tdx: Move bit definitions of TDX_FEATURES0 to public header Xu Yilun
2025-12-02 5:08 ` [PATCH 2/6] x86/virt/tdx: Move read_sys_metadata_field() to where it is called Xu Yilun
2025-12-02 5:08 ` Xu Yilun [this message]
2025-12-02 5:08 ` [PATCH 4/6] x86/virt/tdx: Sanity check the size of each metadata field Xu Yilun
2025-12-02 5:08 ` [PATCH 5/6] x86/virt/tdx: Add generic support for reading array-typed metadata Xu Yilun
2025-12-02 5:08 ` [PATCH 6/6] x86/virt/tdx: Skip unsupported metadata by querying tdx_feature0 Xu Yilun
2025-12-07 7:44 ` [PATCH 0/6] TDX: Stop metadata auto-generation, improve readability dan.j.williams
2025-12-08 10:17 ` Xu Yilun
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=20251202050844.2520762-4-yilun.xu@linux.intel.com \
--to=yilun.xu@linux.intel.com \
--cc=adrian.hunter@intel.com \
--cc=baolu.lu@linux.intel.com \
--cc=chao.gao@intel.com \
--cc=dan.j.williams@intel.com \
--cc=dave.hansen@linux.intel.com \
--cc=kas@kernel.org \
--cc=kvm@vger.kernel.org \
--cc=linux-kernel@vger.kernel.org \
--cc=rick.p.edgecombe@intel.com \
--cc=x86@kernel.org \
--cc=yilun.xu@intel.com \
--cc=zhenzhong.duan@intel.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox
all inboxes | Powered by JetHome®