Re: [edk2-devel] [PATCH V8 13/47] UefiCpuPkg: Enable Tdx support in MpInitLib

2022-03-16 Thread Min Xu
On March 16, 2022 4:37 PM, Ni Ray wrote: > > GetProcessorInfo() can return the information for index#0 CPU. > Other suggestions look good to me. > Agree. It will be updated in the next version. Thanks Min -=-=-=-=-=-=-=-=-=-=-=- Groups.io Links: You receive all messages sent to this group. Vie

Re: [edk2-devel] [PATCH V8 13/47] UefiCpuPkg: Enable Tdx support in MpInitLib

2022-03-16 Thread Ni, Ray
GetProcessorInfo() can return the information for index#0 CPU. Other suggestions look good to me. -Original Message- From: Xu, Min M Sent: Wednesday, March 16, 2022 4:32 PM To: Ni, Ray ; devel@edk2.groups.io Cc: Brijesh Singh ; Aktas, Erdem ; James Bottomley ; Yao, Jiewen ; Tom Lendack

Re: [edk2-devel] [PATCH V8 13/47] UefiCpuPkg: Enable Tdx support in MpInitLib

2022-03-16 Thread Min Xu
On March 15, 2022 3:58 PM, Ni Ray wrote: > > Min, > I don't quite understand. > > Of all MP services, the TDX version only supports: > * GetNumberOfProcessors > * GetProcessorInfo > > And even when TDX MP returns 3 or 4 threads from GetNumberOfProcessors(), > StartupAllAPs() is an NOP. > Will th

Re: [edk2-devel] [PATCH V8 13/47] UefiCpuPkg: Enable Tdx support in MpInitLib

2022-03-15 Thread Ni, Ray
Min, I don't quite understand. Of all MP services, the TDX version only supports: * GetNumberOfProcessors * GetProcessorInfo And even when TDX MP returns 3 or 4 threads from GetNumberOfProcessors(), StartupAllAPs() is an NOP. Will that cause any issue? Can you avoid running MP driver when TDX i