[edk2-devel] [Patch V2 3/3] MdeModulePkg: Add Standalone MM Lockbox Driver.

2024-06-24 Thread Yuanhao Xie
services from the SMM Services Table (gSmst) as opposed to relying on Boot Services. Cc: Liming Gao Cc: Jiaxin Wu Cc: Ray Ni Cc: Ard Biesheuvel Cc: Sami Mujawar Signed-off-by: Yuanhao Xie Reviewed-by: Jiaxin Wu Reviewed-by: Ray Ni --- MdeModulePkg/MdeModulePkg.dsc

[edk2-devel] [Patch V2 2/3] MdeModulePkg: Refactors SmmLockBox.c.

2024-06-24 Thread Yuanhao Xie
Lockbox Driver in an upcoming patch Cc: Liming Gao Cc: Jiaxin Wu Cc: Ray Ni Cc: Ard Biesheuvel Cc: Sami Mujawar Signed-off-by: Yuanhao Xie Reviewed-by: Jiaxin Wu Reviewed-by: Ray Ni --- MdeModulePkg/Universal/LockBox/SmmLockBox/SmmLockBox.c | 361

[edk2-devel] [Patch V2 1/3] StandaloneMmPkg: Add LockBox Dependency DXE Driver

2024-06-24 Thread Yuanhao Xie
tocol installation signifies to the DXE driver that the LockBox API is ready for use. Cc: Liming Gao Cc: Jiaxin Wu Cc: Ray Ni Cc: Ard Biesheuvel Cc: Sami Mujawar Reviewed-by: Jiaxin Wu Signed-off-by: Yuanhao Xie --- StandaloneMmPkg/Library/SmmLockBoxMmDependency/SmmLockBoxMmDependency.c

[edk2-devel] [Patch V2 0/3] Add Standalone MM Lockbox Driver

2024-06-24 Thread Yuanhao Xie
This patch adds a Standalone MM Lockbox Driver xieyuanh (3): StandaloneMmPkg: Add LockBox Dependency DXE Driver MdeModulePkg: Refactors SmmLockBox.c. MdeModulePkg: Add Standalone MM Lockbox Driver. MdeModulePkg/MdeModulePkg.dsc | 1 + MdeModul

[edk2-devel] [PATCH 1/1] UefiCpuPkg: Correct the count of different type of Cache.

2024-06-18 Thread Yuanhao Xie
This patch fixes an error in calculating cache sizes for cores from different Dies. The original code incorrectly cleared cache sizes for different core types during intermediate calculation steps, leading to mistakes in counting duplicate entries. This patch adds a check for cache size to distingu

[edk2-devel] [PATCH 0/1] UefiCpuPkg: Correct the count of different type of Cache.

2024-06-18 Thread Yuanhao Xie
This patch fixes an error in calculating cache sizes for cores from different Dies. The original code incorrectly cleared cache sizes for different core types during intermediate calculation steps, leading to mistakes in counting duplicate entries. This patch adds a check for cache size to distingu

[edk2-devel] [PATCH 1/1] UefiCpuPkg: Correct the count of different type of Cache.

2024-06-18 Thread Yuanhao Xie
This patch fixes an error in calculating cache sizes for cores from different Dies. The original code incorrectly cleared cache sizes for different core types during intermediate calculation steps, leading to mistakes in counting duplicate entries. This patch adds a check for cache size to distingu

[edk2-devel] [PATCH 0/1] UefiCpuPkg: Correct the count of different type of Cache.

2024-06-18 Thread Yuanhao Xie
This patch fixes an error in calculating cache sizes for cores from different Dies. The original code incorrectly cleared cache sizes for different core types during intermediate calculation steps, leading to mistakes in counting duplicate entries. This patch adds a check for cache size to distingu

[edk2-devel] [PATCH 3/3] MdeModulePkg: Add Standalone MM Lockbox Driver.

2024-05-06 Thread Yuanhao Xie
services from the SMM Services Table (gSmst) as opposed to relying on Boot Services. Cc: Liming Gao Cc: Jiaxin Wu Cc: Ray Ni Signed-off-by: Yuanhao Xie --- MdeModulePkg/MdeModulePkg.dsc | 1 + MdeModulePkg/Universal/LockBox/SmmLockBox

[edk2-devel] [PATCH 2/3] MdeModulePkg: Refactors SmmLockBox.c.

2024-05-06 Thread Yuanhao Xie
Lockbox Driver in an upcoming patch Cc: Liming Gao Cc: Jiaxin Wu Cc: Ray Ni Signed-off-by: Yuanhao Xie --- MdeModulePkg/Universal/LockBox/SmmLockBox/SmmLockBox.c | 361

[edk2-devel] [PATCH 1/3] StandaloneMmPkg: Add LockBox Dependency DXE Driver

2024-05-06 Thread Yuanhao Xie
tocol installation signifies to the DXE driver that the LockBox API is ready for use. Cc: Liming Gao Cc: Jiaxin Wu Cc: Ray Ni Signed-off-by: Yuanhao Xie --- StandaloneMmPkg/Library/SmmLockBoxMmDependency/SmmLockBoxMmDependency.c

[edk2-devel] [PATCH 0/3] Add Standalone MM Lockbox Driver

2024-05-06 Thread Yuanhao Xie
The Lockbox Driver allows sensitive data to be securely stored in a designated area, thus protected against unauthorized access. This patch adds a Standalone MM Lockbox Driver with main modifications xieyuanh (3): StandaloneMmPkg: Add LockBox Dependency DXE Driver MdeModulePkg: Refactors SmmLo

[edk2-devel] [Patch V2 1/1] UefiCpuPkg/SmmCpuSyncLib: Add MM_STANDALONE tag.

2024-04-28 Thread Yuanhao Xie
Declares in the .inf file that the current component is an MM_STANDALONE Signed-off-by: Yuanhao Xie Cc: Eric Dong Cc: Ray Ni Cc: Rahul Kumar Cc: Gerd Hoffmann Cc: Jiaxin Wu Reviewed-by: Jiaxin Wu --- UefiCpuPkg/Library/SmmCpuSyncLib/SmmCpuSyncLib.inf | 4 ++-- 1 file changed, 2 insertions

[edk2-devel] [Patch V2 0/1] Add MM_STANDALONE tag.

2024-04-28 Thread Yuanhao Xie
Declares in the .inf file that the current component is an MM_STANDALONE xieyuanh (1): UefiCpuPkg/SmmCpuSyncLib: Add MM_STANDALONE tag. UefiCpuPkg/Library/SmmCpuSyncLib/SmmCpuSyncLib.inf | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) -- 2.39.1.windows.1 -=-=-=-=-=-=-=-=-=-=-=-

Re: [edk2-devel] [PATCH 1/1] UefiCpuPkg/SmmCpuSyncLib: Add MM_STANDALONE tag.

