[edk2] [Patch 2/3] IntelFrameworkPkg UefiLib: Use comparison logic to check UINTN parameter

2018-05-28 Thread Liming Gao
Commit cb96e7d4f7afdbaef0706f9251ae479639d85a28 changes the input parameter from BOOLEAN to UINTN. Its comparison logic should be updated. Contributed-under: TianoCore Contribution Agreement 1.1 Signed-off-by: Liming Gao --- IntelFrameworkPkg/Library/FrameworkUefiLib/UefiLib.c | 6 +++--- 1

[edk2] [Patch 3/3] MdeModulePkg Variable: Use comparison logic to check UINTN parameter

2018-05-28 Thread Liming Gao
Commit 180ac200da84785989443b06bcfa5db343c0bf7e changes the input parameter from BOOLEAN to UINTN. Its comparison logic should be updated. Contributed-under: TianoCore Contribution Agreement 1.1 Signed-off-by: Liming Gao Cc: Star Zeng --- MdeModulePkg/Universal/Variable/EmuRuntimeDxe

[edk2] [Patch 1/3] MdePkg UefiLib: Use comparison logic to check UINTN parameter

2018-05-28 Thread Liming Gao
Commit d2aafe1e410c80d1046f2d1e743055882ead8489 changes the input parameter from BOOLEAN to UINTN. Its comparison logic should be updated. Contributed-under: TianoCore Contribution Agreement 1.1 Signed-off-by: Liming Gao Cc: Michael Kinney --- MdePkg/Library/UefiLib/UefiLib.c | 6 +++--- 1

[edk2] [RFC] Formalize source files to follow DOS format

2018-05-20 Thread Liming Gao
patch per one package. Contributed-under: TianoCore Contribution Agreement 1.1 Signed-off-by: Liming Gao --- BaseTools/Scripts/FormatDosFiles.py | 93 + 1 file changed, 93 insertions(+) create mode 100644 BaseTools/Scripts/FormatDosFiles.py diff --git a/BaseTools

[edk2] [Patch v2 0/5] Remove X86 ASM and S files

2018-05-18 Thread Liming Gao
://github.com/lgao4/edk2/tree/AsmClean Liming Gao (5): IntelFrameworkModulePkg: Remove X86 ASM and S files MdeModulePkg: Remove X86 ASM and S files MdePkg: Remove X86 ASM and S files SourceLevelDebugPkg: Remove X86 ASM and S files UefiCpuPkg: Remove X86 ASM and S files .../Csm

[edk2] [Patch] MdeModulePkg PeiCore: FvCount to be checked for the registered for shadow PEIM

2018-05-15 Thread Liming Gao
The PEIM in all cached FV image may be in registered for shadow status. Current logic CurrentPeimFvCount is not enough. Contributed-under: TianoCore Contribution Agreement 1.1 Signed-off-by: Liming Gao Cc: Star Zeng --- MdeModulePkg/Core/Pei/Dispatcher/Dispatcher.c | 2 +- 1 file changed, 1

[edk2] [Patch 3/3] MdeModulePkg Variable: Fix XCODE5 varargs warning

2018-05-14 Thread Liming Gao
https://bugzilla.tianocore.org/show_bug.cgi?id=741 Change VariableGetBestLanguage() parameter type from BOOLEAN to UINTN Contributed-under: TianoCore Contribution Agreement 1.1 Signed-off-by: Steven Shi Cc: Liming Gao --- MdeModulePkg/Universal/Variable/EmuRuntimeDxe/EmuVariable.c | 2

[edk2] [Patch 0/3] Fix XCODE5 varargs warning in GetBestLanguage API

2018-05-14 Thread Liming Gao
https://bugzilla.tianocore.org/show_bug.cgi?id=741 Change GetBestLanguage() parameter from BOOLEAN to UINTN PI S3SaveState protocol will be updated after new PI spec is public. Liming Gao (3): MdePkg UefiLib: Fix XCODE5 varargs warning IntelFrameworkPkg UefiLib: Fix XCODE5 varargs warning

[edk2] [Patch 2/3] IntelFrameworkPkg UefiLib: Fix XCODE5 varargs warning

2018-05-14 Thread Liming Gao
https://bugzilla.tianocore.org/show_bug.cgi?id=741 Change GetBestLanguage() parameter type from BOOLEAN to UINTN Contributed-under: TianoCore Contribution Agreement 1.1 Signed-off-by: Steven Shi Cc: Liming Gao --- IntelFrameworkPkg/Library/FrameworkUefiLib/UefiLib.c | 2 +- 1 file changed, 1

[edk2] [Patch 1/3] MdePkg UefiLib: Fix XCODE5 varargs warning

2018-05-14 Thread Liming Gao
https://bugzilla.tianocore.org/show_bug.cgi?id=741 Change GetBestLanguage() parameter type from BOOLEAN to UINTN Contributed-under: TianoCore Contribution Agreement 1.1 Signed-off-by: Steven Shi Cc: Liming Gao --- MdePkg/Include/Library/UefiLib.h | 2 +- MdePkg/Library/UefiLib/UefiLib.c | 2

[edk2] [Patch 5/5] UefiCpuPkg: Remove X86 ASM and S files

2018-05-13 Thread Liming Gao
NASM has replaced ASM and S files. Rmove ASM from all modules. Remove S files from the drivers only. After NASM is updated, S files can be removed from Library. Contributed-under: TianoCore Contribution Agreement 1.1 Signed-off-by: Liming Gao --- UefiCpuPkg/CpuDxe/CpuDxe.inf

[edk2] [Patch 1/5] IntelFrameworkModulePkg: Remove X86 ASM and S files

2018-05-13 Thread Liming Gao
NASM has replaced ASM and S files. Rmove ASM from all modules. Remove S files from the drivers only. After NASM is updated, S files can be removed from Library. Contributed-under: TianoCore Contribution Agreement 1.1 Signed-off-by: Liming Gao --- .../Csm/LegacyBiosDxe/IA32/InterruptTable.S

[edk2] [Patch 2/5] MdeModulePkg: Remove X86 ASM and S files

