2022-06-22 11:39:17

by Kai Huang

[permalink] [raw]
Subject: [PATCH v5 05/22] x86/virt/tdx: Prevent hot-add driver managed memory

TDX provides increased levels of memory confidentiality and integrity.
This requires special hardware support for features like memory
encryption and storage of memory integrity checksums. Not all memory
satisfies these requirements.

As a result, the TDX introduced the concept of a "Convertible Memory
Region" (CMR). During boot, the firmware builds a list of all of the
memory ranges which can provide the TDX security guarantees. The list
of these ranges is available to the kernel by querying the TDX module.

However those TDX-capable memory regions are not automatically usable to
the TDX module. The kernel needs to choose which convertible memory
regions to be the TDX-usable memory and pass those regions to the TDX
module when initializing the module. Once those ranges are passed to
the TDX module, the TDX-usable memory regions are fixed during module's
lifetime.

To avoid having to modify the page allocator to distinguish TDX and
non-TDX memory allocation, this implementation guarantees all pages
managed by the page allocator are TDX memory. This means any hot-added
memory to the page allocator will break such guarantee thus should be
prevented.

There are basically two memory hot-add cases that need to be prevented:
ACPI memory hot-add and driver managed memory hot-add. However, adding
new memory to ZONE_DEVICE should not be prevented as those pages are not
managed by the page allocator. Therefore memremap_pages() variants
should be allowed although they internally also use memory hotplug
functions.

ACPI memory hotplug is already prevented. To prevent driver managed
memory and still allow memremap_pages() variants to work, add a __weak
hook to do arch-specific check in add_memory_resource(). Implement the
x86 version to prevent new memory region from being added when TDX is
enabled by BIOS.

The __weak arch-specific hook is used instead of a new CC_ATTR similar
to disable software CPU hotplug. It is because some driver managed
memory resources may actually be TDX-capable (such as legacy PMEM, which
is underneath indeed RAM), and the arch-specific hook can be further
enhanced to allow those when needed.

Note arch-specific hook for __remove_memory() is not required. Both
ACPI hot-removal and driver managed memory removal cannot reach it.

Signed-off-by: Kai Huang <[email protected]>
---
arch/x86/mm/init_64.c | 21 +++++++++++++++++++++
include/linux/memory_hotplug.h | 2 ++
mm/memory_hotplug.c | 15 +++++++++++++++
3 files changed, 38 insertions(+)

diff --git a/arch/x86/mm/init_64.c b/arch/x86/mm/init_64.c
index 96d34ebb20a9..ce89cf88a818 100644
--- a/arch/x86/mm/init_64.c
+++ b/arch/x86/mm/init_64.c
@@ -55,6 +55,7 @@
#include <asm/uv/uv.h>
#include <asm/setup.h>
#include <asm/ftrace.h>
+#include <asm/tdx.h>

#include "mm_internal.h"

@@ -972,6 +973,26 @@ int arch_add_memory(int nid, u64 start, u64 size,
return add_pages(nid, start_pfn, nr_pages, params);
}