2024-04-24 Thread Yuanhao Xie
Ok! From: Ni, Ray Sent: Thursday, April 25, 2024 1:54 PM To: Xie, Yuanhao ; devel@edk2.groups.io Cc: Dong, Eric ; Kumar, Rahul R ; Gerd Hoffmann ; Wu, Jiaxin Subject: Re: [PATCH 1/1] UefiCpuPkg/SmmCpuSyncLib: Add MM_STANDALONE tag. -# Copyright (c) 2023, Intel Corporation. All rights reserved

[edk2-devel] [PATCH 1/1] UefiCpuPkg/SmmCpuSyncLib: Add MM_STANDALONE tag.

2024-04-22 Thread Yuanhao Xie
Declares in the .inf file that the current component is an MM_STANDALONE Signed-off-by: Yuanhao Xie Cc: Eric Dong Cc: Ray Ni Cc: Rahul Kumar Cc: Gerd Hoffmann Cc: Jiaxin Wu --- UefiCpuPkg/Library/SmmCpuSyncLib/SmmCpuSyncLib.inf | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions

[edk2-devel] [PATCH 0/1] UefiCpuPkg/SmmCpuSyncLib: Add MM_STANDALONE tag.

2024-04-22 Thread Yuanhao Xie
Declares in the .inf file that the current component is an MM_STANDALONE xieyuanh (1): UefiCpuPkg/SmmCpuSyncLib: Add MM_STANDALONE tag. UefiCpuPkg/Library/SmmCpuSyncLib/SmmCpuSyncLib.inf | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) base-commit: 86c8d69146310f24069701053a27153ae5

Re: [edk2-devel] [Patch V3] UefiCpuPkg/PiSmmCpuDxeSmm: SmmCpuRendezvous ensure all Aps in Present.

2023-12-18 Thread Yuanhao Xie
Hi Ray, It means that: The "SmmCpuRendezvous" function has a bug in that it summons all APs into SMM, but there is a time gap before they are set as "present." During this window, if using the "setVariable" operation, it can cause issues because "setVariable" requires all APs to be in SMM and m

[edk2-devel] [Patch V3] UefiCpuPkg/PiSmmCpuDxeSmm: SmmCpuRendezvous ensure all Aps in Present.

2023-12-15 Thread Yuanhao Xie
SMM read save state requires AP must be present. This patch aim to avoid the AP not ready for save state check. Furthermore, SmmWaitForApArrival has two callers: SmmCpuRendezvous and BSPHandler, the behavior of the two callers can be consistent by add the while loop to check the present of the APs

Re: [edk2-devel] [Patch V3 1/3] UefiCpuPkg/MpInitLib: Eliminate redundant microcode loading in DXE.

2023-11-23 Thread Yuanhao Xie
Hi Ray, In V4, the GetMicrocodePathInfoFromHob() removal is in patch 1 instead of the patch 3. Comments of MTRRs syncing is a updated. Thanks Yuanhao -Original Message- From: Ni, Ray Sent: Thursday, November 23, 2023 9:18 AM To: Xie, Yuanhao ; devel@edk2.groups.io Cc: Dong, Eric ; Kuma

Re: [edk2-devel] [Patch V3 3/3] UefiCpuPkg/MpInitLib: Extract Dump Microcode Revision as function.

2023-11-23 Thread Yuanhao Xie
Hi Ray, In v4, DEBUG_CODE_BEGIN/END() from DumpMicrocodeRevision() is removed, and calling is updated as: DEBUG_CODE ( DumpMicrocodeRevision (); ); Thanks a lot for the feedback! Regards, Yunahao -Original Message- From: Ni, Ray Sent: Thursday, November 23, 2023 9:24 AM To: Xie, Y

Re: [edk2-devel] [Patch V2 3/3] UefiCpuPkg/MpInitLib: Eliminate redundant microcode loading in DXE.

2023-11-23 Thread Yuanhao Xie
ding in DXE. On 11/21/23 08:39, Yuanhao Xie wrote: > The DXE stage's Microcode loading process has been elimincated by: > > 1. Microcode HOB consumption in MP initialization within the DXE phase. > 2. Restricting MicrocodeDetect to the PEI phase instead of DXE for BSP. &g

[edk2-devel] [Patch V4 3/3] UefiCpuPkg/MpInitLib: Extract Dump Microcode Revision as function.

2023-11-23 Thread Yuanhao Xie
There is no functional changes, only extract DumpMicrocodeRevision since only in PEI BSP will detect, apply microcode, and APs will sync microcode. Cc: Ray Ni Cc: Eric Dong Cc: Rahul Kumar Cc: Tom Lendacky Cc: Laszlo Ersek Signed-off-by: Yuanhao Xie --- UefiCpuPkg/Library/MpInitLib

[edk2-devel] [Patch V4 2/3] UefiCpuPkg/MpInitLib: Store MTRRs settings only when CpuCount>1

2023-11-23 Thread Yuanhao Xie
Store BSP's MTRR setting only when CpuCount>1. Reviewed-by: Ray Ni Cc: Ray Ni Cc: Eric Dong Cc: Rahul Kumar Cc: Tom Lendacky Cc: Laszlo Ersek Signed-off-by: Yuanhao Xie --- UefiCpuPkg/Library/MpInitLib/MpLib.c | 9 - 1 file changed, 4 insertions(+), 5 deletions(-) diff

[edk2-devel] [Patch V4 1/3] UefiCpuPkg/MpInitLib: Eliminate redundant microcode loading in DXE.

2023-11-23 Thread Yuanhao Xie
ff-by: Yuanhao Xie --- UefiCpuPkg/Library/MpInitLib/Microcode.c | 46 -- UefiCpuPkg/Library/MpInitLib/MpLib.c | 39 --- UefiCpuPkg/Library/MpInitLib/MpLib.h | 21 - 3 files changed, 24 inser

[edk2-devel] [Patch V4 0/3] UefiCpuPkg/MpInitLib: Eliminate redundant microcode loading in DXE.

2023-11-23 Thread Yuanhao Xie
The DXE stage's Microcode loading process has been eliminated. Store BSP's MTRR setting only when CpuCount>1. Extract Dump Microcode Revision as a function Compare with V3, this version updates the comments, Move GetMicrocodePatchInfoFromHob removal from patch 3 to patch 1 on xieyuanh (3): Uefi

[edk2-devel] [Patch V3 3/3] UefiCpuPkg/MpInitLib: Extract Dump Microcode Revision as function.

2023-11-21 Thread Yuanhao Xie
There is no functional changes, only extract DumpMicrocodeRevision since only in PEI BSP will detect, apply microcode, and APs will sync microcode. Cc: Ray Ni Cc: Eric Dong Cc: Rahul Kumar Cc: Tom Lendacky Cc: Laszlo Ersek Signed-off-by: Yuanhao Xie --- UefiCpuPkg/Library/MpInitLib

