Re: [edk2-devel] [PATCH v4 1/1] OvmfPkg/Sec: fix stack switch

["Yao, Jiewen" <jiewen.yao@intel.com>]

OK. Let's get it work at first.

Acked-by: Jiewen Yao <Jiewen.yao@intel.com>

> -----Original Message-----
> From: kraxel@redhat.com <kraxel@redhat.com>
> Sent: Tuesday, June 7, 2022 10:45 PM
> To: Yao, Jiewen <jiewen.yao@intel.com>
> Cc: devel@edk2.groups.io; Pawel Polawski <ppolawsk@redhat.com>; Brijesh
> Singh <brijesh.singh@amd.com>; Oliver Steffen <osteffen@redhat.com>; James
> Bottomley <jejb@linux.ibm.com>; Ard Biesheuvel <ardb+tianocore@kernel.org>;
> Aktas, Erdem <erdemaktas@google.com>; Feng, Bob C
> <bob.c.feng@intel.com>; Chen, Christine <yuwei.chen@intel.com>; Gao, Liming
> <gaoliming@byosoft.com.cn>; Xu, Min M <min.m.xu@intel.com>; Tom
> Lendacky <thomas.lendacky@amd.com>; Justen, Jordan L
> <jordan.l.justen@intel.com>; Jiri Slaby <jirislaby@kernel.org>
> Subject: Re: [edk2-devel] [PATCH v4 1/1] OvmfPkg/Sec: fix stack switch
> 
> On Tue, Jun 07, 2022 at 01:43:00PM +0000, Yao, Jiewen wrote:
> > Hello
> > As far as I know the TemporaryRamMigration() is an optional PPI according to
> PI spec, I forget why we add it in the beginning.
> 
> git log isn't very helpful here either, seems to date back to pre-git
> times without descriptive commit messages.
> 
> just commenting it out doesn't work, so probably there is some reason
> which is valid still ...
> 
> take care,
>   Gerd