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]
-=-=-=-=-=-=-=-=-=-=-=-


Reply via email to