Re: [edk2-devel] [Patch 2/3] EmulatorPkg: Remove UNIX_SEC_BUILD/WIN_SEC_BUILD

> -Original Message- > From: devel@edk2.groups.io [mailto:devel@edk2.groups.io] On Behalf Of > Michael D Kinney > Sent: Wednesday, August 07, 2019 12:20 PM > To: devel@edk2.groups.io > Cc: Justen, Jordan L; Andrew Fish; Ni, Ray > Subject: [edk2-devel] [Patch 2/3] EmulatorPkg: Remove >

Re: [edk2-devel] [PATCH] MdeModulePkg/DxeCapsuleLibFmp: Add missing description for parameter

> -Original Message- > From: Zhang, Shenglei > Sent: Tuesday, August 06, 2019 10:18 AM > To: devel@edk2.groups.io > Cc: Wang, Jian J; Wu, Hao A > Subject: [PATCH] MdeModulePkg/DxeCapsuleLibFmp: Add missing > description for parameter > > The description of parameter CapFileName is missing

Re: [edk2-devel] [Patch 1/3] EmulatorPkg: Fix VS20xx IA32 boot failure

> -Original Message- > From: devel@edk2.groups.io [mailto:devel@edk2.groups.io] On Behalf Of > Michael D Kinney > Sent: Wednesday, August 07, 2019 12:20 PM > To: devel@edk2.groups.io > Cc: Justen, Jordan L; Andrew Fish; Ni, Ray > Subject: [edk2-devel] [Patch 1/3] EmulatorPkg: Fix VS20xx

Re: [edk2-devel] [Patch 3/3] EmulatorPkg: Add -D DISABLE_NEW_DEPRECATED_INTERFACES

Hello Mike Some inline comments below: > -Original Message- > From: devel@edk2.groups.io [mailto:devel@edk2.groups.io] On Behalf Of > Michael D Kinney > Sent: Wednesday, August 07, 2019 12:20 PM > To: devel@edk2.groups.io > Cc: Justen, Jordan L; Andrew Fish; Ni, Ray > Subject:

[edk2-devel] [PATCH 1/2] BaseTools/Capsule: Add WindowsCapsuleSupportHelper

Add initial version WindowsCapsuleSupportHelper.py original source comes from https://github.com/microsoft/mu_basecore/blob/dev/201905/BaseTools/Plugin/WindowsCapsuleSupportHelper/WindowsCapsuleSupportHelper.py commit ebd9aad90dee3a3094b575928670cf81b8c685b5 Convert EOL to CRLF Cc: Sean Brogan

[edk2-devel] [PATCH 0/2] Enable Windows Firmware Update Driver Tool

REF: https://bugzilla.tianocore.org/show_bug.cgi?id=1837 The tool makes driver capsule as input, leverage edk2-pytool-library to creates INF, Cat, and signs it. The generated Windows Driver package is composed by drivername.cap, drivername.inf and drivername.cat files. Eric Jin (2):

Re: [edk2-devel] [Patch 2/3] EmulatorPkg: Remove UNIX_SEC_BUILD/WIN_SEC_BUILD

> -Original Message- > From: Kinney, Michael D > Sent: Wednesday, August 07, 2019 3:45 PM > To: Wu, Hao A; devel@edk2.groups.io; Kinney, Michael D > Cc: Justen, Jordan L; Andrew Fish; Ni, Ray > Subject: RE: [edk2-devel] [Patch 2/3] EmulatorPkg: Remove > UNIX_SEC_BUILD/WIN_SEC_BUILD > >

Re: [edk2-devel] [Patch 1/3] EmulatorPkg: Fix VS20xx IA32 boot failure

> -Original Message- > From: Kinney, Michael D > Sent: Wednesday, August 07, 2019 3:43 PM > To: Wu, Hao A; devel@edk2.groups.io; Kinney, Michael D > Cc: Justen, Jordan L; Andrew Fish; Ni, Ray > Subject: RE: [edk2-devel] [Patch 1/3] EmulatorPkg: Fix VS20xx IA32 boot > failure > > I

Re: [edk2-devel] [PATCH v1 0/6] Prevent buffer overruns in acpiview table parsers

For this patch series. Reviewed-by: Sami Mujawar Regards, Sami Mujawar -Original Message- From: Krzysztof Koch Sent: 01 August 2019 09:44 AM To: devel@edk2.groups.io Cc: jaben.car...@intel.com; ray...@intel.com; zhichao@intel.com; Sami Mujawar ; Matteo Carlini ; nd Subject:

Re: [edk2-devel] [Patch 1/3] EmulatorPkg: Fix VS20xx IA32 boot failure

I observed 2 failure modes 1) The range could not be mapped at the request address And an error message was displayed and the app exited. 2) The mapping passed, but when it was accessed by the Sec module, the app generated an exception. Mike > -Original Message- > From: Wu, Hao A

[edk2-devel] [PATCH 2/2] BaseTools/Capsule: Tool to generate Windows Firmware Update Driver

REF: https://bugzilla.tianocore.org/show_bug.cgi?id=1837 The tool is designed to generate Windows Firmware Update Drivers, the input is one drivername.cap with related parameters, the output Windows Driver package are composed by drivername.cap, drivername.inf and drivername.cat to update the

[edk2-devel] [edk2-test][PATCH V3 1/3] Adding HII Config Access Protocol

From: Shrishail Patil Adding HII Config Access Protocol to IHVSCT This patch contains source code which is copied from UEFISCT and added into IHVSCT. Addressed the review comments relating to commented code removal. Contributed-under: TianoCore Contribution Agreement 1.1 Signed-off-by:

Re: [edk2-devel] [Patch 4/4] MdePkg/X64/ProcessorBind.h: Fix EmulatorPkg X64 XCODE5

