On Wed, Mar 03, 2021 at 09:56:00AM -0500, Tobin Feldman-Fitzthum wrote: > > > Hello Tobin, > > > > Just a high level question, why is this patch included in this > > patch series, i don't think you are supporting SEV-ES platform > > migration in this patch-set ? > > You are correct that we don't support migration for SEV-ES machines, > although our approach can potentially be adapted for SEV-ES. I was on the > fence about including this patch, because we don't strictly need it for > migration. I'm not sure if the SEC GHCB would be significant even if we did > support SEV-ES migration. Ultimately it seemed like a good idea because the > SEV firmware build does otherwise support SEV-ES. Since I was introducing > the hypercall in an environment where SEV-ES can be enabled, it seemed > reasonable to include. Syncing page encryption status hypothetically has > uses beyond migration. > > Note that I am not adding full support for the hypercall in OVMF, which > might be a good idea at some point. >
Yes, i don't see the assembler code stub for the hypercall in the OVMF patches, so i was wondering the same. Thanks, Ashish -=-=-=-=-=-=-=-=-=-=-=- Groups.io Links: You receive all messages sent to this group. View/Reply Online (#72389): https://edk2.groups.io/g/devel/message/72389 Mute This Topic: https://groups.io/mt/81036364/21656 Group Owner: devel+ow...@edk2.groups.io Unsubscribe: https://edk2.groups.io/g/devel/unsub [arch...@mail-archive.com] -=-=-=-=-=-=-=-=-=-=-=-