2018-05-13 Thread Liming Gao
NASM has replaced ASM and S files. Rmove ASM from all modules. Remove S files from the drivers only. After NASM is updated, S files can be removed from Library. Contributed-under: TianoCore Contribution Agreement 1.1 Signed-off-by: Liming Gao --- MdeModulePkg/Core/DxeIplPeim/DxeIpl.inf

[edk2] [Patch 4/5] SourceLevelDebugPkg: Remove X86 ASM and S files

2018-05-13 Thread Liming Gao
NASM has replaced ASM and S files. Rmove ASM from all modules. Remove S files from the drivers only. After NASM is updated, S files can be removed from Library. Contributed-under: TianoCore Contribution Agreement 1.1 Signed-off-by: Liming Gao --- .../DebugAgent/DebugAgentCommon/Ia32

[edk2] [Patch 0/5] Remove X86 ASM and S files

2018-05-13 Thread Liming Gao
For IA32 and X64, NASM has replaced ASM and S files. Rmove ASM from all modules. Remove S files from the drivers only. After NASM is updated, S files can be removed from Library. Liming Gao (5): IntelFrameworkModulePkg: Remove X86 ASM and S files MdeModulePkg: Remove X86 ASM and S files

[edk2] [Patch] BaseTools: Retrieve /U and -U CC flags to structure PcdValueInit Makefile

2018-05-03 Thread Liming Gao
/D and -D flags have been added. So, /U and -U flags should be added. Contributed-under: TianoCore Contribution Agreement 1.1 Signed-off-by: Liming Gao Cc: Yonghong Zhu --- BaseTools/Source/Python/Workspace/DscBuildData.py | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a

[edk2] [Patch] BaseTools: Separate HOST and PREFIX env for GCC tool chain

2018-04-25 Thread Liming Gao
Agreement 1.1 Signed-off-by: Liming Gao --- BaseTools/Conf/tools_def.template | 15 --- 1 file changed, 8 insertions(+), 7 deletions(-) diff --git a/BaseTools/Conf/tools_def.template b/BaseTools/Conf/tools_def.template index 5da229a..48a38e1 100755 --- a/BaseTools/Conf

[edk2] [Patch] IntelFrameworkModulePkg IsaSerialDxe: Update algorithm to calculate Divisor

2018-04-17 Thread Liming Gao
To align the way in MdeModulePkg SerialPortLib and PciSioSerialDxe driver, Divisor is added by one when the reminder is more than half (16 * BaudRate). Contributed-under: TianoCore Contribution Agreement 1.1 Signed-off-by: Liming Gao Cc: Ruiyu Ni --- IntelFrameworkModulePkg/Bus/Isa

[edk2] [Patch] BaseTools: Update Rsa2048Sha256Sign to use openssl dgst option

2018-03-27 Thread Liming Gao
Contributed-under: TianoCore Contribution Agreement 1.1 Signed-off-by: Liming Gao Cc: Yonghong Zhu Cc: Qin Long --- BaseTools/Source/Python/Rsa2048Sha256Sign/Rsa2048Sha256Sign.py | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/BaseTools/Source/Python/Rsa2048Sha256Sign

[edk2] [PATCH v2] BaseTools: Update Rsa2048Sha256Sign to use openssl standard options

2018-03-26 Thread Liming Gao
but has sha,sha1. To work with all openssl version, use sha1 -sha256 for it. Contributed-under: TianoCore Contribution Agreement 1.1 Signed-off-by: Liao Jui-peng Signed-off-by: Liming Gao Cc: Michael Kinney Cc: Yonghong Zhu --- BaseTools/Source/Python/Rsa2048Sha256Sign/Rsa2048Sha256Sign.py

[edk2] [Patch] BaseTools: Update Rsa2048Sha256Sign to use openssl standard options

2018-03-26 Thread Liming Gao
sha256 is not the standard option. It should be replaced by sha -sha256. Otherwise, it doesn't work in MAC OS. Contributed-under: TianoCore Contribution Agreement 1.1 Signed-off-by: Liao Jui-peng Signed-off-by: Liming Gao Cc: Michael Kinney Cc: Yonghong Zhu --- BaseTools/Source/P

[edk2] [Patch] NetworkPkg: Correct HttpTlsCipherList.h file format to DOS

2018-03-25 Thread Liming Gao
Contributed-under: TianoCore Contribution Agreement 1.1 Signed-off-by: Liming Gao Cc: Wu Jiaxin Cc: Fu Siyuan --- NetworkPkg/Include/Guid/HttpTlsCipherList.h | 76 ++--- 1 file changed, 38 insertions(+), 38 deletions(-) diff --git a/NetworkPkg/Include/Guid

[edk2] [Patch] BaseTools: Add the missing package include directory in PcdValueInit Makefile

2018-03-22 Thread Liming Gao
Contributed-under: TianoCore Contribution Agreement 1.1 Signed-off-by: Liming Gao Cc: Feng Bob C Cc: Yonghong Zhu --- BaseTools/Source/Python/Workspace/DscBuildData.py | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) diff --git a/BaseTools/Source/Python/Workspace/DscBuildData.py b

[edk2] [Patch] DecSpecification: Add structure PCD definition syntax

2018-03-15 Thread Liming Gao
Contributed-under: TianoCore Contribution Agreement 1.1 Signed-off-by: Liming Gao Cc: Yonghong Zhu Cc: Michael Kinney --- 2_dec_file_overview/210_pcd_usage.md | 7 ++- 3_edk_ii_dec_file_format/310_pcd_sections.md | 10

[edk2] [Patch 1/3] DSCSpecification: Add syntax to support SKU ID inherit from another SKU ID

2018-03-15 Thread Liming Gao
Contributed-under: TianoCore Contribution Agreement 1.1 Signed-off-by: Liming Gao Cc: Michael Kinney Cc: Yonghong Zhu --- 2_dsc_overview/25_[skuids]_section_processing.md | 11 +++ 3_edk_ii_dsc_file_format/37_[skuids]_section.md | 10 ++ 2 files changed, 13 insertions(+), 8

[edk2] [Patch] BuildSpecification: Clarify structure PCD field value assignment precedence

