[edk2] [Patch] BaseTools: Replace ARCH with HOST_ARCH in C Makefile to avoid conflict

2017-11-26 Thread Liming Gao
https://bugzilla.tianocore.org/show_bug.cgi?id=793 ARCH is too generic. It may cause confuse of target arch or host arch. To be clarified, replace it with HOST_ARCH in BaseTools C Makefile. Contributed-under: TianoCore Contribution Agreement 1.1 Signed-off-by: Liming Gao <liming@intel.

[edk2] [Patch 0/2] BaseTools: Update C tools GNUMakefile to support parallel make

2017-11-23 Thread Liming Gao
GNU make supports -j N to trig multiple job at the same time. It can reduce make time. This serial can get 60% improvement on BaseTools C tool clean build with make -j 4. Liming Gao (2): BaseTools: Update C tools top GNUMakefile with the clear dependency BaseTools: Update C tools top

[edk2] [Patch 1/2] BaseTools: Update C tools top GNUMakefile with the clear dependency

2017-11-23 Thread Liming Gao
Contribution Agreement 1.1 Signed-off-by: Liming Gao <liming@intel.com> --- BaseTools/Source/C/GNUmakefile | 11 +-- 1 file changed, 9 insertions(+), 2 deletions(-) diff --git a/BaseTools/Source/C/GNUmakefile b/BaseTools/Source/C/GNUmakefile index 53ddb67439..d067c87081

[edk2] [Patch 2/2] BaseTools: Update C tools top GNUMakefile to adjust tool order

2017-11-23 Thread Liming Gao
Place the tool that takes much build time at the first. This can improve build performance when make -j N used. Contributed-under: TianoCore Contribution Agreement 1.1 Signed-off-by: Liming Gao <liming@intel.com> --- BaseTools/Source/C/GNUmakefile | 6 +++--- 1 file changed, 3 inse

[edk2] [Patch 0/3] Add new PCD PcdInitValueInTempStack

2017-11-01 Thread Liming Gao
This PCD repalces the hard code 0x5AA55AA5 in the different modules. Liming Gao (3): MdeModulePkg: Add new PCD PcdInitValueInTempStack MdeModulePkg: Update PeiCore consumes PCD to get the init value in temp stack Nt32Pkg: Update SecMain consumes PCD to get the init value in temp

[edk2] [Patch 1/3] MdeModulePkg: Add new PCD PcdInitValueInTempStack

2017-11-01 Thread Liming Gao
https://bugzilla.tianocore.org/show_bug.cgi?id=740 Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Liming Gao <liming@intel.com> --- MdeModulePkg/MdeModulePkg.dec | 6 ++ 1 file changed, 6 insertions(+) diff --git a/MdeModulePkg/MdeModulePkg.dec b/MdeMod

[edk2] [Patch 2/3] MdeModulePkg: Update PeiCore consumes PCD to get the init value in temp stack

2017-11-01 Thread Liming Gao
Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Liming Gao <liming@intel.com> --- MdeModulePkg/Core/Pei/Dispatcher/Dispatcher.c | 7 +-- MdeModulePkg/Core/Pei/PeiMain.inf | 1 + 2 files changed, 2 insertions(+), 6 deletions(-) diff --git a/MdeMod

[edk2] [Patch 3/3] Nt32Pkg: Update SecMain consumes PCD to get the init value in temp stack

2017-11-01 Thread Liming Gao
Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Liming Gao <liming@intel.com> --- Nt32Pkg/Sec/SecMain.c | 2 +- Nt32Pkg/Sec/SecMain.inf | 1 + 2 files changed, 2 insertions(+), 1 deletion(-) diff --git a/Nt32Pkg/Sec/SecMain.c b/Nt32Pkg/Sec/SecMain.c index 8053

[edk2] [Patch 2/4] BaseTools: Add VS2017 tool chain in BaseTools tools_def.template

2017-10-18 Thread Liming Gao
VS2017 tool chain enables /WHOLEARCHIVE linker option Contributed-under: TianoCore Contribution Agreement 1.1 Signed-off-by: Liming Gao <liming@intel.com> --- BaseTools/Conf/tools_def.template | 122 ++ 1 file changed, 122 insertions(+) diff

[edk2] [Patch 4/4] Nt32Pkg: Add VS2017 support in SecMain

2017-10-18 Thread Liming Gao
Contributed-under: TianoCore Contribution Agreement 1.1 Signed-off-by: Liming Gao <liming@intel.com> --- Nt32Pkg/Sec/SecMain.inf | 2 ++ 1 file changed, 2 insertions(+) diff --git a/Nt32Pkg/Sec/SecMain.inf b/Nt32Pkg/Sec/SecMain.inf index 423a3c7..48670cf 100644 --- a/Nt32Pkg/Sec/SecMa

[edk2] [Patch 1/4] MdePkg: Disable VS warning 4701 & 4703 for VS2017

2017-10-18 Thread Liming Gao
Contributed-under: TianoCore Contribution Agreement 1.1 Signed-off-by: Liming Gao <liming@intel.com> --- MdePkg/Include/Ia32/ProcessorBind.h | 4 ++-- MdePkg/Include/X64/ProcessorBind.h | 4 ++-- 2 files changed, 4 insertions(+), 4 deletions(-) diff --git a/MdePkg/Includ

[edk2] [Patch 3/4] BaseTools: Update VS batch file to auto detect VS2017

2017-10-18 Thread Liming Gao
This way depends on VS vswhere.exe to find VS2017 installed directory. vswhere.exe starts in Visual Studio 2017 version 15.2. Contributed-under: TianoCore Contribution Agreement 1.1 Signed-off-by: Liming Gao <liming@intel.com> --- BaseTools/get_vsvars.bat| 8 Bas