[edk2-devel] [Patch V3 2/3] UefiCpuPkg/MpInitLib: Store MTRRs settings only when CpuCount>1

2023-11-21 Thread Yuanhao Xie
Store BSP's MTRR setting only when CpuCount>1. Cc: Ray Ni Cc: Eric Dong Cc: Rahul Kumar Cc: Tom Lendacky Cc: Laszlo Ersek Signed-off-by: Yuanhao Xie --- UefiCpuPkg/Library/MpInitLib/MpLib.c | 9 - 1 file changed, 4 insertions(+), 5 deletions(-) diff --git a/UefiCpuPkg

[edk2-devel] [Patch V3 1/3] UefiCpuPkg/MpInitLib: Eliminate redundant microcode loading in DXE.

2023-11-21 Thread Yuanhao Xie
MTRR table to the AP is always essential. During the DXE phase, it cannot be omitted like loading microcode, as the PEI and DXE may be in different bit modes. Cc: Ray Ni Cc: Eric Dong Cc: Rahul Kumar Cc: Tom Lendacky Cc: Laszlo Ersek Signed-off-by: Yuanhao Xie --- UefiCpuPkg/Library/MpIn

[edk2-devel] [Patch V3 0/3] UefiCpuPkg/MpInitLib: Eliminate redundant microcode loading in DXE.

2023-11-21 Thread Yuanhao Xie
The DXE stage's Microcode loading process has been eliminated. Store BSP's MTRR setting only when CpuCount>1. Compare with V2, V3 also Extract Dump Microcode Revision as function xieyuanh (3): UefiCpuPkg/MpInitLib: Eliminate redundant microcode loading in DXE. UefiCpuPkg/MpInitLib: Store MTRRs

[edk2-devel] [Patch V2 3/3] UefiCpuPkg/MpInitLib: Eliminate redundant microcode loading in DXE.

2023-11-20 Thread Yuanhao Xie
ng no longer a part of this process. Cc: Ray Ni Cc: Eric Dong Cc: Rahul Kumar Cc: Tom Lendacky Cc: Laszlo Ersek Signed-off-by: Yuanhao Xie --- UefiCpuPkg/Library/MpInitLib/MpLib.c | 54 +++--- 1 file changed, 35 insertions(+), 19 deletions(-)

[edk2-devel] [Patch V2 1/3] UefiCpuPkg/MpInitLib:Repeat WakeUpAp of loading microcode,Syncing MTRR.

2023-11-20 Thread Yuanhao Xie
, with the ultimate goal of eliminating the microcode loading functionality for the second time Mp initialization. Cc: Ray Ni Cc: Eric Dong Cc: Rahul Kumar Cc: Tom Lendacky Cc: Laszlo Ersek Signed-off-by: Yuanhao Xie --- UefiCpuPkg/Library/MpInitLib/MpLib.c | 4 +++- 1 file changed, 3 insertions

[edk2-devel] [Patch V2 2/3] UefiCpuPkg/MpInitLib: Detect microcode and store MTRR when CpuCount > 1

2023-11-20 Thread Yuanhao Xie
loading functionality for the second time Mp initialization Cc: Ray Ni Cc: Eric Dong Cc: Rahul Kumar Cc: Tom Lendacky Cc: Laszlo Ersek Signed-off-by: Yuanhao Xie --- UefiCpuPkg/Library/MpInitLib/MpLib.c | 18 +- 1 file changed, 9 insertions(+), 9 deletions(-) diff --git a/UefiCpuPkg

[edk2-devel] [Patch V2 0/3] Eliminate redundant microcode loading in DXE.

2023-11-20 Thread Yuanhao Xie
The DXE stage's Microcode loading process has been elimincated. Compare to V1, V2 seperates the patches and adds more comments xieyuanh (3): UefiCpuPkg/MpInitLib:Repeat WakeUpAp of loading microcode,Syncing MTRR. UefiCpuPkg/MpInitLib: Detect microcode and store MTRR when CpuCount > 1

[edk2-devel] [Patch V7 2/2] UefiCpuPkg/MpInitLib: Update the comments of _CPU_MP_DATA.

2023-11-20 Thread Yuanhao Xie
. This patch deletes the unnecessary comments in CpuMpData, since CpuMpData is no longer responsible for passing information from PEI to DXE. Signed-off-by: Yuanhao Xie Cc: Laszlo Ersek ler...@redhat.com Cc: Eric Dong Cc: Ray Ni Cc: Rahul Kumar Cc: Gerd Hoffmann --- UefiCpuPkg/Library/MpInitLib

[edk2-devel] [Patch V7 1/2] UefiCpuPkg/MpInitLib: Enable execute disable bit.

2023-11-20 Thread Yuanhao Xie
From: Yuanhao Xie This patch synchronizes the No-Execute bit in the IA32_EFER register for the APs before the RestoreVolatileRegisters operation. The commit 964a4f0, titled "Eliminate the second INIT-SIPI-SIPI sequence," replaces the second INIT-SIPI-SIPI sequence with the BSP c

[edk2-devel] [Patch V7 0/2]UefiCpuPkg/MpInitLib: Enable execute disable bit.

2023-11-20 Thread Yuanhao Xie
code. It updated the structure comments compared with V6. Yuanhao Xie (1): UefiCpuPkg/MpInitLib: Enable execute disable bit. xieyuanh (1): UefiCpuPkg/MpInitLib: Update the comments of _CPU_MP_DATA. UefiCpuPkg/Library/MpInitLib/MpEqu.inc | 2 ++ UefiCpuPkg/Library/MpInitLib/MpLib.c | 14

[edk2-devel] [Patch V7 2/2] UefiCpuPkg/MpInitLib: Update the comments of _CPU_MP_DATA.

2023-11-20 Thread Yuanhao Xie
. This patch deletes the unnecessary comments in CpuMpData, since CpuMpData is no longer responsible for passing information from PEI to DXE. Signed-off-by: Yuanhao Xie Cc: Laszlo Ersek ler...@redhat.com Cc: Eric Dong Cc: Ray Ni Cc: Rahul Kumar Cc: Gerd Hoffmann --- UefiCpuPkg/Library/MpInitLib

[edk2-devel] [Patch V7 0/2]UefiCpuPkg/MpInitLib: Enable execute disable bit.

2023-11-20 Thread Yuanhao Xie
code. It updated the structure comments compared with V6. Yuanhao Xie (1): UefiCpuPkg/MpInitLib: Enable execute disable bit. xieyuanh (1): UefiCpuPkg/MpInitLib: Update the comments of _CPU_MP_DATA. UefiCpuPkg/Library/MpInitLib/MpEqu.inc | 2 ++ UefiCpuPkg/Library/MpInitLib/MpLib.c | 14

