Re: [edk2-devel] [PATCH v1] UefiCpuPkg/PiSmmCpuDxeSmm: Use processor extended information

2023-11-13 Thread Laszlo Ersek
On 11/13/23 16:38, Laszlo Ersek wrote: > On 11/7/23 03:43, Wu, Jiaxin wrote: >> Processor extended information is filled when >> CPU_V2_EXTENDED_TOPOLOGY is set in parameter ProcessorNumber >> from GetProcessorInfo() (See commit: 1fadd18d). >> >> This filed value is retrieved from CPUID leaf 1FH,

Re: [edk2-devel] [PATCH v1] UefiCpuPkg/PiSmmCpuDxeSmm: Use processor extended information

2023-11-13 Thread Laszlo Ersek
On 11/7/23 03:43, Wu, Jiaxin wrote: > Processor extended information is filled when > CPU_V2_EXTENDED_TOPOLOGY is set in parameter ProcessorNumber > from GetProcessorInfo() (See commit: 1fadd18d). > > This filed value is retrieved from CPUID leaf 1FH, which is > a preferred superset to leaf 0BH.

Re: [edk2-devel] [PATCH v1] UefiCpuPkg/PiSmmCpuDxeSmm: Use processor extended information

2023-11-13 Thread Laszlo Ersek
On 11/8/23 05:11, Wu, Jiaxin wrote: > Hi Laszlo, > >>> >>> The patch looks OK to me, but: >>> >>> - I would like to test it with CPU hotplug (later, likely under v2), and >>> > > Sure, I can wait the update from you. > >>> - I think this should be two patches. >>> >>> First, the

Re: [edk2-devel] [PATCH v1] UefiCpuPkg/PiSmmCpuDxeSmm: Use processor extended information

2023-11-07 Thread Wu, Jiaxin
Hi Laszlo, > > > > The patch looks OK to me, but: > > > > - I would like to test it with CPU hotplug (later, likely under v2), and > > Sure, I can wait the update from you. > > - I think this should be two patches. > > > > First, the SmmAddProcessor() function should be extended just to > >

Re: [edk2-devel] [PATCH v1] UefiCpuPkg/PiSmmCpuDxeSmm: Use processor extended information

2023-11-07 Thread Laszlo Ersek
On 11/7/23 19:40, Laszlo Ersek wrote: > On 11/7/23 03:43, Wu, Jiaxin wrote: >> Processor extended information is filled when >> CPU_V2_EXTENDED_TOPOLOGY is set in parameter ProcessorNumber >> from GetProcessorInfo() (See commit: 1fadd18d). >> >> This filed value is retrieved from CPUID leaf 1FH,

Re: [edk2-devel] [PATCH v1] UefiCpuPkg/PiSmmCpuDxeSmm: Use processor extended information

2023-11-07 Thread Laszlo Ersek
On 11/7/23 03:43, Wu, Jiaxin wrote: > Processor extended information is filled when > CPU_V2_EXTENDED_TOPOLOGY is set in parameter ProcessorNumber > from GetProcessorInfo() (See commit: 1fadd18d). > > This filed value is retrieved from CPUID leaf 1FH, which is > a preferred superset to leaf 0BH.

[edk2-devel] [PATCH v1] UefiCpuPkg/PiSmmCpuDxeSmm: Use processor extended information

2023-11-06 Thread Wu, Jiaxin
Processor extended information is filled when CPU_V2_EXTENDED_TOPOLOGY is set in parameter ProcessorNumber from GetProcessorInfo() (See commit: 1fadd18d). This filed value is retrieved from CPUID leaf 1FH, which is a preferred superset to leaf 0BH. Since Intel recommends first use the CPUID leaf