Reviewed-by: Liming Gao >-Original Message- >From: Kinney, Michael D >Sent: Friday, August 02, 2019 11:23 AM >To: devel@edk2.groups.io >Cc: Gao, Liming ; Justen, Jordan L >; Andrew Fish ; Ni, Ray > >Subject: [Patch 4/4] MdePkg/X64/ProcessorBind.h: Fix EmulatorPkg X64 >XCODE5 >

Re: [edk2-devel] [PATCH v4 06/35] OvmfPkg/XenResetVector: Add new entry point for Xen PVH

On Mon, Jul 29, 2019 at 04:39:15PM +0100, Anthony PERARD wrote: > Add a new entry point for Xen PVH that enter directly in 32bits. > > Information on the expected state of the machine when this entry point > is used can be found at: > https://xenbits.xenproject.org/docs/unstable/misc/pvh.html >

Re: [edk2-devel] [PATCH 0/3] Enhance ESRT to support multiple controllers

On 08/05/19 08:19, Wu, Hao A wrote: >> -Original Message- >> From: devel@edk2.groups.io [mailto:devel@edk2.groups.io] On Behalf Of >> Eric Jin >> Sent: Wednesday, July 31, 2019 10:59 PM >> To: devel@edk2.groups.io >> Subject: [edk2-devel] [PATCH 0/3] Enhance ESRT to support multiple >>

Re: [edk2-devel] [Patch 3/3] BaseTools/PatchCheck: Disable text conversion in 'git show'

On 08/06/19 00:01, Kinney, Michael D wrote: > Laszlo, > > The context of this change is only to the PatchCheck.py tool. > and how that tool uses git show. > > I agree with the summary of very flexible capabilities in git > to help developers review different types of files. All of > those

Re: Microsoft imports - was Re: [edk2-devel] [PATCH V2] BaseTools:Add extra debugging message

Leif: >-Original Message- >From: Leif Lindholm [mailto:leif.lindh...@linaro.org] >Sent: Friday, August 02, 2019 5:55 PM >To: devel@edk2.groups.io >Cc: Feng, Bob C ; Fan, ZhijuX ; >Max Knutsen ; Philippe Mathieu-Daude >; Andrew Fish ; Laszlo Ersek >; Kinney, Michael D ; >Gao, Liming

Re: [edk2-devel] Cancelled Event: TianoCore Design / Bug Triage - EMEA - Wednesday, 7 August 2019 #cal-cancelled

No topics for discussion this week. We will triage bugs in tomorrow's APAC meeting. --S On Wed, Aug 7, 2019 at 6:14 AM devel@edk2.groups.io Calendar wrote: > > Cancelled: TianoCore Design / Bug Triage - EMEA > > This event has been cancelled. > > When: > Wednesday, 7 August 2019 > 8:00am to

Re: [edk2-devel] [PATCH V2] BaseTools:Add the Judgment Method of "--exclude"

Sorry for the missing. Reviewed-by: Liming Gao >-Original Message- >From: Fan, ZhijuX >Sent: Friday, June 28, 2019 3:01 PM >To: devel@edk2.groups.io >Cc: Gao, Liming ; Feng, Bob C >Subject: [PATCH V2] BaseTools:Add the Judgment Method of "--exclude" > >BZ:

Re: [edk2-devel] [edk2-platforms: PATCH v2 1/1] Platform/RPi3: Add Debian 10 installation in Systems.md

Hi Leif, On 2019.08.06 18:25, Leif Lindholm wrote: Minor suggestions inline: On Thu, Jul 25, 2019 at 04:22:48PM +0100, Pete Batard wrote: This documents the installation of vanilla Debian 10.0 ARM64 (netinst), which we validated for both Model B and Model B+. Also fix an erroneous reference

Re: [edk2-devel] [Patch 0/3] BaseTools/PatchCheck: Fix false positives

Mike: For this patch set, I am OK for those changes. Reviewed-by: Liming Gao But, I see there are some discussion on patch 3. After we make the conclusion, you can push it. Thanks Liming >-Original Message- >From: Kinney, Michael D >Sent: Friday, August 02, 2019 8:13 AM >To:

[edk2-devel] Cancelled Event: TianoCore Design / Bug Triage - EMEA - Wednesday, 7 August 2019 #cal-cancelled

BEGIN:VCALENDAR VERSION:2.0 PRODID:-//Groups.io Inc//Groups.io Calendar//EN METHOD:CANCEL CALSCALE:GREGORIAN BEGIN:VEVENT STATUS:CANCELLED UID:calendar.15...@groups.io DTSTAMP:20190807T131407Z ORGANIZER;CN=Stephano Cetola:mailto:stephano.cet...@linux.intel.com

Re: [edk2-devel] [PATCH v4 09/35] OvmfPkg/OvmfXen: use a TimerLib instance that depends only on the CPU

On Mon, Jul 29, 2019 at 04:39:18PM +0100, Anthony PERARD wrote: > The ACPI Timer isn't present in a PVH guest, but local APIC works on > both PVH and HVM. > > Note that the use of SecPeiDxeTimerLibCpu might be an issue with a > driver of type DXE_RUNTIME_DRIVER. I've attempted to find out which

Re: [edk2-devel] [PATCH v4 29/35] OvmfPkg/OvmfXen: Override PcdFSBClock to Xen vLAPIC timer frequency

On Mon, Jul 29, 2019 at 04:39:38PM +0100, Anthony PERARD wrote: > PcdFSBClock is used by SecPeiDxeTimerLibCpu, the TimerLib > implementation. It will also be used by XenTimerDxe. Override > PcdFSBClock to match Xen vLAPIC timer frequency. I'm kind of surprised that his is not automatically

Re: [edk2-devel] UefiCpuPkg/PiSmmCpuDxeSmm: don't free page table pages that are required to handle current page fault

On 08/05/19 23:57, Krzysztof Rusocki wrote: > From: Damian Nikodem > > Reclaim may free page table pages that are required to handle current page > fault. This causes a page leak, and, after sufficent number of specific > page fault+reclaim pairs, we run out of reclaimable pages and hit: > >