[edk2-devel] [Patch V7 1/2] UefiCpuPkg/MpInitLib: Enable execute disable bit.

2023-11-20 Thread Yuanhao Xie
From: Yuanhao Xie This patch synchronizes the No-Execute bit in the IA32_EFER register for the APs before the RestoreVolatileRegisters operation. The commit 964a4f0, titled "Eliminate the second INIT-SIPI-SIPI sequence," replaces the second INIT-SIPI-SIPI sequence with the BSP c

Re: [edk2-devel] [Patch V4] UefiCpuPkg/MpInitLib: Enable execute disable bit.

2023-11-19 Thread Yuanhao Xie
Hi Ray and Laszlo, Thanks a lot for the feedbacks. Please have a review on the patch v6 which: -Kept execute disable bit both in MpExchangeInfo and CpuMpData. -Added another patch in which I updated the comments of CpuMpData. Regards Yuanhao Xie -Original Message- From: Laszlo Ersek

[edk2-devel] [Patch V6 2/2] UefiCpuPkg/MpInitLib: Update the comments of _CPU_MP_DATA.

2023-11-19 Thread Yuanhao Xie
. Signed-off-by: Yuanhao Xie Cc: Laszlo Ersek ler...@redhat.com Cc: Eric Dong Cc: Ray Ni Cc: Rahul Kumar Cc: Gerd Hoffmann --- UefiCpuPkg/Library/MpInitLib/MpLib.h | 6 +- 1 file changed, 5 insertions(+), 1 deletion(-) diff --git a/UefiCpuPkg/Library/MpInitLib/MpLib.h b/UefiCpuPkg/Library

[edk2-devel] [Patch V6 1/2] UefiCpuPkg/MpInitLib: Enable execute disable bit.

2023-11-19 Thread Yuanhao Xie
From: Yuanhao Xie This patch synchronizes the No-Execute bit in the IA32_EFER register for the APs before the RestoreVolatileRegisters operation. The commit 964a4f0, titled "Eliminate the second INIT-SIPI-SIPI sequence," replaces the second INIT-SIPI-SIPI sequence with the BSP c

[edk2-devel] [Patch V6 0/2] UefiCpuPkg/MpInitLib: Enable execute disable bit.

2023-11-19 Thread Yuanhao Xie
code. It also fixed the build error in V5. Yuanhao Xie (1): UefiCpuPkg/MpInitLib: Enable execute disable bit. xieyuanh (1): UefiCpuPkg/MpInitLib: Update the comments of _CPU_MP_DATA. UefiCpuPkg/Library/MpInitLib/MpLib.c | 14 +++--- UefiCpuPkg/Library/MpInitLib/MpLib.h | 7 ++- 2

[edk2-devel] [Patch V5 0/2] UefiCpuPkg/MpInitLib: Enable execute disable bit.

2023-11-19 Thread Yuanhao Xie
code. Yuanhao Xie (1): UefiCpuPkg/MpInitLib: Enable execute disable bit. xieyuanh (1): UefiCpuPkg/MpInitLib: Update the comments of _CPU_MP_DATA. UefiCpuPkg/Library/MpInitLib/MpLib.c | 14 +++--- UefiCpuPkg/Library/MpInitLib/MpLib.h | 7 ++- 2 files changed, 17 insertions(+), 4

[edk2-devel] [Patch V5 1/2] UefiCpuPkg/MpInitLib: Enable execute disable bit.

2023-11-19 Thread Yuanhao Xie
From: Yuanhao Xie This patch synchronizes the No-Execute bit in the IA32_EFER register for the APs before the RestoreVolatileRegisters operation. The commit 964a4f0, titled "Eliminate the second INIT-SIPI-SIPI sequence," replaces the second INIT-SIPI-SIPI sequence with the BSP c

[edk2-devel] [Patch V5 2/2] UefiCpuPkg/MpInitLib: Update the comments of _CPU_MP_DATA.

2023-11-19 Thread Yuanhao Xie
. Signed-off-by: Yuanhao Xie Cc: Laszlo Ersek ler...@redhat.com Cc: Eric Dong Cc: Ray Ni Cc: Rahul Kumar Cc: Gerd Hoffmann --- UefiCpuPkg/Library/MpInitLib/MpLib.h | 6 +- 1 file changed, 5 insertions(+), 1 deletion(-) diff --git a/UefiCpuPkg/Library/MpInitLib/MpLib.h b/UefiCpuPkg/Library

[edk2-devel] [PATCH] UefiCpuPkg/MpInitLib: Remove the duplicate loading of microcode in DXE.

2023-11-19 Thread Yuanhao Xie
: Yuanhao Xie yuanhao@intel.com --- UefiCpuPkg/Library/MpInitLib/MpLib.c | 57 +++- 1 file changed, 40 insertions(+), 17 deletions(-) diff --git a/UefiCpuPkg/Library/MpInitLib/MpLib.c b/UefiCpuPkg/Library/MpInitLib/MpLib.c index 9a6ec5db5c..1c68c803d9 100644 --- a

Re: [edk2-devel] [Patch V4] UefiCpuPkg/MpInitLib: Enable execute disable bit.

2023-11-13 Thread Yuanhao Xie
Hi Gerd, In v4 , InterlockedIncrement called is after RestoreVolatileRegisters to ensure that "finished" reporting from the APs is as later as possible. Here is V3: GetProcessorNumber (CpuMpData, &ProcessorNumber); - RestoreVolatileRegisters (&CpuMpData->CpuData[0].VolatileRegisters, FALSE);

[edk2-devel] [Patch V4] UefiCpuPkg/MpInitLib: Enable execute disable bit.

2023-11-12 Thread Yuanhao Xie
From: Yuanhao Xie This patch synchronizes the No-Execute bit in the IA32_EFER register for the APs before the RestoreVolatileRegisters operation. The commit 964a4f0, titled "Eliminate the second INIT-SIPI-SIPI sequence," replaces the second INIT-SIPI-SIPI sequence with the BSP c

[edk2-devel] [Patch V3] UefiCpuPkg/MpInitLib: Enable execute disable bit.

2023-11-12 Thread Yuanhao Xie
From: Yuanhao Xie This patch synchronizes the No-Execute bit in the IA32_EFER register for the APs before the RestoreVolatileRegisters operation. The commit 964a4f0, titled "Eliminate the second INIT-SIPI-SIPI sequence," replaces the second INIT-SIPI-SIPI sequence with the BSP c

[edk2-devel] [Patch V2] UefiCpuPkg/PiSmmCpuDxeSmm: SmmCpuRendezvous ensure all Aps in Present.

2023-11-12 Thread Yuanhao Xie
SMM read save state requires AP must be present. This patch aim to avoid the AP not ready for save state check. Signed-off-by: Zhihao Li Cc: Ray Ni Cc: Rahul Kumar Cc: Gerd Hoffmann --- UefiCpuPkg/PiSmmCpuDxeSmm/CpuService.c | 25 + UefiCpuPkg/PiSmmCpuDxeSmm/PiSmmC

