Re: [PATCH v3 17/28] x86/sgx: fix a NULL pointer

From: Haitao Huang
Date: Mon Jul 17 2023 - 14:49:56 EST


On Mon, 17 Jul 2023 10:49:03 -0500, Dave Hansen <dave.hansen@xxxxxxxxx> wrote:

On 7/17/23 05:48, Jarkko Sakkinen wrote:
On Wed Jul 12, 2023 at 11:01 PM UTC, Haitao Huang wrote:
Under heavy load, the SGX EPC reclaimers (ksgxd or future EPC cgroup
worker) may reclaim SECS EPC page for an enclave and set
encl->secs.epc_page to NULL. But the SECS EPC page is required for EAUG
in #PF handler and is used without checking for NULL and reloading.

Fix this by checking if SECS is loaded before EAUG and load it if it was
reclaimed.

Signed-off-by: Haitao Huang <haitao.huang@xxxxxxxxxxxxxxx>
A bug fix should be 1/*.

No, bug fixes should not even be _part_ of another series. Send bug
fixes separately, please.


I sent the two bug fixes separately now. Do you want me resend this series without those?
Thanks
Haitao