RE: [PATCH v3 2/3] mm: Change ghes code to allow poison of non-struct pfn
From: Luck, Tony
Date: Tue Oct 21 2025 - 13:19:17 EST
> > pfn = PHYS_PFN(physical_addr);
> > - if (!pfn_valid(pfn) && !arch_is_platform_page(physical_addr)) {
>
> Tony,
>
> I'm not an SGX expert but does this break SGX by removing
> arch_is_platform_page()?
>
> See:
>
> 40e0e7843e23 ("x86/sgx: Add infrastructure to identify SGX EPC pages")
> Cc: Tony Luck <tony.luck@xxxxxxxxx>
>
Ira,
I think this deletion makes the GHES code always call memory_failure()
instead of bailing out here on "bad" page frame numbers.
That centralizes the checks for different types of memory into
memory_failure().
-Tony