[edk2-devel] [Patch V2] UefiCpuPkg/MpInitLib: Enable execute disable bit.

2023-11-10 Thread Yuanhao Xie
From: Yuanhao Xie This patch synchronizes the No-Execute bit in the IA32_EFER register for the APs before the RestoreVolatileRegisters operation. The commit 964a4f0, titled "Eliminate the second INIT-SIPI-SIPI sequence," replaces the second INIT-SIPI-SIPI sequence with the BSP c

[edk2-devel] [PATCH] UefiCpuPkg/MpInitLib: Enable execute disable bit.

2023-11-10 Thread Yuanhao Xie
From: Yuanhao Xie This patch synchronizes the No-Execute bit in the IA32_EFER register for the APs before the RestoreVolatileRegisters operation. The commit 964a4f0, titled "Eliminate the second INIT-SIPI-SIPI sequence," replaces the second INIT-SIPI-SIPI sequence with the BSP c

[edk2-devel] [PATCH] UefiCpuPkg/PiSmmCpuDxeSmm: SmmCpuRendezvous ensure all Aps in Present.

2023-11-09 Thread Yuanhao Xie
From: YuanhaoXie SmmCpuRendezvous has a issue in that it summons all APs into SMM, but there is a time gap before they are set as "present." During this window, if using the "setVariable" operation, it can cause issues since "setVariable" requires all APs to be in SMM and marked as "present." Th

Re: [edk2-devel] [PATCH] UefiCpuPkg: Remove ASSERT checking if FinishedCount equal to CpuCount-1

2023-10-25 Thread Yuanhao Xie
[edk2-devel] [PATCH] UefiCpuPkg: Remove ASSERT checking if FinishedCount equal to CpuCount-1 On 10/25/23 12:07, Yuanhao Xie wrote: > The purpose is to fix an assertion with applying the following patch > series: > > UefiCpuPkg: Refactor the logic for placing APs in HltLoop. > UefiCpuPkg: Re

[edk2-devel] [Patch V3] UefiCpuPkg/MpInitLib: Wait for all APs to finish initialization

2023-10-25 Thread Yuanhao Xie
s in the triggering of the assertion. Solution: Instead of assertion, use while loop to waits until all the APs have incremented the FinishedCount. Fixes: 964a4f032dcd Signed-off-by: Yuanhao Xie Cc: Ray Ni Cc: Eric Dong Cc: Rahul Kumar Cc: Tom Lendacky --- UefiCpuPkg/Library/MpInitLib/MpLib.c | 9

[edk2-devel] [Patch V2] UefiCpuPkg: Remove ASSERT checking if FinishedCount equal to CpuCount-1

2023-10-25 Thread Yuanhao Xie
: Eric Dong Cc: Rahul Kumar Cc: Tom Lendacky Signed-off-by: Yuanhao Xie --- UefiCpuPkg/Library/MpInitLib/MpLib.c | 9 +++-- 1 file changed, 7 insertions(+), 2 deletions(-) diff --git a/UefiCpuPkg/Library/MpInitLib/MpLib.c b/UefiCpuPkg/Library/MpInitLib/MpLib.c index 6f1456cfe1..9a6ec5db5c

[edk2-devel] [PATCH] UefiCpuPkg: Remove ASSERT checking if FinishedCount equal to CpuCount-1

2023-10-25 Thread Yuanhao Xie
have incremented the FinishedCount. Cc: Ray Ni Cc: Eric Dong Cc: Rahul Kumar Cc: Tom Lendacky Signed-off-by: Yuanhao Xie --- UefiCpuPkg/Library/MpInitLib/MpLib.c | 9 +++-- 1 file changed, 7 insertions(+), 2 deletions(-) diff --git a/UefiCpuPkg/Library/MpInitLib/MpLib.c b/UefiCpuPkg

[edk2-devel] [edk2-platforms V2] MinPlatformPkg:Remove memory cache setting for memory under 1MB

2023-10-25 Thread Yuanhao Xie
. Reviewed-by: Eric Dong Cc: Chasel Chiu Cc: Nate DeSimone Cc: Isaac Oram Cc: Liming Gao Cc: Eric Dong Signed-off-by: Yuanhao Xie --- Platform/Intel/MinPlatformPkg/Library/SetCacheMtrrLib/SetCacheMtrrLib.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/Platform/Intel

Re: [edk2-devel] [PATCH] Remove memory cache setting for memory under 1MB

2023-10-25 Thread Yuanhao Xie
1MB Yuanhao, On 10/19/23 05:20, Yuanhao Xie wrote: > With the fact that CSM is not supported, the request is to remove the > cache setting for memory under 1MB. > This can be treated as the missing part of legacy CSM deprecation. > > This patch only set the 0 to 9 and C to

[edk2-devel] [PATCH] Remove memory cache setting for memory under 1MB

2023-10-18 Thread Yuanhao Xie
: Chasel Chiu Cc: Nate DeSimone Cc: Isaac Oram Cc: Liming Gao Cc: Eric Dong Signed-off-by: Yuanhao Xie --- Platform/Intel/MinPlatformPkg/Library/SetCacheMtrrLib/SetCacheMtrrLib.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/Platform/Intel/MinPlatformPkg/Library

[edk2-devel] [PATCH] Remove memory cache setting for memory under 1MB

2023-10-18 Thread Yuanhao Xie
: Chasel Chiu M: Nate DeSimone M: Isaac Oram R: Liming Gao R: Eric Dong Signed-off-by: Yuanhao Xie --- Platform/Intel/MinPlatformPkg/Library/SetCacheMtrrLib/SetCacheMtrrLib.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/Platform/Intel/MinPlatformPkg/Library

[edk2-devel] [Patch V3] UefiCpuPkg/CpuDxe: Eliminate the unused variable.

2023-10-11 Thread Yuanhao Xie
This change does not introduce any functional modifications. Remove the unused mValidMtrrAddressMask and mValidMtrrBitsMask. Reviewed-by: Ray Ni Signed-off-by: Yuanhao Xie Cc: Eric Dong Cc: Rahul Kumar Cc: Gerd Hoffmann Cc: Ray Ni --- UefiCpuPkg/CpuDxe/CpuDxe.c | 16 +++- 1

[edk2-devel] [Patch V2] UefiCpuPkg/CpuDxe: Eliminate the unused variable.

2023-10-11 Thread Yuanhao Xie
This change does not introduce any functional modifications. Remove the unused mValidMtrrAddressMask and mValidMtrrBitsMask. Signed-off-by: Yuanhao Xie Cc: Eric Dong Cc: Rahul Kumar Cc: Gerd Hoffmann Cc: Ray Ni --- UefiCpuPkg/CpuDxe/CpuDxe.c | 17 - 1 file changed, 8

