Ray, Could we add one API (like InitializeApicTimerEx) to intialize CPU local APCI with additional parameter to indicate if interrupt is to be enabled or not? Just like HPE's solution. Jeff fanjianfen

Please correct the commit message which confused me. It should be "Update the PcdCpuStackGuard to support PcdsPatchableInModule, PcdsDynamci and PcdsDynamciEx ... ..." Thanks! Jeff fanjianfeng@byosoft

On 32bit protectd mode, is there any impact if PcdCpuStackGuard is set to TRUE? Even thers is no page table enabling at that time. Best Regards, Jeff fanjianfeng@...

Ray, EFI_SYSTEM_CONTEXT was defined in MdePkg/Include/Protocol/DebugSupport.h Jeff fanjianfeng@...

Ranga, What's CR2 value in Exception dump information？It holds the actual address occurred PF exception. ExceptionData - 0000000000000000 P:0 means no page transition in page table. Jeff fanjianfeng@b

If so, gBS->LocateProtocol (&gEfiMpServiceProtocolGuid, NULL, (VOID **) &Mp) is enough to locate MP Protocol, instead of Thanks! Jeff fanjianfeng@...

Hi，Rebecca I'd like to give two comments on this patch. 1, I don't think there are more than one MP services instances installed reuirement on one system. X86 platform installed one MP service instanc

You may try to enlarger stmm_ns_comm_buf_size in Optee to check if it could solve ASSERT issue. fanjianfeng@...

OVMF did a similare change on Time Driver, please refer to https://github.com/tianocore/edk2/commit/239b50a863704f7960525799eda82de061c7c458 I do not think this will be apply for ArmPkg/TimerDxe. If o

In fact, my concern is that making BaseLib's APIs depend on PeiService & Boot Services is not good. For a instance, AP functions by MP service are not allowed to invoke any PeriServices and Boot Servi

For former caller, they could still keep as is to call the old API in MdeModulePkg. I think Ray's design is compatible change for existing code. Only when the existing code wants to remove the depende

Making baselib implementation depend on MemoryAllocationLib (indirectly on Pei Service and gBS), it may prevent this base API using at some seneraio. i don't think it's better. Add this parameter and

Marvin, ImageBase was saved in DR2 to help HOST debugger to locate the image base more easily, even though the HOST debugger could find the image base through searching the DOS image signagure. The re

Ray, BSP's GDT table is setup in CpuDxe and then MpInitLib re-uses BSP's GDT table for APs. 1, UefiCpuPkg\CpuDxe: gdt = AllocateRuntimePool (sizeof (GdtTemplate) + 8); ..... gdtPtr.Base = (UINT32)(UIN

Ray, Yes. You are right. Acutally, x64 IDT table cannot work correctly on protected mode. :-) But for GDT location, I agree it should be located under 4G space to support AP mode changing. But we coul

" Will EFI_MP_SERVICES_PROTOCOL.StartupAllAPs() make all 8 logical cores do some tasks simultaneously?" All APs logical processors excute AP task, not including BSP. Acutally the answer is 7. Jeff

we will save the current BSP's GDT and IDT for APs at first time APs are waken by BSP as below. APs will start from real mode to protected mode and then to long mode. During protected mode, BSP's GDT/