Re: [edk2-devel] static data in dxe_runtime modules

ummm... not sure why, but I never got this email in my inbox. I only see it in my list folder. I see myself addressed on it as: Laszlo Ersek via Groups.Io which could be the reason. Anyway: On 08/05/19 12:18, Roman Kagan wrote: > On Sat, Aug 03, 2019 at 04:03:04AM +0200, Laszlo Ersek via

Re: [edk2-devel] [PATCH v4 23/35] OvmfPkg/XenPlatformPei: Rework memory detection

On Mon, Jul 29, 2019 at 04:39:32PM +0100, Anthony PERARD wrote: > When running as a Xen PVH guest, there is no CMOS to read the memory > size from. Rework GetSystemMemorySize(Below|Above)4gb() so they can > work without CMOS by reading the e820 table. > > Rework XenPublishRamRegions to also care

Re: [edk2-devel] [PATCH 1/1] UefiCpuPkg/PiSmmCpuDxeSmm: Fix coding style

On 08/07/19 04:09, Shenglei Zhang wrote: > 1. Update CPUStatus to CpuStatus in comments to align comments >with code. > 2. Add "OUT" attribute for "ProcedureArguments" parameter in function >header. > > Cc: Eric Dong > Cc: Ray Ni > Cc: Laszlo Ersek > Signed-off-by: Shenglei Zhang >

Re: [edk2-devel] [Patch v2 0/2] UefiCpuPkg: Remove debug message.

On 08/05/19 08:43, Eric Dong wrote: > This debug message may be called by BSP and APs. It may > caused ASSERT when APs call this debug code. > > In order to avoid system boot assert, Remove this debug > message. > > Cc: Ray Ni > Cc: Laszlo Ersek > > Eric Dong (2): >

Re: [edk2-devel] [PATCH v4 12/35] OvmfPkg/XenPlatformPei: Grab RSDP from PVH guest start of day struct

On Mon, Jul 29, 2019 at 04:39:21PM +0100, Anthony PERARD wrote: > Check if there's a start of the day struct provided to PVH guest, save > the ACPI RSDP address for later. > > This patch import import arch-x86/hvm/start_info.h from xen.git. You seem to change the types when importing

Re: [edk2-devel] [PATCH v4 33/35] OvmfPkg: Introduce XenIoPvhDxe to initialize Grant Tables

On Mon, Jul 29, 2019 at 04:39:42PM +0100, Anthony PERARD wrote: > XenIoPvhDxe use XenIoMmioLib to reserve some space to be use by the > Grant Tables. > > The call is only done if it is necessary, we simply detect if the > guest is PVH, as in this case there is currently no PCI bus, and no > PCI

[edk2-devel] [PATCH 0/4] Add EDKII_UFS_HC_PLATFORM_PROTOCOL to support platform specific programming of UFS host controllers

To cover additional host controller programming mentioned in the UFS specification we have added an additional protocol that allows the UEFI driver to give control to platform driver. This allows the platform to perform any additional steps needed for the stable operation. Test coverage: Tested

[edk2-devel] [PATCH 1/4] MdeModulePkg: Add definition of the EDKII_UFS_HC_PLATFORM_PROTOCOL

https://bugzilla.tianocore.org/show_bug.cgi?id=1343 EDKII_UFS_HC_PLATFORM_PROTOCOL will allow the platform to inject platform specific logic into standard UFS flows. Right now we support callbacks pre and post host controller enable and pre and post link startup. Provided callbacks allow the

Re: [edk2-devel] [edk2-platforms: PATCH v2 1/1] Platform/RPi3: Add Debian 10 installation in Systems.md

Hi Pete, On Wed, Aug 07, 2019 at 03:24:10PM +0100, Pete Batard wrote: > > > +Below are steps you can follow to install Debian 10 onto an SD card: > > > +* Partition the media as MBR and create a ~300 MB partition on it with > > > MBR type `0x0e`. > > > + __Note:__ Make sure that the partition

Re: [edk2-devel] [PATCH] MdePkg: Add MmAccess and MmControl definition.

On 08/06/19 12:08, Ni, Ray wrote: > Laszlo, > Do you want to avoid touching the OvmfPkg due the file removal in > MdeModulePkg? My main problem with this *specific* update proposal is that the identifiers (type names and such) that would be subject to removal were publicly specified in earlier

Re: [edk2-devel] [PATCH v4 35/35] OvmfPkg/OvmfXen: use RealTimeClockRuntimeDxe from EmbeddedPkg

On Mon, Jul 29, 2019 at 04:39:44PM +0100, Anthony PERARD wrote: > A Xen PVH guest doesn't have a RTC that OVMF would expect, so > PcatRealTimeClockRuntimeDxe fails to initialize and prevent the > firmware from finish to boot. To prevent that, we will use > XenRealTimeClockLib which simply always

Re: [edk2-devel] [PATCH v4 20/35] OvmfPkg/XenPlatformPei: Introduce XenPvhDetected

On Mon, Jul 29, 2019 at 04:39:29PM +0100, Anthony PERARD wrote: > XenPvhDetected() can be used to figure out if OVMF has started via the > Xen PVH entry point. > > Ref: https://bugzilla.tianocore.org/show_bug.cgi?id=1689 > Signed-off-by: Anthony PERARD > Acked-by: Laszlo Ersek Thanks, I've got

Re: [edk2-devel] [PATCH v4 22/35] OvmfPkg/XenPlatformPei: no hvmloader: get the E820 table via hypercall

On Mon, Jul 29, 2019 at 04:39:31PM +0100, Anthony PERARD wrote: > When the Xen PVH entry point has been used, hvmloader hasn't run and > hasn't prepared an E820 table. The only way left to get an E820 table > is to ask Xen via an hypercall, the call can only be made once so keep > the result

Re: [edk2-devel] static data in dxe_runtime modules