[edk2-devel] [PATCH] UefiCpuPkg/CpuDxe: Eliminate the unused variable.

2023-10-11 Thread Yuanhao Xie
This change does not introduce any functional modifications. Recover the modification that was accidentally performanced during cherry-pick commit 1f78add. 1. Remove the unused variable. 2. reinstate the comment message. Signed-off-by: Yuanhao Xie Cc: Eric Dong Cc: Rahul Kumar Cc

[edk2-devel] [PATCH] UefiCpuPkg: Correct file description for MpHandOff header file

2023-10-06 Thread Yuanhao Xie
Cc: Eric Dong Cc: Rahul Kumar Cc: Tom Lendacky Signed-off-by: Yuanhao Xie --- UefiCpuPkg/Library/MpInitLib/MpHandOff.h | 5 - 1 file changed, 4 insertions(+), 1 deletion(-) diff --git a/UefiCpuPkg/Library/MpInitLib/MpHandOff.h b/UefiCpuPkg/Library/MpInitLib/MpHandOff.h index 83e4055ec9

[edk2-devel] [PATCH 16/16] UefiCpuPkg/CpuDxe: Update RefreshMemoryAttributesFromMtrr.

2023-09-12 Thread Yuanhao Xie
From: Ray Ni Old implementation of RefreshMemoryAttributesFromMtrr directly retrieves the MTRR register content and applies the MTRR cache type to GCD database following the precedence order defined by SDM. The code can updated to simply get all the memory cache types for all memory through newl

[edk2-devel] [PATCH 15/16] UefiCpuPkg/MtrrUnitTest: Add Unit test of setting/getting memory attributes

2023-09-12 Thread Yuanhao Xie
Add Unit test of trrGetMemoryAttributesInMtrrSettings and MtrrSetMemoryAttributesInMtrrSettings. Signed-off-by: Ray Ni Signed-off-by: Yuanhao Xie Cc: Eric Dong Cc: Rahul Kumar Cc: Gerd Hoffmann --- UefiCpuPkg/Library/MtrrLib/UnitTest/MtrrLibUnitTest.c | 36

[edk2-devel] [PATCH 14/16] UefiCpuPkg/MtrrLib: Improve MtrrDebugPrintAllMtrrsWorker.

2023-09-12 Thread Yuanhao Xie
Improve MtrrDebugPrintAllMtrrsWorker by making use of MtrrGetMemoryAttributesInMtrrSettings. Signed-off-by: Ray Ni Cc: Eric Dong Cc: Rahul Kumar Cc: Gerd Hoffmann --- UefiCpuPkg/Library/MtrrLib/MtrrLib.c | 45 ++--- 1 file changed, 10 insertions(+), 35

[edk2-devel] [PATCH 13/16] UefiCpuPkg/MtrrLib: Add API MtrrGetMemoryAttributesInMtrrSettings.

2023-09-12 Thread Yuanhao Xie
MtrrGetMemoryAttributesInMtrrSettings parses the MTRR settings either from hardware or from the parameter and returns an array containing the memory cache types of all memory addresses. This API could elinimate the needs of following APIs: 1. MtrrGetMemoryAttributeInVariableMtr 2. MtrrGetFixedMtrr

[edk2-devel] [PATCH 12/16] UefiCpuPkg/MtrrLib: Update APIs related to set memory attributes.

2023-09-12 Thread Yuanhao Xie
From: Ray Ni Update APIs related to set memory attributes to handle the fixed MTRR is not always supported. There are 3 APIs in MtrrLib that can set memory attributes: 1. MtrrSetMemoryAttributesInMtrrSettings 2. MtrrSetMemoryAttributeInMtrrSettings 3. MtrrSetMemoryAttribute The general idea app

[edk2-devel] [PATCH 11/16] UefiCpuPkg/MtrrUnitTest: Update UnitTestMtrrSetAllMtrrs().

2023-09-12 Thread Yuanhao Xie
Update UnitTestMtrrSetAllMtrrs() for the case that fixed MtrrLib is not always supported. Signed-off-by: Yuanhao Xie Cc: Eric Dong Cc: Rahul Kumar Cc: Gerd Hoffmann Cc: Ray Ni --- UefiCpuPkg/Library/MtrrLib/UnitTest/MtrrLibUnitTest.c | 35 --- 1 file changed

[edk2-devel] [PATCH 10/16] UefiCpuPkg/MtrrLib: Fix MtrrSetAllMtrrs to handle absent fixed MTRRs.

2023-09-12 Thread Yuanhao Xie
From: Ray Ni Update MtrrSetAllMtrrs to not access fixed MTRRs if CPU doesn't support them. Signed-off-by: Ray Ni Signed-off-by: Yuanhao Xie Cc: Eric Dong Cc: Rahul Kumar Cc: Gerd Hoffmann --- UefiCpuPkg/Include/Library/MtrrLib.h | 9 ++--- UefiCpuPkg/Library/MtrrLib/MtrrLib.c

[edk2-devel] [PATCH 09/16] UefiCpuPkg/MtrrUnitTest: Update UnitTestMtrrGetFixedMtrr().

2023-09-12 Thread Yuanhao Xie
UnitTestMtrrGetFixedMtrr updated for the case that fixed MtrrLib is not always supported. Signed-off-by: Yuanhao Xie Cc: Eric Dong Cc: Rahul Kumar Cc: Gerd Hoffmann Cc: Ray Ni --- UefiCpuPkg/Library/MtrrLib/UnitTest/MtrrLibUnitTest.c | 13 + 1 file changed, 13 insertions

[edk2-devel] [PATCH 08/16] UefiCpuPkg/MtrrLib: Update MtrrGetFixedMtrr().

2023-09-12 Thread Yuanhao Xie
IsMtrrSupported() return true when either fixed mtrr supported or variable mtrr suppored. In this case, rather than checking whether MTRR is supported or not, we should specifically check for fixed MTRR support. Signed-off-by: Yuanhao Xie Cc: Eric Dong Cc: Rahul Kumar Cc: Gerd Hoffmann Cc

[edk2-devel] [PATCH 07/16] UefiCpuPkg/MtrrUnitTest: Update UnitTestMtrrGetAllMtrrs().

2023-09-12 Thread Yuanhao Xie
Update UnitTestMtrrGetAllMtrrs() for the case that fixed MtrrLib is not always supported Signed-off-by: Yuanhao Xie Cc: Eric Dong Cc: Rahul Kumar Cc: Gerd Hoffmann Cc: Ray Ni --- UefiCpuPkg/Library/MtrrLib/UnitTest/MtrrLibUnitTest.c | 99

[edk2-devel] [PATCH 06/16] UefiCpuPkg/MtrrLib: Fix MtrrGetAllMtrrs to return correct MTRR setting.

