[edk2-devel] [PATCH] Maintainers.txt: Add 'Zhichao Gao' for ShellPkg review

2019-05-16 Thread Gao, Zhichao
Add 'Zhichao Gao' as a reviewer for ShellPkg. Cc: Jaben Carsey Cc: Ray Ni Cc: Liming Gao Signed-off-by: Zhichao Gao --- Maintainers.txt | 1 + 1 file changed, 1 insertion(+) diff --git a/Maintainers.txt b/Maintainers.txt index 7315241b6e..6a5f5d6200 100644 --- a/Maintainers.txt +++

Re: [edk2-devel] [PATCH v1 1/1] ShellPkg/UefiShellAcpiViewCommandLib: Fix PPTT cache attributes validation

2019-05-16 Thread Gao, Zhichao
Reviewed-by: Zhichao Gao > -Original Message- > From: devel@edk2.groups.io [mailto:devel@edk2.groups.io] On Behalf Of > Sami Mujawar > Sent: Thursday, May 9, 2019 9:37 PM > To: devel@edk2.groups.io > Cc: Carsey, Jaben ; Ni, Ray ; > Matteo Carlini ; Stephanie Hughes-Fitt > ; nd ;

Re: [edk2-devel] [PATCH v2 1/1] ShellPkg: acpiview: Add GT Frame Number validation to GTDT parser

2019-05-16 Thread Gao, Zhichao
Reviewed-by: Zhichao Gao > -Original Message- > From: Krzysztof Koch [mailto:krzysztof.k...@arm.com] > Sent: Thursday, May 16, 2019 4:44 PM > To: devel@edk2.groups.io > Cc: Carsey, Jaben ; Ni, Ray ; > Gao, Zhichao ; sami.muja...@arm.com; > n...@arm.com > Subject:

[edk2-devel] [PATCH] SecurityPkg/AuthSeriableLib: Always delete variable in certdb

2019-05-12 Thread Gao, Zhichao
REF: https://bugzilla.tianocore.org/show_bug.cgi?id=1783 While cleaning the certdb, always delete the variable data in the certdb regardless of its attribute. Cc: Chao Zhang Cc: Jiewen Yao Cc: Jian Wang Cc: Star Zeng Cc: Liming Gao Signed-off-by: Zhichao Gao ---

Re: [edk2-devel] [PATCH v1 1/1] ShellPkg: acpiview: Add GT Frame Number validation to GTDT parser

2019-05-15 Thread Gao, Zhichao
Sorry for missing this email. Some minor comments below. > -Original Message- > From: devel@edk2.groups.io [mailto:devel@edk2.groups.io] On Behalf Of > Krzysztof Koch > Sent: Wednesday, May 8, 2019 9:44 PM > To: devel@edk2.groups.io > Cc: Carsey, Jaben ; Ni, Ray ; > sami.muja...@arm.com;

[edk2-devel] [PATCH 1/2] MdeModulePkg: Add a pcd to set the OS indications bit

2019-06-03 Thread Gao, Zhichao
From: Bret Barkelew REF: https://bugzilla.tianocore.org/show_bug.cgi?id=1678 Add a pcd PcdPlatformRecoverySupport to conditionally set an OS indications bit and set the 'PlatformRecovery' variable. This pcd would also control whether the BDS supports platform recovery or not. Cc: Jian J

[edk2-devel] [PATCH 0/2] Use a pcd to control Platform Recovery behavior

2019-06-03 Thread Gao, Zhichao
V1: Add a pcd PcdPlatformRecoverySupport to control the variable PlatformRecovery and the EFI_OS_INDICATIONS_START_PLATFORM_RECOVERY bit of variable "OsIndicationsSupported". V2: While PcdPlatformRecoverySupport is FALSE, do not set a PlatformRecovery Variable. But remain boot from a

[edk2-devel] [PATCH 2/2] MdeModulePkg/BdsDxe: Use a pcd to control PlatformRecovery

2019-06-03 Thread Gao, Zhichao
REF: https://bugzilla.tianocore.org/show_bug.cgi?id=1678 Use the PcdPlatformRecoverySupport to control the function of platform recovery in BDS. First, set the variable's ("OsIndicationsSupported") EFI_OS_INDICATIONS_START_PLATFORM_RECOVERY bit base on the pcd. It would affect the variable

Re: [edk2-devel] [PATCH 2/2] MdeModulePkg/BdsDxe: Use a pcd to control PlatformRecovery

2019-06-03 Thread Gao, Zhichao
> -Original Message- > From: Ni, Ray > Sent: Monday, June 3, 2019 2:59 PM > To: Gao, Zhichao ; devel@edk2.groups.io > Cc: Wang, Jian J ; Wu, Hao A ; > Zeng, Star ; Gao, Liming ; > Sean Brogan ; Michael Turner > ; Bret Barkelew > > Subject: RE: [PATCH 2/2]

Re: [edk2-devel] [PATCH 2/2] MdeModulePkg/BdsDxe: Use a pcd to control PlatformRecovery

2019-06-03 Thread Gao, Zhichao
> -Original Message- > From: Ni, Ray > Sent: Monday, June 3, 2019 3:27 PM > To: Gao, Zhichao ; devel@edk2.groups.io > Cc: Wang, Jian J ; Wu, Hao A ; > Zeng, Star ; Gao, Liming ; > Sean Brogan ; Michael Turner > ; Bret Barkelew > > Subject: RE: [PATCH 2/2]

Re: [edk2-devel] [PATCH] MdeModulePkg/CapsulePei: Optimize the CapsulePei

2019-06-03 Thread Gao, Zhichao
> -Original Message- > From: devel@edk2.groups.io [mailto:devel@edk2.groups.io] On Behalf Of > Leif Lindholm > Sent: Saturday, June 1, 2019 12:44 AM > To: devel@edk2.groups.io; Gao, Zhichao > Cc: Bret Barkelew ; Wang, Jian J > ; Wu, Hao A ; Ni, Ray > ; Zeng, Sta

[edk2-devel] [PATCH v4 1/2] MdeModulePkg/CapsulePei: Optimize the CapsulePei

2019-06-04 Thread Gao, Zhichao
From: Bret Barkelew REF: https://bugzilla.tianocore.org/show_bug.cgi?id=1853 Code change form Mu project: https://github.com/microsoft/mu_basecore/blob/release/201903/ MdeModulePkg/Universal/CapsulePei/UefiCapsule.c#L801 Separate the capsule check function from GetCapsuleDescriptors to

