Re: [edk2-devel] [PATCH] MdePkg-UefiSpec.h: Add UEFI 2.8 new memory attributes

2019-10-29 Thread Liming Gao
Qing: Thanks for your update. Reviewed-by: Liming Gao Thanks Liming >-Original Message- >From: Huang, Qing >Sent: Wednesday, October 30, 2019 12:55 PM >To: devel@edk2.groups.io >Cc: Gao, Liming >Subject: [PATCH] MdePkg-UefiSpec.h: Add UEFI 2.8 new memory attributes > >BZ:

Re: [edk2-devel] [PATCH] MdeModulePkg/BdsDxe: Make PlatformRecovery work regardless of OsIndications

2019-10-29 Thread Wang, Jian J
Pushed at b15646484eaffcf7cc464fdea0214498f26addc2 Regards, Jian > -Original Message- > From: Wang, Sunny (HPS SW) > Sent: Wednesday, October 23, 2019 7:20 PM > To: devel@edk2.groups.io; Ni, Ray ; Wang, Jian J > ; Gao, Zhichao ; > ler...@redhat.com; Wu, Hao A > Cc: Li, Walon ; Wang,

[edk2-devel] [PATCH] MdePkg-UefiSpec.h: Add UEFI 2.8 new memory attributes

2019-10-29 Thread Huang, Qing
-=-=-=-=-=-=-=-=-=-=-=- Groups.io Links: You receive all messages sent to this group. View/Reply Online (#49631): https://edk2.groups.io/g/devel/message/49631 Mute This Topic: https://groups.io/mt/36927089/21656 Group Owner: devel+ow...@edk2.groups.io Unsubscribe:

Re: [edk2-devel] [edk2] [EmulatorPkg] using __debugbreak()

2019-10-29 Thread Liming Gao
Kilian: I normally modify the source code and insert CpuBreakpoint() function. That will trig VS debugger for the source code. Thanks Liming From: devel@edk2.groups.io [mailto:devel@edk2.groups.io] On Behalf Of Kilian Kegel Sent: Monday, October 28, 2019 9:01 PM To: Gao, Liming Cc:

Re: [edk2-devel] [PATCH] MdePkg-UefiSpec.h: Add UEFI 2.8 new memory attributes

2019-10-29 Thread Liming Gao
Qing: Please send patch mail instead of the attachment. Thanks Liming >-Original Message- >From: Huang, Qing >Sent: Wednesday, October 30, 2019 11:35 AM >To: Gao, Liming ; devel@edk2.groups.io >Subject: RE: [PATCH] MdePkg-UefiSpec.h: Add UEFI 2.8 new memory >attributes > >Add V2 patch

Re: [edk2-devel] [PATCH] MdePkg-UefiSpec.h: Add UEFI 2.8 new memory attributes

2019-10-29 Thread Huang, Qing
Add V2 patch to add some description for these new memory type from UEFI 2.8 spec. Thanks & Best regards, Huang, Qing > -Original Message- > From: Gao, Liming > Sent: Wednesday, October 30, 2019 9:17 AM > To: Huang, Qing ; devel@edk2.groups.io > Subject: RE: [PATCH] MdePkg-UefiSpec.h:

[edk2-devel] [PATCH] Support skipping automatic BM enumeration

2019-10-29 Thread Ashish Singhal
Right now, any and every handle with a BlockIO or SimpleFileSystem or LoadFile protocol installed on the system is used to enumerate BM automatically. There may be cases where on a platform, some of these are not desirable to be enumerated automatically. This patch adds support for skipping this

[edk2-devel] [PATCH] MdeModulePkg/UefiBootManagerLib: Support skipping BM enumeration

2019-10-29 Thread Ashish Singhal
Allow support for skipping auto enumeration of a BlockIO or SimpleFileSystem or LoadFile protocol based on a new protocol installed on the same handle. EdkiiSkipBmAutoEnumerate protocol has been added for that purpose. Signed-off-by: Ashish Singhal --- .../Include/Protocol/SkipBmAutoEnumerate.h

[edk2-devel] [PATCH] EmbeddedPkg/DtPlatformDxe: Add DT/ACPI Default Flexibility

2019-10-29 Thread Ashish Singhal
Add a PCD to govern whether to use DT or ACPI in case the variable governing this is not found or is not valid. Signed-off-by: Ashish Singhal --- EmbeddedPkg/Drivers/DtPlatformDxe/DtPlatformDxe.c | 16 ++-- EmbeddedPkg/Drivers/DtPlatformDxe/DtPlatformDxe.inf | 3 +++

[edk2-devel] [PATCH] Add DT/ACPI Default Flexibility

2019-10-29 Thread Ashish Singhal
Some platforms which support both DT as well as ACPI may need the flexibility to allow either of these 2 as default preference. This change introduces a boolean PCD which tells whether DT is the default preference or not. Ashish Singhal (1): EmbeddedPkg/DtPlatformDxe: Add DT/ACPI Default

[edk2-devel] [PATCH v3] CryptoPkg: Upgrade OpenSSL to 1.1.1d

2019-10-29 Thread Zhang, Shenglei
Update openssl from 1.1.1b to 1.1.1d. Something needs to be noticed is that, there is a bug existing in the released 1_1_1d version(894da2fb7ed5d314ee5c2fc9fd2d9b8b74111596), which causes build failure. So we switch the code base to a usable version, which is 2 commits later than the stable tag.

Re: [edk2-devel] [Patch v3 03/22] requirements.txt: Add python pip requirements file

2019-10-29 Thread Leif Lindholm
OK, if it makes a difference for tools (and security updates), let's try to keep it. (*grumble*) *But* given its too-generic name, can we add a big bold comment header to the file explaining what it is? Some quick searching suggests lines starting with # are ignored, so hopefully this should be

Re: [edk2-devel] [Patch v3 22/22] Readme.md: Add CI build status badges

2019-10-29 Thread Leif Lindholm
On Tue, Oct 29, 2019 at 12:55:17PM -0700, Michael D Kinney wrote: > https://bugzilla.tianocore.org/show_bug.cgi?id=2315 > > Update Readme.md with status badges from the most recent > EDK II Continuous Integration(CI) build. > > Cc: Andrew Fish > Cc: Laszlo Ersek > Cc: Leif Lindholm >

Re: [edk2-devel] [Patch v3 03/22] requirements.txt: Add python pip requirements file

2019-10-29 Thread Sean via Groups.Io
It is a convention for projects using python.  It definitely isn't required but there are some features that come for free when using that filename. https://github.blog/2018-07-12-security-vulnerability-alerts-for-python/ and

Re: [edk2-devel] [edk2-platforms][PATCH V1 2/2] WhiskeylakeOpenBoardPkg: Correct Platform Hook Library references

2019-10-29 Thread Chiu, Chasel
Reviewed-by: Chasel Chiu > -Original Message- > From: Kubacki, Michael A > Sent: Wednesday, October 30, 2019 7:05 AM > To: devel@edk2.groups.io > Cc: Chiu, Chasel ; Desimone, Nathaniel L > > Subject: [edk2-platforms][PATCH V1 2/2] WhiskeylakeOpenBoardPkg: Correct > Platform Hook

Re: [edk2-devel] [edk2-platforms][PATCH V1 1/2] KabylakeOpenBoardPkg: Correct Platform Hook Library references

2019-10-29 Thread Chiu, Chasel
Reviewed-by: Chasel Chiu > -Original Message- > From: Kubacki, Michael A > Sent: Wednesday, October 30, 2019 7:05 AM > To: devel@edk2.groups.io > Cc: Chiu, Chasel ; Desimone, Nathaniel L > ; Jeremy Soller > Subject: [edk2-platforms][PATCH V1 1/2] KabylakeOpenBoardPkg: Correct >

Re: [edk2-devel] [PATCH] CryptoPkg/OpensslLib: Update process_files.pl to generate .h files

2019-10-29 Thread Wang, Jian J
Pushed at 9f4fbd56d43054cc73d722c1643659f9741c0fcf Regards, Jian > -Original Message- > From: Lu, XiaoyuX > Sent: Tuesday, October 29, 2019 6:01 PM > To: Zhang, Shenglei ; devel@edk2.groups.io > Cc: Wang, Jian J > Subject: RE: [PATCH] CryptoPkg/OpensslLib: Update process_files.pl to >

Re: [edk2-devel] [Patch v3 01/22] Maintainers.txt: Add continuous integration(CI) directories

2019-10-29 Thread Leif Lindholm
On Tue, Oct 29, 2019 at 12:54:56PM -0700, Michael D Kinney wrote: > https://bugzilla.tianocore.org/show_bug.cgi?id=2315 > > Add maintainers and reviewers for the directories associated > with continuous integration steps. > * .azurepipelines > * .mergify > * .pytool > > Cc: Andrew Fish > Cc:

Re: [edk2-devel] [Patch v3 02/22] .gitignore: Ignore python compiled files, extdeps, and vscode

2019-10-29 Thread Leif Lindholm
On Tue, Oct 29, 2019 at 12:54:57PM -0700, Michael D Kinney wrote: > From: Sean Brogan > > https://bugzilla.tianocore.org/show_bug.cgi?id=2315 > > Update .gitignore to ignore .pyc files and __pycache__ > directories. Python based plugins can be added to any > package or platform, so these files

Re: [edk2-devel] [Patch v3 03/22] requirements.txt: Add python pip requirements file

2019-10-29 Thread Leif Lindholm
On Tue, Oct 29, 2019 at 12:54:58PM -0700, Michael D Kinney wrote: > From: Sean Brogan > > https://bugzilla.tianocore.org/show_bug.cgi?id=2315 > > Add pip requirements file that is used to install the > python pip modules build from the edk2-pytool-library and > edk2-pytool-extensions

Re: [edk2-devel] [PATCH] MdePkg-UefiSpec.h: Add UEFI 2.8 new memory attributes

2019-10-29 Thread Liming Gao
Reviewed-by: Liming Gao >-Original Message- >From: Huang, Qing >Sent: Thursday, October 24, 2019 10:34 AM >To: devel@edk2.groups.io >Cc: Gao, Liming >Subject: [PATCH] MdePkg-UefiSpec.h: Add UEFI 2.8 new memory attributes > >BZ: https://bugzilla.tianocore.org/show_bug.cgi?id=2309 >Add

[edk2-devel] [edk2-platforms][PATCH V1 0/2] Correct Platform Hook Library references

2019-10-29 Thread Kubacki, Michael A
Many file descriptions reference "Platform Hook Library" when the file is not related to the PlatformHookLib library class. This change updates those references in KabylakeOpenBoardPkg and WhiskeylakeOpenBoardPkg to a more accurate description. Cc: Chasel Chiu Cc: Nate DeSimone Cc: Jeremy

[edk2-devel] [edk2-platforms][PATCH V1 2/2] WhiskeylakeOpenBoardPkg: Correct Platform Hook Library references

2019-10-29 Thread Kubacki, Michael A
Many file descriptions reference "Platform Hook Library" when the file is not related to the PlatformHookLib library class. This change updates those references in WhiskeylakeOpenBoardPkg to a more accurate description. Cc: Chasel Chiu Cc: Nate DeSimone Signed-off-by: Michael Kubacki ---

[edk2-devel] [edk2-platforms][PATCH V1 1/2] KabylakeOpenBoardPkg: Correct Platform Hook Library references

2019-10-29 Thread Kubacki, Michael A
Many file descriptions reference "Platform Hook Library" when the file is not related to the PlatformHookLib library class. This change updates those references in KabylakeOpenBoardPkg to a more accurate description. Cc: Chasel Chiu Cc: Nate DeSimone Cc: Jeremy Soller Signed-off-by: Michael

[edk2-devel] [edk2-platforms] [PATCH] SimicsOpenBoardPkg: Add CmosAccessLib to BoardX58Ich10

2019-10-29 Thread Agyeman, Prince
REF: https://bugzilla.tianocore.org/show_bug.cgi?id=2164 Replaced Cmos.c and Cmos.h with BoardModulePkg's Cmos library CmosAccessLib Cc: Ankit Sinha Cc: Nate DeSimone Cc: Kubacki Michael A Signed-off-by: Prince Agyeman --- .../BoardX58Ich10/OpenBoardPkg.dsc| 2 +

[edk2-devel] [edk2-platforms] [PATCH] SimicsOpenBoardPkg: Change Build Output Directory

2019-10-29 Thread Agyeman, Prince
REF: https://bugzilla.tianocore.org/show_bug.cgi?id=2250 Changed BoardX558ich10's build directory to Build/SimicsOpenBoardPkg/BoardX58Ich10 to align it with align with the other Intel board packages Cc: Ankit Sinha Cc: Nate DeSimone Cc: Kubacki Michael A Signed-off-by: Prince Agyeman ---

[edk2-devel] [Patch v3 02/22] .gitignore: Ignore python compiled files, extdeps, and vscode

2019-10-29 Thread Michael D Kinney
From: Sean Brogan https://bugzilla.tianocore.org/show_bug.cgi?id=2315 Update .gitignore to ignore .pyc files and __pycache__ directories. Python based plugins can be added to any package or platform, so these files and directories may be present outside of BaseTools. Ignore _extdep

[edk2-devel] [Patch v3 22/22] Readme.md: Add CI build status badges

2019-10-29 Thread Michael D Kinney
https://bugzilla.tianocore.org/show_bug.cgi?id=2315 Update Readme.md with status badges from the most recent EDK II Continuous Integration(CI) build. Cc: Andrew Fish Cc: Laszlo Ersek Cc: Leif Lindholm Signed-off-by: Michael D Kinney --- Readme.md | 29 ++--- 1 file

[edk2-devel] [Patch v3 08/22] CryptoPkg: Add YAML file for CI builds

2019-10-29 Thread Michael D Kinney
https://bugzilla.tianocore.org/show_bug.cgi?id=2315 Add YAML file to the package directory with the configuration of the checks to perform during a CI build. Cc: Jian J Wang Cc: Xiaoyu Lu Signed-off-by: Kinney --- CryptoPkg/CryptoPkg.ci.yaml | 48 + 1 file

[edk2-devel] [Patch v3 19/22] .pytool: Add CISettings.py and Readme.md

2019-10-29 Thread Michael D Kinney
From: Sean Brogan https://bugzilla.tianocore.org/show_bug.cgi?id=2315 Add main python script for EDK II Continuous Integration (CI) builds along with a Readme.md that provides a summary of the packages, platforms, and checks performs during a CI build. Cc: Sean Brogan Cc: Bret Barkelew Cc:

[edk2-devel] [Patch v3 09/22] FatPkg: Add YAML file for CI builds

2019-10-29 Thread Michael D Kinney
https://bugzilla.tianocore.org/show_bug.cgi?id=2315 Add YAML file to the package directory with the configuration of the checks to perform during a CI build. Cc: Ray Ni Signed-off-by: Michael D Kinney --- FatPkg/FatPkg.ci.yaml | 50 +++ 1 file changed,

[edk2-devel] [Patch v3 04/22] BaseTools/tools_def.template: Add RC_PATH define

2019-10-29 Thread Michael D Kinney
From: Sean Brogan https://bugzilla.tianocore.org/show_bug.cgi?id=2315 Add use of RC_PATH define that provides the path to the resource compiler that is typically provided in a Windows SDK. The path changes with different Windows SDK releases. This define is set to the WINSDK_PATH_FOR_RC_EXE

[edk2-devel] [Patch v3 03/22] requirements.txt: Add python pip requirements file

2019-10-29 Thread Michael D Kinney
From: Sean Brogan https://bugzilla.tianocore.org/show_bug.cgi?id=2315 Add pip requirements file that is used to install the python pip modules build from the edk2-pytool-library and edk2-pytool-extensions repositories. These python modules provide the extensions required to perform EDK II

[edk2-devel] [Patch v3 17/22] UefiCpuPkg: Add YAML file for CI builds

2019-10-29 Thread Michael D Kinney
https://bugzilla.tianocore.org/show_bug.cgi?id=2315 Add YAML file to the package directory with the configuration of the checks to perform during a CI build. Cc: Eric Dong Cc: Ray Ni Cc: Laszlo Ersek Signed-off-by: Michael D Kinney --- UefiCpuPkg/UefiCpuPkg.ci.yaml | 51

[edk2-devel] [Patch v3 18/22] SignedCapsulePkg: Use BaseCryptLibNull to reduce package CI time

2019-10-29 Thread Michael D Kinney
https://bugzilla.tianocore.org/show_bug.cgi?id=2315 Use BaseCryptLibNull for package CI builds to reduce package build times. Enabled with PYTOOL_CONTINUOUS_INTEGRATION in YAML files. By default PYTOOL_CONTINUOUS_INTEGRATION is not defined, and the original lib mappings are preserved. Cc:

[edk2-devel] [Patch v3 15/22] SecurityPkg: Add YAML files for CI builds

2019-10-29 Thread Michael D Kinney
https://bugzilla.tianocore.org/show_bug.cgi?id=2315 Add YAML file to the package directory with the configuration of the checks to perform during a CI build. Use BaseCryptLibNull for package CI builds to reduce package build times. Enabled with CONTINUOUS_INTEGRATION in YAML files. By default

[edk2-devel] [Patch v3 10/22] FmpDevicePkg: Add YAML file for CI builds

2019-10-29 Thread Michael D Kinney
https://bugzilla.tianocore.org/show_bug.cgi?id=2315 Add YAML file to the package directory with the configuration of the checks to perform during a CI build. Use BaseCryptLibNull for package CI builds to reduce package build times. Enabled with CONTINUOUS_INTEGRATION in YAML files. By default

[edk2-devel] [Patch v3 20/22] .azurepipelines: Add Azure Pipelines YML configuration files

2019-10-29 Thread Michael D Kinney
From: Sean Brogan https://bugzilla.tianocore.org/show_bug.cgi?id=2315 Add YML configuration files used to run the EDK II Continuous Integration (CI) checks on Azure Pipelines agents. Cc: Sean Brogan Cc: Bret Barkelew Cc: Liming Gao Signed-off-by: Michael D Kinney ---

[edk2-devel] [Patch v3 12/22] MdePkg: Add YAML file for CI builds

2019-10-29 Thread Michael D Kinney
https://bugzilla.tianocore.org/show_bug.cgi?id=2315 Add YAML file to the package directory with the configuration of the checks to perform during a CI build. Cc: Liming Gao Signed-off-by: Michael D Kinney --- MdePkg/MdePkg.ci.yaml | 90 +++ 1 file

[edk2-devel] [Patch v3 01/22] Maintainers.txt: Add continuous integration(CI) directories

2019-10-29 Thread Michael D Kinney
https://bugzilla.tianocore.org/show_bug.cgi?id=2315 Add maintainers and reviewers for the directories associated with continuous integration steps. * .azurepipelines * .mergify * .pytool Cc: Andrew Fish Cc: Laszlo Ersek Cc: Leif Lindholm Signed-off-by: Michael D Kinney --- Maintainers.txt |

[edk2-devel] [Patch v3 16/22] ShellPkg: Add YAML file for CI builds

2019-10-29 Thread Michael D Kinney
https://bugzilla.tianocore.org/show_bug.cgi?id=2315 Add YAML file to the package directory with the configuration of the checks to perform during a CI build. Cc: Ray Ni Cc: Zhichao Gao Signed-off-by: Michael D Kinney --- ShellPkg/ShellPkg.ci.yaml | 55 +++

[edk2-devel] [Patch v3 14/22] PcAtChipsetPkg: Add YAML files for CI builds

2019-10-29 Thread Michael D Kinney
https://bugzilla.tianocore.org/show_bug.cgi?id=2315 Add YAML file to the package directory with the configuration of the checks to perform during a CI build. Cc: Ray Ni Signed-off-by: Michael D Kinney --- PcAtChipsetPkg/PcAtChipsetPkg.ci.yaml | 46 +++ 1 file changed,

[edk2-devel] [Patch v3 07/22] .pytool/Plugin: Add CI plugins

2019-10-29 Thread Michael D Kinney
From: Sean Brogan https://bugzilla.tianocore.org/show_bug.cgi?id=2315 Add .pytool directory to the edk2 repository with the following plugins. These plugins are in a top level directory because that can be used with all packages and platforms. * CharEncodingCheck * CompilerPlugin *

[edk2-devel] [Patch v3 06/22] BaseTools: Add BaseTools plugins to support CI

2019-10-29 Thread Michael D Kinney
From: Sean Brogan https://bugzilla.tianocore.org/show_bug.cgi?id=2315 Add the following plugins that are required to support EDK II Continuous Integration (CI) builds. These plugins are added to BaseTools because that support EDK II BaseTools features. * BuildToolsReportGenerator *

[edk2-devel] [Patch v3 13/22] NetworkPkg: Add YAML file for CI builds

2019-10-29 Thread Michael D Kinney
https://bugzilla.tianocore.org/show_bug.cgi?id=2315 Add YAML file to the package directory with the configuration of the checks to perform during a CI build. Use BaseCryptLibNull and TlsLibNull for package CI builds to reduce package build times. Enabled with CONTINUOUS_INTEGRATION in YAML

[edk2-devel] [Patch v3 05/22] BaseTools: Add YAML files with path env and tool extdeps

2019-10-29 Thread Michael D Kinney
From: Sean Brogan https://bugzilla.tianocore.org/show_bug.cgi?id=2315 Cc: Bob Feng Cc: Liming Gao Signed-off-by: Michael D Kinney --- BaseTools/Bin/gcc_aarch64_linux_ext_dep.yaml | 21 +++ BaseTools/Bin/gcc_arm_linux_ext_dep.yaml | 21 +++ BaseTools/Bin/iasl_ext_dep.yaml

[edk2-devel] [Patch v3 21/22] .merify: Add Megify YML pull request rules configuration file

2019-10-29 Thread Michael D Kinney
https://bugzilla.tianocore.org/show_bug.cgi?id=2315 Add directory for the Mergify YML configuration files that provides rules and actions used to process a pull request. * Auto commit a PR from EDK II Maintainer with 'push' label set and all CI checks pass * Auto close a PR from any developers

[edk2-devel] [Patch v3 11/22] MdeModulePkg: Add YAML file for CI builds

2019-10-29 Thread Michael D Kinney
https://bugzilla.tianocore.org/show_bug.cgi?id=2315 Add YAML file to the package directory with the configuration of the checks to perform during a CI build. Cc: Jian J Wang Cc: Hao A Wu Signed-off-by: Michael D Kinney --- MdeModulePkg/MdeModulePkg.ci.yaml | 81

[edk2-devel] How /sys/firmware/fdt getting created

2019-10-29 Thread Prabhakar Kushwaha
Hi All, I am working on Ubuntu-18.04 with UEFI on ARM64(64 bit) platform. The UEFI used is having ACPI tables. I am trying to understand where and how /sys/firmware/fdt is getting created. is it created by UEFI or grub and passed to Linux? below is the dts format of /sys/firmware/fdt.

[edk2-devel] [edk2] NOOPT and PcdOptimizeCompilerEnable

2019-10-29 Thread Tiger Liu(BJ-RD)
Hi, All: I have a question about NOOPT and PcdOptimizeCompilerEnable. If Target = NOOPT, then will disable all optimizations. And PcdOptimizeCompilerEnable has the same function. So, What's the relationship between PcdOptimizeCompilerEnable and NOOPT? Thanks Best wishes, ?

Re: [edk2-devel] [PATCH] CryptoPkg/OpensslLib: Update process_files.pl to generate .h files

2019-10-29 Thread Wang, Jian J
Reviewed-by: Jian J Wang Regards, Jian > -Original Message- > From: Zhang, Shenglei > Sent: Tuesday, October 29, 2019 3:43 PM > To: devel@edk2.groups.io > Cc: Wang, Jian J ; Lu, XiaoyuX > Subject: [PATCH] CryptoPkg/OpensslLib: Update process_files.pl to generate .h > files > >

[edk2-devel] [PATCH] CryptoPkg/OpensslLib: Update process_files.pl to generate .h files

2019-10-29 Thread Zhang, Shenglei
There are missing headers added into INF files at 8906f076de35b222a.. They are now manually added but not auto-generated. So we update the perl script to enable this feature. Meanwhile, update the order of the .h files in INF files, which are auto-generated now.