Re: [edk2-devel] [PATCH v4 00/11] Measured SEV boot with kernel/initrd/cmdline

2021-07-26 Thread Yao, Jiewen
Xu, Min M ; Tom Lendacky > ; Leif Lindholm ; Sami > Mujawar > Subject: Re: [edk2-devel] [PATCH v4 00/11] Measured SEV boot with > kernel/initrd/cmdline > > On Mon, 2021-07-26 at 00:55 +, Yao, Jiewen wrote: > > Hi James > > "However, this ran into problems whe

Re: [edk2-devel] [PATCH v4 00/11] Measured SEV boot with kernel/initrd/cmdline

2021-07-26 Thread James Bottomley
On Mon, 2021-07-26 at 00:55 +, Yao, Jiewen wrote: > Hi James > "However, this ran into problems when it was decided AmdSev shouldn't > have it's own Library." > > I am not clear on the history. Would you please clarify why AmdSev > should not have its own library? The history predates me. It

Re: [edk2-devel] [PATCH v4 00/11] Measured SEV boot with kernel/initrd/cmdline

2021-07-25 Thread Yao, Jiewen
Tobin Feldman-Fitzthum > ; Jim Cadden ; Hubertus Franke > ; Ard Biesheuvel ; Justen, > Jordan L ; Ashish Kalra ; > Brijesh Singh ; Erdem Aktas > ; Xu, Min M ; Tom Lendacky > ; Leif Lindholm ; Sami > Mujawar > Subject: Re: [edk2-devel] [PATCH v4 00/11] Measured SEV boot with &

Re: [edk2-devel] [PATCH v4 00/11] Measured SEV boot with kernel/initrd/cmdline

2021-07-25 Thread James Bottomley
On Sun, 2021-07-25 at 10:52 +0300, Dov Murik wrote: > And I do have one question: > > May I know what is criteria to put a SEV module to OvmfPkg\AmdSev > > or OvmfPkg directly? > > > > My original understanding is: > > If a module is required by OvmfPkg{Ia32,Ia32X64,X64}.{dsc,fdf}, > > then it sho

Re: [edk2-devel] [PATCH v4 00/11] Measured SEV boot with kernel/initrd/cmdline

2021-07-25 Thread Dov Murik
> Cc: Tobin Feldman-Fitzthum ; Tobin Feldman-Fitzthum >> ; Jim Cadden ; James Bottomley >> ; Hubertus Franke ; Ard Biesheuvel >> ; Justen, Jordan L ; >> Ashish Kalra ; Brijesh Singh ; >> Erdem Aktas ; Xu, Min M ; >> Tom Lendacky ; Leif Lindholm >> ; Sami Mujawa

Re: [edk2-devel] [PATCH v4 00/11] Measured SEV boot with kernel/initrd/cmdline

2021-07-25 Thread Yao, Jiewen
Biesheuvel > ; Justen, Jordan L ; > Ashish Kalra ; Brijesh Singh ; > Erdem Aktas ; Xu, Min M ; > Tom Lendacky ; Leif Lindholm > ; Sami Mujawar ; Dov Murik > > Subject: Re: [edk2-devel] [PATCH v4 00/11] Measured SEV boot with > kernel/initrd/cmdline > > Hi Jiewen, > &g

Re: [edk2-devel] [PATCH v4 00/11] Measured SEV boot with kernel/initrd/cmdline

2021-07-25 Thread Dov Murik
Hi Jiewen, On 25/07/2021 5:43, Yao, Jiewen wrote: > Hi > I am reviewing this patch series. I am OK with most parts. Thank you. > > And I do have one question: > May I know what is criteria to put a SEV module to OvmfPkg\AmdSev or OvmfPkg > directly? > > My original understanding is: > If a mo

Re: [edk2-devel] [PATCH v4 00/11] Measured SEV boot with kernel/initrd/cmdline

2021-07-24 Thread Yao, Jiewen
Hi I am reviewing this patch series. I am OK with most parts. And I do have one question: May I know what is criteria to put a SEV module to OvmfPkg\AmdSev or OvmfPkg directly? My original understanding is: If a module is required by OvmfPkg{Ia32,Ia32X64,X64}.{dsc,fdf}, then it should be OvmfPk