In V2, add comments on why uses __ prefix in enum type name.
For field name in structure, its first character should be upper case.
Cc: Michael Kinney <michael.d.kin...@intel.com>
Contributed-under: TianoCore Contribution Agreement 1.0
Signed-off-by: Liming Gao <liming@intel.com>
https://bugzilla.tianocore.org/show_bug.cgi?id=201
Cc: Giri P Mudusuru <giri.p.mudus...@intel.com>
Contributed-under: TianoCore Contribution Agreement 1.0
Signed-off-by: Liming Gao <liming@intel.com>
---
MdePkg/Include/IndustryStandard/SdramSpd.h | 1682
For field name in structure, its first character should be upper case.
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 | 6 +++---
1 file changed, 3 inse
All CpuExceptionHandlerLib library instances use nasm source files.
Cc: Reviewed-by: Jeff Fan <jeff@intel.com>
Contributed-under: TianoCore Contribution Agreement 1.0
Signed-off-by: Liming Gao <liming@intel.com>
---
.../Library/CpuExceptionHandlerLib/PeiCpuExceptionHandlerLi
Contributed-under: TianoCore Contribution Agreement 1.0
Signed-off-by: Liming Gao <liming@intel.com>
---
.../LzmaCustomDecompressLib/LZMA-SDK-README.txt| 6 +-
.../LzmaArchCustomDecompressLib.inf| 10 +-
.../LzmaCustomDecompressLib.inf
Contributed-under: TianoCore Contribution Agreement 1.0
Signed-off-by: Liming Gao <liming@intel.com>
---
.../LzmaCustomDecompressLib/LZMA-SDK-README.txt| 6 +-
.../LzmaArchCustomDecompressLib.inf| 10 +-
.../LzmaCustomDecompressLib.inf
New version LZMA improves the compression performance, and has no change on
the compression algorithm. So, the decompression library can work with
the old and new LZMA SDK. To be consistent, LZMA tool and library are both
updated to new 16.04 version.
Liming Gao (3):
BaseTools LzmaCompress
Move checkNt32Flag before set BASE_TOOLS_PATH env to make sure it be set
Cc: Yonghong Zhu <yonghong@intel.com>
Contributed-under: TianoCore Contribution Agreement 1.0
Signed-off-by: Liming Gao <liming@intel.com>
---
edksetup.bat | 3 ++-
1 file changed, 2 insertions(+)
Fix issue: https://bugzilla.tianocore.org/show_bug.cgi?id=94
Cc: Jiewen Yao <jiewen@intel.com>
Contributed-under: TianoCore Contribution Agreement 1.0
Signed-off-by: Liming Gao <liming@intel.com>
---
MdePkg/Include/IndustryStandard/Acpi60.h | 6 +++---
1 file changed, 3 inse
/MD option will introduce the specific version dll dependency. It will cause
the compiled C tools not work on some system.
Contributed-under: TianoCore Contribution Agreement 1.0
Signed-off-by: Liming Gao <liming@intel.com>
---
BaseTools/Source/C/Makefiles/ms.common | 4 ++--
1 file c
Cc: Ard Biesheuvel <ard.biesheu...@linaro.org>
Contributed-under: TianoCore Contribution Agreement 1.0
Signed-off-by: Liming Gao <liming@intel.com>
---
MdePkg/Library/BaseMemoryLibOptDxe/Arm/MemLibGuid.c | 13 +
1 file changed, 13 insertions(+)
diff --git a/Md
Contributed-under: TianoCore Contribution Agreement 1.0
Signed-off-by: Liming Gao <liming@intel.com>
---
MdeModulePkg/Universal/LoadFileOnFv2/LoadFileOnFv2.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/MdeModulePkg/Universal/LoadFileOnFv2/LoadFileOnFv
Fix issue: https://bugzilla.tianocore.org/show_bug.cgi?id=95
Cc: Jiewen Yao <jiewen@intel.com>
Contributed-under: TianoCore Contribution Agreement 1.0
Signed-off-by: Liming Gao <liming@intel.com>
---
MdePkg/Include/IndustryStandard/Acpi51.h | 8
1 file changed,
Remove extra qword in nasm code to make it pass build.
This file is only built in INTEL ICC compiler. So, there is missing
build check for it.
Contributed-under: TianoCore Contribution Agreement 1.0
Signed-off-by: Liming Gao <liming@intel.com>
---
.../BaseSynchronizationLi
--nt32
build -p Nt32Pkg\Nt32Pkg.dsc -a X64 # for 64
build -p Nt32Pkg\Nt32Pkg.dsc -a IA32 # for 32
Contributed-under: TianoCore Contribution Agreement 1.0
Signed-off-by: David Chan <chan...@sina.com>
Tested-by: Liming Gao <liming@intel.com>
---
Nt32Pkg/Sec/SecMain.inf | 6 +++---
The function with the variable parameters should have EFIAPI.
Cc: Ruiyu Ni <ruiyu...@intel.com>
Contributed-under: TianoCore Contribution Agreement 1.0
Signed-off-by: Liming Gao <liming@intel.com>
---
DuetPkg/DxeIpl/Debug.c | 1 +
DuetPkg/DxeIpl/Debug.h | 1 +
DuetPkg/EfiLdr
The function with the variable parameters should have EFIAPI.
Cc: Feng Tian <feng.t...@intel.com>
Cc: Cinnamon Shia <cinnamon.s...@hpe.com>
Cc: Cecil Sheng <cecil.sh...@hpe.com>
Contributed-under: TianoCore Contribution Agreement 1.0
Signed-off-by: Liming Gao <
From: Hao Wu <hao.a...@intel.com>
In Linux, Command needs to be String instead of list when Command run
as shell with True.
Contributed-under: TianoCore Contribution Agreement 1.0
Signed-off-by: Liming Gao <liming@intel.com>
---
BaseTools/Source/Python/build/build.py | 6 ++
Contributed-under: TianoCore Contribution Agreement 1.0
Signed-off-by: Liming Gao <liming@intel.com>
---
BaseTools/Source/C/GenVtf/GenVtf.c | 1 +
1 file changed, 1 insertion(+)
diff --git a/BaseTools/Source/C/GenVtf/GenVtf.c
b/BaseTools/Source/C/GenVtf/GenVtf.c
index 9a3f508..f
Enable O2 option to generate fast code for performance improvement.
Contributed-under: TianoCore Contribution Agreement 1.0
Signed-off-by: Liming Gao <liming@intel.com>
---
BaseTools/Source/C/Makefiles/ms.app| 2 +-
BaseTools/Source/C/Makefiles/ms.common | 4 ++--
2 files chan
Enable Ofast option to generate fast code for performance improvement.
Contributed-under: TianoCore Contribution Agreement 1.0
Signed-off-by: Liming Gao <liming@intel.com>
---
BaseTools/Source/C/Makefiles/header.makefile | 6 +++---
1 file changed, 3 insertions(+), 3 deletions(-)
diff
Contributed-under: TianoCore Contribution Agreement 1.0
Signed-off-by: Liming Gao <liming@intel.com>
---
BaseTools/Source/C/EfiLdrImage/EfiLdrImage.c | 1 -
1 file changed, 1 deletion(-)
diff --git a/BaseTools/Source/C/EfiLdrImage/EfiLdrImage.c
b/BaseTools/Source/C/EfiLdrImage/EfiLdrI
0.107s
LzmaCompress (VS Od) 3.047s 0.210s
LzmaCompress (VS O2) 1.551s 0.126s
Liming Gao (4):
BaseTools EfiLdrImage: Remove unnecessary exit (0)
BaseTools Makefile: Enable O2 option to replace Od for VS tool chain
BaseTools GenVtf: Initialize the return
ms.common is included by every tool Makefile. it is not necessary to be placed
in ms.app again.
Contributed-under: TianoCore Contribution Agreement 1.0
Signed-off-by: Liming Gao <liming@intel.com>
---
BaseTools/Source/C/Makefiles/ms.app | 2 --
1 file changed, 2 deletions(-)
diff
Contributed-under: TianoCore Contribution Agreement 1.0
Signed-off-by: Liming Gao <liming@intel.com>
---
BaseTools/Source/C/Makefiles/app.makefile | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/BaseTools/Source/C/Makefiles/app.makefile
b/BaseTools/Source/C/Mak
EDKII implementation protocol should be with EDKII_ prefix.
V2: add gEdkiiFormBrowserExProtocolGuid to align its structure name.
Cc: Eric Dong <eric.d...@intel.com>
Cc: Feng Tian <feng.t...@intel.com>
Contributed-under: TianoCore Contribution Agreement 1.0
Signed-off-by: Liming
on it to
generate the binary HII image package.
Liming Gao (2):
MdePkg UefiHii: Add IMAGE_TOKEN macro to access image resource in C
and VFR
MdeModulePkg HiiDatabaseDxe: Ignore new EFI_HII_IIBT_IMAGE_PNG type
Image
Yonghong Zhu (1):
BaseTools: support generating image package from BMP/JPEG/PNG
ileName".
2) New IMAGE_TOKEN macro is used to refer to IMAGE_ID.
3) New AutoGen header file $(MODULE_NAME)ImgDefs.h to include the
generated ImageId definition.
4) New $(MODULE_NAME)Idf.hpk or $(MODULE_NAME)Images are generated
as the output binary HII image package.
Cc: Liming Gao <l
HiiImage protocol implementation doesn't support EFI_HII_IIBT_IMAGE_PNG.
Cc: Eric Dong <eric.d...@intel.com>
Cc: Dandan Bi <dandan...@intel.com>
Contributed-under: TianoCore Contribution Agreement 1.0
Signed-off-by: Liming Gao <liming@intel.com>
---
MdeModulePkg/Univer
Cc: Eric Dong <eric.d...@intel.com>
Cc: Dandan Bi <dandan...@intel.com>
Contributed-under: TianoCore Contribution Agreement 1.0
Signed-off-by: Liming Gao <liming@intel.com>
---
MdePkg/Include/Uefi/UefiInternalFormRepresentation.h | 8 +++-
1 file changed, 7 inserti
oCore Contribution Agreement 1.0
Signed-off-by: Liming Gao <liming@intel.com>
---
BaseTools/Source/Python/Common/VpdInfoFile.py | 7 ---
BaseTools/Source/Python/build/build.py| 5 ++---
2 files changed, 6 insertions(+), 6 deletions(-)
diff --git a/BaseTools/Source/Python/
d-under: TianoCore Contribution Agreement 1.0
Signed-off-by: Liming Gao <liming@intel.com>
---
BaseTools/toolsetup.bat | 101 +---
1 file changed, 44 insertions(+), 57 deletions(-)
diff --git a/BaseTools/toolsetup.bat b/BaseTools/toolsetup.ba
Add 13 windows batch files for every python tool.
Cc: Yonghong Zhu <yonghong@intel.com>
Cc: Michael Kinney <michael.d.kin...@intel.com>
Cc: Erik Bjorge <erik.c.bjo...@intel.com>
Contributed-under: TianoCore Contribution Agreement 1.0
Signed-off-by: Liming Gao <
If PYTHON_FREEZER_PATH is not set, Python tools will not be freeze.
Cc: Yonghong Zhu <yonghong@intel.com>
Cc: Michael Kinney <michael.d.kin...@intel.com>
Cc: Erik Bjorge <erik.c.bjo...@intel.com>
Contributed-under: TianoCore Contribution Agreement 1.0
Signed-off-by:
When Python tool exe files don't exist, toolsetup.bat will configure
PYTHONPATH to python source code, and set BaseTools\BinWrappers\WindowsLike
into system PATH directory. Then, windows batch files will be used to
run python tools from source files.
Liming Gao (4):
BaseTools: Add Windows
This change is also applied to VfrCompile Pccts antlr and dlg tool.
In V2, add the missing C rules.
Cc: Giri P Mudusuru <giri.p.mudus...@intel.com>
Cc: Yonghong Zhu <yonghong@intel.com>
Contributed-under: TianoCore Contribution Agreement 1.0
Signed-off-by: Liming Gao <limin
The change is missing in VfrComile GNUmakefile.
Cc: Giri P Mudusuru <giri.p.mudus...@intel.com>
Cc: Yonghong Zhu <yonghong@intel.com>
Contributed-under: TianoCore Contribution Agreement 1.0
Signed-off-by: Liming Gao <liming@intel.com>
---
BaseTools/Source/C/VfrCompil
t 1.0
Signed-off-by: Liming Gao <liming@intel.com>
---
BaseTools/Source/C/Makefiles/footer.makefile | 2 +-
BaseTools/Source/C/Makefiles/header.makefile | 1 +
2 files changed, 2 insertions(+), 1 deletion(-)
diff --git a/BaseTools/Source/C/Makefiles/footer.makefile
b/BaseTools/
The following 4 patches clean up the remaining issues in GNU makefile.
With those changes, all GNU makefiles have been updated.
Liming Gao (4):
BaseTools GNU makefile: Add BUILD_CXXFLAGS to align make built-in rule
BaseTools GNU makefile: remove unused .S rule
BaseTools VfrCompile GNU
BootManagerMenu boot option is handled by EfiBootManagerGetBootManagerMenu.
Don't need to handle it again when parse LoadFile protocol.
In V2, use "BootManagerMenu" instead of "BootMenuApp".
Contributed-under: TianoCore Contribution Agreement 1.0
Signed-off-by: Liming Gao &
Rename local function name BootMenuApp to BootManagerMenu to align to
other public function name.
In V2, use "BootManagerMenu" instead of "BootMenuApp".
Contributed-under: TianoCore Contribution Agreement 1.0
Signed-off-by: Liming Gao <liming@intel.com>
Re
Rename local function name BootMenuApp to BootManagerMenuApp to align
to other public function name.
Cc: Ruiyu Ni <ruiyu...@intel.com>
Contributed-under: TianoCore Contribution Agreement 1.0
Signed-off-by: Liming Gao <liming@intel.com>
---
MdeModulePkg/Library/UefiBootManager
BootManagerMenuApp boot option is handled by EfiBootManagerGetBootManagerMenu.
Don't need to handle it again when parse LoadFile protocol.
Cc: Ruiyu Ni <ruiyu...@intel.com>
Cc: Eric Dong <eric.d...@intel.com>
Contributed-under: TianoCore Contribution Agreement 1.0
Signed-off-by
To avoid the conflict with the default GCC flag name, BUILD_ prefix is added.
Cc: Giri P Mudusuru <giri.p.mudus...@intel.com>
Cc: Yonghong Zhu <yonghong@intel.com>
Contributed-under: TianoCore Contribution Agreement 1.0
Signed-off-by: Liming Gao <liming@intel.com>
---
Cc: Giri P Mudusuru <giri.p.mudus...@intel.com>
Cc: Yonghong Zhu <yonghong@intel.com>
Contributed-under: TianoCore Contribution Agreement 1.0
Signed-off-by: Liming Gao <liming@intel.com>
---
BaseTools/Source/C/Makefiles/footer.makefile | 5 -
1 file changed,
. But,
it is missing in BaseTools BasePeCoffLib.
Cc: Ard Biesheuvel <ard.biesheu...@linaro.org>
Cc: Yonghong Zhu <yonghong@intel.com>
Contributed-under: TianoCore Contribution Agreement 1.0
Signed-off-by: Liming Gao <liming@intel.com>
---
BaseTools/Source/C/Common/BasePeCoff.c | 4 ++
Add two name files IntrinsicLib Ia32 MathLShiftS64.nasm and MathRShiftU64.nasm
Cc: Qin Long <qin.l...@intel.com>
Contributed-under: TianoCore Contribution Agreement 1.0
Signed-off-by: Liming Gao <liming@intel.com>
---
.../Library/IntrinsicLib/Ia32/MathLShiftS64
Check NULL pointer before access it.
Contributed-under: TianoCore Contribution Agreement 1.0
Signed-off-by: Liming Gao <liming@intel.com>
---
MdeModulePkg/Universal/LoadFileOnFv2/LoadFileOnFv2.c | 6 ++
1 file changed, 6 insertions(+)
diff --git a/MdeModulePkg/Universal/LoadFil
Contributed-under: TianoCore Contribution Agreement 1.0
Signed-off-by: Liming Gao <liming@intel.com>
---
MdeModulePkg/Universal/LoadFileOnFv2/LoadFileOnFv2.c | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/MdeModulePkg/Universal/LoadFileOnFv2/LoadFileOnFv
Cc: Yonghong Zhu <yonghong@intel.com>
Contributed-under: TianoCore Contribution Agreement 1.0
Signed-off-by: Liming Gao <liming@intel.com>
---
BaseTools/ReadMe.txt | 6 +++---
1 file changed, 3 insertions(+), 3 deletions(-)
diff --git a/BaseTools/ReadMe.txt b/BaseTools/Read
edk2 project source files are CRLF line ending. BaseTools\Scripts\PatchCheck.py
script can detect Line ending issue in patch.
Liming Gao (3):
BaseTools: Correct ReadMe.txt file with CRLF line ending
MdePkg PeiDxePostCodeLibReportStatusCode: Correct files with CRLF line
ending
ShellPkg
Cc: Ruiyu Ni <ruiyu...@intel.com>
Cc: Jaben Carsey <jaben.car...@intel.com>
Contributed-under: TianoCore Contribution Agreement 1.0
Signed-off-by: Liming Gao <liming@intel.com>
---
.../UefiHandleParsingLib/UefiHandleParsingLib.h| 6 ++--
.../UefiHandleParsingLib/UefiHa
Contributed-under: TianoCore Contribution Agreement 1.0
Signed-off-by: Liming Gao <liming@intel.com>
---
.../PeiDxePostCodeLibReportStatusCode.inf | 4 ++--
.../PeiDxePostCodeLibReportStatusCode.uni | 4 ++--
2 files chan
Edk2 enables Os option and ms_va_list in GCC tool chain.
This change makes VA_LIST is not pointer, and cause GCC failure.
Contributed-under: TianoCore Contribution Agreement 1.0
Signed-off-by: Liming Gao <liming@intel.com>
---
.../EdkIIGlueLib/Library/DxeReportStatusC
New LoadFileOnFv2 driver will install LoadFile protocol based on FV file.
Update UefiBootManagerLib to find BootMenuApp with LoadFile protocol.
Cc: Ruiyu Ni <ruiyu...@intel.com>
Contributed-under: TianoCore Contribution Agreement 1.0
Signed-off-by: Liming Gao <liming@
handle Shell application.
Cc: Ruiyu Ni <ruiyu...@intel.com>
Contributed-under: TianoCore Contribution Agreement 1.0
Signed-off-by: Liming Gao <liming@intel.com>
---
.../Universal/LoadFileOnFv2/LoadFileOnFv2.c| 421 +
.../Universal/LoadFileOnFv2/LoadF
Update boot description to support LoadFile protocol based on FV file.
Cc: Ruiyu Ni <ruiyu...@intel.com>
Contributed-under: TianoCore Contribution Agreement 1.0
Signed-off-by: Liming Gao <liming@intel.com>
---
.../Library/UefiBootManagerLib/BmBootDescription.c | 47 +
Cc: Ruiyu Ni <ruiyu...@intel.com>
Contributed-under: TianoCore Contribution Agreement 1.0
Signed-off-by: Liming Gao <liming@intel.com>
---
MdeModulePkg/MdeModulePkg.dsc | 1 +
1 file changed, 1 insertion(+)
diff --git a/MdeModulePkg/MdeModulePkg.dsc b/MdeModulePkg/MdeModuleP
-under: TianoCore Contribution Agreement 1.0
Signed-off-by: Liming Gao <liming@intel.com>
---
.../Library/UefiBootManagerLib/BmLoadOption.c | 44 --
1 file changed, 41 insertions(+), 3 deletions(-)
diff --git a/MdeModulePkg/Library/UefiBootManagerLib/BmLoa
This function abstracts the common logic to find BootMenuApp file.
Cc: Ruiyu Ni <ruiyu...@intel.com>
Contributed-under: TianoCore Contribution Agreement 1.0
Signed-off-by: Liming Gao <liming@intel.com>
---
MdeModulePkg/Library/UefiBootManagerLib/BmBoot.c | 49 +++-
PatchCheck script prints File and Line information in error message.
This patch adds line number in the error message to help detect the issue.
Contributed-under: TianoCore Contribution Agreement 1.0
Signed-off-by: Liming Gao <liming@intel.com>
---
BaseTools/Scripts/PatchCheck.p
Package DSC is used to verify the module source build.
Cc: Jiewen Yao <jiewen@intel.com>
Cc: Giri Mudusuru <giri.p.mudus...@intel.com>
Contributed-under: TianoCore Contribution Agreement 1.0
Signed-off-by: Liming Gao <liming@intel.com>
---
IntelFsp2Pkg/IntelFsp2Pkg.
Package DSC is used to verify the module source build.
Cc: Jiewen Yao <jiewen@intel.com>
Cc: Giri Mudusuru <giri.p.mudus...@intel.com>
Contributed-under: TianoCore Contribution Agreement 1.0
Signed-off-by: Liming Gao <liming@intel.com>
---
IntelFsp2WrapperPkg/IntelFsp2
Package DSC is used to verify the module source build.
Liming Gao (3):
IntelFsp2WrapperPkg SecFspWrapperPlatformSecLibSample:Update code to
pass build
IntelFsp2WrapperPkg: Add missing modules in Package DSC
IntelFsp2Pkg: Add missing modules in Package DSC
IntelFsp2Pkg/IntelFsp2Pkg.dsc
Package DSC is used to verify the module source build.
Liming Gao (3):
IntelFsp2WrapperPkg SecFspWrapperPlatformSecLibSample:Update code to
pass build
IntelFsp2WrapperPkg: Add missing modules in Package DSC
IntelFsp2Pkg: Add missing modules in Package DSC
IntelFsp2Pkg/IntelFsp2Pkg.dsc
1. Update its library class to PlatformSecLib
2. Update source code to refer to the matched header file
Cc: Jiewen Yao <jiewen@intel.com>
Cc: Giri Mudusuru <giri.p.mudus...@intel.com>
Contributed-under: TianoCore Contribution Agreement 1.0
Signed-off-by: Liming Gao <limin
BaseLib Ia32 InternalSwitchStack.S has no matched InternalSwitchStack.nasm.
Use ObjDump to verify the output object files be same.
Contributed-under: TianoCore Contribution Agreement 1.0
Signed-off-by: Liming Gao <liming@intel.com>
---
MdePkg/Library/BaseLib/BaseLib.inf
PlatformSecLib.h is not used and removed.
Cc: Giri P Mudusuru <giri.p.mudus...@intel.com>
Cc: Jiewen Yao <jiewen@intel.com>
Contributed-under: TianoCore Contribution Agreement 1.0
Signed-off-by: Liming Gao <liming@intel.com>
---
IntelFsp2WrapperPkg/FspmWrapperPeim/Fsp
Use BasePeCoffLib PeCoffLoaderGetImageInfo() to check the PE/COFF image.
In V2, add specific ImageRead() to make sure the PE/COFF image content
read is within the image buffer.
Contributed-under: TianoCore Contribution Agreement 1.0
Signed-off-by: Liming Gao <liming@intel.com>
Re
ImageRead() to make sure the PE/COFF image content
read is within the image buffer.
Liming Gao (5):
SecurityPkg SecureBootConfigDxe: Add check for the external PE/COFF
image.
SecurityPkg TrEEDxe: Add check for the PE/COFF image.
SecurityPkg Tcg2Dxe: Add check for the PE/COFF image
The input PeImage in TcgMeasurePeImage() has been checked.
Cc: Jiewen Yao <jiewen@intel.com>
Cc: Chao Zhang <chao.b.zh...@intel.com>
Contributed-under: TianoCore Contribution Agreement 1.0
Signed-off-by: Liming Gao <liming@intel.com>
---
SecurityPkg/Library/Dx
Use BasePeCoffLib PeCoffLoaderGetImageInfo() to check the PE/COFF image.
In V2, add specific ImageRead() to make sure the PE/COFF image content
read is within the image buffer.
Contributed-under: TianoCore Contribution Agreement 1.0
Signed-off-by: Liming Gao <liming@intel.com>
Re
The input PeImage in HashPeImage() has been checked.
Contributed-under: TianoCore Contribution Agreement 1.0
Signed-off-by: Liming Gao <liming@intel.com>
Reviewed-by: Jiewen Yao <jiewen@intel.com>
Reviewed-by: Chao Zhang <chao.b.zh...@intel.com>
---
.../Library/DxeIm
Use BasePeCoffLib PeCoffLoaderGetImageInfo() to check the PE/COFF image.
In V2, add specific ImageRead() to make sure the PE/COFF image content
read is within the image buffer.
Contributed-under: TianoCore Contribution Agreement 1.0
Signed-off-by: Liming Gao <liming@intel.com>
Re
The input PeImage in TcgMeasurePeImage() has been checked.
Cc: Jiewen Yao <jiewen@intel.com>
Cc: Chao Zhang <chao.b.zh...@intel.com>
Contributed-under: TianoCore Contribution Agreement 1.0
Signed-off-by: Liming Gao <liming@intel.com>
---
SecurityPkg/Library/Dx
External PE/COFF image may not be valid and cause memory corruption.
These patches uses PeCoffLib PeCoffLoaderGetImageInfo() to check the PE format.
If this API has been used to check PE format, the addtional comments will
be added to describe PE image has been checked.
Liming Gao (4
Use BasePeCoffLib PeCoffLoaderGetImageInfo() to check the PE/COFF image.
Cc: Jiewen Yao <jiewen@intel.com>
Cc: Chao Zhang <chao.b.zh...@intel.com>
Contributed-under: TianoCore Contribution Agreement 1.0
Signed-off-by: Liming Gao <liming@intel.com>
---
Secur
Use BasePeCoffLib PeCoffLoaderGetImageInfo() to check the PE/COFF image.
Cc: Jiewen Yao <jiewen@intel.com>
Cc: Chao Zhang <chao.b.zh...@intel.com>
Contributed-under: TianoCore Contribution Agreement 1.0
Signed-off-by: Liming Gao <liming@intel.com>
---
SecurityP
The input PeImage in HashPeImage() has been checked.
Cc: Jiewen Yao <jiewen@intel.com>
Cc: Chao Zhang <chao.b.zh...@intel.com>
Contributed-under: TianoCore Contribution Agreement 1.0
Signed-off-by: Liming Gao <liming@intel.com>
---
.../Library/DxeIm
Use BasePeCoffLib PeCoffLoaderGetImageInfo() to check the PE/COFF image.
Cc: Jiewen Yao <jiewen@intel.com>
Cc: Chao Zhang <chao.b.zh...@intel.com>
Contributed-under: TianoCore Contribution Agreement 1.0
Signed-off-by: Liming Gao <liming@intel.com>
---
Secur
Cc: Jeff Fan <jeff@intel.com>
Contributed-under: TianoCore Contribution Agreement 1.0
Signed-off-by: Liming Gao <liming@intel.com>
---
UefiCpuPkg/CpuIo2Dxe/CpuIo2Dxe.inf | 12 +---
1 file changed, 5 insertions(+), 7 deletions(-)
diff --git a/UefiCpuPkg/CpuIo2Dxe/CpuI
NASM is used as the default assembly source file.
Liming Gao (2):
UefiCpuPkg CpuIo2Dxe: Update INF to refer to NASM source file
UefiCpuPkg CpuMpPei: Update INF to refer to NASM source file
UefiCpuPkg/CpuIo2Dxe/CpuIo2Dxe.inf | 12 +---
UefiCpuPkg/CpuMpPei/CpuMpPei.inf | 10
Cc: Jeff Fan <jeff@intel.com>
Contributed-under: TianoCore Contribution Agreement 1.0
Signed-off-by: Liming Gao <liming@intel.com>
---
UefiCpuPkg/CpuMpPei/CpuMpPei.inf | 10 --
1 file changed, 4 insertions(+), 6 deletions(-)
diff --git a/UefiCpuPkg/CpuMpPei/Cp
Cc: Jeff Fan <jeff@intel.com>
Contributed-under: TianoCore Contribution Agreement 1.0
Signed-off-by: Liming Gao <liming@intel.com>
---
UefiCpuPkg/SecCore/SecCore.inf | 6 ++
1 file changed, 2 insertions(+), 4 deletions(-)
diff --git a/UefiCpuPkg/SecCore/SecCore.inf
Keep NASM file only for new added modules.
Cc: Jiewen Yao <jiewen@intel.com>
Cc: Giri Mudusuru <giri.p.mudus...@intel.com>
Contributed-under: TianoCore Contribution Agreement 1.0
Signed-off-by: Liming Gao <liming@intel.com>
---
.../BaseFspSwitchStackLib.inf
Keep NASM file only for new added modules.
Cc: Jiewen Yao <jiewen@intel.com>
Cc: Giri Mudusuru <giri.p.mudus...@intel.com>
Contributed-under: TianoCore Contribution Agreement 1.0
Signed-off-by: Liming Gao <liming@intel.com>
---
.../SecFspSecPlatformLibNull/Ia32/Flat
Keep NASM file only for new added modules.
Cc: Jiewen Yao <jiewen@intel.com>
Cc: Giri Mudusuru <giri.p.mudus...@intel.com>
Contributed-under: TianoCore Contribution Agreement 1.0
Signed-off-by: Liming Gao <liming@intel.com>
---
.../BaseFspDebugLibSerialPort.inf
Keep NASM file only for new added modules.
Cc: Jiewen Yao <jiewen@intel.com>
Cc: Giri Mudusuru <giri.p.mudus...@intel.com>
Contributed-under: TianoCore Contribution Agreement 1.0
Signed-off-by: Liming Gao <liming@intel.com>
---
.../Ia32/PeiCoreEntry.S
.
Here, I would like to propose to add NASM only for IA32 and X64 assembly files
in new modules and packages. If there is the real request for MASM or GAS,
we can consider to add them later.
Liming Gao (5):
IntelFsp2WrapperPkg BaseFspWrapperApiLib: Remove MASM/GAS files
IntelFsp2WrapperPkg
Keep NASM file only for new added modules.
Cc: Jiewen Yao <jiewen@intel.com>
Cc: Giri Mudusuru <giri.p.mudus...@intel.com>
Contributed-under: TianoCore Contribution Agreement 1.0
Signed-off-by: Liming Gao <liming@intel.com>
---
.../BaseFspWrapperApiLib/BaseFspWrappe
RUNTIME_MEMORY_STATUSCODE_HEADER has been moved into MdeModulePkg public header
file Include/Guid/MemoryStatusCodeRecord.h. It should be removed from the
driver.
Cc: Cinnamon Shia <cinnamon.s...@hpe.com>
Contributed-under: TianoCore Contribution Agreement 1.0
Signed-off-by: Liming Gao &
Liming Gao (2):
PcAtChipsetPkg: Add two PCDs for Reset Control Register and Value
PcAtChipsetPkg: Update ResetSystemLib with PCDs
PcAtChipsetPkg/Library/ResetSystemLib/ResetSystemLib.c | 6 +++---
PcAtChipsetPkg/Library/ResetSystemLib/ResetSystemLib.inf | 6 +-
PcAtChipsetPkg
PcdResetControlRegister for Reset Control Register address.
PcdResetControlValueColdReset for Reset Control Register code reset value.
Contributed-under: TianoCore Contribution Agreement 1.0
Signed-off-by: Liming Gao <liming@intel.com>
---
PcAtChipsetPkg/PcAtChipsetPkg.de
Update ResetSystemLib with PCDs for Reset Control Register and Value
instead of hard code 0x64 and 0xFE.
Contributed-under: TianoCore Contribution Agreement 1.0
Signed-off-by: Liming Gao <liming@intel.com>
---
PcAtChipsetPkg/Library/ResetSystemLib/ResetSystemLib.c
ser wants this
behavior, he can still configure PcdUserPhysicalPresence value to TRUE in
the platform DSC file.
Contributed-under: TianoCore Contribution Agreement 1.0
Signed-off-by: Liming Gao <liming@intel.com>
---
.../PlatformSecureLibNull/PlatformSecureLibNul
This PCD supports all configuration type. Its default value is FALSE.
Contributed-under: TianoCore Contribution Agreement 1.0
Signed-off-by: Liming Gao <liming@intel.com>
---
SecurityPkg/SecurityPkg.dec | 6 ++
SecurityPkg/SecurityPkg.uni | 9 +
2 files changed, 15 inse
Liming Gao (2):
SecurityPkg: Add PcdUserPhysicalPresence to indicate use physical
presence.
SecurityPkg: Update PlatformSecureLibNull with PCD to get physical
presence.
.../PlatformSecureLibNull/PlatformSecureLibNull.c | 29 --
.../PlatformSecureLibNull.inf
-off-by: Liming Gao <liming@intel.com>
Cc: Marvin Haeuser <marvin.haeu...@outlook.com>
---
MdeModulePkg/Core/Dxe/DxeMain/DxeMain.c | 1 -
MdeModulePkg/Core/Pei/PeiMain.h | 1 -
MdeModulePkg/Core/Pei/PeiMain/PeiMain.c | 1 -
3 files changed, 3 deletions(-)
diff --git a/MdeMo
These patches are available in https://github.com/lgao4/edk2.git nasm-v1.
The nasm-v1 branch sha1 is 42bec457c575b6cb7c9fc1730cbea263ffce7b1c
Contributed-under: TianoCore Contribution Agreement 1.0
Signed-off-by: Liming Gao <liming@intel.com>
Signed-off-by: Jordan Justen <jor
GenFds GenSection will search the output file based on the file extension.
If the output file has no extension, it should be skip.
Cc: Andrew Fish <af...@apple.com>
Contributed-under: TianoCore Contribution Agreement 1.0
Signed-off-by: Liming Gao <liming@intel.com>
---
Base
301 - 400 of 485 matches
Mail list logo