> On Aug 7, 2019, at 10:29 AM, Laszlo Ersek wrote: > > ummm... not sure why, but I never got this email in my inbox. I only see > it in my list folder. I see myself addressed on it as: > > Laszlo Ersek via Groups.Io > > > which could be the reason. > >

[edk2-devel] [PATCH 2/4] MdeModulePkg/UfsPassThruDxe: Refactor UfsExecUicCommand function

https://bugzilla.tianocore.org/show_bug.cgi?id=1343 UfsExecUicCommand function has been refactored to allow the caller to check the command results which is important for commands such as UIC read. Cc: Hao A Wu Signed-off-by: Mateusz Albecki ---

[edk2-devel] [PATCH 3/4] MdeModulePkg/UfsPassThruDxe: Refactor private data to use EDKII_UFS_HC_INFO

https://bugzilla.tianocore.org/show_bug.cgi?id=1343 Private data has been refactored to use EDKII_UFS_HC_INFO structure to store host controller capabilities and version information. Getting host controller data has been moved into single place and is done before host controller enable. Cc: Hao

[edk2-devel] [PATCH 4/4] MdeModulePkg/UfsPassThruDxe: Implement EDKII_UFS_HC_PLATFORM_PROTOCOL

https://bugzilla.tianocore.org/show_bug.cgi?id=1343 This commit adds EDKII_UFS_HC_PLATFORM_PROTOCOL implementation in UfsPassThruDxe driver in version 1. Driver assumes that at most one instance of the protocol exists in the system. Presence of the protocol is not mandatory. Cc: Hao A Wu

Re: [edk2-devel] [Patch v2 1/2] UefiCpuPkg/RegisterCpuFeaturesLib: Remove debug message.

On 2019-08-05 00:43, Dong, Eric wrote: > This debug message may be called by BSP and APs. It may > caused ASSERT when APs call this debug code. > > In order to avoid system boot assert, Remove this debug > message. > > Signed-off-by: Eric Dong > Cc: Ray Ni > Cc: Laszlo Ersek I can't find it

Re: [edk2-devel] [tianocore-docs EDK_II_Secure_Coding_Guide PATCH] Add Appendix: Threat Mode for EDK II.

(+ Laurie) On 08/05/19 09:47, Yao, Jiewen wrote: > This patch adds "Threat model for EDK II" as the appendix section > > of "EDK II secure coding guide" document. > > > The threat model discussed here is a general guide and serves as the baseline > of > > the EDK II firmware. For each

Re: [edk2-devel] [tianocore-docs EDK_II_Secure_Coding_Guide PATCH] Add Appendix: Threat Mode for EDK II.

Laurie created https://github.com/tianocore/tianocore.github.io/wiki/EDK-II-Security-White-Papers for the security collateral. Vincent -Original Message- From: devel@edk2.groups.io On Behalf Of Laszlo Ersek Sent: Wednesday, August 7, 2019 10:46 AM To: Yao, Jiewen Cc:

[edk2-devel] [edk2-platforms: PATCH v2] Marvell/Drivers: XenonDxe: Explicitly disable HS400

On another SoC revision, the capability register marks HS400 support as enabled. However in case the interface itself is powered with 3.3V this flag must be unset by the SdMmcOverride protocol callback - otherwise the generic EmmcSwitchToHS400 () would be executed with a failure. Ensure that in

Re: [edk2-devel] [Patch 3/3] BaseTools/PatchCheck: Disable text conversion in 'git show'

On 2019-08-05 15:01:14, Kinney, Michael D wrote: > Laszlo, > > The context of this change is only to the PatchCheck.py tool. > and how that tool uses git show. > > I agree with the summary of very flexible capabilities in git > to help developers review different types of files. All of > those

Re: [edk2-devel] [Patch v2 0/2] UefiCpuPkg: Remove debug message.

On 08/07/19 19:11, Laszlo Ersek wrote: > On 08/05/19 08:43, Eric Dong wrote: >> This debug message may be called by BSP and APs. It may >> caused ASSERT when APs call this debug code. >> >> In order to avoid system boot assert, Remove this debug >> message. >> >> Cc: Ray Ni >> Cc: Laszlo Ersek

[edk2-devel] [edk2-platforms][PATCH V1 2/2] KabylakeOpenBoardPkg: Fix duplicate GUID in DSC

REF:https://bugzilla.tianocore.org/show_bug.cgi?id=2063 Assigns a unique GUID to PLATFORM_GUID. Cc: Chasel Chiu Cc: Nate DeSimone Cc: Liming Gao Signed-off-by: Michael Kubacki --- Platform/Intel/KabylakeOpenBoardPkg/KabylakeRvp3/OpenBoardPkg.dsc | 2 +- 1 file changed, 1 insertion(+), 1

[edk2-devel] [edk2-platforms][PATCH V1 1/2] ClevoOpenBoardPkg: Fix duplicate GUID in DSC

REF:https://bugzilla.tianocore.org/show_bug.cgi?id=2064 Assigns a unique GUID to PLATFORM_GUID. Cc: Ankit Sinha Cc: Nate DeSimone Cc: Liming Gao Signed-off-by: Michael Kubacki --- Platform/Intel/ClevoOpenBoardPkg/N1xxWU/OpenBoardPkg.dsc | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-)

[edk2-devel] [edk2-platforms][PATCH V1 0/2] Fix Duplicated Platform DSC GUID

REF:https://bugzilla.tianocore.org/show_bug.cgi?id=2063 REF:https://bugzilla.tianocore.org/show_bug.cgi?id=2064 The PLATFORM_GUID currently the same in the following DSC files: * Platform/Intel/ClevoOpenBoardPkg/N1xxWU/OpenBoardPkg.dsc *

[edk2-devel] [edk2-platforms: PATCH 0/9] Marvell Octeon CN913X SoC family support

Hi, Marvell Octeon CN913X SoC is a new device, which is built of upgraded hardware blocks known from previously supported line of SoCs. It is avaialble in 3 variants - CN9130/CN9131/CN9132. CN9130 is made of a single Application Processor unit (AP807) and one internal south bridge (CP115). It