2023-09-12 Thread Yuanhao Xie
the portion exceeding the maximum number remains uninitialized. Signed-off-by: Ray Ni Signed-off-by: Yuanhao Xie Cc: Eric Dong Cc: Rahul Kumar Cc: Gerd Hoffmann --- UefiCpuPkg/Library/MtrrLib/MtrrLib.c | 30 ++ 1 file changed, 22 insertions(+), 8 deletions

[edk2-devel] [PATCH 05/16] UefiCpuPkg/MtrrUnitTest: Update test to cover no-fixed-mtrr cases.

2023-09-12 Thread Yuanhao Xie
From: Ray Ni Signed-off-by: Ray Ni Cc: Eric Dong Cc: Rahul Kumar Cc: Gerd Hoffmann --- UefiCpuPkg/Library/MtrrLib/UnitTest/MtrrLibUnitTest.c | 43 --- UefiCpuPkg/Library/MtrrLib/UnitTest/Support.c | 24 +++- 2 files change

[edk2-devel] [PATCH 04/16] UefiCpuPkg/MtrrUnitTest: Update UnitTestMtrrGetDefaultMemoryType.

2023-09-12 Thread Yuanhao Xie
Update UnitTestMtrrGetDefaultMemoryType for the case the when Fixed MTRRs are not supported. The original implementation returns FALSE when either fixed MTRR isn't supported or the number of variable MTRRs is 0. The correct behavior should return FALSE only when both fixed MTRR isn't supported and

[edk2-devel] [PATCH 03/16] UefiCpuPkg/MtrrUnitTest: Update UnitTestGetFirmwareVariableMtrrCount.

2023-09-12 Thread Yuanhao Xie
Update UnitTestGetFirmwareVariableMtrrCount for the case the when Fixed MTRRs are not supported. The original implementation returns FALSE when either fixed MTRR isn't supported or the number of variable MTRRs is 0. The correct behavior should return FALSE only when both fixed MTRR isn't supported

[edk2-devel] [PATCH 02/16] UefiCpuPkg/MtrrUnitTest: Update the Unit Test for IsMtrrSupported().

2023-09-12 Thread Yuanhao Xie
The previous implementation returns FALSE if either fixed MTRR is unsupported or the number of variable MTRRs is 0. The correct behavior is to return FALSE only when both fixed MTRR is unsupported and the number of variable MTRRs is 0. Signed-off-by: Ray Ni Cc: Eric Dong Cc: Rahul Kumar Cc: Ger

[edk2-devel] [PATCH 01/16] UefiCpuPkg/MtrrLib: Add internal function MtrrLibIsMtrrSupported.

2023-09-12 Thread Yuanhao Xie
From: Ray Ni Add internal function MtrrLibIsMtrrSupported and update IsMtrrSupported to call the new internal function. Signed-off-by: Ray Ni Cc: Eric Dong Cc: Rahul Kumar Cc: Gerd Hoffmann --- UefiCpuPkg/Library/MtrrLib/MtrrLib.c | 83 +

[edk2-devel] [PATCH 00/16] MtrrLib modules and Unit test Enhancement

2023-09-12 Thread Yuanhao Xie
Update MtrrLib modules to handle the absense of Fixed MTRRs Update unit test of Update for the absense of Fixed MTRRs Ray Ni (6): UefiCpuPkg/MtrrLib: Add internal function MtrrLibIsMtrrSupported. UefiCpuPkg/MtrrUnitTest: Update test to cover no-fixed-mtrr cases. UefiCpuPkg/MtrrLib: Fix MtrrG

[edk2-devel] [PATCH] OvmfPkg: Disable PcdFirstTimeWakeUpAPsBySipi.

2023-08-21 Thread Yuanhao Xie
Disable PcdFirstTimeWakeUpAPsBySipi for IntelTdx, Microvm, and Xen to preserve the original execution of INIT-SIPI-SIPI. Cc: Eric Dong Cc: Ray Ni Cc: Rahul Kumar Cc: Gerd Hoffmann Cc: Ard Biesheuvel Cc: Jiewen Yao Cc: Jordan Justen Signed-off-by: Yuanhao Xie --- OvmfPkg/IntelTdx

[edk2-devel] [PATCH] OvmfPkg: Disable PcdFirstTimeWakeUpAPsBySipi.

2023-08-16 Thread Yuanhao Xie
Deactivate PcdFirstTimeWakeUpAPsBySipi for AMD SEV and SNP to preserve the original execution of INIT-SIPI-SIPI. Cc: Eric Dong Cc: Ray Ni Cc: Rahul Kumar Cc: Gerd Hoffmann Cc: Ard Biesheuvel Cc: Jiewen Yao Cc: Jordan Justen Signed-off-by: Yuanhao Xie --- OvmfPkg/AmdSev/AmdSevX64.dsc | 7

[edk2-devel] [Patch V7 4/4] UefiCpuPkg:Wake up APs after power-up or RESET through SIPI.

2023-07-30 Thread Yuanhao Xie
: Gerd Hoffmann Signed-off-by: Ray Ni Signed-off-by: Yuanhao Xie Reviewed-by: Ray Ni --- UefiCpuPkg/Library/MpInitLib/MpLib.c | 9 - 1 file changed, 8 insertions(+), 1 deletion(-) diff --git a/UefiCpuPkg/Library/MpInitLib/MpLib.c b/UefiCpuPkg/Library/MpInitLib/MpLib.c index e7054adbcc

[edk2-devel] [Patch V7 3/4] OvmfPkg: Disable PcdFirstTimeWakeUpAPsBySipi.

2023-07-30 Thread Yuanhao Xie
Disable PcdFirstTimeWakeUpAPsBySipi for OVMF to let BSP wake up APs by INIT-SIPI-SIPI. Cc: Eric Dong Cc: Ray Ni Cc: Rahul Kumar Cc: Gerd Hoffmann Cc: Ard Biesheuvel Cc: Jiewen Yao Cc: Jordan Justen Signed-off-by: Yuanhao Xie Reviewed-by: Ray Ni --- OvmfPkg/OvmfPkgIa32.dsc| 9

[edk2-devel] [Patch V7 2/4] UefiCpuPkg: Add PcdFirstTimeWakeUpAPsBySipi

2023-07-30 Thread Yuanhao Xie
Add PcdFirstTimeWakeUpAPsBySipi to check if it is in the OVMF environment and necessary to wake up APs by INIT-SIPI-SIPI. Cc: Eric Dong Cc: Ray Ni Cc: Rahul Kumar Cc: Gerd Hoffmann Signed-off-by: Yuanhao Xie Reviewed-by: Ray Ni --- UefiCpuPkg/Library/MpInitLib/DxeMpInitLib.inf | 1

