Re: [PATCH v2 06/12] x86/sev: Replace occurrences of sev_active() with prot_guest_has()

From: Borislav Petkov
Date: Tue Aug 17 2021 - 14:42:29 EST


On Tue, Aug 17, 2021 at 10:26:18AM -0500, Tom Lendacky wrote:
> >> /*
> >> - * If SME is active we need to be sure that kexec pages are
> >> - * not encrypted because when we boot to the new kernel the
> >> + * If host memory encryption is active we need to be sure that kexec
> >> + * pages are not encrypted because when we boot to the new kernel the
> >> * pages won't be accessed encrypted (initially).
> >> */
> >
> > That hunk belongs logically into the previous patch which removes
> > sme_active().
>
> I was trying to keep the sev_active() changes separate... so even though
> it's an SME thing, I kept it here. But I can move it to the previous
> patch, it just might look strange.

Oh I meant only the comment because it is a SME-related change. But not
too important so whatever.

--
Regards/Gruss,
Boris.

https://people.kernel.org/tglx/notes-about-netiquette