[edk2] [Patch 0/4] Add VS2017 tool chain for evaluation

2017-10-18 Thread Liming Gao
, VS2017xASL and VS2017x86xASL tool chain will be provided. And, to avoid more duplicated informations be introduced, the proposal to simplify tools_def.template will be provided. Liming Gao (4): MdePkg: Disable VS warning 4701 & 4703 for VS2017 BaseTools: Add VS2017 tool chain in BaseT

[edk2] [Patch] MdeModulePkg: Update RuntimeDxe Crc32 to check the input parameter

2017-10-11 Thread Liming Gao
This is the regression issue. After apply CalculateCrc32(), the parameter check is missing. Contributed-under: TianoCore Contribution Agreement 1.1 Signed-off-by: Liming Gao <liming@intel.com> Cc: Wu Hao A <hao.a...@intel.com> --- MdeModulePkg/Core/RuntimeDxe/Crc32.c | 4 1

[edk2] [Patch] SourceLevelDebugPkg: Update SmmDebugAgentLib to restore APIC timer

2017-10-10 Thread Liming Gao
In enter SMI, APIC timer may be initialized. After exit SMI, APIC timer will be restore. Contributed-under: TianoCore Contribution Agreement 1.1 Signed-off-by: Liming Gao <liming@intel.com> Cc: Ruiyu Ni <ruiyu...@intel.com> --- .../DebugAgent/SmmDebugAgent/SmmDebugAgentLi

[edk2] [Patch 11/11] MdeModulePkg: Update Ppi/Protocol/Guid usage in INF files

2017-09-25 Thread Liming Gao
Contributed-under: TianoCore Contribution Agreement 1.1 Signed-off-by: Liming Gao <liming@intel.com> Cc: Star Zeng <star.z...@intel.com> Cc: Eric Dong <eric.d...@intel.com> --- MdeModulePkg/Core/Dxe/DxeMain.inf | 2 +- MdeModulePkg/Core/PiSmm

[edk2] [Patch 09/11] IntelFsp2Pkg: Update Section Name in INF files

2017-09-25 Thread Liming Gao
Contributed-under: TianoCore Contribution Agreement 1.1 Signed-off-by: Liming Gao <liming@intel.com> Cc: Jiewen Yao <jiewen@intel.com> --- IntelFsp2Pkg/FspNotifyPhase/FspNotifyPhasePeim.inf | 3 +-- .../Library/BaseCacheAsRamLibNull/BaseCacheAsRamLibNull.inf

[edk2] [Patch 10/11] IntelFsp2WrapperPkg: Update Protocol/Guid usage in INF files

2017-09-25 Thread Liming Gao
Contributed-under: TianoCore Contribution Agreement 1.1 Signed-off-by: Liming Gao <liming@intel.com> Cc: Jiewen Yao <jiewen@intel.com> --- IntelFsp2WrapperPkg/FspWrapperNotifyDxe/FspWrapperNotifyDxe.inf | 6 +++--- IntelFsp2WrapperPkg/FspmWrapperPeim/FspmWrapperPeim.inf

[edk2] [Patch 08/11] SecurityPkg: Update Guid usage in INF file to match source code logic

2017-09-25 Thread Liming Gao
Contributed-under: TianoCore Contribution Agreement 1.1 Signed-off-by: Liming Gao <liming@intel.com> Cc: Chao Zhang <chao.b.zh...@intel.com> --- .../FmpAuthenticationLibRsa2048Sha256.inf | 6 +++--- .../Library/HashLibBaseCryptoRouter/HashLibBaseCryptoR

[edk2] [Patch 07/11] SignedCapsulePkg: Update Guid usage in INF file to match source code logic

2017-09-25 Thread Liming Gao
Contributed-under: TianoCore Contribution Agreement 1.1 Signed-off-by: Liming Gao <liming@intel.com> Cc: Jiewen Yao <jiewen@intel.com> --- .../Library/EdkiiSystemCapsuleLib/EdkiiSystemCapsuleLib.inf | 12 ++-- 1 file changed, 6 insertions(+), 6 deletions(-)

[edk2] [Patch 06/11] NetworkPkg: Update Protocol/Guid usage in INF file to match source code logic

2017-09-25 Thread Liming Gao
Contributed-under: TianoCore Contribution Agreement 1.1 Signed-off-by: Liming Gao <liming@intel.com> Cc: Siyuan Fu <siyuan...@intel.com> Cc: Jiaxin Wu <jiaxin...@intel.com> --- NetworkPkg/HttpDxe/HttpDxe.inf | 2 +- NetworkPkg/IScsiDxe/IScsiDxe.inf

[edk2] [Patch 04/11] UefiCpuPkg: Update Guid usage in DxeMpInitLib INF to match source code logic

2017-09-25 Thread Liming Gao
Contributed-under: TianoCore Contribution Agreement 1.1 Signed-off-by: Liming Gao <liming@intel.com> Cc: Eric Dong <eric.d...@intel.com> --- UefiCpuPkg/Library/MpInitLib/DxeMpInitLib.inf | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/UefiCpuPkg/Libra

[edk2] [Patch 05/11] ShellPkg: Update Guid usage in ShellLib INF to match source code logic

2017-09-25 Thread Liming Gao
Contributed-under: TianoCore Contribution Agreement 1.1 Signed-off-by: Liming Gao <liming@intel.com> Cc: Jaben Carsey <jaben.car...@intel.com> Cc: Ruiyu Ni <ruiyu...@intel.com> --- ShellPkg/Library/UefiHandleParsingLib/UefiHandleParsingLib.inf | 2 +- 1 file changed,