2018-03-15 Thread Liming Gao
Contributed-under: TianoCore Contribution Agreement 1.1 Signed-off-by: Liming Gao Cc: Michael Kinney Cc: Yonghong Zhu --- 8_pre-build_autogen_stage/82_auto-generation_process.md | 5 +++-- 1 file changed, 3 insertions(+), 2 deletions(-) diff --git a/8_pre-build_autogen_stage/82_auto

[edk2] [Patch 3/3] DscSpecification: Add structure PCD field value assignment syntax

2018-03-15 Thread Liming Gao
Contributed-under: TianoCore Contribution Agreement 1.1 Signed-off-by: Liming Gao Cc: Michael Kinney Cc: Yonghong Zhu --- 2_dsc_overview/28_pcd_section_processing.md | 10 ++ 3_edk_ii_dsc_file_format/310_pcd_sections.md | 19

[edk2] [Patch 2/3] DscSpecification: Add DefaultStores section to describe the default setting

2018-03-15 Thread Liming Gao
Contributed-under: TianoCore Contribution Agreement 1.1 Signed-off-by: Liming Gao Cc: Michael Kinney Cc: Yonghong Zhu --- .../213_[defaultstores]_section_processing.md | 49 3_edk_ii_dsc_file_format/310_pcd_sections.md | 86 -- .../313_[defaultstores

[edk2] [Patch 0/3] Update DscSpecification for structure PCD support

2018-03-15 Thread Liming Gao
Liming Gao (3): DSCSpecification: Add syntax to support SKU ID inherit from another SKU ID DscSpecification: Add DefaultStores section to describe the default setting DscSpecification: Add structure PCD field value assignment syntax .../213_[defaultstores]_section_processing.md

[edk2] [Patch] UefiCpuPkg CpuExceptionHandlerLib: use FixedPcdGetSize() as the macro value

2018-03-15 Thread Liming Gao
FixedPcdGetSize() is used as the macro value, PcdGetSize() is used as global variable or function. Here usage is to access macro value. Contributed-under: TianoCore Contribution Agreement 1.1 Signed-off-by: Liming Gao Cc: Wang Jian J Cc: Ruiyu Ni Cc: Michael Kinney --- UefiCpuPkg/Library

[edk2] [Patch] MdePkg BaseStackCheckLib: Correct style of file header

2018-03-15 Thread Liming Gao
Contributed-under: TianoCore Contribution Agreement 1.1 Signed-off-by: Liming Gao Cc: Michael Kinney --- MdePkg/Library/BaseStackCheckLib/BaseStackCheckNull.c | 7 ++- 1 file changed, 2 insertions(+), 5 deletions(-) diff --git a/MdePkg/Library/BaseStackCheckLib/BaseStackCheckNull.c b

[edk2] [Patch] MdeModulePkg BrotliLib: Rename function with the specific lib name

2018-03-13 Thread Liming Gao
This change is to avoid the function conflict. Contributed-under: TianoCore Contribution Agreement 1.1 Signed-off-by: Liming Gao Cc: Star Zeng --- .../Library/BrotliCustomDecompressLib/BrotliDecompress.c | 14 +++--- .../BrotliDecompressLibInternal.h | 6

[edk2] [Patch] MdeModulePkg PCD: Remove unused PCD attribute PCD_TYPE_SKU_ENABLED

2018-03-07 Thread Liming Gao
PcdDb optimization has handled PCD DB for each SKU, not for single PCD. So, this PCD attribute is not used any more. Contributed-under: TianoCore Contribution Agreement 1.1 Signed-off-by: Liming Gao Cc: Reviewed-by: Liming Gao --- MdeModulePkg/Include/Guid/PcdDataBaseSignatureGuid.h | 3

[edk2] [Patch] BaseTools: Align WIN_CERTIFICATE_UEFI_GUID definition to MdePkg one.

2018-02-28 Thread Liming Gao
Contributed-under: TianoCore Contribution Agreement 1.1 Signed-off-by: Liming Gao --- BaseTools/Source/C/Include/Common/UefiMultiPhase.h | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/BaseTools/Source/C/Include/Common/UefiMultiPhase.h b/BaseTools/Source/C/Include/Common

[edk2] [Patch] BaseTools: Resolve BaseTools C tool build failure

2018-02-27 Thread Liming Gao
New GUID definition is conflicted with GUID in Windows Kits guiddef.h. GUID definition will be defined when it is undefined. Contributed-under: TianoCore Contribution Agreement 1.1 Signed-off-by: Liming Gao --- BaseTools/Source/C/GenFv/GenFvInternalLib.c | 2 +- BaseTools/Source/C/Include

[edk2] [Patch] MdeModulePkg PCD: Add more debug message to show SkuId update

2018-02-26 Thread Liming Gao
Contributed-under: TianoCore Contribution Agreement 1.1 Signed-off-by: Liming Gao Cc: Star Zeng --- MdeModulePkg/Universal/PCD/Dxe/Pcd.c | 6 +- MdeModulePkg/Universal/PCD/Pei/Pcd.c | 6 +- 2 files changed, 10 insertions(+), 2 deletions(-) diff --git a/MdeModulePkg/Universal/PCD/Dxe

[edk2] [Patch] MdeModulePkg PCD: Fix the issue to set the big SkuId

2018-02-26 Thread Liming Gao
Contributed-under: TianoCore Contribution Agreement 1.1 Signed-off-by: Liming Gao Cc: Star Zeng --- MdeModulePkg/Universal/PCD/Dxe/Service.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/MdeModulePkg/Universal/PCD/Dxe/Service.c b/MdeModulePkg/Universal/PCD/Dxe/Service.c

[edk2] [Patch] BaseTools: Add more error message when PcdValue is wrong

2018-02-25 Thread Liming Gao
For structure PCD, its field name is wrong and cause build failure. Its build error message will output to let user aware what's wrong. Contributed-under: TianoCore Contribution Agreement 1.1 Signed-off-by: Liming Gao Cc: Feng Bob C --- BaseTools/Source/Python/Workspace/DscBuildData.p

[edk2] [Patch] BaseTools: Update GenFw to correct DebugEntry Offset when convert XIP image

2018-02-25 Thread Liming Gao
DebugEntry FileOffset is required to be updated to the virtual address if the input image is converted to XIP image. Contributed-under: TianoCore Contribution Agreement 1.1 Signed-off-by: Liming Gao --- BaseTools/Source/C/GenFw/GenFw.c | 6 ++ 1 file changed, 6 insertions(+) diff --git a

[edk2] [Patch] EmulatorPkg: Undefine CR3 macro in Host.h

2018-02-22 Thread Liming Gao
CR3 has been used as structure field name in BaseLib IA32_TASK_STATE_SEGMENT. Undefine CR3 to make sure there is no conflict to it. Contributed-under: TianoCore Contribution Agreement 1.1 Signed-off-by: Liming Gao Cc: Andrew Fish --- EmulatorPkg/Unix/Host/Host.h | 2 +- 1 file changed, 1

[edk2] [Patch] BaseTools: Add the missing basic definition in C BaseType.h

2018-02-08 Thread Liming Gao
Contributed-under: TianoCore Contribution Agreement 1.1 Signed-off-by: Liming Gao --- BaseTools/Source/C/Include/Common/BaseTypes.h | 25 + 1 file changed, 25 insertions(+) diff --git a/BaseTools/Source/C/Include/Common/BaseTypes.h b/BaseTools/Source/C/Include/Common

[edk2] [Patch] MdeModulePkg PeiCore: Add error message to describe PEIM load failure

2018-02-08 Thread Liming Gao
If PEIM image address doesn't meet with its section alignment, it will load fail. PeiCore adds more debug message to report it. Contributed-under: TianoCore Contribution Agreement 1.1 Signed-off-by: Liming Gao Cc: Star Zeng --- MdeModulePkg/Core/Pei/Image/Image.c | 7 ++- 1 file chang

[edk2] [Patch] BaseTools GenFv: Update error message to describe PE image alignment

2018-02-07 Thread Liming Gao
Contributed-under: TianoCore Contribution Agreement 1.1 Signed-off-by: Liming Gao --- BaseTools/Source/C/GenFv/GenFvInternalLib.c | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/BaseTools/Source/C/GenFv/GenFvInternalLib.c b/BaseTools/Source/C/GenFv/GenFvInternalLib.c

[edk2] [Patch] MdePkg BaseCpuLib: Make it pass VS ARM build

2018-02-06 Thread Liming Gao
Contributed-under: TianoCore Contribution Agreement 1.1 Signed-off-by: Liming Gao --- MdePkg/Library/BaseCpuLib/BaseCpuLib.inf | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) diff --git a/MdePkg/Library/BaseCpuLib/BaseCpuLib.inf b/MdePkg/Library/BaseCpuLib/BaseCpuLib.inf index 637a3c5

