Re: [PATCH v2 0/5] UefiCpuPkg/DxeMpLib: Allocate new safe stack < 4GB

[Laszlo Ersek <lersek@redhat.com>]

On 11/25/16 07:03, Jeff Fan wrote:
> Allocate safe AP stack under 4GB and make sure BSP wait till all APs running in
> safe code.  
> 
> v2:
>   1. Update #1 to address the comments in
>        https://lists.01.org/pipermail/edk2-devel/2016-November/005136.html
>   2. Update #2 to address the comments in
>        https://lists.01.org/pipermail/edk2-devel/2016-November/005137.html
>   3. Update #3 to address the comments in
>        https://lists.01.org/pipermail/edk2-devel/2016-November/005138.html
>   4. Add #4 to fix getting target C-State bug.
>   5. Add #5 to remove ret instruction.
> 
> Cc: Laszlo Ersek <lersek@redhat.com>
> Cc: Feng Tian <feng.tian@intel.com>
> Cc: Michael D Kinney <michael.d.kinney@intel.com>
> Contributed-under: TianoCore Contribution Agreement 1.0
> Signed-off-by: Jeff Fan <jeff.fan@intel.com>
> 
> Jeff Fan (5):
>   UefiCpuPkg/DxeMpLib: Get safe AP loop handler from global variable
>   UefiCpuPkg/DxeMpLib: Allocate new safe stack < 4GB
>   UefiCpuPkg/DxeMpLib: Make sure APs in safe loop code
>   UefiCpuPkg/DxeMpLib: Fix bug when getting target C-State from eax
>   UefiCpuPkg/DxeMpLib: Remove unnecessary ret instruction
> 
>  UefiCpuPkg/Library/MpInitLib/DxeMpLib.c        | 29 ++++++++++++++++++++++----
>  UefiCpuPkg/Library/MpInitLib/Ia32/MpFuncs.nasm | 16 ++++++++++----
>  UefiCpuPkg/Library/MpInitLib/MpLib.h           |  4 +++-
>  UefiCpuPkg/Library/MpInitLib/X64/MpFuncs.nasm  |  8 ++++---
>  4 files changed, 45 insertions(+), 12 deletions(-)
> 

Tested-by: Laszlo Ersek <lersek@redhat.com>