hi maobibo: > > > On 2024/3/25 下午5:25, xianglai wrote: >> From: Bibo Mao <maob...@loongson.cn> >> >> Temparory stack memory on PEI is hardcoded now, also minimium memory >> [......] >> + >> + QemuFwCfgSelectItem (FwCfgItem); >> + for (Processed = 0; Processed < FwCfgSize; Processed += sizeof >> MemoryMapEntry) { >> + QemuFwCfgReadBytes (sizeof MemoryMapEntry, &MemoryMapEntry); >> + if (MemoryMapEntry.Type != EfiAcpiAddressRangeMemory) { >> + continue; >> + } >> + >> + /* >> + * Find memory map entry where PEI temp stack is located >> + */ >> + if ((MemoryMapEntry.BaseAddr <= Base) && >> + (Base < (MemoryMapEntry.BaseAddr + MemoryMapEntry.Length))) { >> + RamTop = MemoryMapEntry.BaseAddr + MemoryMapEntry.Length; > Xianglai, > > Will it be better if there is one "break" sentence?
Ok,I will fix it in the next version. Thanks, Xianglai. > > Regards > Bibo Mao > >> + } >> + } >> + >> + if (RamTop == 0) { >> + DEBUG ((DEBUG_ERROR, "ERROR: No memory map entry contains temp stack >> \n")); >> + ASSERT (FALSE); >> + } >> -=-=-=-=-=-=-=-=-=-=-=- Groups.io Links: You receive all messages sent to this group. View/Reply Online (#117144): https://edk2.groups.io/g/devel/message/117144 Mute This Topic: https://groups.io/mt/105134610/21656 Group Owner: devel+ow...@edk2.groups.io Unsubscribe: https://edk2.groups.io/g/devel/unsub [arch...@mail-archive.com] -=-=-=-=-=-=-=-=-=-=-=-