[edk2] [Patch 00/11] Correct Ppi/Protocol/Guid usage in module INF files

2017-09-25 Thread Liming Gao
Update Ppi/Protocol/Guid usage described in INF file to match its source logic. Update Module base name in INF file to match its uni file. Update Section name in INF file to follow INF spec. Liming Gao (11): MdePkg: Correct BaseS3PciSegmentLib module name to match its uni file FatPkg: Update

[edk2] [Patch 02/11] FatPkg: Update Ppi/Protocol usage in Fat INF to match source code logic

2017-09-25 Thread Liming Gao
Contributed-under: TianoCore Contribution Agreement 1.1 Signed-off-by: Liming Gao <liming@intel.com> Cc: Ruiyu Ni <ruiyu...@intel.com> --- FatPkg/EnhancedFatDxe/Fat.inf | 2 +- FatPkg/FatPei/FatPei.inf | 10 +++--- 2 files changed, 8 insertions(+), 4 deletions(-)

[edk2] [Patch 01/11] MdePkg: Correct BaseS3PciSegmentLib module name to match its uni file

2017-09-25 Thread Liming Gao
Contributed-under: TianoCore Contribution Agreement 1.1 Signed-off-by: Liming Gao <liming@intel.com> Cc: Ruiyu Ni <ruiyu...@intel.com> --- MdePkg/Library/BaseS3PciSegmentLib/BaseS3PciSegmentLib.inf | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/Md

[edk2] [Patch 03/11] PcAtChipsetPkg: Update GUID usage in PcRtc INF to match the source code

2017-09-25 Thread Liming Gao
Contributed-under: TianoCore Contribution Agreement 1.1 Signed-off-by: Liming Gao <liming@intel.com> Cc: Ruiyu Ni <ruiyu...@intel.com> --- .../PcatRealTimeClockRuntimeDxe/PcatRealTimeClockRuntimeDxe.inf | 9 +++-- 1 file changed, 7 insertions(+), 2 deletions(-)

[edk2] [Patch] ShellPkg: Update the parameter type to match function interface in DmpStore

2017-09-22 Thread Liming Gao
This fix can resolve CLANG38 build failure. Contributed-under: TianoCore Contribution Agreement 1.1 Signed-off-by: Liming Gao <liming@intel.com> Cc: Ruiyu Ni <ruiyu...@intel.com> --- ShellPkg/Library/UefiShellDebug1CommandsLib/DmpStore.c | 2 +- 1 file changed, 1 insertion(+)

[edk2] [Patch 0/2] Correct PI EfiGcdMemoryTypePersistent definition

2017-09-19 Thread Liming Gao
Liming Gao (2): MdePkg: Correct EfiGcdMemoryTypePersistent name to follow PI spec MdeModulePkg: Update DxeCore to consume PI EfiGcdMemoryTypePersistent MdeModulePkg/Core/Dxe/Gcd/Gcd.c | 4 ++-- MdeModulePkg/Core/Dxe/Mem/Page.c | 4 ++-- MdePkg/Include/Pi/PiDxeCis.h | 6 +- 3 files

[edk2] [Patch 1/2] MdePkg: Correct EfiGcdMemoryTypePersistent name to follow PI spec

2017-09-19 Thread Liming Gao
PI spec defines EfiGcdMemoryTypePersistent name, MdePkg uses EfiGcdMemoryTypePersistentMemory name. So, EfiGcdMemoryTypePersistent is added. And, EfiGcdMemoryTypePersistentMemory is kept for compatility. Contributed-under: TianoCore Contribution Agreement 1.1 Signed-off-by: Liming Gao <lim

[edk2] [Patch 2/2] MdeModulePkg: Update DxeCore to consume PI EfiGcdMemoryTypePersistent

2017-09-19 Thread Liming Gao
Contributed-under: TianoCore Contribution Agreement 1.1 Signed-off-by: Liming Gao <liming@intel.com> --- MdeModulePkg/Core/Dxe/Gcd/Gcd.c | 4 ++-- MdeModulePkg/Core/Dxe/Mem/Page.c | 4 ++-- 2 files changed, 4 insertions(+), 4 deletions(-) diff --git a/MdeModulePkg/Core/Dxe/Gcd/G

[edk2] [Patch 0/2] Add CalculateCrc32() API in MdePkg BaseLib

2017-09-07 Thread Liming Gao
Add CalculateCrc32() as the public API for code sharing. Liming Gao (2): MdePkg BaseLib: Add new API CalculateCrc32() MdeModulePkg: Update modules to consume CalculateCrc32() MdeModulePkg/Core/RuntimeDxe/Crc32.c | 74 +- MdeModulePkg/Core/RuntimeDxe/Runtime.c

[edk2] [Patch 1/2] MdePkg BaseLib: Add new API CalculateCrc32()

2017-09-07 Thread Liming Gao
-by: Liming Gao <liming@intel.com> --- MdePkg/Include/Library/BaseLib.h | 18 +++ MdePkg/Library/BaseLib/CheckSum.c | 294 ++ 2 files changed, 312 insertions(+) diff --git a/MdePkg/Include/Library/BaseLib.h b/MdePkg/Include/Library/BaseLib.h index 4

[edk2] [Patch 2/2] MdeModulePkg: Update modules to consume CalculateCrc32()

2017-09-07 Thread Liming Gao
Contributed-under: TianoCore Contribution Agreement 1.1 Signed-off-by: Liming Gao <liming@intel.com> --- MdeModulePkg/Core/RuntimeDxe/Crc32.c | 74 +- MdeModulePkg/Core/RuntimeDxe/Runtime.c | 5 - MdeModulePkg/Core/RuntimeDxe/Run