[edk2] [Patch] MdePkg SafeIntLib: Update API definition to use the same output name

2018-01-31 Thread Liming Gao
In SafeUintnToChar8(), update its output parameter name. Update pui8Result --> Result to match its library implementation Contributed-under: TianoCore Contribution Agreement 1.1 Signed-off-by: Liming Gao --- MdePkg/Include/Library/SafeIntLib.h | 12 ++-- 1 file changed, 6 inserti

[edk2] [Patch 0/2] BaseTools GNUmakefile: Move HOST_ARCH detection into common makefile

2018-01-31 Thread Liming Gao
Liming Gao (2): BaseTools GNUmakefile: Move HOST_ARCH detection into common makefile BaseTools GNUmakefile: Remove HOST_ARCH in every tool Makefile BaseTools/Source/C/BootSectImage/GNUmakefile| 3 +-- BaseTools/Source/C/BrotliCompress/GNUmakefile | 3 +-- BaseTools/Source/C/Common

[edk2] [Patch 1/2] BaseTools GNUmakefile: Move HOST_ARCH detection into common makefile

2018-01-31 Thread Liming Gao
With this change, enter single tool directory, make can pass. Contributed-under: TianoCore Contribution Agreement 1.1 Signed-off-by: Liming Gao --- BaseTools/Source/C/Makefiles/header.makefile | 28 ++-- 1 file changed, 26 insertions(+), 2 deletions(-) diff --git a

[edk2] [Patch 2/2] BaseTools GNUmakefile: Remove HOST_ARCH in every tool Makefile

2018-01-31 Thread Liming Gao
HOST_ARCH has been moved into the common header.makefile Contributed-under: TianoCore Contribution Agreement 1.1 Signed-off-by: Liming Gao --- BaseTools/Source/C/BootSectImage/GNUmakefile| 3 +-- BaseTools/Source/C/BrotliCompress/GNUmakefile | 3 +-- BaseTools/Source/C/Common/GNUmakefile

[edk2] [Patch] UefiCpuPkg: Remove the unused file ResetVec.asm16

2018-01-31 Thread Liming Gao
ResetVec.nasmb is used. ResetVec.asm16 can be retired. Contributed-under: TianoCore Contribution Agreement 1.1 Signed-off-by: Liming Gao --- UefiCpuPkg/SecCore/Ia32/ResetVec.asm16 | 106 - 1 file changed, 106 deletions(-) delete mode 100644 UefiCpuPkg/SecCore

[edk2] [Patch] BaseTools CommonLib: Remove the unnecessary print message in PcdValueCommon

2018-01-31 Thread Liming Gao
Contributed-under: TianoCore Contribution Agreement 1.1 Signed-off-by: Liming Gao --- BaseTools/Source/C/Common/PcdValueCommon.c | 10 -- 1 file changed, 10 deletions(-) diff --git a/BaseTools/Source/C/Common/PcdValueCommon.c b/BaseTools/Source/C/Common/PcdValueCommon.c index 92328da

[edk2] [Patch] SecurityPkg: Don't build AuthVariableLib for EBC arch

