| lists.openwall.net | lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC | |
|
Open Source and information security mailing list archives
| ||
|
Date: Wed, 6 Apr 2022 16:49:29 +1200
From: Kai Huang <kai.huang@...el.com>
To: linux-kernel@...r.kernel.org, kvm@...r.kernel.org
Cc: seanjc@...gle.com, pbonzini@...hat.com, dave.hansen@...el.com,
len.brown@...el.com, tony.luck@...el.com,
rafael.j.wysocki@...el.com, reinette.chatre@...el.com,
dan.j.williams@...el.com, peterz@...radead.org, ak@...ux.intel.com,
kirill.shutemov@...ux.intel.com,
sathyanarayanan.kuppuswamy@...ux.intel.com,
isaku.yamahata@...el.com, kai.huang@...el.com
Subject: [PATCH v3 17/21] x86/virt/tdx: Configure global KeyID on all packages
Before TDX module can use the global KeyID to access TDX metadata, the
key of the global KeyID must be configured on all physical packages via
TDH.SYS.KEY.CONFIG. This SEAMCALL cannot run concurrently on different
cpus since it exclusively acquires the TDX module.
Implement a helper to run SEAMCALL on one (any) cpu for all packages in
serialized way, and run TDH.SYS.KEY.CONFIG on all packages using the
helper.
The TDX module uses the global KeyID to initialize its metadata (PAMTs).
Before TDX module can do that, all cachelines of PAMTs must be flushed.
Otherwise, they may silently corrupt the PAMTs later initialized by the
TDX module.
Use wbinvd to flush cache as PAMTs can be potentially large (~1/256th of
system RAM).
Flush cache before configuring the global KeyID on all packages, as
suggested by TDX specification. In practice, the current generation of
TDX doesn't use the global KeyID in TDH.SYS.KEY.CONFIG. Therefore in
practice flushing cache can be done after configuring the global KeyID
is done on all packages. But the future generation of TDX may change
this behaviour, so just follow TDX specification's suggestion to flush
cache before configuring the global KeyID on all packages.
Signed-off-by: Kai Huang <kai.huang@...el.com>
---
arch/x86/virt/vmx/tdx/tdx.c | 96 ++++++++++++++++++++++++++++++++++++-
arch/x86/virt/vmx/tdx/tdx.h | 1 +
2 files changed, 96 insertions(+), 1 deletion(-)
diff --git a/arch/x86/virt/vmx/tdx/tdx.c b/arch/x86/virt/vmx/tdx/tdx.c
index 2bf49d3d7cfe..bb15122fb8bd 100644
--- a/arch/x86/virt/vmx/tdx/tdx.c
+++ b/arch/x86/virt/vmx/tdx/tdx.c
@@ -23,6 +23,7 @@
#include <asm/virtext.h>
#include <asm/e820/api.h>
#include <asm/pgtable.h>
+#include <asm/smp.h>
#include <asm/tdx.h>
#include "tdx.h"
@@ -398,6 +399,46 @@ static int seamcall_on_each_cpu(struct seamcall_ctx *sc)
return atomic_read(&sc->err);
}
+/*
+ * Call the SEAMCALL on one (any) cpu for each physical package in
+ * serialized way. Return immediately in case of any error while
+ * calling SEAMCALL on any cpu.
+ *
+ * Note for serialized calls 'struct seamcall_ctx::err' doesn't have
+ * to be atomic, but for simplicity just reuse it instead of adding
+ * a new one.
+ */
+static int seamcall_on_each_package_serialized(struct seamcall_ctx *sc)
+{
+ cpumask_var_t packages;
+ int cpu, ret = 0;
+
+ if (!zalloc_cpumask_var(&packages, GFP_KERNEL))
+ return -ENOMEM;
+
+ for_each_online_cpu(cpu) {
+ if (cpumask_test_and_set_cpu(topology_physical_package_id(cpu),
+ packages))
+ continue;
+
+ ret = smp_call_function_single(cpu, seamcall_smp_call_function,
+ sc, true);
+ if (ret)
+ break;
+
+ /*
+ * Doesn't have to use atomic_read(), but it doesn't
+ * hurt either.
+ */
+ ret = atomic_read(&sc->err);
+ if (ret)
+ break;
+ }
+
+ free_cpumask_var(packages);
+ return ret;
+}
+
static inline bool p_seamldr_ready(void)
{
return !!p_seamldr_info.p_seamldr_ready;
@@ -1320,6 +1361,21 @@ static int config_tdx_module(struct tdmr_info **tdmr_array, int tdmr_num,
return ret;
}
+static int config_global_keyid(void)
+{
+ struct seamcall_ctx sc = { .fn = TDH_SYS_KEY_CONFIG };
+
+ /*
+ * Configure the key of the global KeyID on all packages by
+ * calling TDH.SYS.KEY.CONFIG on all packages.
+ *
+ * TDH.SYS.KEY.CONFIG may fail with entropy error (which is
+ * a recoverable error). Assume this is exceedingly rare and
+ * just return error if encountered instead of retrying.
+ */
+ return seamcall_on_each_package_serialized(&sc);
+}
+
static int init_tdx_module(void)
{
struct tdmr_info **tdmr_array;
@@ -1370,6 +1426,37 @@ static int init_tdx_module(void)
if (ret)
goto out_free_pamts;
+ /*
+ * The same physical address associated with different KeyIDs
+ * has separate cachelines. Before using the new KeyID to access
+ * some memory, the cachelines associated with the old KeyID must
+ * be flushed, otherwise they may later silently corrupt the data
+ * written with the new KeyID. After cachelines associated with
+ * the old KeyID are flushed, CPU speculative fetch using the old
+ * KeyID is OK since the prefetched cachelines won't be consumed
+ * by CPU core.
+ *
+ * TDX module initializes PAMTs using the global KeyID to crypto
+ * protect them from malicious host. Before that, the PAMTs are
+ * used by kernel (with KeyID 0) and the cachelines associated
+ * with the PAMTs must be flushed. Given PAMTs are potentially
+ * large (~1/256th of system RAM), just use WBINVD on all cpus to
+ * flush the cache.
+ *
+ * In practice, the current generation of TDX doesn't use the
+ * global KeyID in TDH.SYS.KEY.CONFIG. Therefore in practice,
+ * the cachelines can be flushed after configuring the global
+ * KeyID on all pkgs is done. But the future generation of TDX
+ * may change this, so just follow the suggestion of TDX spec to
+ * flush cache before TDH.SYS.KEY.CONFIG.
+ */
+ wbinvd_on_all_cpus();
+
+ /* Config the key of global KeyID on all packages */
+ ret = config_global_keyid();
+ if (ret)
+ goto out_free_pamts;
+
/*
* Return -EFAULT until all steps of TDX module
* initialization are done.
@@ -1380,8 +1467,15 @@ static int init_tdx_module(void)
* Free PAMTs allocated in construct_tdmrs() when TDX module
* initialization fails.
*/
- if (ret)
+ if (ret) {
+ /*
+ * Part of PAMTs may already have been initialized by
+ * TDX module. Flush cache before returning them back
+ * to kernel.
+ */
+ wbinvd_on_all_cpus();
tdmrs_free_pamt_all(tdmr_array, tdmr_num);
+ }
out_free_tdmrs:
/*
* TDMRs are only used during initializing TDX module. Always
diff --git a/arch/x86/virt/vmx/tdx/tdx.h b/arch/x86/virt/vmx/tdx/tdx.h
index d8e2800397af..bba8cabea4bb 100644
--- a/arch/x86/virt/vmx/tdx/tdx.h
+++ b/arch/x86/virt/vmx/tdx/tdx.h
@@ -122,6 +122,7 @@ struct tdmr_info {
/*
* TDX module SEAMCALL leaf functions
*/
+#define TDH_SYS_KEY_CONFIG 31
#define TDH_SYS_INFO 32
#define TDH_SYS_INIT 33
#define TDH_SYS_LP_INIT 35
--
2.35.1
Powered by blists - more mailing lists