[edk2-devel] [PATCH v4 0/2] MdeModulePkg/CapsulePei: Optimize the CapsulePei

2019-06-04 Thread Gao, Zhichao
v1: Sperate the capsule check function from GetCapsuleDescriptors and name it to AreCapsulesStaged. Rename GetCapsuleDescriptors to GetScatterGatherHeadEntries. And optimize its to remove the duplicated code. V2: Clarify the changes of the patch and descript in the commit message. V3: Add the

[edk2-devel] [PATCH v4 2/2] MdeMoudlePkg/CapsulePei: Substantial change on UefiCapsule.c

2019-06-04 Thread Gao, Zhichao
REF: https://bugzilla.tianocore.org/show_bug.cgi?id=1853 AreCapsulesStaged do not need to return the status, only boolean result is useful. So directly return a boolean value. Cannot initialize the variable at its definition. GetScatterGatherHeadEntries: use allocated buffer instead of fixed

Re: [edk2-devel] [patch] ShellPkg/Debug1CommandsLib: Fix bugs in func DisplaySysEventLogData

2019-05-28 Thread Gao, Zhichao
Reviewed-by: Zhichao Gao > -Original Message- > From: Bi, Dandan > Sent: Tuesday, May 28, 2019 10:25 AM > To: devel@edk2.groups.io; oleks...@ami.com > Cc: Carsey, Jaben ; Ni, Ray ; > Gao, Zhichao > Subject: [patch] ShellPkg/Debug1CommandsLib: Fix bugs in func >

[edk2-devel] [PATCH] MdeModulePkg: Clarify the lib supported ARCH

2019-05-28 Thread Gao, Zhichao
From: Bret Barkelew REF: https://bugzilla.tianocore.org/show_bug.cgi?id=1856 SmmLockBoxPeiLib only support IA32 and X64 ARCH. So put this lib to a specific [LibraryClasses.IA32.PEIM, LibraryClasses.X64.PEIM] section. Cc: Jian J Wang Cc: Hao Wu Cc: Ray Ni Cc: Star Zeng Cc: Liming Gao Cc:

[edk2-devel] [PATCH] MdeModulePkg/CapsulePei: Optimize the CapsulePei

2019-05-28 Thread Gao, Zhichao
From: Bret Barkelew REF: https://bugzilla.tianocore.org/show_bug.cgi?id=1853 Sperate the capsule check function from GetCapsuleDescriptors and name it to AreCapsulesStaged. Rename GetCapsuleDescriptors to GetScatterGatherHeadEntries. And optimize its to remove the duplicated code. Cc: Jian J

Re: [edk2-devel] [PATCH] MdeModulePkg/CapsulePei: Optimize the CapsulePei

2019-05-30 Thread Gao, Zhichao
> -Original Message- > From: Leif Lindholm [mailto:leif.lindh...@linaro.org] > Sent: Wednesday, May 29, 2019 11:09 PM > To: devel@edk2.groups.io; Gao, Zhichao > Cc: Bret Barkelew ; Wang, Jian J > ; Wu, Hao A ; Ni, Ray > ; Zeng, Star ; Gao, Liming > ; Sean B

[edk2-devel] [PATCH V2] MdeModulePkg/CapsulePei: Optimize the CapsulePei

2019-05-30 Thread Gao, Zhichao
From: Bret Barkelew REF: https://bugzilla.tianocore.org/show_bug.cgi?id=1853 Optimize some function in CapsulePei to make it easier to maintian. 1. Separate the capsule check function form GetCapsuleDescriptors to AreCapsulesStaged. The original logic is unclear. 2. Avoid querying the capsule

Re: [edk2-devel] [PATCH v1 1/1] ShellPkg: acpiview: ACPI 6.3 update for MADT parser

2019-05-30 Thread Gao, Zhichao
Sorry for late update. Some minor comments below. > -Original Message- > From: devel@edk2.groups.io [mailto:devel@edk2.groups.io] On Behalf Of > Krzysztof Koch > Sent: Friday, May 17, 2019 12:06 AM > To: devel@edk2.groups.io > Cc: sami.muja...@arm.com; Carsey, Jaben ; Ni, > Ray ;

Re: [edk2-devel] [PATCH] MdeModulePkg/CapsulePei: Optimize the CapsulePei