2018-01-29 Thread Liming Gao
or EBC. #define MAX_UINTN ((UINTN) ~0) ==> #define MAX_UINTN ((UINTN)(~0ULL >> (64 - sizeof (INTN) * 8))) Contributed-under: TianoCore Contribution Agreement 1.1 Signed-off-by: Liming Gao --- SecurityPkg/SecurityPkg.dsc | 1 + 1 file changed, 1 insertion(+) diff --git a/SecurityPkg/Se

[edk2] [Patch] MdeModulePkg: Don't build VarCheckUefiLib and DxeCore for EBC arch

2018-01-29 Thread Liming Gao
fix is not to build VarCheckUefiLib and DxeCore for EBC arch. #define MAX_UINTN ((UINTN) ~0) ==> #define MAX_UINTN ((UINTN)(~0ULL >> (64 - sizeof (INTN) * 8))) Contributed-under: TianoCore Contribution Agreement 1.1 Signed-off-by: Liming Gao --- MdeModulePkg/MdeModulePkg.dsc | 13 +

[edk2] [Patch] BaseTools: CommonLib Fix Crash to write the last byte

2018-01-24 Thread Liming Gao
Contributed-under: TianoCore Contribution Agreement 1.1 Signed-off-by: Liming Gao --- BaseTools/Source/C/Common/PcdValueCommon.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/BaseTools/Source/C/Common/PcdValueCommon.c b/BaseTools/Source/C/Common/PcdValueCommon.c index

[edk2] [PATCH v2] PcAtChipsetPkg: Add PeiAcpiTimerLib to save PerformanceCounterFrequency in HOB

2018-01-22 Thread Liming Gao
share the same PerformanceCounterFrequency. Contributed-under: TianoCore Contribution Agreement 1.1 Signed-off-by: Liming Gao Cc: Star Zeng --- PcAtChipsetPkg/Library/AcpiTimerLib/AcpiTimerLib.c | 4 +- .../Library/AcpiTimerLib/DxeAcpiTimerLib.c | 57 +- .../Library

[edk2] [Patch] PcAtChipsetPkg: Add PeiAcpiTimerLib to save PerformanceCounterFrequency in HOB

2018-01-21 Thread Liming Gao
PeiAcpiTimerLib caches PerformanceCounterFrequency in HOB, then Pei and Dxe AcpiTimerLib can share the same PerformanceCounterFrequency. Contributed-under: TianoCore Contribution Agreement 1.1 Signed-off-by: Liming Gao Cc: Star Zeng --- PcAtChipsetPkg/Library/AcpiTimerLib/AcpiTimerLib.c | 4

[edk2] [Patch] MdePkg: Update DEC and DSC version from 1.07 to 1.08

2018-01-17 Thread Liming Gao
UEFI2.7/PI1.6 have been added in MdePkg. Update DEC and DSC version to reflect those changes in MdePkg. Contributed-under: TianoCore Contribution Agreement 1.1 Signed-off-by: Liming Gao --- MdePkg/MdePkg.dec | 4 ++-- MdePkg/MdePkg.dsc | 4 ++-- 2 files changed, 4 insertions(+), 4 deletions

[edk2] [Patch] IntelFsp2WrapperPkg: Update BaseFspWrapperApiLib to pass XCODE5 build

2018-01-17 Thread Liming Gao
XCODE5 doesn't support absolute addressing in the assembly code. This change uses lea instruction to get the address. Contributed-under: TianoCore Contribution Agreement 1.1 Signed-off-by: Liming Gao Cc: Jiewen Yao --- .../Library/BaseFspWrapperApiLib/X64/Thunk64To32.nasm

[edk2] [PATCH v2 5/7] UefiCpuPkg: Update SmmCpuFeatureLib pass XCODE5 tool chain

2018-01-11 Thread Liming Gao
TianoCore Contribution Agreement 1.1 Signed-off-by: Liming Gao Cc: Andrew Fish Cc: Jiewen Yao Cc: Eric Dong Cc: Laszlo Ersek Cc: Michael Kinney --- .../Library/SmmCpuFeaturesLib/Ia32/SmiEntry.nasm | 6 +++- UefiCpuPkg/Library/SmmCpuFeaturesLib/SmmStm.c | 8 +++-- UefiCpuPkg/Library/SmmCpuF

[edk2] [PATCH v2 6/7] UefiCpuPkg: Update PiSmmCpuDxeSmm pass XCODE5 tool chain

2018-01-11 Thread Liming Gao
changeInfo to record InitializeFloatingPointUnits. This way is same to MpInitLib. Contributed-under: TianoCore Contribution Agreement 1.1 Signed-off-by: Liming Gao Cc: Andrew Fish Cc: Jiewen Yao Cc: Eric Dong Cc: Laszlo Ersek Cc: Michael Kinney --- UefiCpuPkg/PiSmmCpuDxeSmm/CpuS3.c

[edk2] [PATCH v2 4/7] UefiCpuPkg: Update CpuExceptionHandlerLib pass XCODE5 tool chain

2018-01-11 Thread Liming Gao
In V2, use mov rax, strict qword 0 to replace the hard code db. Use the dummy address as jmp destination, and add the logic to fix up the address to the absolute address at boot time. Contributed-under: TianoCore Contribution Agreement 1.1 Signed-off-by: Liming Gao Cc: Andrew Fish Cc: Jiewen

[edk2] [Patch] Edk2: Retire Edk2Setup.bat

2018-01-11 Thread Liming Gao
Edk2Setup.bat is out of maintain. It is a draft version batch file, and not implement all features listed in its header file. To avoid the confuse, drop it from edk2 project, and keep edksetup.bat only. Contributed-under: TianoCore Contribution Agreement 1.1 Signed-off-by: Liming Gao

[edk2] [Patch 7/7] OvmfPkg: Don't add -mno-mmx -mno-sse option for XCODE5 tool chain

2018-01-10 Thread Liming Gao
issue. To work around it, don't add these two options for XCODE5 tool chain. Contributed-under: TianoCore Contribution Agreement 1.1 Signed-off-by: Liming Gao Cc: Andrew Fish Cc: Laszlo Ersek --- OvmfPkg/OvmfPkgIa32.dsc| 4 +++- OvmfPkg/OvmfPkgIa32X64.dsc | 4 +++- OvmfPkg/OvmfPkgX6

