[edk2-devel] [PATCH] UefiCpuPkg/MpInitLib: Remove Executable attribute from MpLib.h

2020-04-22 Thread Leo Duran
REF: https://bugzilla.tianocore.org/show_bug.cgi?id=2678 This patch fixes a file permission issue introduced by accident. Cc: Eric Dong Cc: Ray Ni Cc: Laszlo Ersek Signed-off-by: Leo Duran --- UefiCpuPkg/Library/MpInitLib/MpLib.h | 0 1 file changed, 0 insertions(+), 0 deletions(-) mode

[edk2-devel] [PATCH v4] UefiCpuPkg: Fix bug in MpInitLib

2020-02-29 Thread Leo Duran
This patch fixes an issue introduced recently in MpInitLib, where we read a PlatformId MSR that is not implemented on AMD processors. The patch uses CPUID signature check to skip reading the PlatformId MSR. Changes since v3: Add header file entry for CPUID definitions: Register/Amd/Cpuid.h Leo

[edk2-devel] [PATCH] UefiCpuPkg/MpInitLib: Skip reading PlatformId on AMD processors.

2020-02-29 Thread Leo Duran
the PlatformId dependency, PcdCpuMicrocodePatchAddress and PcdCpuMicrodePatchRegionSize must be set to 0 (default value), in order to bypass microcode loading code paths. Cc: Eric Dong Cc: Ray Ni Cc: Laszlo Ersek Signed-off-by: Leo Duran --- UefiCpuPkg/Library/MpInitLib/MpLib.c | 34

[edk2-devel] [PATCH] UefiCpuPkg/MpInitLib: Skip reading PlatformId on AMD processors.

2020-02-28 Thread Leo Duran
the PlatformId dependency, PcdCpuMicrocodePatchAddress and PcdCpuMMicrodePatchRegionSize must be set to 0 (default value), in order to bypass microcode loading code paths. Cc: Eric Dong Cc: Ray Ni Cc: Laszlo Ersek Signed-off-by: Leo Duran --- UefiCpuPkg/Library/MpInitLib/MpLib.c | 34

[edk2-devel] [PATCH v3] UefiCpuPkg: Fix bug in MpInitLib

2020-02-28 Thread Leo Duran
This patch fixes an issue introduced recently in MpInitLib, where we read a PlatformId MSR that is not implemented on AMD processors. The patch uses CPUID signature check to skip reading the PlatformId MSR. Changes since v2: Fix typo (PCD name) on commit log. Leo Duran (1): UefiCpuPkg

[edk2-devel] [PATCH] UefiCpuPkg/MpInitLib: Skip reading PlatformId on AMD processors.

2020-02-28 Thread Leo Duran
the PlatformId dependency, PcdCpuMicrocodePatchAddress and PcdCpuMicrodePatchRegionSize must be set to 0 (default value), in order to bypass microcode loading code paths. Cc: Eric Dong Cc: Ray Ni Cc: Laszlo Ersek Signed-off-by: Leo Duran --- UefiCpuPkg/Library/MpInitLib/MpLib.c | 34

[edk2-devel] [PATCH v2] UefiCpuPkg: Fix bug in MpInitLib

2020-02-28 Thread Leo Duran
. - Undo changes to MpInitLib/Microcode.c and rely on existing PCDs to manage loading of microcode patches. Leo Duran (1): UefiCpuPkg/MpInitLib: Skip reading PlatformId on AMD processors. UefiCpuPkg/Library/MpInitLib/MpLib.c | 34 -- 1 file changed, 32 insertions(+), 2

[edk2-devel] [PATCH 1/2] UefiCpuPkg: LocalApicLib: Export StandardSignatureIsAuthenticAMD function

2020-02-25 Thread Leo Duran
LocalApicLib. Cc: Eric Dong Cc: Ray Ni Cc: Laszlo Ersek Signed-off-by: Leo Duran --- UefiCpuPkg/Include/Library/LocalApicLib.h | 15 +++ UefiCpuPkg/Library/BaseXApicLib/BaseXApicLib.c | 47 +++--- .../BaseXApicX2ApicLib/BaseXApicX2ApicLib.c| 47

[edk2-devel] [PATCH 0/2] UefiCpuPkg/Library: Fix bug in MpInitLib

2020-02-25 Thread Leo Duran
platforms as the flow never gets that far, since the Detect routine bails out early when it finds the size of the patch is zero. Leo Duran (2): UefiCpuPkg: LocalApicLib: Export StandardSignatureIsAuthenticAMD function UefiCpuPkg: MpInitLib: Exclude code no pertinent to AMD processors. UefiCpuPkg

[edk2-devel] [PATCH 2/2] UefiCpuPkg: MpInitLib: Exclude code no pertinent to AMD processors.

2020-02-25 Thread Leo Duran
-based platforms. Cc: Eric Dong Cc: Ray Ni Cc: Laszlo Ersek Signed-off-by: Leo Duran --- UefiCpuPkg/Library/MpInitLib/Microcode.c | 17 +++-- UefiCpuPkg/Library/MpInitLib/MpLib.c | 11 +-- 2 files changed, 24 insertions(+), 4 deletions(-) diff --git a/UefiCpuPkg/Library