[edk2-devel] [edk2-platforms: PATCH 4/9] Marvell/Library: ArmadaSoCDescLib: Extend Xenon information

Hitherto SoC description library code assumed that there could be only two Xenon SdMmc controller instances in the SoC. Remove this limitation, so that to support CN913x SoCs, which may have up to 4 of such interfaces. Signed-off-by: Marcin Wojtas ---

[edk2-devel] [edk2-platforms: PATCH 3/9] Marvell/Cn9130Db: Introduce board support

This patch introduces all necessary components required for building EDK2 firmware for CN9130-DB setup A. Because the board is modular and can be extended to support also CN9131 and CN9132 SoC variants, extract common part into .dsc.inc file, which will be included by them. Signed-off-by: Marcin

[edk2-devel] [edk2-platforms: PATCH 9/9] Marvell/Drivers: SmbiosPlatformDxe: Use more generic board name

SmbiosPlatformDxe is used both by Armada 7k8k and CN913x platforms. Replace board name placeholder in order to avoid confusion. Signed-off-by: Marcin Wojtas --- Silicon/Marvell/Drivers/SmbiosPlatformDxe/SmbiosPlatformDxe.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git

[edk2-devel] [PATCH] ShellPkg/UefiShellLevel2CommansLib: Pointer Resonse should be checked

REF: https://bugzilla.tianocore.org/show_bug.cgi?id=2049 ShellPkg\Library\UefiShellLevel2CommandsLib\Cp.c line 104 and ShellPkg\Library\UefiShellLevel2CommandsLib\Mv.c line 640, the pointer variable Response may be a NULL pointer. So we should make sure that it isn't NULL before dereference it.

Re: [edk2-devel] [edk2-platforms][PATCH V1 2/2] KabylakeOpenBoardPkg: Fix duplicate GUID in DSC

Reviewed-by: Chasel Chiu > -Original Message- > From: Kubacki, Michael A > Sent: Thursday, August 8, 2019 5:15 AM > To: devel@edk2.groups.io > Cc: Chiu, Chasel ; Desimone, Nathaniel L > ; Gao, Liming > Subject: [edk2-platforms][PATCH V1 2/2] KabylakeOpenBoardPkg: Fix duplicate > GUID

Re: [edk2-devel] [tianocore-docs EDK_II_Secure_Coding_Guide PATCH] Add Appendix: Threat Mode for EDK II.

Right. You can get it from below step: 1) goto https://www.tianocore.org/ 2) click "document" on the left, then goto https://www.tianocore.org/docs/ 3) click "EDK II Documents wiki page", then goto https://github.com/tianocore/tianocore.github.io/wiki/EDK-II-Documents 4) click "EDK II Security

[edk2-devel] [edk2-platforms: PATCH 1/9] Marvell/Cn9130Db: Add ACPI tables

This patch adds ACPI tables and necessary headers, which are common for Cn913x SoCs and the CN9130 development board (variant A). Wiring up of support will be done in the follow-up commits. Signed-off-by: Marcin Wojtas --- Silicon/Marvell/OcteonTx/AcpiTables/T91/Cn9130DbA.inf | 56

[edk2-devel] [edk2-platforms: PATCH 2/9] Marvell/Cn9130Db: Add DeviceTree

This patch adds device tree sources which are common for Cn913x SoCs and the CN9130 development board (variant A). Wiring up of support will be done in the follow-up commits. Signed-off-by: Marcin Wojtas --- Silicon/Marvell/OcteonTx/DeviceTree/T91/Cn9130DbA.inf | 22 +

[edk2-devel] [edk2-platforms: PATCH 7/9] Marvell/Cn9131Db: Introduce board support

This patch introduces all necessary components required for building EDK2 firmware for CN9131-DB setup A. Signed-off-by: Marcin Wojtas --- Platform/Marvell/Cn913xDb/Cn9131DbA.dsc.inc | 72 ++ Platform/Marvell/Cn913xDb/Cn9131DbA.dsc

[edk2-devel] [edk2-platforms: PATCH 5/9] Marvell/Library: MppLib: Allow to configure more Xenon PHYs

Hitherto MppLib code assumed that there could be only two Xenon SdMmc controllers' PHYs. Remove this limitation, so that to support CN913x SoCs, which may have up to 4 of such interfaces. Signed-off-by: Marcin Wojtas --- Silicon/Marvell/Library/MppLib/MppLib.c | 4 +--- 1 file changed, 1

[edk2-devel] [edk2-platforms: PATCH 6/9] Marvell/Library: IcuLib: Fix debug information

In case the number of CP11x components exceeded the maximum of currently supported, the user is informed with the information. It turned out that the print arguments were incorrect - fix it. Signed-off-by: Marcin Wojtas --- Silicon/Marvell/Library/IcuLib/IcuLib.c | 4 ++-- 1 file changed, 2

Re: [edk2-devel] [Patch 1/1] BaseTools: Fixed an issue that build raise exception

Reviewed-by: Liming Gao >-Original Message- >From: devel@edk2.groups.io [mailto:devel@edk2.groups.io] On Behalf Of >Bob Feng >Sent: Friday, August 02, 2019 3:22 PM >To: devel@edk2.groups.io >Cc: Gao, Liming ; Feng, Bob C >Subject: [edk2-devel] [Patch 1/1] BaseTools: Fixed an issue that

[edk2-devel] [PATCH V3 01/15] MdeModulePkg: Add guids for capsule update

From: Bret Barkelew REF: https://bugzilla.tianocore.org/show_bug.cgi?id=1772 And add two guids gEdkiiCapsuleArmedResetGuid and gEdkiiCapsuleUpdateCompleteResetGuid for capsule Update. Cc: Jian J Wang Cc: Hao Wu Cc: Ray Ni Cc: Star Zeng Cc: Liming Gao Cc: Sean Brogan Cc: Michael Turner