[edk2] [Patch 6/7] UefiCpuPkg: Update PiSmmCpuDxeSmm pass XCODE5 tool chain

2018-01-10 Thread Liming Gao
MpInitLib. Contributed-under: TianoCore Contribution Agreement 1.1 Signed-off-by: Liming Gao Cc: Andrew Fish Cc: Jiewen Yao Cc: Eric Dong Cc: Laszlo Ersek Cc: Michael Kinney --- UefiCpuPkg/PiSmmCpuDxeSmm/CpuS3.c | 6 - UefiCpuPkg/PiSmmCpuDxeSmm/Ia32/SmiEntry.nasm| 5

[edk2] [Patch 0/7] EDK2: Enable XCODE5 tool chain with NASM source

2018-01-10 Thread Liming Gao
. Known limitation is XCODE5 doesn't support HII resource section generation. So, new UEFI shell application tftp can't be used. Old shell is used. Liming Gao (7): BaseTools: Disable -Wno-unused-const-variable in XCODE5 RELEASE target BaseTools: Use nasm as the preferred assem

[edk2] [Patch 5/7] UefiCpuPkg: Update SmmCpuFeatureLib pass XCODE5 tool chain

2018-01-10 Thread Liming Gao
1. Use lea instruction to get the address instead of mov instruction. 2. Use the dummy address as jmp destination, and add the logic to fix up the address to the absolute address at boot time. Contributed-under: TianoCore Contribution Agreement 1.1 Signed-off-by: Liming Gao Cc: Andrew Fish Cc

[edk2] [Patch 4/7] UefiCpuPkg: Update CpuExceptionHandlerLib pass XCODE5 tool chain

2018-01-10 Thread Liming Gao
Use the dummy address as jmp destination, and add the logic to fix up the address to the absolute address at boot time. Contributed-under: TianoCore Contribution Agreement 1.1 Signed-off-by: Liming Gao Cc: Andrew Fish Cc: Jiewen Yao Cc: Eric Dong Cc: Laszlo Ersek Cc: Michael Kinney

[edk2] [Patch 2/7] BaseTools: Use nasm as the preferred assembly source files for XCODE5 tool

2018-01-10 Thread Liming Gao
Contributed-under: TianoCore Contribution Agreement 1.1 Signed-off-by: Liming Gao Cc: Andrew Fish Cc: Yonghong Zhu --- BaseTools/Conf/tools_def.template | 1 - 1 file changed, 1 deletion(-) diff --git a/BaseTools/Conf/tools_def.template b/BaseTools/Conf/tools_def.template index a961048288

[edk2] [Patch 3/7] MdeModulePkg: Update DebugSupportDxe to pass XCODE5 build

2018-01-10 Thread Liming Gao
XCODE5 doesn't support absolute addressing in the assembly code. This change uses lea instruction to get the address. Contributed-under: TianoCore Contribution Agreement 1.1 Signed-off-by: Liming Gao Cc: Andrew Fish Cc: Star Zeng --- MdeModulePkg/Universal/DebugSupportDxe/X64/AsmFuncs

[edk2] [Patch 1/7] BaseTools: Disable -Wno-unused-const-variable in XCODE5 RELEASE target

2018-01-10 Thread Liming Gao
Contributed-under: TianoCore Contribution Agreement 1.1 Signed-off-by: Liming Gao Cc: Yonghong Zhu Cc: Andrew Fish --- BaseTools/Conf/tools_def.template | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/BaseTools/Conf/tools_def.template b/BaseTools/Conf

[edk2] [Patch] MdeModulePkg DxeIpl: remove the hard code alignment adjustment in Decompress()

2018-01-09 Thread Liming Gao
Section data alignment should be made in the build generation. Contributed-under: TianoCore Contribution Agreement 1.1 Signed-off-by: Liming Gao Cc: Star Zeng --- MdeModulePkg/Core/DxeIplPeim/DxeLoad.c | 16 +++- 1 file changed, 3 insertions(+), 13 deletions(-) diff --git a

[edk2] [Patch 0/2] MdeModulePkg: Remove the hard code alignment adjustment.

2018-01-09 Thread Liming Gao
Liming Gao (2): MdeModulePkg DxeIpl: remove the hard code alignment adjustment. MdeModulePkg SectionExtractionPei: remove the hard code alignment adjustment MdeModulePkg/Core/DxeIplPeim/DxeLoad.c | 9 ++--- .../Universal/SectionExtractionPei

[edk2] [Patch 2/2] MdeModulePkg SectionExtractionPei: remove the hard code alignment adjustment

2018-01-09 Thread Liming Gao
Section data alignment should be made in the build generation. Contributed-under: TianoCore Contribution Agreement 1.1 Signed-off-by: Liming Gao Cc: Star Zeng --- .../Universal/SectionExtractionPei/SectionExtractionPei.c| 9 ++--- 1 file changed, 2 insertions(+), 7 deletions

[edk2] [Patch 1/2] MdeModulePkg DxeIpl: remove the hard code alignment adjustment.

2018-01-09 Thread Liming Gao
Section data alignment should be made in the build generation. Contributed-under: TianoCore Contribution Agreement 1.1 Signed-off-by: Liming Gao Cc: Star Zeng --- MdeModulePkg/Core/DxeIplPeim/DxeLoad.c | 9 ++--- 1 file changed, 2 insertions(+), 7 deletions(-) diff --git a/MdeModulePkg

[edk2] [Patch] BaseTools CommonLib: Fix printf %llx issue on UINT64

2018-01-02 Thread Liming Gao
UINT64 is defined as the different type for the different ARCHs. To let it work for all archs and compilers, add (unsigned long long) for the input value together with %llx. Contributed-under: TianoCore Contribution Agreement 1.1 Signed-off-by: Liming Gao --- BaseTools/Source/C/Common

[edk2] [Patch] MdeModulePkg DxePrintLibPrint2Protocol: Fix GCC5 build failure

2017-12-28 Thread Liming Gao
Contributed-under: TianoCore Contribution Agreement 1.1 Signed-off-by: Liming Gao Cc: Eric Dong --- MdeModulePkg/Library/DxePrintLibPrint2Protocol/PrintLib.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/MdeModulePkg/Library/DxePrintLibPrint2Protocol/PrintLib.c b