+int arch_memory_add_precheck(int nid, u64 start, u64 size, mhp_t mhp_flags)
+{
+ if (!platform_tdx_enabled())
+ return 0;
+
+ /*
+ * TDX needs to guarantee all pages managed by the page allocator
+ * are TDX memory in order to not have to distinguish TDX and
+ * non-TDX memory allocation. The kernel needs to pass the
+ * TDX-usable memory regions to the TDX module when it gets
+ * initialized. After that, the TDX-usable memory regions are
+ * fixed. This means any memory hot-add to the page allocator
+ * will break above guarantee thus should be prevented.
+ */
+ pr_err("Unable to add memory [0x%llx, 0x%llx) on TDX enabled platform.\n",
+ start, start + size);
+
+ return -EINVAL;
+}
+
static void __meminit free_pagetable(struct page *page, int order)
{
unsigned long magic;
diff --git a/include/linux/memory_hotplug.h b/include/linux/memory_hotplug.h
index 1ce6f8044f1e..306ef4ceb419 100644
--- a/include/linux/memory_hotplug.h
+++ b/include/linux/memory_hotplug.h
@@ -325,6 +325,8 @@ extern int add_memory_resource(int nid, struct resource *resource,
extern int add_memory_driver_managed(int nid, u64 start, u64 size,
const char *resource_name,
mhp_t mhp_flags);
+extern int arch_memory_add_precheck(int nid, u64 start, u64 size,
+ mhp_t mhp_flags);
extern void move_pfn_range_to_zone(struct zone *zone, unsigned long start_pfn,
unsigned long nr_pages,
struct vmem_altmap *altmap, int migratetype);
diff --git a/mm/memory_hotplug.c b/mm/memory_hotplug.c
index 416b38ca8def..2ad4b2603c7c 100644
--- a/mm/memory_hotplug.c
+++ b/mm/memory_hotplug.c
@@ -1296,6 +1296,17 @@ bool mhp_supports_memmap_on_memory(unsigned long size)
IS_ALIGNED(remaining_size, (pageblock_nr_pages << PAGE_SHIFT));
}

+/*
+ * Pre-check whether hot-add memory is allowed before arch_add_memory().
+ *
+ * Arch to provide replacement version if required.
+ */
+int __weak arch_memory_add_precheck(int nid, u64 start, u64 size,
+ mhp_t mhp_flags)
+{
+ return 0;
+}
+
/*
* NOTE: The caller must call lock_device_hotplug() to serialize hotplug
* and online/offline operations (triggered e.g. by sysfs).
@@ -1319,6 +1330,10 @@ int __ref add_memory_resource(int nid, struct resource *res, mhp_t mhp_flags)
if (ret)
return ret;

+ ret = arch_memory_add_precheck(nid, start, size, mhp_flags);
+ if (ret)
+ return ret;
+
if (mhp_flags & MHP_NID_IS_MGID) {
group = memory_group_find_by_id(nid);
if (!group)
--
2.36.1


2022-06-24 02:36:29

by Chao Gao

[permalink] [raw]
Subject: Re: [PATCH v5 05/22] x86/virt/tdx: Prevent hot-add driver managed memory

On Wed, Jun 22, 2022 at 11:16:19PM +1200, Kai Huang wrote:
>@@ -55,6 +55,7 @@
> #include <asm/uv/uv.h>
> #include <asm/setup.h>
> #include <asm/ftrace.h>
>+#include <asm/tdx.h>
>
> #include "mm_internal.h"
>
>@@ -972,6 +973,26 @@ int arch_add_memory(int nid, u64 start, u64 size,
> return add_pages(nid, start_pfn, nr_pages, params);
> }
>
>+int arch_memory_add_precheck(int nid, u64 start, u64 size, mhp_t mhp_flags)
>+{
>+ if (!platform_tdx_enabled())
>+ return 0;

add a new cc attribute (if existing ones don't fit) for TDX host platform and
check the attribute here. So that the code here can be reused by other cc
platforms if they have the same requirement.

2022-06-24 11:42:02

by Kai Huang

[permalink] [raw]
Subject: Re: [PATCH v5 05/22] x86/virt/tdx: Prevent hot-add driver managed memory

On Fri, 2022-06-24 at 10:12 +0800, Chao Gao wrote:
> On Wed, Jun 22, 2022 at 11:16:19PM +1200, Kai Huang wrote:
> > @@ -55,6 +55,7 @@
> > #include <asm/uv/uv.h>
> > #include <asm/setup.h>
> > #include <asm/ftrace.h>
> > +#include <asm/tdx.h>
> >
> > #include "mm_internal.h"
> >
> > @@ -972,6 +973,26 @@ int arch_add_memory(int nid, u64 start, u64 size,
> > return add_pages(nid, start_pfn, nr_pages, params);
> > }
> >
> > +int arch_memory_add_precheck(int nid, u64 start, u64 size, mhp_t mhp_flags)
> > +{
> > + if (!platform_tdx_enabled())
> > + return 0;
>
> add a new cc attribute (if existing ones don't fit) for TDX host platform and
> check the attribute here. So that the code here can be reused by other cc
> platforms if they have the same requirement.

Please see my explanation in the commit message:

The __weak arch-specific hook is used instead of a new CC_ATTR similar
to disable software CPU hotplug. It is because some driver managed
memory resources may actually be TDX-capable (such as legacy PMEM, which
is underneath indeed RAM), and the arch-specific hook can be further
enhanced to allow those when needed.


--
Thanks,
-Kai


2022-06-24 19:24:08

by Dave Hansen

[permalink] [raw]
Subject: Re: [PATCH v5 05/22] x86/virt/tdx: Prevent hot-add driver managed memory

On 6/22/22 04:16, Kai Huang wrote:
> @@ -1319,6 +1330,10 @@ int __ref add_memory_resource(int nid, struct resource *res, mhp_t mhp_flags)
> if (ret)
> return ret;
>
> + ret = arch_memory_add_precheck(nid, start, size, mhp_flags);
> + if (ret)
> + return ret;

Shouldn't a patch that claims to be only for "driver managed memory" be
patching add_memory_driver_managed()?

2022-06-27 05:44:08

by Kai Huang

[permalink] [raw]
Subject: Re: [PATCH v5 05/22] x86/virt/tdx: Prevent hot-add driver managed memory

On Fri, 2022-06-24 at 12:01 -0700, Dave Hansen wrote:
> On 6/22/22 04:16, Kai Huang wrote:
> > @@ -1319,6 +1330,10 @@ int __ref add_memory_resource(int nid, struct resource *res, mhp_t mhp_flags)
> > if (ret)
> > return ret;
> >
> > + ret = arch_memory_add_precheck(nid, start, size, mhp_flags);
> > + if (ret)
> > + return ret;
>
> Shouldn't a patch that claims to be only for "driver managed memory" be
> patching add_memory_driver_managed()?

Right given the ACPI memory hotplug is handled in a separate patch. Will move
to add_memory_driver_managed().


--
Thanks,
-Kai