[edk2-devel] [Patch V2 0/3] EmulatorPkg: Fix VS20xx IA32 boot and simplify build config

New in V2 = * Fix size values in safe string function calls. * Update POSIX sources to use AsciiStrCpyS() and AsciiStrCatS(). * Verify that no exceptions occur if EMU_MAGIC_PAGE() can not be mapped. An error message is generated and the host app exits normally. * Update EmulatorPkg DEC

[edk2-devel] [Patch V2 1/3] EmulatorPkg: Fix VS20xx IA32 boot failure

https://bugzilla.tianocore.org/show_bug.cgi?id=2056 The IA32 build of the EmulatorPkg for VS20xx does not boot because the default value of PCD PcdPeiServicesTablePage rarely succeeds to be mapped on IA32 Windows Host Environments. Change the DEC default value for this PCD to a value that is

[edk2-devel] [Patch V2 3/3] EmulatorPkg: Add -D DISABLE_NEW_DEPRECATED_INTERFACES

https://bugzilla.tianocore.org/show_bug.cgi?id=162 Update EmulatorPkg specific modules and libraries to use safe string functions in BaseLib and safe PcdSetxx() functions in PcdLib. With these updates, the define DISABLE_NEW_DEPRECATED_INTERFACES is enabled in the DSC file. Cc: Jordan Justen

[edk2-devel] [Patch V2 2/3] EmulatorPkg: Remove UNIX_SEC_BUILD/WIN_SEC_BUILD

https://bugzilla.tianocore.org/show_bug.cgi?id=2055 Remove the use of the defines UNIX_SEC_BUILD and WIN_SEC_BUILD. This simplifies the build command for the EmulatorPkg. Instead, use !if statements in the DSC file using $(ARCH) and $(FAMILY) to determine if the build is for a Windows or POSIX

Re: [edk2-devel] [PATCH V3 11/15] UefiPayloadPkg: Add ResetUtilityLib to UefiPayloadPkgIa32X64.dsc

Reviewed-by: Maurice Ma Regards, Maurice > -Original Message- > From: Gao, Zhichao > Sent: Wednesday, August 7, 2019 18:08 > To: devel@edk2.groups.io > Cc: Ma, Maurice ; Dong, Guo > ; You, Benjamin ; Gao, > Liming > Subject: [PATCH V3 11/15] UefiPayloadPkg: Add ResetUtilityLib to >

Re: [edk2-devel] [PATCH V3 10/15] UefiPayloadPkg/UefiPayloadPkgIa32.dsc: Add ResetUtilityLib to dsc file

Reviewed-by: Maurice Ma Regards, Maurice > -Original Message- > From: Gao, Zhichao > Sent: Wednesday, August 7, 2019 18:08 > To: devel@edk2.groups.io > Cc: Ma, Maurice ; Dong, Guo > ; You, Benjamin ; Gao, > Liming > Subject: [PATCH V3 10/15] UefiPayloadPkg/UefiPayloadPkgIa32.dsc: Add >

Re: [edk2-devel] [Patch 1/3] EmulatorPkg: Fix VS20xx IA32 boot failure

Hao Wu, I could not reproduce the exception case today. There must have been something incorrect about my build environment. I have verified an error message with normal exit always occurs. Mike > -Original Message- > From: Kinney, Michael D > Sent: Wednesday, August 7, 2019 8:52 AM >

Re: [edk2-devel] [PATCH 1/4] MdeModulePkg: Add definition of the EDKII_UFS_HC_PLATFORM_PROTOCOL

> -Original Message- > From: Albecki, Mateusz > Sent: Thursday, August 08, 2019 12:51 AM > To: devel@edk2.groups.io > Cc: Albecki, Mateusz; Wu, Hao A > Subject: [PATCH 1/4] MdeModulePkg: Add definition of the > EDKII_UFS_HC_PLATFORM_PROTOCOL > >

Re: [edk2-devel] [PATCH 3/4] MdeModulePkg/UfsPassThruDxe: Refactor private data to use EDKII_UFS_HC_INFO