[edk2] [Patch] MdeModulePkg: Update PiDxeS3BootScriptLib Internal function name

2017-09-07 Thread Liming Gao
To avoid the function name conflict, update the internal function name to be the specific one. Contributed-under: TianoCore Contribution Agreement 1.1 Signed-off-by: Liming Gao <liming@intel.com> --- MdeModulePkg/Library/PiDxeS3BootScriptLib/BootScriptExecute.c | 4 ++-- 1 file chan

[edk2] [Patch] Nt32Pkg: Clean up DSC to remove unnecessary build option in SecMain

2017-09-05 Thread Liming Gao
Contributed-under: TianoCore Contribution Agreement 1.1 Signed-off-by: Liming Gao <liming@intel.com> Cc: Ruiyu Ni <ruiyu...@intel.com> --- Nt32Pkg/Nt32Pkg.dsc | 6 +- 1 file changed, 1 insertion(+), 5 deletions(-) diff --git a/Nt32Pkg/Nt32Pkg.dsc b/Nt32Pkg/Nt32Pkg.dsc i

[edk2] [Patch] BaseTools: Update toolsetup.bat to support the case without EDK_TOOLS_BIN

2017-08-25 Thread Liming Gao
Contribution Agreement 1.1 Signed-off-by: Liming Gao <liming@intel.com> Cc: Yonghong Zhu <yonghong@intel.com> --- BaseTools/toolsetup.bat | 15 +-- 1 file changed, 5 insertions(+), 10 deletions(-) diff --git a/BaseTools/toolsetup.bat b/BaseTools/toolsetup.bat index 3801

[edk2] [Patch] BaseTools: Enable --whole-archive in GCC tool chain as the default option

2017-08-24 Thread Liming Gao
https://bugzilla.tianocore.org/show_bug.cgi?id=581 Contributed-under: TianoCore Contribution Agreement 1.1 Signed-off-by: Liming Gao <liming@intel.com> Cc: Yonghong Zhu <yonghong@intel.com> --- BaseTools/Conf/tools_def.template | 4 ++-- 1 file changed, 2 insertions(+),

[edk2] [Patch] BaseTools: Support /WHOLEARCHIVE option in VS2015 tool chain

2017-08-23 Thread Liming Gao
Agreement 1.1 Signed-off-by: Liming Gao <liming@intel.com> Cc: Yonghong Zhu <yonghong@intel.com> --- BaseTools/Conf/build_rule.template | 1 + BaseTools/Conf/tools_def.template | 6 +- 2 files changed, 6 insertions(+), 1 deletion(-) diff --git a/BaseTools/Conf/build_rule

[edk2] [Patch] BaseTools: Update tools_def to remove /Gw option in VS NOOPT target

2017-08-23 Thread Liming Gao
To remove /Gw option is to disable size optimization. Contributed-under: TianoCore Contribution Agreement 1.1 Signed-off-by: Liming Gao <liming@intel.com> Cc: Yonghong Zhu <yonghong@intel.com> --- BaseTools/Conf/tools_def.template | 32 1 file

[edk2] [Patch] BaseTools: Add the missing -pie link option in GCC tool chain

2017-08-23 Thread Liming Gao
the value 1 for '-fpie' and 2 for '-fPIE'. Contributed-under: TianoCore Contribution Agreement 1.1 Signed-off-by: Liming Gao <liming@intel.com> --- BaseTools/Conf/tools_def.template | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/BaseTools/Conf/tools_def.te

[edk2] [Patch] Maintainers.txt: Update the Maintainers for IntelFrameworkXXXPkg

2017-08-16 Thread Liming Gao
Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Liming Gao <liming@intel.com> --- Maintainers.txt | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/Maintainers.txt b/Maintainers.txt index cdd9a88..6f16227 100644 --- a/Maintainers.txt

[edk2] [Patch] UefiCpuPkg: Update RegisterCpuFeaturesLib to consume PcdGetSize with UINTN

2017-07-10 Thread Liming Gao
PcdGetSize() returns UINTN data type. The consumer code should use UINTN data to get its size. This issue is found when PcdCpuFeaturesSupport is configured as patchable. Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Liming Gao <liming@intel.com> --- Uefi

[edk2] [Patch] MdeModulePkg: Update NonDiscoverableDeviceRegistrationLib file header format

2017-07-10 Thread Liming Gao
Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Liming Gao <liming@intel.com> --- .../NonDiscoverableDeviceRegistrationLib.inf | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/MdeModulePkg/L

[edk2] [Patch] UefiCpuPkg: Update RegisterCpuFeaturesLib module UNI to match it

2017-07-10 Thread Liming Gao
Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Liming Gao <liming@intel.com> --- .../DxeRegisterCpuFeaturesLib.inf | 2 +- .../PeiRegisterCpuFeaturesLib.inf | 2 +- .../RegisterCpuFeaturesDxe.uni

[edk2] [Patch] MdePkg: Declare _ReturnAddress() in Base.h for MSFT tool chain

2017-07-04 Thread Liming Gao
https://bugzilla.tianocore.org/show_bug.cgi?id=590 Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Liming Gao <liming@intel.com> Cc: Michael Kinney <michael.d.kin...@intel.com> --- MdePkg/Include/Base.h | 1 + 1 file changed, 1 insertion(+) diff --git a/Md

[edk2] [Patch] BaseTools: Update tools_def.template to remove old XCLANG and XCODE32

2017-07-02 Thread Liming Gao
Per Andrew Fish recommendation, XCLANG and XCODE32 are not used anymore. https://bugzilla.tianocore.org/show_bug.cgi?id=562 https://bugzilla.tianocore.org/show_bug.cgi?id=563 Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Liming Gao <liming@intel.com> Cc: Andre