[edk2] [Patch] MdeModulePkg PeiCore: Update debug message to print the correct FV length

2017-12-27 Thread Liming Gao
Contributed-under: TianoCore Contribution Agreement 1.1 Signed-off-by: Liming Gao Cc: Star Zeng --- MdeModulePkg/Core/Pei/FwVol/FwVol.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/MdeModulePkg/Core/Pei/FwVol/FwVol.c b/MdeModulePkg/Core/Pei/FwVol/FwVol.c index 7cb295c

[edk2] [Patch] MdeModulePkg HiiDataBase: Fix the potential NULL pointer access

2017-12-27 Thread Liming Gao
Contributed-under: TianoCore Contribution Agreement 1.1 Signed-off-by: Liming Gao Cc: Star Zeng --- MdeModulePkg/Universal/HiiDatabaseDxe/Database.c | 11 --- 1 file changed, 8 insertions(+), 3 deletions(-) diff --git a/MdeModulePkg/Universal/HiiDatabaseDxe/Database.c b/MdeModulePkg

[edk2] [Patch] SecurityPkg: Convert source file to DOS format

2017-12-27 Thread Liming Gao
Contributed-under: TianoCore Contribution Agreement 1.1 Signed-off-by: Liming Gao Cc: Qin Long --- .../Include/Ppi/FirmwareVolumeInfoPrehashedFV.h| 148 ++--- 1 file changed, 74 insertions(+), 74 deletions(-) diff --git a/SecurityPkg/Include/Ppi

[edk2] [Patch 4/4] SecurityPkg: Convert source file to DOS format

2017-12-27 Thread Liming Gao
Contributed-under: TianoCore Contribution Agreement 1.1 Signed-off-by: Liming Gao Cc: Qin Long --- .../SecureBootConfigDxe/SecureBootConfigImpl.c | 26 +++--- 1 file changed, 13 insertions(+), 13 deletions(-) diff --git a/SecurityPkg/VariableAuthenticated

[edk2] [Patch 3/4] SourceLevelDebugPkg: Convert source file to DOS format

2017-12-27 Thread Liming Gao
Contributed-under: TianoCore Contribution Agreement 1.1 Signed-off-by: Liming Gao Cc: Wu Hao A --- .../Library/DebugAgent/SecPeiDebugAgent/SecPeiDebugAgentLib.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/SourceLevelDebugPkg/Library/DebugAgent

[edk2] [Patch 2/4] NetworkPkg: Convert source file to DOS format

2017-12-27 Thread Liming Gao
Contributed-under: TianoCore Contribution Agreement 1.1 Signed-off-by: Liming Gao Cc: Wu Jiaxin --- NetworkPkg/HttpDxe/HttpProto.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/NetworkPkg/HttpDxe/HttpProto.c b/NetworkPkg/HttpDxe/HttpProto.c index 925281a9c0..d7fe271168

[edk2] [Patch 1/4] IntelFspWrapperPkg: Convert INF file to dos format

2017-12-27 Thread Liming Gao
Contributed-under: TianoCore Contribution Agreement 1.1 Signed-off-by: Liming Gao Cc: Jiewen Yao --- .../Library/PeiFspWrapperApiTestLib/PeiFspWrapperApiTestLib.inf | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/IntelFsp2WrapperPkg/Library/PeiFspWrapperApiTestLib

[edk2] [Patch 0/4] Update source file as DOS format

2017-12-27 Thread Liming Gao
Liming Gao (4): IntelFspWrapperPkg: Convert INF file to dos format NetworkPkg: Convert source file to DOS format SourceLevelDebugPkg: Convert source file to DOS format SecurityPkg: Convert source file to DOS format .../PeiFspWrapperApiTestLib.inf| 2 +- NetworkPkg

[edk2] [Patch] MdePkg: Update PI spec version to 1.6 in MdePkg.dec

2017-12-27 Thread Liming Gao
Contributed-under: TianoCore Contribution Agreement 1.1 Signed-off-by: Liming Gao --- MdePkg/MdePkg.dec | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/MdePkg/MdePkg.dec b/MdePkg/MdePkg.dec index 603e498..3c61a8f 100644 --- a/MdePkg/MdePkg.dec +++ b/MdePkg/MdePkg.dec @@ -2,7

[edk2] [Patch] MdeModulePkg Variable: Update expression to directly use boolean type value

2017-12-26 Thread Liming Gao
Contributed-under: TianoCore Contribution Agreement 1.1 Signed-off-by: Liming Gao --- MdeModulePkg/Universal/Variable/RuntimeDxe/Variable.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/MdeModulePkg/Universal/Variable/RuntimeDxe/Variable.c b/MdeModulePkg/Universal/Variable

[edk2] [Patch] MdeModulePkg HiiDataBase: Fix the potential NULL pointer reference

2017-12-26 Thread Liming Gao
Contributed-under: TianoCore Contribution Agreement 1.1 Signed-off-by: Liming Gao Cc: Star Zeng --- MdeModulePkg/Universal/HiiDatabaseDxe/Database.c | 10 ++ 1 file changed, 6 insertions(+), 4 deletions(-) diff --git a/MdeModulePkg/Universal/HiiDatabaseDxe/Database.c b/MdeModulePkg

[edk2] [Patch] BaseTools: Update Python Makefile to include the new added python files

2017-12-26 Thread Liming Gao
Contributed-under: TianoCore Contribution Agreement 1.1 Signed-off-by: Liming Gao Cc: Yonghong Zhu Cc: Bob Feng --- BaseTools/Source/Python/Makefile | 4 1 file changed, 4 insertions(+) diff --git a/BaseTools/Source/Python/Makefile b/BaseTools/Source/Python/Makefile index 7ffe211

[edk2] [PATCH v2] MdeModulePkg Variable: Update Pointer Minus with UINTN type