Hello Mateusz, One inline comment below: > -Original Message- > From: devel@edk2.groups.io [mailto:devel@edk2.groups.io] On Behalf Of > Albecki, Mateusz > Sent: Thursday, August 08, 2019 12:51 AM > To: devel@edk2.groups.io > Cc: Albecki, Mateusz; Wu, Hao A > Subject: [edk2-devel] [PATCH

Re: [edk2-devel] [PATCH 4/4] MdeModulePkg/UfsPassThruDxe: Implement EDKII_UFS_HC_PLATFORM_PROTOCOL

Hello Mateusz, One inline comment below: > -Original Message- > From: Albecki, Mateusz > Sent: Thursday, August 08, 2019 12:51 AM > To: devel@edk2.groups.io > Cc: Albecki, Mateusz; Wu, Hao A > Subject: [PATCH 4/4] MdeModulePkg/UfsPassThruDxe: Implement > EDKII_UFS_HC_PLATFORM_PROTOCOL >

Re: [edk2-devel] [PATCH 0/4] Add EDKII_UFS_HC_PLATFORM_PROTOCOL to support platform specific programming of UFS host controllers

Dear stewards and Liming, Sorry for top-posting. I plan to add the below new feature which is targeting for the upcoming edk2-stable201908 tag: * Add EDKII_UFS_HC_PLATFORM_PROTOCOL to support platform specific programming of UFS host controllers The BZ tracker for the feature request is at:

Re: [edk2-devel] [PATCH v4 5/8] MdePkg/Cpuid.h: Move Cpuid.h from UefiCpuPkg to MdePkg

Reviewed-by: Liming Gao >-Original Message- >From: Ni, Ray >Sent: Thursday, August 01, 2019 5:58 PM >To: devel@edk2.groups.io >Cc: Kinney, Michael D ; Gao, Liming >; Dong, Eric ; Laszlo Ersek > >Subject: [PATCH v4 5/8] MdePkg/Cpuid.h: Move Cpuid.h from UefiCpuPkg to >MdePkg > >REF:

[edk2-devel] [edk2-test][PATCH V3 2/3] Adding HII to IHVSCT

From: Shrishail Patil Adding HII Config Access Protocol to IHVSCT This patch contains the changes to build/compilation Changes. Addressed the review comments relating to adding space between sections. Contributed-under: TianoCore Contribution Agreement 1.1 Signed-off-by: shrishail patil

[edk2-devel] [edk2-test][PATCH V3 2/3] Adding FMP to IHVSCT.

From: Shrishail Patil Adding Firmware Management Protocol to IHVSCT. This patch contains the changes to build/compilation Changes. Addressed the review comments relating to adding space between sections. Contributed-under: TianoCore Contribution Agreement 1.1 Signed-off-by: shrishail

Re: [edk2-devel] [Patch v2 0/2] UefiCpuPkg: Remove debug message.

Hi Laszlo, Good catch, I will follow up to clean it. Thanks. Thanks, Eric > -Original Message- > From: Laszlo Ersek [mailto:ler...@redhat.com] > Sent: Thursday, August 8, 2019 2:14 AM > To: Dong, Eric ; devel@edk2.groups.io > Cc: Ni, Ray > Subject: Re: [Patch v2 0/2] UefiCpuPkg: Remove

[edk2-devel][edk2-platforms: 10/22] Platform/DeveloperBox.dsc: Add ResetUtilityLib to dsc file

REF: https://bugzilla.tianocore.org/show_bug.cgi?id=1772 ResetUtilityLib would be consumed by CapsuleRuntimeDxe. So add it for the platform dsc file. Cc: Ard Biesheuvel Cc: Leif Lindholm Cc: Michael D Kinney Cc: Liming Gao Signed-off-by: Zhichao Gao ---

[edk2-devel] [PATCH V3 05/15] ArmVirtPkg/ArmVirtXen.dsc: Add ResetUtilityLib to dsc file

REF: https://bugzilla.tianocore.org/show_bug.cgi?id=1772 ResetUtilityLib would be consumed by CapsuleRuntimeDxe. So add it for the platform dsc file. Signed-off-by: Zhichao Gao --- ArmVirtPkg/ArmVirtXen.dsc | 1 + 1 file changed, 1 insertion(+) diff --git a/ArmVirtPkg/ArmVirtXen.dsc

[edk2-devel] [PATCH V3 09/15] OvmfPkg/OvmfPkgX64.dsc: Add ResetUtilityLib to dsc file

REF: https://bugzilla.tianocore.org/show_bug.cgi?id=1772 ResetUtilityLib would be consumed by CapsuleRuntimeDxe. So add it for the platform dsc file. Cc: Jordan Justen Cc: Laszlo Ersek Cc: Ard Biesheuvel Cc: Liming Gao Signed-off-by: Zhichao Gao --- OvmfPkg/OvmfPkgX64.dsc | 2 ++ 1 file

[edk2-devel][edk2-platforms: 00/22] Add ResetUtilityLib to dsc files that contian CapsuleRuntimeDxe

The ResetUtilityLib would be consumed by CapsuleRuntimeDxe. So add it to the related platform dsc files. Cc: Ard Biesheuvel Cc: Leif Lindholm Cc: Michael D Kinney Cc: Michael Kubacki Cc: Chasel Chiu Cc: Nate DeSimone Cc: Zailiang Sun Cc: Yi Qian Cc: Kelly Steele Zhichao Gao (22):

[edk2-devel][edk2-platforms: 09/22] Platform/D06.dsc: Add ResetUtilityLib to dsc file

REF: https://bugzilla.tianocore.org/show_bug.cgi?id=1772 ResetUtilityLib would be consumed by CapsuleRuntimeDxe. So add it for the platform dsc file. Cc: Ard Biesheuvel Cc: Leif Lindholm Cc: Michael D Kinney Cc: Liming Gao Signed-off-by: Zhichao Gao --- Platform/Hisilicon/D06/D06.dsc | 2

[edk2-devel][edk2-platforms: 07/22] Platform/D03.dsc: Add ResetUtilityLib to dsc file

REF: https://bugzilla.tianocore.org/show_bug.cgi?id=1772 ResetUtilityLib would be consumed by CapsuleRuntimeDxe. So add it for the platform dsc file. Cc: Ard Biesheuvel Cc: Leif Lindholm Cc: Michael D Kinney Cc: Liming Gao Signed-off-by: Zhichao Gao --- Platform/Hisilicon/D03/D03.dsc | 2

[edk2-devel][edk2-platforms: 12/22] Platform/HiKey960.dsc: Add ResetUtilityLib to dsc file

REF: https://bugzilla.tianocore.org/show_bug.cgi?id=1772 ResetUtilityLib would be consumed by CapsuleRuntimeDxe. So add it for the platform dsc file. Cc: Ard Biesheuvel Cc: Leif Lindholm Cc: Michael D Kinney Cc: Liming Gao Signed-off-by: Zhichao Gao ---

[edk2-devel] [PATCH V3 13/15] MdeModulePkg/CapsuleLib: Transfer reset data

From: Bret Barkelew REF: https://bugzilla.tianocore.org/show_bug.cgi?id=1772 Transfer reset data start with a null sting and followed by gEdkiiCapsuleUpdateCompleteResetGuid for DoResetSystem function. Cc: Jian J Wang Cc: Hao Wu Cc: Ray Ni Cc: Star Zeng Cc: Liming Gao Cc: Sean Brogan Cc:

[edk2-devel][edk2-platforms: 04/22] Platform/BeagleBoardPkg.dsc: Add ResetUtilityLib to dsc file

REF: https://bugzilla.tianocore.org/show_bug.cgi?id=1772 ResetUtilityLib would be consumed by CapsuleRuntimeDxe. So add it for the platform dsc file. Cc: Ard Biesheuvel Cc: Leif Lindholm Cc: Michael D Kinney Cc: Liming Gao Signed-off-by: Zhichao Gao ---

[edk2-devel] [PATCH V3 07/15] OvmfPkg/OvmfPkgIa32.dsc: Add ResetUtilityLib to dsc file

REF: https://bugzilla.tianocore.org/show_bug.cgi?id=1772 ResetUtilityLib would be consumed by CapsuleRuntimeDxe. So add it for the platform dsc file. Cc: Jordan Justen Cc: Laszlo Ersek Cc: Ard Biesheuvel Cc: Liming Gao Signed-off-by: Zhichao Gao --- OvmfPkg/OvmfPkgIa32.dsc | 2 ++ 1 file

[edk2-devel] [PATCH V3 15/15] MdePkg/UefiRuntimeLib: Change the comment

Change the comment of EfiResetSystem base on the UEFI spec 2.8 Cc: Michael D Kinney Cc: Liming Gao Signed-off-by: Zhichao Gao --- MdePkg/Library/UefiRuntimeLib/RuntimeLib.c | 8 1 file changed, 4 insertions(+), 4 deletions(-) diff --git a/MdePkg/Library/UefiRuntimeLib/RuntimeLib.c

[edk2-devel][edk2-platforms: 14/22] Platform/OverdriveBoard.dsc: Add ResetUtilityLib to dsc file

REF: https://bugzilla.tianocore.org/show_bug.cgi?id=1772 ResetUtilityLib would be consumed by CapsuleRuntimeDxe. So add it for the platform dsc file. Cc: Ard Biesheuvel Cc: Leif Lindholm Cc: Michael D Kinney Cc: Liming Gao Signed-off-by: Zhichao Gao ---

[edk2-devel] [PATCH V3 11/15] UefiPayloadPkg: Add ResetUtilityLib to UefiPayloadPkgIa32X64.dsc

REF: https://bugzilla.tianocore.org/show_bug.cgi?id=1772 ResetUtilityLib would be consumed by CapsuleRuntimeDxe. So add it for the platform dsc file. Cc: Maurice Ma Cc: Guo Dong Cc: Benjamin You Cc: Liming Gao Signed-off-by: Zhichao Gao --- UefiPayloadPkg/UefiPayloadPkgIa32X64.dsc | 2 ++

[edk2-devel] [PATCH V3 03/15] ArmVirtPkg/ArmVirtQemu.dsc: Add ResetUtilityLib to dsc file

REF: https://bugzilla.tianocore.org/show_bug.cgi?id=1772 ResetUtilityLib would be consumed by CapsuleRuntimeDxe. So add it for the platform dsc file. Cc: Laszlo Ersek Cc: Ard Biesheuvel Cc: Leif Lindholm Cc: Liming Gao Signed-off-by: Zhichao Gao --- ArmVirtPkg/ArmVirtQemu.dsc | 1 + 1 file

[edk2-devel][edk2-platforms: 02/22] Platform/ArmVExpress-CTA15-A7.dsc: Add ResetUtilityLib to dsc file

REF: https://bugzilla.tianocore.org/show_bug.cgi?id=1772 ResetUtilityLib would be consumed by CapsuleRuntimeDxe. So add it for the platform dsc file. Cc: Ard Biesheuvel Cc: Leif Lindholm Cc: Michael D Kinney Cc: Liming Gao Signed-off-by: Zhichao Gao ---

[edk2-devel] [PATCH V3 00/15] MdeModulePkg: Transfer reset data

Indicate two guids gCapsuleArmedResetGuid and gCapsuleUpdateCompleteResetGuid for capsule update. And define a struct which start with a null string and followed by a EFI_GUID. V2: Add prefix Edkii to the new guids. Move the new struct definition to ResetSystemLib.h. Fix the wrong reset data to

[edk2-devel] [PATCH V3 14/15] MdePkg/UefiRuntimeLib.h: Change the comment

Change the comment of EfiResetSystem base on the UEFI spec 2.8 Cc: Michael D Kinney Cc: Liming Gao Signed-off-by: Zhichao Gao --- MdePkg/Include/Library/UefiRuntimeLib.h | 8 1 file changed, 4 insertions(+), 4 deletions(-) diff --git a/MdePkg/Include/Library/UefiRuntimeLib.h

[edk2-devel] [PATCH V3 02/15] MdeModulePkg: Add ResetUtilityLib to dsc file

REF: https://bugzilla.tianocore.org/show_bug.cgi?id=1772 ResetUtilityLib would be consumed by CapsuleRuntimeDxe. So add it for the platform dsc file. Cc: Jian J Wang Cc: Hao Wu Cc: Ray Ni Cc: Star Zeng Cc: Liming Gao Signed-off-by: Zhichao Gao --- MdeModulePkg/MdeModulePkg.dsc | 1 + 1

[edk2-devel] [PATCH V3 08/15] OvmfPkg/OvmfPkgIa32X64.dsc: Add ResetUtilityLib to dsc file

REF: https://bugzilla.tianocore.org/show_bug.cgi?id=1772 ResetUtilityLib would be consumed by CapsuleRuntimeDxe. So add it for the platform dsc file. Cc: Jordan Justen Cc: Laszlo Ersek Cc: Ard Biesheuvel Cc: Liming Gao Signed-off-by: Zhichao Gao --- OvmfPkg/OvmfPkgIa32X64.dsc | 2 ++ 1

[edk2-devel][edk2-platforms: 01/22] Platform/ARM/JunoPkg/ArmJuno.dsc: Add ResetUtilityLib to dsc file

REF: https://bugzilla.tianocore.org/show_bug.cgi?id=1772 ResetUtilityLib would be consumed by CapsuleRuntimeDxe. So add it for the platform dsc file. Cc: Ard Biesheuvel Cc: Leif Lindholm Cc: Michael D Kinney Cc: Liming Gao Signed-off-by: Zhichao Gao --- Platform/ARM/JunoPkg/ArmJuno.dsc | 2

  1   2   >