[edk2] [Patch] BaseTools: Update GenFw to clear unused debug entry generated by VS tool chain

2017-07-02 Thread Liming Gao
https://bugzilla.tianocore.org/show_bug.cgi?id=600 Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Liming Gao <liming@intel.com> --- BaseTools/Source/C/GenFw/GenFw.c | 27 ++- 1 file changed, 22 insertions(+), 5 deletions(-) diff

[edk2] [PATCH v2 staging][BaseToolsOpt 4/4] Update Readme.MD to include multiple driver combination.

2017-06-15 Thread Liming Gao
Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Liming Gao <liming@intel.com> --- Readme.MD | 3 +++ 1 file changed, 3 insertions(+) diff --git a/Readme.MD b/Readme.MD index a436dd5..ba64853 100644 --- a/Readme.MD +++ b/Readme.MD @@ -15,6 +15,9 @@ iden

[edk2] [PATCH v2 staging][BaseToolsOpt 0/4] Enable multiple driver combination

2017-06-15 Thread Liming Gao
in BaseTools. Liming Gao (4): BaseTools: Merge multiple drivers into one for size and link performance SamplePkg: Add it to show the edk2 usage case. SamplePkg: Combine two drivers into one Update Readme.MD to include multiple driver combination. BaseTools/Source/Python/AutoGen/GenC.py

[edk2] [PATCH v2 staging][BaseToolsOpt 1/4] BaseTools: Merge multiple drivers into one for size and link performance

2017-06-15 Thread Liming Gao
combining some drivers, the compile failure may happen, because the same function name are used in the different drivers. Those drivers are required to be clean up first, then be combined. Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Liming Gao <liming@intel.

[edk2] [PATCH v2 staging][BaseToolsOpt 3/4] SamplePkg: Combine two drivers into one

2017-06-15 Thread Liming Gao
This is an example to show the driver combination in DSC. Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Liming Gao <liming@intel.com> --- SamplePkg/SamplePkg.dsc | 4 1 file changed, 4 insertions(+) diff --git a/SamplePkg/SamplePkg.dsc b/SamplePkg/SampleP

[edk2] [PATCH v2 staging][BaseToolsOpt 2/4] SamplePkg: Add it to show the edk2 usage case.

2017-06-15 Thread Liming Gao
Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Liming Gao <liming@intel.com> --- SamplePkg/SamplePkg.dec | 21 SamplePkg/SamplePkg.dsc | 260 2 files changed, 281 insertions(+) create mode 100644 Sam

[edk2] [PATCH staging][BaseToolsOpt 0/4] Enable multiple driver combination

2017-06-08 Thread Liming Gao
Combine more drivers into the single one can reduce the image size and compile link time. This patch adds this support in BaseTools. Liming Gao (4): BaseTools: Merge multiple drivers into one for size and link performance OvmfPkg: Update QemuVideo and VirtioGpuDxe to use NULL

[edk2] [PATCH staging][BaseToolsOpt 4/4] Update Readme.MD to include multiple driver combination.

2017-06-08 Thread Liming Gao
Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Liming Gao <liming@intel.com> --- Readme.MD | 1 + 1 file changed, 1 insertion(+) diff --git a/Readme.MD b/Readme.MD index a436dd5..b5646be 100644 --- a/Readme.MD +++ b/Readme.MD @@ -15,6 +15,7 @@ iden

[edk2] [PATCH staging][BaseToolsOpt 2/4] OvmfPkg: Update QemuVideo and VirtioGpuDxe to use NULL as DriverBindingHandle

2017-06-08 Thread Liming Gao
To combine two drivers into one, they can't share ImageHandle as DriverBindingHandle. So, update their code to use NULL as DriverBindingHandle. Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Liming Gao <liming@intel.com> --- OvmfPkg/QemuVideoDxe/Driver.c

[edk2] [PATCH staging][BaseToolsOpt 3/4] OvmfPkg: Combine QemuVideoDxe and VirtioGpuDxe to one driver

2017-06-08 Thread Liming Gao
This is an example to show the driver combination in Platform.dsc. Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Liming Gao <liming@intel.com> --- OvmfPkg/OvmfPkgIa32X64.dsc | 6 -- OvmfPkg/OvmfPkgIa32X64.fdf | 2 +- 2 files changed, 5 insertions(+), 3 del

[edk2] [PATCH staging][BaseToolsOpt 1/4] BaseTools: Merge multiple drivers into one for size and link performance

2017-06-08 Thread Liming Gao
up first, then be combined. Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Liming Gao <liming@intel.com> --- BaseTools/Source/Python/AutoGen/GenC.py | 24 +++- 1 file changed, 19 insertions(+), 5 deletions(-) diff --git a/BaseTools/Source/

[edk2] [Patch] MdeModulePkg: Update PiSmmCore to set correct ImageAddress into LoadedImage

2017-04-24 Thread Liming Gao
Cc: Star Zeng <star.z...@intel.com> Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Liming Gao <liming@intel.com> --- MdeModulePkg/Core/PiSmmCore/Dispatcher.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/MdeModulePkg/Co

[edk2] [Patch] MdeModulePkg PiSmmIpl: Fix the issue in LMFA feature

2017-04-24 Thread Liming Gao
SmramBase should be got from mLMFAConfigurationTable. Cc: Star Zeng <star.z...@intel.com> Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Liming Gao <liming@intel.com> --- MdeModulePkg/Core/PiSmmCore/PiSmmIpl.c | 12 ++-- 1 file changed, 6 inse

