Re: [PATCH 35/62] x86/sev-es: Setup per-cpu GHCBs for the runtime handler

From: Joerg Roedel
Date: Wed Feb 12 2020 - 10:16:18 EST


On Tue, Feb 11, 2020 at 02:46:11PM -0800, Andy Lutomirski wrote:
> On Tue, Feb 11, 2020 at 5:53 AM Joerg Roedel <joro@xxxxxxxxxx> wrote:
> > +/* Runtime GHCBs */
> > +static DEFINE_PER_CPU_DECRYPTED(struct ghcb, ghcb_page) __aligned(PAGE_SIZE);
>
> Hmm. This is a largeish amount of memory on large non-SEV-ES systems.
> Maybe store a pointer instead? It would be even better if it could be
> DEFINE_PER_CPU like this but be discarded if we don't need it, but I
> don't think we have the infrastructure for that.

Yeah, discarding is not easily possible right now, but I changed it to
only store a pointer and allocating the pages only when running as an
SEV-ES guest.

Regards,

Joerg