On 08/14/19 16:00, Gao, Liming wrote: > Laszlo: > > I cherry pick this patch from Mu project with the minimal change. > I can update the comment style.
Yes, please. Thanks! >> The gEfiMemoryTypeInformationGuid HOB is supposed to be built -- if it >> is built at all -- no later than in the DXE IPL PEIM (if VariablePei is >> included in the platform, and the underlying UEFI variable exists). Is >> that correct? >> > gEfiMemoryTypeInformationGuid HOB is installed by platform PEI. Yes, that's what I meant by "no later than in the DXE IPL PEIM". > If the platform PEI doesn't install this HOB, it means this feature is > disabled. PlatformPei is supposed to build the HOB in the first place, yes. However, if it doesn't, then there still may be a feedback loop between the DXE IPL PEIM and BDS. The former builds the HOB from the UEFI variable, and the latter updates the variable (as I understand) for future boots. Thanks Laszlo -=-=-=-=-=-=-=-=-=-=-=- Groups.io Links: You receive all messages sent to this group. View/Reply Online (#45614): https://edk2.groups.io/g/devel/message/45614 Mute This Topic: https://groups.io/mt/32821535/21656 Group Owner: devel+ow...@edk2.groups.io Unsubscribe: https://edk2.groups.io/g/devel/unsub [arch...@mail-archive.com] -=-=-=-=-=-=-=-=-=-=-=-