[edk2] [PATCH v2] MdeModulePkg BrotliLib: Fix the regression logic issue in loop

2017-04-13 Thread Liming Gao
In V2, change logic to avoid use mtf[-1] style to get value. Roll back to previous logic, and use point + offset to get byte value. Cc: Bell Song <binx.s...@intel.com> Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Liming Gao <liming@intel.com> ---

[edk2] [Patch] MdeModulePkg BrotliLib: Fix the regression logic issue in loop

2017-04-13 Thread Liming Gao
Roll back to previous logic, and use point + offset to get byte value. Cc: Bell Song <binx.s...@intel.com> Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Liming Gao <liming@intel.com> --- MdeModulePkg/Library/BrotliCustomDecompressLib/dec/decode.c | 10 +

[edk2] [Patch] MdeModulePkg DxeCore: Fix issue to print GUID value %g without pointer

2017-04-13 Thread Liming Gao
https://bugzilla.tianocore.org/show_bug.cgi?id=474 Cc: Star Zeng <star.z...@intel.com> Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Liming Gao <liming@intel.com> --- MdeModulePkg/Core/Dxe/Dispatcher/Dispatcher.c | 2 +- 1 file changed, 1 insertion(+)

[edk2] [RFC 3/3] TestPkg: Show Structure PCD value assignment

2017-04-11 Thread Liming Gao
Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Liming Gao <liming@intel.com> --- TestPkg/Include/Guid/Test.h| 31 ++ TestPkg/StructuredPcdValueGenerator.py | 702 + TestPkg/TestPkg.dec| 44 +++ T

[edk2] [RFC 2/3] BaseTools: Add PcdValueCommon logic into CommonLib

2017-04-11 Thread Liming Gao
Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Liming Gao <liming@intel.com> --- BaseTools/Source/C/Common/GNUmakefile | 3 +- BaseTools/Source/C/Common/Makefile | 3 +- BaseTools/Source/C/Common/PcdValueCommon.c

[edk2] [RFC 1/3] BaseTools: Update Makefile to work at absolute path

2017-04-11 Thread Liming Gao
Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Liming Gao <liming@intel.com> --- BaseTools/Source/C/Makefiles/app.makefile | 2 +- BaseTools/Source/C/Makefiles/ms.app | 4 ++-- 2 files changed, 3 insertions(+), 3 deletions(-) diff --git a/BaseTools/So

[edk2] [RFC 0/3] Structure PCD value assignment in DEC/DSC

2017-04-11 Thread Liming Gao
on TestPkg/StructuredPcdValueGenerator.py -p TestPkg/TestPkg.dec -d TestPkg/TestPkg.dsc After run StructuredPcdValueGenerator.py, Structure PCD value will output as byte array. Liming Gao (3): BaseTools: Update Makefile to work at absolute path BaseTools: Add PcdValueCommon logic into CommonLib TestPkg: Show Str

[edk2] [Patch] BaseTools: Add option in CLANG38 to disable warning unknown-warning-option

2017-04-09 Thread Liming Gao
https://bugzilla.tianocore.org/show_bug.cgi?id=466 Cc: Yonghong Zhu <yonghong@intel.com> Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Liming Gao <liming@intel.com> --- BaseTools/Conf/tools_def.template | 2 +- 1 file changed, 1 insertion(+), 1 delet

[edk2] [Patch] BaseTools: Convert BrotliCompress.bat to DOS format

2017-04-07 Thread Liming Gao
Cc: Yonghong Zhu <yonghong@intel.com> Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Liming Gao <liming@intel.com> --- BaseTools/Source/C/BrotliCompress/BrotliCompress.bat | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/BaseTo

[edk2] [Patch] CryptoPkg IntrinsicLib: Remove GCC -fno-builtin option

2017-04-06 Thread Liming Gao
GCC -fno-builtin option is added into tools_def.template. So, there is no need to set it in module INF file. Cc: Qin Long <qin.l...@intel.com> Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Liming Gao <liming@intel.com> --- CryptoPkg/Library/

[edk2] [Patch] MdeModulePkg RegularExpressionDxe: Remove GCC -fno-builtin option

2017-04-06 Thread Liming Gao
GCC -fno-builtin option is added into tools_def.template. So, there is no need to set it in module INF file. Cc: Star Zeng <star.z...@intel.com> Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Liming Gao <liming@intel.com> --- MdeModuleP

[edk2] [Patch] BaseTools: Update tools_def.template to add -fno-builtin in GCC tool chain

2017-04-05 Thread Liming Gao
r: TianoCore Contribution Agreement 1.0 Signed-off-by: Liming Gao <liming@intel.com> Signed-off-by: Laszlo Ersek <ler...@redhat.com> --- BaseTools/Conf/tools_def.template | 14 +++--- 1 file changed, 7 insertions(+), 7 deletions(-) diff --git a/BaseTools/Conf/tools_def.te

[edk2] [Patch] BaseTools: Add the missing copyrights in BrotliCompress.bat

2017-04-04 Thread Liming Gao
Cc: Yonghong Zhu <yonghong@intel.com> Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Liming Gao <liming@intel.com> --- BaseTools/Source/C/BrotliCompress/BrotliCompress.bat | 13 + 1 file changed, 13 insertions(+) diff --git a/BaseTo

[edk2] [Patch] MdeModulePkg: Remove unsupported PcdExpression usage in module INF

2017-03-30 Thread Liming Gao
Contribution Agreement 1.0 Signed-off-by: Liming Gao <liming@intel.com> --- MdeModulePkg/Application/HelloWorld/HelloWorld.inf | 6 +++--- MdeModulePkg/Core/DxeIplPeim/DxeIpl.inf | 12 ++-- .../Universal/CapsuleRuntimeDxe/CapsuleRuntimeD

