Re: [PATCH] x86/sev: Do RMP memory coverage check after max_pfn has been set

From: Tom Lendacky
Date: Fri Jun 21 2024 - 11:02:53 EST


On 6/21/24 09:49, Borislav Petkov wrote:
> On Fri, Jun 21, 2024 at 09:37:46AM -0500, Tom Lendacky wrote:
>> Ok, I'll remove the new static and resubmit. There is also a logic error
>> in the original check which should be using PFN_UP instead of PHYS_PFN, so
>> I'll include that, too.
>
> So we said this fix should not go to stable because SNP host is not upstream
> yet.

Correct.

>
>> Do you want a single patch or two patches, one to fix the PHYS_PFN to
>> PFN_UP and one to move the check?
>
> Since this is snp_rmptable_init() and that is also SNP host then I think
> a single patch is fine.

Will do.

Thanks,
Tom

>
> Thx.
>