[edk2-devel] [Patch V7 1/4] UefiCpuPkg: Add SendStartupIpiAllExcludingSelf

2023-07-30 Thread Yuanhao Xie
From: Yuanhao Xie Add new API SendStartupIpiAllExcludingSelf(), and modify SendInitSipiSipiAllExcludingSelf() by let it call the new API. Cc: Eric Dong Cc: Rahul Kumar Cc: Gerd Hoffmann Cc: Ray Ni Signed-off-by: Ray Ni Signed-off-by: Yuanhao Xie Reviewed-by: Ray Ni --- UefiCpuPkg

[edk2-devel] [Patch V7 0/4] Wake up APs after power-up or RESET through SIPI.

2023-07-30 Thread Yuanhao Xie
minimal self-configuration, then wait for SIPI from the BSP processor Yuanhao Xie (1): UefiCpuPkg: Add SendStartupIpiAllExcludingSelf YuanhaoXie (3): UefiCpuPkg: Add PcdFirstTimeWakeUpAPsBySipi OvmfPkg: Disable PcdFirstTimeWakeUpAPsBySipi. UefiCpuPkg:Wake up APs after power-up or RESET

[edk2-devel] [Patch V6 4/4] UefiCpuPkg:Wake up APs after power-up or RESET through SIPI.

2023-07-27 Thread Yuanhao Xie
: Gerd Hoffmann Signed-off-by: Ray Ni Signed-off-by: Yuanhao Xie Reviewed-by: Ray Ni --- UefiCpuPkg/Library/MpInitLib/MpLib.c | 9 - 1 file changed, 8 insertions(+), 1 deletion(-) diff --git a/UefiCpuPkg/Library/MpInitLib/MpLib.c b/UefiCpuPkg/Library/MpInitLib/MpLib.c index e7054adbcc

[edk2-devel] [Patch V6 3/4] OvmfPkg: Disable PcdFirstTimeWakeUpAPsBySipi.

2023-07-27 Thread Yuanhao Xie
Disable PcdFirstTimeWakeUpAPsBySipi for OVMF to let BSP wake up APs by INIT-SIPI-SIPI. Cc: Eric Dong Cc: Ray Ni Cc: Rahul Kumar Cc: Gerd Hoffmann Cc: Ard Biesheuvel Cc: Jiewen Yao Cc: Jordan Justen Signed-off-by: Yuanhao Xie Reviewed-by: Ray Ni --- OvmfPkg/OvmfPkgIa32.dsc| 10

[edk2-devel] [Patch V6 2/4] UefiCpuPkg: Add PcdFirstTimeWakeUpAPsBySipi

2023-07-27 Thread Yuanhao Xie
Add PcdFirstTimeWakeUpAPsBySipi to check if it is in the OVMF environment and necessary to wake up APs by INIT-SIPI-SIPI. Cc: Eric Dong Cc: Ray Ni Cc: Rahul Kumar Cc: Gerd Hoffmann Signed-off-by: Yuanhao Xie Reviewed-by: Ray Ni --- UefiCpuPkg/Library/MpInitLib/DxeMpInitLib.inf | 1

[edk2-devel] [Patch V6 1/4] UefiCpuPkg: Add SendStartupIpiAllExcludingSelf

2023-07-27 Thread Yuanhao Xie
From: Yuanhao Xie Add new API SendStartupIpiAllExcludingSelf(), and modify SendInitSipiSipiAllExcludingSelf() by let it call the new API. Cc: Eric Dong Cc: Rahul Kumar Cc: Gerd Hoffmann Cc: Ray Ni Signed-off-by: Ray Ni Signed-off-by: Yuanhao Xie Reviewed-by: Ray Ni --- UefiCpuPkg

[edk2-devel] [Patch V6 0/4]Wake up APs after power-up or RESET through SIPI.

2023-07-27 Thread Yuanhao Xie
minimal self-configuration, then wait for SIPI from the BSP processor. Yuanhao Xie (1): UefiCpuPkg: Add SendStartupIpiAllExcludingSelf YuanhaoXie (3): UefiCpuPkg: Add PcdFirstTimeWakeUpAPsBySipi OvmfPkg: Disable PcdFirstTimeWakeUpAPsBySipi. UefiCpuPkg:Wake up APs after power-up or RESET

Re: [edk2-devel] [Patch V4 3/4] OvmfPkg: Disable PcdFirstTimeWakeUpAPsBySipi.

2023-07-27 Thread Yuanhao Xie
TimeWakeUpAPsBySipi for OVMF to let BSP wake up APs by INIT-SIPI-SIPI. Cc: Eric Dong Cc: Ray Ni Cc: Rahul Kumar Cc: Gerd Hoffmann Cc: Ard Biesheuvel Cc: Jiewen Yao Cc: Jordan Justen Signed-off-by: Yuanhao Xie Reviewed-by: Ray Ni --- OvmfPkg/OvmfPkgIa32.dsc| 9 - OvmfPkg

[edk2-devel] [Patch V5 4/4] UefiCpuPkg:Wake up APs after power-up or RESET through SIPI.

2023-07-27 Thread Yuanhao Xie
: Gerd Hoffmann Signed-off-by: Ray Ni Signed-off-by: Yuanhao Xie Reviewed-by: Ray Ni --- UefiCpuPkg/Library/MpInitLib/MpLib.c | 9 - 1 file changed, 8 insertions(+), 1 deletion(-) diff --git a/UefiCpuPkg/Library/MpInitLib/MpLib.c b/UefiCpuPkg/Library/MpInitLib/MpLib.c index e7054adbcc

[edk2-devel] [Patch V5 3/4] OvmfPkg: Disable PcdFirstTimeWakeUpAPsBySipi.

2023-07-27 Thread Yuanhao Xie
Disable PcdFirstTimeWakeUpAPsBySipi for OVMF to let BSP wake up APs by INIT-SIPI-SIPI. Cc: Eric Dong Cc: Ray Ni Cc: Rahul Kumar Cc: Gerd Hoffmann Cc: Ard Biesheuvel Cc: Jiewen Yao Cc: Jordan Justen Signed-off-by: Yuanhao Xie Reviewed-by: Ray Ni --- OvmfPkg/OvmfPkgIa32.dsc| 9

[edk2-devel] [Patch V5 2/4] UefiCpuPkg: Add PcdFirstTimeWakeUpAPsBySipi

2023-07-27 Thread Yuanhao Xie
Add PcdFirstTimeWakeUpAPsBySipi to check if it is in the OVMF environment and necessary to wake up APs by INIT-SIPI-SIPI. Cc: Eric Dong Cc: Ray Ni Cc: Rahul Kumar Cc: Gerd Hoffmann Signed-off-by: Yuanhao Xie Reviewed-by: Ray Ni --- UefiCpuPkg/Library/MpInitLib/DxeMpInitLib.inf | 1

  1   2   3   >