[edk2] [Patch] SignedCapsulePkg: Update RecoveryModuleLoadPei to report the correct FvInfo

2017-03-28 Thread Liming Gao
Update logic to install FvInfo PPI with its file system guid. Cc: Jiewen Yao <jiewen@intel.com> Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Liming Gao <liming@intel.com> --- .../Universal/RecoveryModuleLoadPei/RecoveryModuleLoadPei.c | 4

[edk2] [Patch] MdeModulePkg UefiBootManagerLib: Correct usages of GUID and Protocol

2017-02-15 Thread Liming Gao
https://bugzilla.tianocore.org/show_bug.cgi?id=316 Cc: Ruiyu Ni <ruiyu...@intel.com> Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Liming Gao <liming@intel.com> --- .../UefiBootManagerLib/UefiBootManagerLib.inf | 54 -- 1 file

[edk2] [PATCH v2] MdeModulePkg PiSmmIpl: Update SmmCommunication comments to match code logic

2017-02-08 Thread Liming Gao
Cc: Star Zeng <star.z...@intel.com> Cc: Feng Tian <feng.t...@intel.com> Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Liming Gao <liming@intel.com> --- MdeModulePkg/Core/PiSmmCore/PiSmmIpl.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deleti

[edk2] [Patch] BaseTools: Add NOOPT target in CLANG38 tool chain

2017-02-08 Thread Liming Gao
https://bugzilla.tianocore.org/show_bug.cgi?id=310 Cc: Yonghong Zhu <yonghong@intel.com> Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Liming Gao <liming@intel.com> --- BaseTools/Conf/tools_def.template | 32 1 file

[edk2] [Patch 1/2] MdeModulePkg PeiCore: Reset PeimNeedingDispatch when its security violation

2017-02-08 Thread Liming Gao
When PEIM is security violation, its matched extraction ppi may not be installed. So, its PeimNeedingDispatch will still reset to TRUE. Cc: Star Zeng <star.z...@intel.com> Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Liming Gao <liming@intel.com> ---

[edk2] [Patch] BaseTools: Update top VS Makefile with the absolute path

2017-02-02 Thread Liming Gao
After this update, BaseTools/Makefile can be trig at the other directory. Cc: Yonghong Zhu <yonghong@intel.com> Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Liming Gao <liming@intel.com> --- BaseTools/Makefile | 18 +++--- 1 file

[edk2] [Patch] MdeModulePkg PiSmmIpl: Update SmmCommunication comments to match code logic

2017-02-02 Thread Liming Gao
Cc: Star Zeng <star.z...@intel.com> Cc: Feng Tian <feng.t...@intel.com> Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Liming Gao <liming@intel.com> --- MdeModulePkg/Core/PiSmmCore/PiSmmIpl.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deleti

[edk2] [Patch] Nt32Pkg FDF: Move StatusCode Handler run earlier in DXE phase

2017-01-20 Thread Liming Gao
Add StatusCode Handler into APRIORI DXE so that they can run early, then more DEBUG message can print. Cc: Ruiyu Ni <ruiyu...@intel.com> Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Liming Gao <liming@intel.com> --- Nt32Pkg/Nt32Pkg.fdf | 3 +++ 1 fil

[edk2] [Patch] MdeModulePkg UefiBootManagerLib: Correct Guid and Protocol usage in INF

2016-12-29 Thread Liming Gao
to SOMETIMES_CONSUMES. Cc: Ruiyu Ni <ruiyu...@intel.com> Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Liming Gao <liming@intel.com> --- .../UefiBootManagerLib/UefiBootManagerLib.inf | 28 -- 1 file changed, 16 insertions(+), 12 deleti

[edk2] [Patch 2/2] BaseTools toolsetup.bat: set PATH to include Bin Win32 directory

2016-12-28 Thread Liming Gao
Bin Win32 directory includes antlr.exe. This tool is used when compile BaseTools source code. So, set it into PATH first. Cc: Yonghong Zhu <yonghong@intel.com> Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Liming Gao <liming@intel.com> --- BaseTools/t

[edk2] [Patch] BaseTools GCC makefile: disable unused-result warning for CPP file

2016-12-15 Thread Liming Gao
From: Hao Wu <hao.a...@intel.com> This warning has been disabled for C file. To be same, it is also disabled for CPP file. Cc: Yonghong Zhu <yonghong@intel.com> Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Liming Gao <liming@intel.com> ---

[edk2] [Patch] MdeModulePkg EbcDxe: Fix CLANG38 build failure

2016-12-13 Thread Liming Gao
Change structure value assignment with CopyMem() API. Cc: Jiewen Yao <jiewen@intel.com> Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Liming Gao <liming@intel.com> --- MdeModulePkg/Universal/EbcDxe/EbcDebugger/EdbHook.c | 7 +-- 1 file changed,

[edk2] [Patch 2/2] MdeModulePkg PiSmmCore: Retrieve Smram base address from system table

2016-11-30 Thread Liming Gao
PiSmmIpl records LoadModuleAtFixAddressSmramBase in LMFAConfigurationTable. Update PiSmmCore to directly get the address from this system table. Cc: Star Zeng <star.z...@intel.com> Cc: Jiewen Yao <jiewen@intel.com> Contributed-under: TianoCore Contribution Agreement 1.0 Signed-of

[edk2] [Patch 1/2] MdeModulePkg SmmIpl: Fill Smram range for SMM driver when LMFA enable

2016-11-30 Thread Liming Gao
Allocate the additional Smram range to describe the reserved smram for SMM core and driver when LMFA feature is enabled. Cc: Star Zeng <star.z...@intel.com> Cc: Jiewen Yao <jiewen@intel.com> Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Liming