2019-05-29 Thread Gao, Zhichao
> -Original Message- > From: Wu, Hao A > Sent: Wednesday, May 29, 2019 2:55 PM > To: Gao, Zhichao ; devel@edk2.groups.io > Cc: Bret Barkelew ; Wang, Jian J > ; Ni, Ray ; Zeng, Star > ; Gao, Liming ; Sean Brogan > ; Michael Turner > > Subject: RE: [

Re: [edk2-devel] [PATCH] MdeModulePkg/CapsulePei: Optimize the CapsulePei

2019-05-29 Thread Gao, Zhichao
> Sent: Wednesday, May 29, 2019 7:12 PM > To: devel@edk2.groups.io; Gao, Zhichao > Cc: Bret Barkelew ; Wang, Jian J > ; Wu, Hao A ; Ni, Ray > ; Zeng, Star ; Gao, Liming > ; Sean Brogan ; > Michael Turner > Subject: Re: [edk2-devel] [PATCH] MdeModulePkg/CapsulePei: Optimize the &g

[edk2-devel] [PATCH v3] MdeModulePkg/CapsulePei: Optimize the CapsulePei

2019-06-03 Thread Gao, Zhichao
From: Bret Barkelew REF: https://bugzilla.tianocore.org/show_bug.cgi?id=1853 Change form Mu repo: https://github.com/microsoft/mu_basecore/blob/release/201903/ MdeModulePkg/Universal/CapsulePei/UefiCapsule.c#L801 Minor changes on it: 1. Change the AreCapsulesStaged: directly return the BOOLEAN

[edk2-devel] [PATCH V3 1/2] MdeModulePkg: Add a pcd to set the OS indications bit

2019-06-03 Thread Gao, Zhichao
From: Bret Barkelew REF: https://bugzilla.tianocore.org/show_bug.cgi?id=1678 Add a pcd PcdPlatformRecoverySupport to conditionally set an OS indications bit and set the 'PlatformRecovery' variable. This pcd would also control whether the BDS support platform recovery or not. Cc: Jian J

[edk2-devel] [PATCH V3 2/2] MdeModulePkg/BdsDxe: Use a pcd to control PlatformRecovery

2019-06-03 Thread Gao, Zhichao
REF: https://bugzilla.tianocore.org/show_bug.cgi?id=1678 Use the PcdPlatformRecoverySupport to control the function of platform recovery in BDS. First, set the variable's ("OsIndicationsSupported") EFI_OS_INDICATIONS_START_PLATFORM_RECOVERY bit base on the pcd. It would affect the variable

[edk2-devel] [PATCH V3 0/2] Use a pcd to control Platform Recovery behavior

2019-06-03 Thread Gao, Zhichao
V1: Add a pcd PcdPlatformRecoverySupport to control the variable PlatformRecovery and the EFI_OS_INDICATIONS_START_PLATFORM_RECOVERY bit of variable "OsIndicationsSupported". V2: While PcdPlatformRecoverySupport is FALSE, do not set a PlatformRecovery Variable. But remain boot from a

Re: [edk2-devel] [PATCH V3 2/2] MdeModulePkg/BdsDxe: Use a pcd to control PlatformRecovery

2019-06-13 Thread Gao, Zhichao
> -Original Message- > From: Wu, Hao A > Sent: Thursday, June 13, 2019 3:39 PM > To: devel@edk2.groups.io; Gao, Zhichao > Cc: Wang, Jian J ; Ni, Ray ; Zeng, > Star ; Gao, Liming ; Sean > Brogan ; Michael Turner > ; Bret Barkelew > > Subject:

[edk2-devel] [PATCH] MdeModulePkg/PeiMain: Substantial change for PeiAllocatePool

2019-06-11 Thread Gao, Zhichao
From: Bret Barkelew REF: https://bugzilla.tianocore.org/show_bug.cgi?id=1901 The original logic is ASSERT if fail to create HOB. But that doesn't make sense for release version. So it is required to set the Buffer to null to indicate the failure. Cc: Jian J Wang Cc: Hao Wu Cc: Ray Ni Cc:

[edk2-devel] [PATCH] ShellPkg/UefiShellLib: Set input pointer parameter to null if failure

2019-06-12 Thread Gao, Zhichao
From: Bret Barkelew REF: https://bugzilla.tianocore.org/show_bug.cgi?id=1906 While failed to allocate memory to save the response, set the input/output parameter 'Response'(VOID **) to NULL to indicate the failure not only depend on the returned status. Cc: Jaben Carsey Cc: Ray Ni Cc: Liming

Re: [edk2-devel] [PATCH v4 2/2] MdeMoudlePkg/CapsulePei: Substantial change on UefiCapsule.c

2019-06-09 Thread Gao, Zhichao
> -Original Message- > From: Leif Lindholm [mailto:leif.lindh...@linaro.org] > Sent: Thursday, June 6, 2019 7:18 PM > To: Gao, Zhichao > Cc: devel@edk2.groups.io; Wang, Jian J ; Wu, Hao A > ; Ni, Ray ; Zeng, Star > ; Gao, Liming ; Sean Brogan > ; Michael

[edk2-devel] [PATCH v5 2/5] MdeMoudlePkg/CapsulePei: Fix coding style issue

2019-06-09 Thread Gao, Zhichao
REF: https://bugzilla.tianocore.org/show_bug.cgi?id=1853 Remove API specifier AreCapsulesStaged and GetScatterGatherHeadEntries because they are internal used. Add space between function name and bracket. Cc: Jian J Wang Cc: Hao A Wu Cc: Ray Ni Cc: Star Zeng Cc: Liming Gao Cc: Sean Brogan

[edk2-devel] [PATCH v5 4/5] MdeMoudlePkg/CapsulePei: Optimize GetScatterGatherHeadEntries

2019-06-09 Thread Gao, Zhichao
REF: https://bugzilla.tianocore.org/show_bug.cgi?id=1853 Rename the MACRO from MAX_SG_LIST_HEADS to DEFAULT_SG_LIST_HEADS. GetScatterGatherHeadEntries: use allocated buffer instead of fixed array to handle the condition which the SG list is larger then the array size. Cc: Jian J Wang Cc: Hao A

[edk2-devel] [PATCH v5 0/5] MdeModulePkg/CapsulePei: Optimize the CapsulePei

2019-06-09 Thread Gao, Zhichao
v1: Sperate the capsule check function from GetCapsuleDescriptors and name it to AreCapsulesStaged. Rename GetCapsuleDescriptors to GetScatterGatherHeadEntries. And optimize its to remove the duplicated code. V2: Clarify the changes of the patch and descript in the commit message. V3: Add the

[edk2-devel] [PATCH v5 1/5] MdeModulePkg/CapsulePei: Optimize the CapsulePei

2019-06-09 Thread Gao, Zhichao
From: Bret Barkelew REF: https://bugzilla.tianocore.org/show_bug.cgi?id=1853 Code change form Mu project: https://github.com/microsoft/mu_basecore/blob/release/201903/ MdeModulePkg/Universal/CapsulePei/UefiCapsule.c#L801 Separate the capsule check function from GetCapsuleDescriptors to

[edk2-devel] [PATCH v5 5/5] MdeMoudlePkg/CapsulePei: No need to remain space for null-terminate

2019-06-09 Thread Gao, Zhichao
REF: https://bugzilla.tianocore.org/show_bug.cgi?id=1853 UnicodeValueToStringS would remain two bytes for the first parameter to set the null- terminate. So remove this change in Mu. Cc: Jian J Wang Cc: Hao A Wu Cc: Ray Ni Cc: Star Zeng Cc: Liming Gao Cc: Sean Brogan Cc: Michael Turner

[edk2-devel] [PATCH v5 3/5] MdeMoudlePkg/CapsulePei: Optimize AreCapsulesStaged

2019-06-09 Thread Gao, Zhichao
REF: https://bugzilla.tianocore.org/show_bug.cgi?id=1853 AreCapsulesStaged do not need to return the status, only boolean result is useful. So directly return a boolean value. Cannot initialize the variable at its definition. Cc: Jian J Wang Cc: Hao A Wu Cc: Ray Ni Cc: Star Zeng Cc: Liming

Re: [edk2-devel] [PATCH v2 1/1] ShellPkg: acpiview: ACPI 6.3 update for MADT parser

2019-06-09 Thread Gao, Zhichao
Sorry for late update. > -Original Message- > From: Krzysztof Koch [mailto:krzysztof.k...@arm.com] > Sent: Friday, June 7, 2019 4:48 PM > To: devel@edk2.groups.io > Cc: sami.muja...@arm.com; Carsey, Jaben ; Ni, > Ray ; Gao, Zhichao ; > matteo.carl...@arm.com; stephani

Re: [edk2-devel] [PATCH v1 1/1] ShellPkg/acpiview: GTDT updates for ACPI 6.3

2019-06-09 Thread Gao, Zhichao
Reviewed-by: Zhichao Gao I would collect the R-B and send to the maintainer to push. Thanks, Zhichao > -Original Message- > From: PierreGondois [mailto:pierre.gond...@arm.com] > Sent: Thursday, June 6, 2019 8:29 PM > To: devel@edk2.groups.io > Cc: Carsey, Jaben ;

[edk2-devel] [PATCH 0/6] Change the performance code

2019-06-10 Thread Gao, Zhichao
Add and rdjust the performance code. Cc: Jian J Wang Cc: Hao A Wu Cc: Ray Ni Cc: Star Zeng Cc: Liming Gao Cc: Sean Brogan Cc: Michael Turner Cc: Bret Barkelew Cc: Chao Zhang Cc: Jiewen Yao Bret Barkelew (6): MdeModulePkg/DxeMain: Add performance code MdeModule/PeiMain: Add

[edk2-devel] [PATCH 2/6] MdeModule/PeiMain: Add performance code

2019-06-10 Thread Gao, Zhichao
From: Bret Barkelew REF: https://bugzilla.tianocore.org/show_bug.cgi?id=1888 Add performance code for PeiDisaptcher function and Image function. Cc: Jian J Wang Cc: Hao A Wu Cc: Ray Ni Cc: Star Zeng Cc: Liming Gao Signed-off-by: Zhichao Gao ---

[edk2-devel] [PATCH 4/6] SecurityPkg/Tcg2Dxe: Change performance code

2019-06-10 Thread Gao, Zhichao
From: Bret Barkelew REF: https://bugzilla.tianocore.org/show_bug.cgi?id=1888 Use PERF_FUNCTION_BEGIN and PERF_FUNCTION_END to replace PERF_START_EX and PERF_FUNCTION_END. Cc: Chao Zhang Cc: Jiewen Yao Cc: Jian Wang Signed-off-by: Zhichao Gao --- SecurityPkg/Tcg/Tcg2Dxe/Tcg2Dxe.c | 6

[edk2-devel] [PATCH 6/6] IntelSiliconPkg/IntelVtdDxe: Change the performance code

2019-06-10 Thread Gao, Zhichao
From: Bret Barkelew REF: https://bugzilla.tianocore.org/show_bug.cgi?id=1888 Use PERF_INMODULE_BEGIN and PERF_INMODULE_END to replace PERF_CODE. Cc: Ray Ni Cc: Rangasai V Chaganty Signed-off-by: Zhichao Gao --- .../Feature/VTd/IntelVTdDxe/IntelVTdDxe.c | 13 +++-- 1 file

[edk2-devel] [PATCH 1/6] MdeModulePkg/DxeMain: Add performance code

2019-06-10 Thread Gao, Zhichao
From: Bret Barkelew REF: https://bugzilla.tianocore.org/show_bug.cgi?id=1888 Add missing PERF_FUNCTION_END before the return statement. Cc: Jian J Wang Cc: Hao A Wu Cc: Ray Ni Cc: Star Zeng Cc: Liming Gao Cc: Sean Brogan Cc: Michael Turner Cc: Bret Barkelew Signed-off-by: Zhichao Gao

[edk2-devel] [PATCH 3/6] MdeModulePkg/UefiBootManagerLib: Change performance code

2019-06-10 Thread Gao, Zhichao
From: Bret Barkelew REF: https://bugzilla.tianocore.org/show_bug.cgi?id=1888 Use PERF_INMODULE_BEGIN and PERF_INMODULE_END to replace PERF_START_EX, PERF_CODE and PERF_END_EX. Use PERF_CROSSMODULE_END and PERF_CROSSMODULE_BEGIN to get the info of one boot image's performance. Cc: Jian J Wang

[edk2-devel] [PATCH 5/6] SecurityPkg/Tcg2Pei: Change performance code

2019-06-10 Thread Gao, Zhichao
From: Bret Barkelew REF: https://bugzilla.tianocore.org/show_bug.cgi?id=1888 Use PERF_FUNCTION_BEGIN and PERF_FUNCTION_END to replace PERF_START_EX and PERF_FUNCTION_END. Cc: Chao Zhang Cc: Jiewen Yao Cc: Jian Wang Signed-off-by: Zhichao Gao --- SecurityPkg/Tcg/Tcg2Pei/Tcg2Pei.c | 8

[edk2-devel] [PATCH V4 2/2] MdeModulePkg/BdsDxe: Use a pcd to control PlatformRecovery

2019-06-18 Thread Gao, Zhichao
REF: https://bugzilla.tianocore.org/show_bug.cgi?id=1678 Use the PcdPlatformRecoverySupport to control the function of platform recovery in BDS. First, set the variable's ("OsIndicationsSupported") EFI_OS_INDICATIONS_START_PLATFORM_RECOVERY bit base on the pcd. It would affect the variable

[edk2-devel] [PATCH V4 1/2] MdeModulePkg: Add a pcd to set the OS indications bit

2019-06-18 Thread Gao, Zhichao
From: Bret Barkelew REF: https://bugzilla.tianocore.org/show_bug.cgi?id=1678 Add a pcd PcdPlatformRecoverySupport to conditionally set an OS indications bit and set the 'PlatformRecovery' variable. This pcd would also control whether the BDS support platform recovery or not. Update the uni

[edk2-devel] [PATCH V4 0/2] Use a pcd to control Platform Recovery behavior

2019-06-18 Thread Gao, Zhichao
V1: Add a pcd PcdPlatformRecoverySupport to control the variable PlatformRecovery and the EFI_OS_INDICATIONS_START_PLATFORM_RECOVERY bit of variable "OsIndicationsSupported". V2: While PcdPlatformRecoverySupport is FALSE, do not set a PlatformRecovery Variable. But remain boot from a

[edk2-devel] [PATCH] MdeModulePkg/CapsulePei: Add memory pointer check

2019-06-24 Thread Gao, Zhichao
REF: https://bugzilla.tianocore.org/show_bug.cgi?id=1935 Before use the memory that is allocated through AllocateZeroPool, we should check the memory pointer is valid to avoid using the NULL pointer. Add check for VariableArrayAddress that is returned from GetScatterGatherHeadEntries. If it is

[edk2-devel] [PATCH V2] UefiCpuPkg/MpInitLib: MicrocodeDetect: Ensure checked range is valid

2019-06-25 Thread Gao, Zhichao
REF: https://bugzilla.tianocore.org/show_bug.cgi?id=1934 V1: Originally, the checksum part would done before verfiy the microcode data. Which meas the checksum would be done for a meaningless data. It would cause a incorrect TotalSize (the size of microcode data), then incorrect checksum and

Re: [edk2-devel] [PATCH] CryptoPkg: Add lacking instances for build only

2019-06-12 Thread Gao, Zhichao
@edk2.groups.io; Gao, Zhichao > Cc: Bret Barkelew ; Wang, Jian J > ; Ye, Ting ; Gao, Liming > ; Sean Brogan ; > Michael Turner > Subject: Re: [edk2-devel] [PATCH] CryptoPkg: Add lacking instances for build > only > > On 06/12/19 04:39, Gao, Zhichao wrote: >

Re: [edk2-devel] [PATCH] MdeModulePkg/PeiMain: Substantial change for PeiAllocatePool

2019-06-12 Thread Gao, Zhichao
To: devel@edk2.groups.io; Gao, Zhichao > Cc: Bret Barkelew ; Wang, Jian J > ; Wu, Hao A ; Ni, Ray > ; Zeng, Star ; Gao, Liming > ; Sean Brogan ; > Michael Turner > Subject: Re: [edk2-devel] [PATCH] MdeModulePkg/PeiMain: Substantial > change for PeiAllocatePool > > Hi Zhi

[edk2-devel] [PATCH v2] SecurityPkg: Add missing instances for build only

2019-06-14 Thread Gao, Zhichao
From: Bret Barkelew REF: https://bugzilla.tianocore.org/show_bug.cgi?id=1902 Add the missing instances to the [Components] of dsc file for build only. Cc: Chao Zhang Cc: Jiewen Yao Cc: Jian Wang Cc: Liming Gao Cc: Sean Brogan Cc: Michael Turner Cc: Bret Barkelew Cc: Leif Lindholm

Re: [edk2-devel] [PATCH V4 1/2] MdeModulePkg: Add a pcd to set the OS indications bit

2019-06-19 Thread Gao, Zhichao
> -Original Message- > From: Ni, Ray > Sent: Wednesday, June 19, 2019 1:06 PM > To: Gao, Zhichao ; devel@edk2.groups.io > Cc: Bret Barkelew ; Wang, Jian J > ; Wu, Hao A ; Zeng, Star > ; Gao, Liming ; Sean Brogan > ; Michael Turner > > Subject: RE: [PATCH V

[edk2-devel] [PATCH v5 1/2] MdeModulePkg: Add a pcd to set the OS indications bit

2019-06-19 Thread Gao, Zhichao
From: Bret Barkelew REF: https://bugzilla.tianocore.org/show_bug.cgi?id=1678 Add a pcd PcdPlatformRecoverySupport to conditionally set an OS indications bit and set the 'PlatformRecovery' variable. This pcd would also control whether the BDS support platform recovery or not. Update the uni

[edk2-devel] [PATCH v5 2/2] MdeModulePkg/BdsDxe: Use a pcd to control PlatformRecovery

2019-06-19 Thread Gao, Zhichao
REF: https://bugzilla.tianocore.org/show_bug.cgi?id=1678 Use the PcdPlatformRecoverySupport to control the function of platform recovery in BDS. First, set the variable's ("OsIndicationsSupported") EFI_OS_INDICATIONS_START_PLATFORM_RECOVERY bit base on the pcd. It would affect the variable

[edk2-devel] [PATCH v5 0/2] Use a pcd to control Platform Recovery behavior

2019-06-19 Thread Gao, Zhichao
V1: Add a pcd PcdPlatformRecoverySupport to control the variable PlatformRecovery and the EFI_OS_INDICATIONS_START_PLATFORM_RECOVERY bit of variable "OsIndicationsSupported". V2: While PcdPlatformRecoverySupport is FALSE, do not set a PlatformRecovery Variable. But remain boot from a

Re: [edk2-devel] [PATCH v1 2/2] ShellPkg: acpiview: Update SRAT parser to ACPI 6.3

2019-06-20 Thread Gao, Zhichao
f Koch [mailto:krzysztof.k...@arm.com] > Sent: Thursday, June 20, 2019 10:54 PM > To: devel@edk2.groups.io; Carsey, Jaben ; Ni, Ray > ; Gao, Zhichao > Cc: Sami Mujawar ; nd > Subject: RE: [edk2-devel] [PATCH v1 2/2] ShellPkg: acpiview: Update SRAT > parser to ACPI 6.3 > > Hi, &g

[edk2-devel] [PATCH] UefiCpuPkg/MpInitLib: Move checksum part for valid microcode data

2019-06-24 Thread Gao, Zhichao
REF: https://bugzilla.tianocore.org/show_bug.cgi?id=1934 Originally, the checksum part would done before verfiy the microcode data. Which meas the checksum would be done for a meaningless data. It would cause a incorrect TotalSize (the size of microcode data), then incorrect checksum and

Re: [edk2-devel] [PATCH v2 1/1] ShellPkg: acpiview: ACPI 6.3 update for MADT parser

2019-06-12 Thread Gao, Zhichao
Thanks for the interpret. Seems I read the old version spec. Now it is clear for me. Reviewed-by: Zhichao Gao > -Original Message- > From: devel@edk2.groups.io [mailto:devel@edk2.groups.io] On Behalf Of > Krzysztof Koch > Sent: Wednesday, June 12, 2019 6:44 PM > T

[edk2-devel] [PATCH] CryptoPkg: Add lacking instances for build only

2019-06-11 Thread Gao, Zhichao
From: Bret Barkelew REF: https://bugzilla.tianocore.org/show_bug.cgi?id=1900 Add the lacking instance to [Componnets] of dsc file for build only. Cc: Jian Wang Cc: Ting Ye Cc: Liming Gao Cc: Sean Brogan Cc: Michael Turner Cc: Bret Barkelew Signed-off-by: Zhichao Gao ---

[edk2-devel] [PATCH] SecurityPkg: Add lacking instances for build only

2019-06-11 Thread Gao, Zhichao
From: Bret Barkelew REF: https://bugzilla.tianocore.org/show_bug.cgi?id=1902 Add the lacking instance to the [Components] of dsc file for build only. Cc: Chao Zhang Cc: Jiewen Yao Cc: Jian Wang Cc: Liming Gao Cc: Sean Brogan Cc: Michael Turner Cc: Bret Barkelew Cc: Leif Lindholm

[edk2-devel] [PATCH] SourceLevelDebugPkg: Add lacking instances for build only

2019-06-11 Thread Gao, Zhichao
From: Bret Barkelew REF: https://bugzilla.tianocore.org/show_bug.cgi?id=1902 Add the lacking instance to the [Components] of dsc file for build only. Cc: Hao A Wu Cc: Liming Gao Cc: Sean Brogan Cc: Michael Turner Cc: Bret Barkelew Cc: Leif Lindholm Signed-off-by: Zhichao gao ---

[edk2-devel] [PATCH] MdeModulePkg: Add lacking instances for build only

2019-06-11 Thread Gao, Zhichao
From: Bret Barkelew REF: https://bugzilla.tianocore.org/show_bug.cgi?id=1857 Add the lacking instances to the [Components] of dsc file for build only. Cc: Jian J Wang Cc: Hao Wu Cc: Ray Ni Cc: Star Zeng Cc: Liming Gao Cc: Sean Brogan Cc: Michael Turner Cc: Bret Barkelew Signed-off-by:

Re: [edk2-devel] [PATCH v2] SecurityPkg: Add missing instances for build only

2019-06-17 Thread Gao, Zhichao
Added: https://bugzilla.tianocore.org/show_bug.cgi?id=1919 Thanks, Zhichao > -Original Message- > From: Ni, Ray > Sent: Friday, June 14, 2019 4:50 PM > To: devel@edk2.groups.io; Yao, Jiewen ; Gao, > Zhichao > Cc: Bret Barkelew ; Zhang, Chao B > ; Wang, Jian J ;

Re: [edk2-devel] [PATCH V1 1/1] ShellPkg/acpiview: GTDT updates for ACPI 6.3

2019-05-21 Thread Gao, Zhichao
el@edk2.groups.io > Cc: sami.muja...@arm.com; Carsey, Jaben ; Ni, > Ray ; Gao, Zhichao ; > matteo.carl...@arm.com; stephanie.hughes-f...@arm.com; n...@arm.com > Subject: [PATCH V1 1/1] ShellPkg/acpiview: GTDT updates for ACPI 6.3 > > From: Pierre Gondois > > The ACPI 6.

Re: [edk2-devel] [PATCH 4/6] MdePkg: Add gEfiCpu2ProtocolGuid and header file

2019-05-23 Thread Gao, Zhichao
I have found it in UEFP spec and PI spec. It is not defined in these specs. So I think maybe no spec has already defined it. Thanks, Zhichao > -Original Message- > From: Gao, Liming > Sent: Friday, May 24, 2019 1:20 PM > To: Gao, Zhichao ; devel@edk2.groups.io > Cc: Sean

[edk2-devel] [PATCH 1/6] MdeModulePkg: Add gEdkiiCommonEventProtocolGuid for event

2019-05-23 Thread Gao, Zhichao
From: Sean Brogan REF: https://bugzilla.tianocore.org/show_bug.cgi?id=1400 Add common event protocol to support all TPL event: Add the guid and header file of it. Cc: Jian J Wang Cc: Hao A Wu Cc: Ray Ni Cc: Star Zeng Cc: Liming gao Cc: Sean Brogan Cc: Michael Turner Cc: Bret Barkelew

[edk2-devel] [PATCH 2/6] MdePkg/BaseLib.h: Add EnableInterruptsAndSleep function declare

2019-05-23 Thread Gao, Zhichao
From: Sean Brogan REF: https://bugzilla.tianocore.org/show_bug.cgi?id=1400 Add EnableInterruptsAndSleep function. Cc: Michael D Kinney Cc: Liming Gao Cc: Sean Brogan Cc: Michael Turner Cc: Bret Barkelew Signed-off-by: Zhichao Gao --- MdePkg/Include/Library/BaseLib.h | 11 +++ 1

[edk2-devel] [PATCH 4/6] MdePkg: Add gEfiCpu2ProtocolGuid and header file

2019-05-23 Thread Gao, Zhichao
From: Sean Brogan REF: https://bugzilla.tianocore.org/show_bug.cgi?id=1400 Add gEfiCpu2ProtocolGuid to MdePkg.dec. Add the header file of Cpu2 protocol: it has one interface to enable interrupt and put cpu to sleep to wait for an interrupt. Cc: Michael D Kinney Cc: Liming Gao Cc: Sean Brogan

[edk2-devel] [PATCH 0/6] Fix race condition and add event protocol

2019-05-23 Thread Gao, Zhichao
There is a race condition in CoreWaitForEvent function: If an interrupt happens between CheckEvent and gIdleLoopEvent, there would be a event pending during cpu sleep. So it is required to check the gEventPending with the interrupt disabled. Implement a gEfiCpu2ProtocolGuid to fix that. The

[edk2-devel] [PATCH 6/6] MdeModulePkg/DxeMain: Implement common event protocol

2019-05-23 Thread Gao, Zhichao
From: Sean Brogan REF: https://bugzilla.tianocore.org/show_bug.cgi?id=1400 If an interrupt happens between CheckEvent and gIdleLoopEvent, there would be a event pending during cpu sleep. So it is required to check the gEventPending with the interrupt disabled. Implement a protocol

[edk2-devel] [PATCH 3/6] MdePkg/BaseLib: Implement EnableInterruptsAndSleep

2019-05-23 Thread Gao, Zhichao
From: Sean Brogan REF: https://bugzilla.tianocore.org/show_bug.cgi?id=1400 Implement EnableInterruptsAndSleep. Cc: Michael D Kinney Cc: Liming Gao Cc: Sean Brogan Cc: Michael Turner Cc: Bret Barkelew Signed-off-by: Zhichao Gao --- MdePkg/Library/BaseLib/Ia32/EnableInterrupts.c | 18

[edk2-devel] [PATCH V4 0/2] MdeModulePkg: Make the screen seamless

2019-05-23 Thread Gao, Zhichao
For now most platforms support display function at PEI phase. But the conspliter and graphics console driver would clear the screen at BDS connect console phase. Maybe some platforms would show logo in the next or maybe not. For consumers, it looks like the screen flashed. So change the behavior

[edk2-devel] [PATCH V4 2/2] MdeModulePkg/GraphicsConsoleDxe: Initialize the output mode

2019-05-23 Thread Gao, Zhichao
REF: https://bugzilla.tianocore.org/show_bug.cgi?id=1412 Original logic: Connect the graphics device -> connect it as graphics consoles and initialize its parameters(Mode = -1, invalid) -> connect it as console spliter and add the device to the list(use SetMode to set mode to the user defined

[edk2-devel] [PATCH V4 1/2] MdeModulePkg/ConSplitterDxe: Optimize the ConSplitterTextOutSetMode

2019-05-23 Thread Gao, Zhichao
From: Aaron Antone REF: https://bugzilla.tianocore.org/show_bug.cgi?id=1412 For Console Out device, it would always set all present devices' text out mode again through ConSplitterTextOutSetMode while adding devices. That may cause the screen cleared for serval times. So add a BOOLEAN to judge

[edk2-devel] [PATCH 5/6] UefiCpuPkg/CpuDxe: Implement Cpu2 protocol

2019-05-23 Thread Gao, Zhichao
REF: https://bugzilla.tianocore.org/show_bug.cgi?id=1400 Implement Cp2 protocol: it has one interface to enable the interrupt and put cpu to sleep and wait for an interrupt. Cc: Eric Dong Cc: Ray Ni Cc: Laszlo Ersek Cc: Liming Gao Cc: Sean Brogan Cc: Michael Turner Cc: Bret Barkelew

Re: [edk2-devel] [PATCH 4/6] MdePkg: Add gEfiCpu2ProtocolGuid and header file

2019-05-24 Thread Gao, Zhichao
I would move the definition to MdeModulePkg and change the prefix. Thanks, Zhichao > -Original Message- > From: Gao, Liming > Sent: Friday, May 24, 2019 2:02 PM > To: Gao, Zhichao ; devel@edk2.groups.io > Cc: Sean Brogan ; Kinney, Michael D > ; Michael Turner

Re: [edk2-devel] [PATCH 0/6] Fix race condition and add event protocol

2019-05-24 Thread Gao, Zhichao
ng == TRUE { Enable interrupt and goto CheckEvent to handle } else { Enable interrupt and hlt (this should be 'atomic' so that no race condition would happen) } Thanks, Zhichao > -Original Message- > From: Ni, Ray > Sent: Friday, May 24, 2019 1:18 PM > To: Gao, Zhichao ; dev

Re: [edk2-devel] [Patch 1/2] ShellPkg: Update DSC to use NetworkPkg's include fragment file

2019-05-19 Thread Gao, Zhichao
Reviewed-by: Zhichao Gao > -Original Message- > From: devel@edk2.groups.io [mailto:devel@edk2.groups.io] On Behalf Of > Liming Gao > Sent: Friday, May 17, 2019 4:51 PM > To: devel@edk2.groups.io > Cc: Gao, Zhichao ; Ni, Ray ; > Carsey, Jaben > Subject: [edk2-dev

Re: [edk2-devel] [PATCH 1/1] ShellPkg/CommandLib: avoid NULL derefence and memory leak

2019-04-08 Thread Gao, Zhichao
The patch code looks good to me. But while I apply this patch, it will shows "error: corrupt patch at line 24" and " error: could not build fake ancestor". Maybe you direct change the patch file and remove some blank lines. For Uefi BdsDxe driver, the variable "PlatfomLang" would be initialize

Re: [edk2-devel] [PATCH 1/1] ShellPkg/CommandLib: avoid NULL derefence and memory leak

2019-04-09 Thread Gao, Zhichao
g. Thanks, Zhichao > -Original Message- > From: Heinrich Schuchardt [mailto:xypron.g...@gmx.de] > Sent: Wednesday, April 10, 2019 12:37 AM > To: Gao, Zhichao ; devel@edk2.groups.io; Carsey, > Jaben ; Ni, Ray ; Leif Lindholm > > Cc: Gao, Liming > Subject: Re:

Re: [edk2-devel] [PATCH 2/2] MdeModulePkg/BdsDxe: Use a pcd to set OS indications bit

2019-04-09 Thread Gao, Zhichao
ril 9, 2019 5:32 PM > To: Gao, Zhichao ; 'devel@edk2.groups.io' > > Cc: Bret Barkelew ; Wang, Jian J > ; Zeng, Star ; Gao, Liming > ; Sean Brogan ; > Michael Turner > Subject: RE: [PATCH 2/2] MdeModulePkg/BdsDxe: Use a pcd to set OS > indications bit > > Resend to grou

[edk2-devel] [PATCH 3/5] MdeModulePkg/UefiDebugLibDebugPortProtocol: Make it runtime safe

2019-04-10 Thread Gao, Zhichao
From: Aaron Antone REF: https://bugzilla.tianocore.org/show_bug.cgi?id=1416 After ExitBootServices, some pointer would be invalid such as the Protocol pointer and gBS. The function depend on those should be prevent. So disable the related function while after ExitBootServices. Change the gBS to

[edk2-devel] [PATCH 1/5] MdeModulePkg: Change the SMM debug lib instance

2019-04-10 Thread Gao, Zhichao
REF: https://bugzilla.tianocore.org/show_bug.cgi?id=1416 The UefiDebugLibConOut will not support DXE_SMM_DRIVER, change UefiDebugLibConOut to PeiDxeDebugLibReportStatusCode. Cc: Jian J Wang Cc: Ray Ni Cc: Star Zeng Cc: Liming Gao Cc: Sean Brogan Cc: Michael Turner Cc: Bret Barkelew

[edk2-devel] [PATCH 0/5] Make some DebugLib instance runtime safe

2019-04-10 Thread Gao, Zhichao
Some pointer will be invalid after ExitBootServices, such as protocol pointer, gST, gBS and so on. Disable the functions which used that pointer. Remove SMM support of UefiDebugLibDebugPortProtocol, UefidebugLibConOut and UefiDebugLibStdErr. Before removing, should change the SMM instance in the

[edk2-devel] [PATCH 4/5] MdeModulePkg/UefidebugLibConOut: Make it runtime safe

2019-04-10 Thread Gao, Zhichao
From: Aaron Antone REF: https://bugzilla.tianocore.org/show_bug.cgi?id=1416 After ExitBootServices, some pointer would be invalid such as the Protocol pointer and gBS. The function depend on those should be prevent. So disable the related function while after ExitBootServices. Change the gST to

[edk2-devel] [PATCH 5/5] MdeModule/UefiDebugLibStdErr: Make it runtime safe

2019-04-10 Thread Gao, Zhichao
From: Aaron Antone REF: https://bugzilla.tianocore.org/show_bug.cgi?id=1416 After ExitBootServices, some pointer would be invalid such as the Protocol pointer and gBS. The function depend on those should be prevent. So disable the related function while after ExitBootServices. Change the gST to

[edk2-devel] [PATCH 2/5] SignedCapsulePkg: Change the SMM debug lib instance

2019-04-10 Thread Gao, Zhichao
REF: https://bugzilla.tianocore.org/show_bug.cgi?id=1416 The UefiDebugLibConOut will not support DXE_SMM_DRIVER, change UefiDebugLibConOut to PeiDxeDebugLibReportStatusCode. Cc: Jian J Wang Cc: Ray Ni Cc: Star Zeng Cc: Liming Gao Cc: Sean Brogan Cc: Michael Turner Cc: Bret Barkelew

Re: [edk2-devel] [edk2] [PATCH] MdeModulePkg/CapsulePei: Update the debug code to print 64bit data

2019-04-11 Thread Gao, Zhichao
Resend to gourps.io. And add the missing people. Thanks, Zhichao > -Original Message- > From: edk2-devel [mailto:edk2-devel-boun...@lists.01.org] On Behalf Of > Zhichao Gao > Sent: Tuesday, April 2, 2019 1:50 PM > To: edk2-de...@lists.01.org > Cc: Bret Barkelew ; Michael Turner > ; Gao,

Re: [edk2-devel] [edk2] [PATCH] MdeModulePkg/CapsulePei: Update the debug code to print 64bit data

2019-04-11 Thread Gao, Zhichao
thieu-Daudé [mailto:phi...@redhat.com] > Sent: Tuesday, April 2, 2019 11:36 PM > To: Gao, Zhichao ; edk2-de...@lists.01.org > Cc: Bret Barkelew ; Michael Turner > ; Gao, Liming ; > Zeng, Star > Subject: Re: [edk2] [PATCH] MdeModulePkg/CapsulePei: Update the debug > code to print 64bit d

Re: [edk2-devel] [PATCH 1/5] MdeModulePkg: Change the SMM debug lib instance

2019-04-11 Thread Gao, Zhichao
> -Original Message- > From: Wu, Hao A > Sent: Thursday, April 11, 2019 4:45 PM > To: devel@edk2.groups.io; Gao, Zhichao > Cc: Wang, Jian J ; Ni, Ray ; Zeng, > Star ; Gao, Liming ; Sean > Brogan ; Michael Turner > ; Bret Barkelew > > Subject: RE: [edk2-d

Re: [edk2-devel] [edk2] [PATCH] MdeModulePkg/CapsulePei: Update the debug code to print 64bit data

2019-04-11 Thread Gao, Zhichao
Thanks for you advices. I would better to keep the 'UINTN' with '%x' because PEIM is compiled as 32 bit object. And lots of PEIMs keep this combine. Thanks, Zhichao > -Original Message- > From: Wu, Hao A > Sent: Thursday, April 11, 2019 4:12 PM > To: Gao, Zhichao ; Phil

[edk2-devel] [PATCH V2 1/2] MdeModulePkg/CapsulePei: Change debug MACRO

2019-04-11 Thread Gao, Zhichao
REF: https://bugzilla.tianocore.org/show_bug.cgi?id=1677 Change debug printlevel from EFI_D_ to DEBUG_. DEBUG_ version is recommanded to use. Cc: Jian J Wang Cc: Hao Wu Cc: Ray Ni Cc: Star Zeng Cc: Liming Gao Cc: Sean Brogan Cc: Michael Turner Cc: Bret Barkelew Signed-off-by: Zhichao Gao

[edk2-devel] [PATCH V2 2/2] MdeModulePkg/CapsulePei: Update the debug code to print 64bit data

2019-04-11 Thread Gao, Zhichao
From: Bret Barkelew REF: https://bugzilla.tianocore.org/show_bug.cgi?id=1677 For some pointer and UINT64 data, the debug code print with '%x'. Which would loss the upper 32bit data. So update '%x' to '%lx' for these data. And change the pointer value from '%x' to '%p'. Cc: Jian J Wang Cc: Hao

[edk2-devel] [PATCH V2 0/2] MdeModulePkg/CapsulePei: Update the debug code to print 64bit data

2019-04-11 Thread Gao, Zhichao
Change the EFI_D_ version debug MACRO to DEBUG_ version. And change the UINT64 value from '%x' to '%lx'. V2: Seperate one patch to two to view different change goal. Change the pointer value from '%x' to '%p'. Cc: Jian J Wang Cc: Hao Wu Cc: Ray Ni Cc: Star Zeng Cc: Liming Gao Cc: Sean

[edk2-devel] [PATCH V2 2/5] SignedCapsulePkg: Change the SMM debug lib instance

2019-04-11 Thread Gao, Zhichao
REF: https://bugzilla.tianocore.org/show_bug.cgi?id=1416 The UefiDebugLibConOut will not support DXE_SMM_DRIVER, change UefiDebugLibConOut to BaseDebugLibNull. Cc: Jiewen Yao Cc: Chao Zhang Cc: Liming Gao Cc: Sean Brogan Cc: Michael Turner Cc: Bret Barkelew Signed-off-by: Zhichao Gao ---

[edk2-devel] [PATCH V2 4/5] MdePkg/UefidebugLibConOut: Make it runtime safe

2019-04-11 Thread Gao, Zhichao
From: Aaron Antone REF: https://bugzilla.tianocore.org/show_bug.cgi?id=1416 After ExitBootServices, some pointer would be invalid such as the Protocol pointer and gST. The function depend on those should be prevent. So disable the related function while after ExitBootServices. Change the gST to

[edk2-devel] [PATCH V2 0/5] Make some DebugLib instance runtime safe

2019-04-11 Thread Gao, Zhichao
REF: https://bugzilla.tianocore.org/show_bug.cgi?id=1416 Some pointer will be invalid after ExitBootServices, such as protocol pointer, gST, gBS and so on. Disable the functions which used that pointer. Remove SMM support of UefiDebugLibDebugPortProtocol, UefidebugLibConOut and

[edk2-devel] [PATCH V2 5/5] MdePkg/UefiDebugLibStdErr: Make it runtime safe

2019-04-11 Thread Gao, Zhichao
From: Aaron Antone REF: https://bugzilla.tianocore.org/show_bug.cgi?id=1416 After ExitBootServices, some pointer would be invalid such as the Protocol pointer and gST. The function depend on those should be prevent. So disable the related function while after ExitBootServices. Change the gST to

  1   2   3   4   5   >