[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH v7 07/13] confidential guest support: Introduce cgs "ready" f
Re: [PATCH v7 07/13] confidential guest support: Introduce cgs "ready" flag
Tue, 19 Jan 2021 09:16:08 +0100
On Mon, 18 Jan 2021 19:47:30 +0000
"Dr. David Alan Gilbert" <firstname.lastname@example.org> wrote:
> * David Gibson (email@example.com) wrote:
> > The platform specific details of mechanisms for implementing
> > confidential guest support may require setup at various points during
> > initialization. Thus, it's not really feasible to have a single cgs
> > initialization hook, but instead each mechanism needs its own
> > initialization calls in arch or machine specific code.
> > However, to make it harder to have a bug where a mechanism isn't
> > properly initialized under some circumstances, we want to have a
> > common place, relatively late in boot, where we verify that cgs has
> > been initialized if it was requested.
> > This patch introduces a ready flag to the ConfidentialGuestSupport
> > base type to accomplish this, which we verify just before the machine
> > specific initialization function.
> You may find you need to define 'ready' and the answer might be a bit
> variable; for example, on SEV there's a setup bit and then you may end
> up doing an attestation and receiving some data before you actaully let
> the guest execute code. Is it ready before it's received the
> attestation response or only when it can run code?
> Is a Power or 390 machine 'ready' before it's executed the magic
> instruction to enter the confidential mode?
I would consider those machines where the guest makes the transition
itself "ready" as soon as everything is set up so that the guest can
actually initiate the transition. Otherwise, those machines would never
be "ready" if the guest does not transition.
Maybe we can define "ready" as "the guest can start to execute in
secure mode", where "guest" includes the bootloader and everything that
runs in a guest context, and "start to execute" implies that some setup
may be done only after the guest has kicked it off?
[PATCH v7 01/13] qom: Allow optional sugar props, David Gibson, 2021/01/13
[PATCH v7 03/13] sev: Remove false abstraction of flash encryption, David Gibson, 2021/01/13
[PATCH v7 07/13] confidential guest support: Introduce cgs "ready" flag, David Gibson, 2021/01/13
[PATCH v7 05/13] confidential guest support: Rework the "memory-encryption" property, David Gibson, 2021/01/13
[PATCH v7 04/13] confidential guest support: Move side effect out of machine_set_memory_encryption(), David Gibson, 2021/01/13
[PATCH v7 06/13] sev: Add Error ** to sev_kvm_init(), David Gibson, 2021/01/13
[PATCH v7 09/13] confidential guest support: Update documentation, David Gibson, 2021/01/13
- Re: [PATCH v7 02/13] confidential guest support: Introduce new confidential guest support class, (continued)