[edk2] [Patch 0/2] MdeModulePkg SmmCore: Fix hang issue when LMFA enable

2016-11-30 Thread Liming Gao
Liming Gao (2): MdeModulePkg SmmIpl: Fill Smram range for SMM driver when LMFA enable MdeModulePkg PiSmmCore: Retrieve Smram base address from system table MdeModulePkg/Core/PiSmmCore/PiSmmCore.h | 1 + MdeModulePkg/Core/PiSmmCore/PiSmmCore.inf | 2 ++ MdeModulePkg/Core/PiSmmCore

[edk2] [Patch] MdeModulePkg PiSmmCore: Update FreePages to handle zero address and pages

2016-11-30 Thread Liming Gao
https://bugzilla.tianocore.org/show_bug.cgi?id=278 Zero memory address or zero number pages are invalid to SmmFreePages(). Cc: Star Zeng <star.z...@intel.com> Cc: Jiewen Yao <jiewen@intel.com> Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Liming

[edk2] [Patch] MdeModulePkg PiSmmCore: Update comments in InitializeMemoryServices

2016-11-30 Thread Liming Gao
Add the comments to describe Free and Allocated SMRAM are added separately. Cc: Star Zeng <star.z...@intel.com> Cc: Jiewen Yao <jiewen@intel.com> Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Liming Gao <liming@intel.com> --- MdeModulePkg/Cor

[edk2] [Patch] MdeModulePkg PeiCore: Make SetPeiServicesTablePointer() early in EntryPoint

2016-11-22 Thread Liming Gao
intel.com> Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Liming Gao <liming@intel.com> --- MdeModulePkg/Core/Pei/PeiMain/PeiMain.c | 18 +- 1 file changed, 9 insertions(+), 9 deletions(-) diff --git a/MdeModulePkg/Core/Pei/PeiMain/PeiMain.c b/Mde

[edk2] [PATCH v2] MdePkg IndustryStandard: Add DDR3, DDR4 and LPDDR definition per SPD spec

2016-11-17 Thread Liming Gao
bution Agreement 1.0 Signed-off-by: Liming Gao <liming@intel.com> --- MdePkg/Include/IndustryStandard/SdramSpd.h | 31 +- MdePkg/Include/IndustryStandard/SdramSpdDdr3.h | 769 +++ MdePkg/Include/IndustryStandard/SdramSpdDdr4.h | 964 Md

[edk2] [Patch] MdePkg Base.h: Add add comments on __ prefix in enum VERIFY_UINTXX_ENUM_SIZE

2016-11-15 Thread Liming Gao
Cc: Michael Kinney <michael.d.kin...@intel.com> Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Liming Gao <liming@intel.com> --- MdePkg/Include/Base.h | 5 + 1 file changed, 5 insertions(+) diff --git a/MdePkg/Include/Base.h b/MdePkg/Include/Base.h i

[edk2] [Patch] OvmfPkg AcpiTables: Use PcdDebugIoPort to describe QEMU debug console

2016-11-15 Thread Liming Gao
Cc: Laszlo Ersek <ler...@redhat.com> Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Liming Gao <liming@intel.com> --- OvmfPkg/AcpiTables/AcpiTables.inf | 4 OvmfPkg/AcpiTables/Dsdt.asl | 2 +- 2 files changed, 5 insertions(+), 1 deletion(-)

[edk2] [PATCH v2] BaseTools gitignore: Ignore VS intermediate files *.obj and *.pdb

2016-11-15 Thread Liming Gao
In V2, ignore .exe and .ilk files Cc: Yonghong Zhu <yonghong@intel.com> Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Liming Gao <liming@intel.com> --- BaseTools/.gitignore | 4 1 file changed, 4 insertions(+) diff --git a/BaseTools/.gitignore

[edk2] [Patch] BaseTools gitignore: Ignore VS intermediate files *.obj and *.pdb

2016-11-14 Thread Liming Gao
Cc: Yonghong Zhu <yonghong@intel.com> Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Liming Gao <liming@intel.com> --- BaseTools/.gitignore | 2 ++ 1 file changed, 2 insertions(+) diff --git a/BaseTools/.gitignore b/BaseTools/.gitignore index 5ca0

[edk2] [Patch 0/2] BaseTools tools_def.txt: Update GCC ASLPP_FLAGS

2016-11-13 Thread Liming Gao
Update GCC ASLPP_FLAGS to align to VS ASLPP_FLAGS so that the same ASL file can support GCC and VS tool chains both. Liming Gao (2): BaseTools tools_def.txt: Remove -P option in GCC ASLPP_FLAGS BaseTools tools_def.txt: Include AutoGen.h in GCC ASLPP_FLAGS BaseTools/Conf/tools_def.template

[edk2] [Patch 2/2] BaseTools tools_def.txt: Include AutoGen.h in GCC ASLPP_FLAGS

2016-11-13 Thread Liming Gao
https://bugzilla.tianocore.org/show_bug.cgi?id=227 Refer to VS ASLPP_FLAGS, force include AutoGen.h so that ASL code can use FixedPcdGetXX to get FixedPcd value. Cc: Yonghong Zhu <yonghong@intel.com> Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Liming Gao &

[edk2] [Patch 1/2] BaseTools tools_def.txt: Remove -P option in GCC ASLPP_FLAGS

2016-11-13 Thread Liming Gao
chains. Cc: Yonghong Zhu <yonghong@intel.com> Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Liming Gao <liming@intel.com> --- BaseTools/Conf/tools_def.template | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/BaseTools/Conf/tools_def

<    1   2   3   4   5   >