Re: [PATCH Part2 v6 14/49] crypto: ccp: Handle the legacy TMR allocation when SNP is enabled

Borislav Petkov
2022-11-02T11:29:38Z

On Mon, Oct 31, 2022 at 04:58:38PM -0500, Kalra, Ashish wrote:
> if (snp_lookup_rmpentry(pfn, &rmp_level)) {
> do_sigbus(regs, error_code, address, VM_FAULT_SIGBUS);
> return RMP_PF_RETRY;

Does this issue some halfway understandable error message why the
process got killed?

> Will look at adding our own recovery function for the same, but that will
> again mark the pages as poisoned, right ?

Well, not poisoned but PG_offlimits or whatever the mm folks agree upon.
Semantically, it'll be handled the same way, ofc.

> Still waiting for some/more feedback from mm folks on the same.

Just send the patch and they'll give it.

Thx.

--
Regards/Gruss,
Boris.