On 8/16/22 02:57, Yuanhao Xie via groups.io wrote:
To remove the dependency of CPU register, 4/8 byte at the top of the stack
is occupied for CpuMpData. BIST information is also taken care here.
This modification is only for PEI phase, since in DXE phase CpuMpData is
accessed via global variable.
> - MpBuffer= AllocatePages (EFI_SIZE_TO_PAGES (BufferSize));
> + //
> + // Allocate extra ApStackSize to let stack align on ApStackSize bounday
> + //
> + MpBuffer = AllocatePages (EFI_SIZE_TO_PAGES (BufferSize));
1. Above comments are not necessary. Can you please remove them?
>ASS
