Dov: Can you submit one BZ for this new feature? I will add it into edk2 202108 stable tag planning.
Thanks Liming > -----邮件原件----- > 发件人: devel@edk2.groups.io <devel@edk2.groups.io> 代表 Laszlo Ersek > 发送时间: 2021年6月9日 21:54 > 收件人: Dov Murik <dovmu...@linux.ibm.com>; devel@edk2.groups.io; Ard > Biesheuvel <ardb+tianoc...@kernel.org> > 抄送: Tobin Feldman-Fitzthum <to...@linux.ibm.com>; Tobin > Feldman-Fitzthum <to...@ibm.com>; Jim Cadden <jcad...@ibm.com>; > James Bottomley <j...@linux.ibm.com>; Hubertus Franke > <fran...@us.ibm.com>; Jordan Justen <jordan.l.jus...@intel.com>; Ashish > Kalra <ashish.ka...@amd.com>; Brijesh Singh <brijesh.si...@amd.com>; > Erdem Aktas <erdemak...@google.com>; Jiewen Yao > <jiewen....@intel.com>; Min Xu <min.m...@intel.com>; Tom Lendacky > <thomas.lenda...@amd.com> > 主题: Re: [edk2-devel] [PATCH v1 0/8] Measured SEV boot with > kernel/initrd/cmdline > > On 06/09/21 14:25, Dov Murik wrote: > > > > > > On 08/06/2021 18:59, Laszlo Ersek wrote: > >> On 06/08/21 14:09, Dov Murik wrote: > >>> On 08/06/2021 13:59, Laszlo Ersek wrote: > >>>> On 06/08/21 11:57, Dov Murik wrote: > >> > > > >>> > >>> But if we go with (1) -- do you (and Ard) prefer: > >>> > >>> (a) leave X86QemuLoadImageLib as it is in master; > >>> > >>> -or- > >>> > >>> (b) modify X86QemuLoadImageLib the "main" path to use the > >>> QemuKernelLoaderFs (what I started doing) and leave the "legacy" path > >>> with QemuFwCfg > >>> > >>> ? > >> > >> I prefer option (a), with the extension that we need to update the > >> following file-top comment in the files under > >> "OvmfPkg/Library/X86QemuLoadImageLib": > >> > >> X86 specific implementation of QemuLoadImageLib library class > interface > >> with support for loading mixed mode images and non-EFI stub images > >> > > > > First attempt at this is submitted to the mailing list: > > https://edk2.groups.io/g/devel/message/76265 > > > > > >> We should add a warning there that this library instance (a) depends on > >> fw_cfg directly, and (b) is therefore unsuitable for blob verification > >> purposes. > > > > I'll add the warning (b) when I add the blob verification feature. > > That makes sense to me, thanks. > Laszlo > > > > > -=-=-=-=-=-=-=-=-=-=-=- Groups.io Links: You receive all messages sent to this group. View/Reply Online (#76322): https://edk2.groups.io/g/devel/message/76322 Mute This Topic: https://groups.io/mt/83440567/21656 Group Owner: devel+ow...@edk2.groups.io Unsubscribe: https://edk2.groups.io/g/devel/unsub [arch...@mail-archive.com] -=-=-=-=-=-=-=-=-=-=-=-
