On Wed, Feb 23, 2022 at 12:34:21AM +0000, Min Xu wrote: > On February 22, 2022 9:10 PM, Gerd Hoffmann wrote: > > > > > > Add null instance of PlatformInitLib. > > > > What is this library needed for? > > In #25 (OvmfPkg: Update PlatformInitLib to process Tdx hoblist) > ProcessTdxHobList is added in PlatformInitLib to process the Tdx HobList. > This function is called in OvmfPkg/Sec/SecMain.c (#28 OvmfPkg: Update Sec to > support Tdx). ProcessTdxHobList is called by OvmfPkg/IntelTdx/Sec/SecMain.c > as well. See https://edk2.groups.io/g/devel/message/86120 > > OvmfPkg/Sec/SecMain.c is used by > OvmfXen.dsc/AmdSevX64.dsc/BhyveX64.dsc/CloudHvX64.dsc/MicrovmX64.dsc. So > PlatformInitLibNull is needed by these platforms.
OK. So this is probably temporary thing until the other ovmf variants are switched over to use PlatformInitLib too? If so, can you add a comment saying so? With that added: Acked-by: Gerd Hoffmann <kra...@redhat.com> -=-=-=-=-=-=-=-=-=-=-=- Groups.io Links: You receive all messages sent to this group. View/Reply Online (#86884): https://edk2.groups.io/g/devel/message/86884 Mute This Topic: https://groups.io/mt/89252053/21656 Group Owner: devel+ow...@edk2.groups.io Unsubscribe: https://edk2.groups.io/g/devel/unsub [arch...@mail-archive.com] -=-=-=-=-=-=-=-=-=-=-=-