2017-12-26 Thread Liming Gao
In V2, reduce unnecessary () in the expression. Contributed-under: TianoCore Contribution Agreement 1.1 Signed-off-by: Liming Gao Cc: Star Zeng --- MdeModulePkg/Universal/Variable/RuntimeDxe/Variable.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/MdeModulePkg/Universal

[edk2] [Patch] MdeModulePkg Variable: Update Pointer Minus with UINTN type

2017-12-26 Thread Liming Gao
Contributed-under: TianoCore Contribution Agreement 1.1 Signed-off-by: Liming Gao Cc: Star Zeng --- MdeModulePkg/Universal/Variable/RuntimeDxe/Variable.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/MdeModulePkg/Universal/Variable/RuntimeDxe/Variable.c b/MdeModulePkg

[edk2] [Patch] MdeModulePkg HiiDataBaseDxe: Add the check for the memory allocation return

2017-12-26 Thread Liming Gao
Contributed-under: TianoCore Contribution Agreement 1.1 Signed-off-by: Liming Gao Cc: Star Zeng --- MdeModulePkg/Universal/HiiDatabaseDxe/Database.c | 9 + 1 file changed, 9 insertions(+) diff --git a/MdeModulePkg/Universal/HiiDatabaseDxe/Database.c b/MdeModulePkg/Universal

[edk2] [Patch] BaseTools: Update SkuId checker to make sure it be valid UINT64 value

2017-12-25 Thread Liming Gao
Contributed-under: TianoCore Contribution Agreement 1.1 Signed-off-by: Liming Gao Cc: Yonghong Zhu Cc: Signed-off-by: Bob Feng --- BaseTools/Source/Python/Common/Misc.py | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/BaseTools/Source/Python/Common/Misc.py b

[edk2] [Patch] MdeModulePkg HiiDataBaseDxe: Initialize the local variable before reference

2017-12-25 Thread Liming Gao
Make VS2012x86 and GCC48 pass build Contributed-under: TianoCore Contribution Agreement 1.1 Signed-off-by: Liming Gao --- MdeModulePkg/Universal/HiiDatabaseDxe/Database.c | 3 +++ 1 file changed, 3 insertions(+) diff --git a/MdeModulePkg/Universal/HiiDatabaseDxe/Database.c b/MdeModulePkg

[edk2] [Patch] BaseTools: Disable warning varargs in XCODE5 align to CLANG38

2017-12-08 Thread Liming Gao
https://bugzilla.tianocore.org/show_bug.cgi?id=741 Contributed-under: TianoCore Contribution Agreement 1.1 Signed-off-by: Liming Gao --- BaseTools/Conf/tools_def.template | 12 ++-- 1 file changed, 6 insertions(+), 6 deletions(-) diff --git a/BaseTools/Conf/tools_def.template b

[edk2] [Patch] BaseTools: Update BrotliCompress script to handle the different input format

2017-12-07 Thread Liming Gao
After this update, BrotliCompress can support below styles. BrotliCompress -e InputFile -o OutputFile BrotliCompress -e -o OutputFile InputFile Contributed-under: TianoCore Contribution Agreement 1.1 Signed-off-by: Liming Gao --- BaseTools/BinWrappers/PosixLike/BrotliCompress | 61

[edk2] [RFC] Update EDKII Performance infrastructure based on ACPI FPDT table

2017-12-07 Thread Liming Gao
nto FPDT FBBP Table. If so, OS utility can parse FPDT FBBP table to get S3 boot performance data. 5. Update PerformancePkg Dp_App to parse ACPI FPDT table and get FBBP Table, then retrieve edkii string event records, convert them into the performance entry log and dump them as current outpu

[edk2] [Patch] SourceLevelDebugPkg/SecPeiDebugAgentLib: Fix duplicate symbol

2017-12-06 Thread Liming Gao
From: Michael Kinney https://bugzilla.tianocore.org/show_bug.cgi?id=573 https://bugzilla.tianocore.org/show_bug.cgi?id=796 The same issue is reported again by GCC. Resend this patch again. This patch renames the duplicated function name to fix it. The SecPeiDebugAgentLib uses the global variabl

[edk2] [Patch] UefiCpuPkg: SmmCpuFeaturesLib Add the missing ASM_PFX in nasm code

2017-12-06 Thread Liming Gao
Contributed-under: TianoCore Contribution Agreement 1.1 Signed-off-by: Liming Gao Cc: Eric Dong --- UefiCpuPkg/Library/SmmCpuFeaturesLib/X64/SmiEntry.nasm | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/UefiCpuPkg/Library/SmmCpuFeaturesLib/X64/SmiEntry.nasm b

[edk2] [Patch] UefiCpuPkg: PiSmmCpuDxeSmm Add the missing ASM_PFX in nasm code

2017-12-06 Thread Liming Gao
Contributed-under: TianoCore Contribution Agreement 1.1 Signed-off-by: Liming Gao Cc: Eric Dong --- UefiCpuPkg/PiSmmCpuDxeSmm/X64/SmiEntry.nasm | 10 +- 1 file changed, 5 insertions(+), 5 deletions(-) diff --git a/UefiCpuPkg/PiSmmCpuDxeSmm/X64/SmiEntry.nasm b/UefiCpuPkg/PiSmmCpuDxeSmm

[edk2] [Patch] UefiCpuPkg: Update SecCore to get BFV size based on BFV header FvLength

2017-12-06 Thread Liming Gao
Contributed-under: TianoCore Contribution Agreement 1.1 Signed-off-by: Liming Gao Cc: Eric Dong Cc: Star Zeng --- UefiCpuPkg/SecCore/SecMain.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/UefiCpuPkg/SecCore/SecMain.c b/UefiCpuPkg/SecCore/SecMain.c index 173bbfcfcb

[edk2] [Patch 00/14] Enable Structure PCD support in edk2

2017-12-01 Thread Liming Gao
shown in build report. 8) Multiple SKU PCD values is shown in build report. Liming Gao (14): BaseTools: Update Makefile to work at absolute path BaseTools: Add PcdValueCommon logic into C source CommonLib MdeModulePkg: Update PCD driver to support Dynamic PcdVpdBaseAddress MdeModulePkg

<    1   2   3   4   5   6   >