Re: [edk2-devel] [PATCH v2 1/1] UefiCpuPkg/MpInitLib: Reduce reset vector memory pressure

2020-10-02 Thread Lendacky, Thomas
On 9/24/20 10:03 AM, Laszlo Ersek wrote: On 09/24/20 15:30, Tom Lendacky wrote: On 9/24/20 1:22 AM, Laszlo Ersek wrote: On 09/23/20 20:04, Tom Lendacky wrote: From: Tom Lendacky The AP reset vector stack allocation is only required if running as an SEV-ES guest. Since the reset vector

Re: [edk2-devel] [PATCH v2 1/1] UefiCpuPkg/MpInitLib: Reduce reset vector memory pressure

2020-09-25 Thread Lendacky, Thomas
On 9/25/20 11:09 AM, Brian J. Johnson wrote: On 9/24/20 1:22 AM, Laszlo Ersek wrote: - I don't remember if it's required that the APIC ID space be densely populated. For example, if we have a topology with 7 possible (= maximum) logical CPUs, I'm unsure if a spec forbids any of those CPUs from

Re: [edk2-devel] [PATCH v2 1/1] UefiCpuPkg/MpInitLib: Reduce reset vector memory pressure

2020-09-25 Thread Brian J. Johnson
On 9/24/20 1:22 AM, Laszlo Ersek wrote: - I don't remember if it's required that the APIC ID space be densely populated. For example, if we have a topology with 7 possible (= maximum) logical CPUs, I'm unsure if a spec forbids any of those CPUs from having APIC ID 7. That could cause a problem

Re: [edk2-devel] [PATCH v2 1/1] UefiCpuPkg/MpInitLib: Reduce reset vector memory pressure

2020-09-24 Thread Laszlo Ersek
On 09/24/20 15:30, Tom Lendacky wrote: > On 9/24/20 1:22 AM, Laszlo Ersek wrote: >> On 09/23/20 20:04, Tom Lendacky wrote: >>> From: Tom Lendacky >>> >>> The AP reset vector stack allocation is only required if running as an >>> SEV-ES guest. Since the reset vector allocation is below 1MB in

Re: [edk2-devel] [PATCH v2 1/1] UefiCpuPkg/MpInitLib: Reduce reset vector memory pressure

2020-09-24 Thread Lendacky, Thomas
On 9/24/20 1:22 AM, Laszlo Ersek wrote: On 09/23/20 20:04, Tom Lendacky wrote: From: Tom Lendacky The AP reset vector stack allocation is only required if running as an SEV-ES guest. Since the reset vector allocation is below 1MB in memory, eliminate the requirement for bare-metal systems and

Re: [edk2-devel] [PATCH v2 1/1] UefiCpuPkg/MpInitLib: Reduce reset vector memory pressure

2020-09-24 Thread Laszlo Ersek
On 09/23/20 20:04, Tom Lendacky wrote: > From: Tom Lendacky > > The AP reset vector stack allocation is only required if running as an > SEV-ES guest. Since the reset vector allocation is below 1MB in memory, > eliminate the requirement for bare-metal systems and non SEV-ES guests > to allocate

[edk2-devel] [PATCH v2 1/1] UefiCpuPkg/MpInitLib: Reduce reset vector memory pressure

2020-09-23 Thread Lendacky, Thomas
From: Tom Lendacky The AP reset vector stack allocation is only required if running as an SEV-ES guest. Since the reset vector allocation is below 1MB in memory, eliminate the requirement for bare-metal systems and non SEV-ES guests to allocate the extra stack area, which can be large if the