Thank you Jiewen :) ________________________________ From: Yao, Jiewen Sent: Friday, February 25, 2022 3:21 PM To: Boeuf, Sebastien ; kraxel@redhat.com ; devel@edk2.groups.io Cc: Justen, Jordan L Subject: RE: [edk2-devel] [PATCH 0/3] CloudHv: Rely on PVH boot specification I updated release planning page for you. Thank you Yao Jiewen > -----Original Message----- > From: Boeuf, Sebastien > Sent: Friday, February 25, 2022 7:10 PM > To: kraxel@redhat.com; Yao, Jiewen ; > devel@edk2.groups.io > Cc: Justen, Jordan L > Subject: Re: [edk2-devel] [PATCH 0/3] CloudHv: Rely on PVH boot specification > > On Wed, 2022-02-23 at 14:14 +0000, Yao, Jiewen wrote: > > Yes, justification in bugzilla and README, please. > > I've just created a bug in Bugzilla: > https://bugzilla.tianocore.org/show_bug.cgi?id=3848 > > But I don't have the permissions to edit the wiki > https://github.com/tianocore/tianocore.github.io/wiki/EDK-II-Release-Planning > > Do I need to send a PR to update the wiki? > > > > > I suggestion you send a different email to ask the different topic - > > not distract people. > > > > > > > -----Original Message----- > > > From: Boeuf, Sebastien > > > Sent: Wednesday, February 23, 2022 10:03 PM > > > To: kraxel@redhat.com; Yao, Jiewen ; > > > devel@edk2.groups.io > > > Cc: Justen, Jordan L > > > Subject: Re: [edk2-devel] [PATCH 0/3] CloudHv: Rely on PVH boot > > > specification > > > > > > On Wed, 2022-02-23 at 13:11 +0000, Yao, Jiewen wrote: > > > > If you want to support PVH-only, that means you *defeature* the > > > > CloudHv in *edk2-stable202202* tag according to > > > > https://github.com/tianocore/tianocore.github.io/wiki/EDK-II-Release > > > > - > > > Planning > > > > . > > > > I hope that is stated clearly, with justification why we choose > > > > PVH- > > > > only. Something like: "In edk2-stable202202, CloudHv supported xxx. > > > > In edk2-stable202205 or future, CloudHv for non-TDX will only > > > > support > > > > PVH, because xxxxxx. The CloudFv for TDX will continue support > > > > xxx." > > > > An ASCII table is preferred to clarify the combination. > > > > > > Sounds good. So all the justification should be part of the Bugzilla > > > issue, right? > > > > > > BTW, completely different topic, but wouldn't it be easier to use > > > Github for tracking issues? I mean especially since it's already used > > > for CI and Wiki. > > > > > > > > > > > If possible, please create a similar README under > > > > https://github.com/tianocore/edk2/tree/master/OvmfPkg/CloudHv to > > > > record such info. (configuration, feature, supported v.s. > > > > unsupported, URL link, how to build, how to launch, etc) > > > > > > Of course :) > > > > > > > > > > > > > > > FYI: The readme in Microvm is a good example - > > > > > https://github.com/tianocore/edk2/blob/master/OvmfPkg/Microvm/README > > > > . > > > > > > > > Thank you > > > > Yao Jiewen > > > > > > > > > -----Original Message----- > > > > > From: Boeuf, Sebastien > > > > > Sent: Wednesday, February 23, 2022 8:20 PM > > > > > To: kraxel@redhat.com; devel@edk2.groups.io > > > > > Cc: Yao, Jiewen ; Justen, Jordan L > > > > > > > > > > Subject: Re: [edk2-devel] [PATCH 0/3] CloudHv: Rely on PVH boot > > > > > specification > > > > > > > > > > On Wed, 2022-02-23 at 13:02 +0100, kraxel@redhat.com wrote: > > > > > > Hi, > > > > > > > > > > > > > Well that's a good question. If we expect the same target > > > > > > > (CloudHv) > > > > > > > to > > > > > > > support both TDX and non-TDX, that means the generated TDVF > > > > > > > will be > > > > > > > a > > > > > > > PVH ELF binary, which will require some special handling from > > > > > > > Cloud > > > > > > > Hypervisor. > > > > > > > Having two separate targets would simplify things a lot. > > > > > > > What's > > > > > > > the > > > > > > > plan for QEMU? Will the same OVMF target cover both use > > > > > > > cases? > > > > > > > > > > > > Yes, there will be a single binary supporting both tdx and non- > > > > > > tdx, > > > > > > some > > > > > > configs add sev to the mix. Doing the same for cloudhv > > > > > > shouldn't > > > > > > be > > > > > > much > > > > > > of a problem I think. > > > > > > > > > > > > In tdx mode the firmware uses the tdhob for memory detection, > > > > > > in > > > > > > non- > > > > > > tdx > > > > > > mode qemu fw_cfg is used instead. The cloudhv build could > > > > > > switch > > > > > > between tdhob and pvhinfo in a simliar way. > > > > > > > > > > Sounds good :) > > > > > > > > > > > > > > > > > take care, > > > > > > Gerd > > > > > > > > > > > > --------------------------------------------------------------------- Intel Corporation SAS (French simplified joint stock company) Registered headquarters: "Les Montalets"- 2, rue de Paris, 92196 Meudon Cedex, France Registration Number: 302 456 199 R.C.S. NANTERRE Capital: 4,572,000 Euros This e-mail and any attachments may contain confidential material for the sole use of the intended recipient(s). Any review or distribution by others is strictly prohibited. If you are not the intended recipient, please contact the sender and delete all copies.