Re: [edk2-devel] [edk2][PATCH 1/1] MdePkg/Include: Add DCC and BCM2835 SPCR UART types

2019-12-11 Thread Ard Biesheuvel
On Thu, 12 Dec 2019 at 01:26, Gao, Liming wrote: > > Reviewed-by: Liming Gao > Thanks Pushed as bfb141cf19dd6f9b8df8b9d0914a5b3b15e1a798 > > -Original Message- > > From: Ard Biesheuvel > > Sent: Wednesday, December 11, 2019 6:01 PM > > To: Pete Batard ; Kinney, Michael D > > ; Gao,

Re: [edk2-devel] [PATCH] ShellPkg/ShellProtocol: Return error code while fail parsing cmd-line

2019-12-11 Thread Ni, Ray
Reviewed-by: Ray Ni > -Original Message- > From: Gao, Zhichao > Sent: Monday, December 2, 2019 8:54 AM > To: devel@edk2.groups.io > Cc: Ni, Ray ; Augustine, Linson > Subject: [PATCH] ShellPkg/ShellProtocol: Return error code while fail parsing > cmd-line > > REF:

Re: [edk2-devel] [RFC PATCH v3 35/43] UefiCpuPkg/MpInitLib: Add a CPU MP data flag to indicate if SEV-ES is enabled

2019-12-11 Thread Ni, Ray
OK I see now the Enable PCD is defined in MdeModulePkg, produced in platform and consumed in MpInitLib. Is there a way to easily detect whether SEV-ES is enabled? (without triggering CPUID as what SEC does) If no, can you define the PCD in UefiCpuPkg? > -Original Message- > From: Tom

Re: [edk2-devel] [RFC PATCH v3 33/43] MdeModulePkg: Reserve a 16-bit protected mode code segment descriptor

2019-12-11 Thread Ni, Ray
Is the 16bit code segment descriptor needed in early DXE before CpuDxe (DXE MP)? > -Original Message- > From: Tom Lendacky > Sent: Thursday, November 21, 2019 4:07 AM > To: devel@edk2.groups.io > Cc: Justen, Jordan L ; Laszlo Ersek > ; Ard Biesheuvel > ; Kinney, Michael D ; > Gao,

Re: [edk2-devel] [RFC PATCH v3 28/43] OvmfPkg: Create GHCB pages for use during Pei and Dxe phase

2019-12-11 Thread Ni, Ray
> + // Allocate GHCB and per-CPU variable pages. > + // > + GhcbPageCount = mMaxCpuCount * 2; > + GhcbBase = AllocatePages (GhcbPageCount); > + ASSERT (GhcbBase != NULL); > + > + GhcbBasePa = (PHYSICAL_ADDRESS)(UINTN) GhcbBase; > + > + DecryptStatus = MemEncryptSevClearPageEncMask ( > +

Re: [edk2-devel] [RFC PATCH v3 04/43] MdeModulePkg/DxeIplPeim: Support GHCB pages when creating page tables

2019-12-11 Thread Ni, Ray
Tom, When are GHCP pages are allocated? Can DxeIpl gets the address by reading the GHCB MSR? Can the GHCB PCD be eliminated by updating all GHCB address consumer to read the GHCB MSR? Thanks, Ray > -Original Message- > From: devel@edk2.groups.io On Behalf Of Lendacky, > Thomas >

Re: [edk2-devel] [RFC PATCH v3 22/43] UefiCpuPkg/CpuExceptionHandler: Add support for DR7 Read/Write NAE events

2019-12-11 Thread Ni, Ray
Tom, Why all DR registers are not pushed to stack in VC handler? I thought only DR7 pushing is skipped. Thanks, Ray > -Original Message- > From: Tom Lendacky > Sent: Thursday, November 21, 2019 4:07 AM > To: devel@edk2.groups.io > Cc: Justen, Jordan L ; Laszlo Ersek > ; Ard Biesheuvel

Re: [edk2-devel] [RFC PATCH v3 01/43] MdePkg: Create PCDs to be used in support of SEV-ES

2019-12-11 Thread Ni, Ray
Do you really need to define the PCD in MdePkg? General guide lines are: 1. Avoid UefiCpuPkg depend on MdeModulePkg. 2. Do not define platform level PCD in core pkgs (MdePkg, MdeModulePkg, UefiCpuPkg, etc) PcdSevEsIsEnabled seems to be used in OVMF pkg only so how about define that in

Re: [edk2-devel] [RFC PATCH v3 22/43] UefiCpuPkg/CpuExceptionHandler: Add support for DR7 Read/Write NAE events

2019-12-11 Thread Ni, Ray
> > ;; UINT64 Dr0, Dr1, Dr2, Dr3, Dr6, Dr7; > +cmp qword [rbp + 8], 29 Can you define a macro instead of using 29? > +je VcDebugRegs ; For SEV-ES (#VC) Debug registers ignored -=-=-=-=-=-=-=-=-=-=-=- Groups.io Links: You receive all messages sent to this group.

Re: [edk2-devel] [PATCH] ShellPkg/ShellProtocol: Return error code while fail parsing cmd-line

2019-12-11 Thread Gao, Zhichao
Hi Ray, Can you help to review the patch? Thanks, Zhichao > -Original Message- > From: devel@edk2.groups.io [mailto:devel@edk2.groups.io] On Behalf Of > Philippe Mathieu-Daudé > Sent: Monday, December 2, 2019 7:10 PM > To: Gao, Zhichao ; devel@edk2.groups.io > Cc: Ni, Ray ; Augustine,

Re: [edk2-devel] [PATCH V2 0/2] MdePkg/UefiDevicePathLib: Separate the lib instances

2019-12-11 Thread Gao, Zhichao
Actually I didn't do the functionality test of Mandatory one, only do the build test and logic check. It is part of the Optional one. I assume the optional one is working fine. Then there would be no problem with Mandatory one. I would write a test for both of them to make sure they are both

Re: [edk2-devel] [PATCH V2 0/2] MdePkg/UefiDevicePathLib: Separate the lib instances

2019-12-11 Thread Liming Gao
Zhichao: The change is good. What functionality test is done? Thanks Liming >-Original Message- >From: Gao, Zhichao >Sent: Thursday, December 12, 2019 10:09 AM >To: devel@edk2.groups.io >Cc: Kinney, Michael D ; Gao, Liming >; Vitaly Cheptsov >Subject: [PATCH V2 0/2]

[edk2-devel] [PATCH V2 0/2] MdePkg/UefiDevicePathLib: Separate the lib instances

2019-12-11 Thread Gao, Zhichao
REF: https://bugzilla.tianocore.org/show_bug.cgi?id=2298 The UefiDevicePathLibOptionalDevicePathProtocolConstructor's implementation isn't match with its instance name. Remove the ASSERT and depex of the gEfiDevicePathUtilitiesProtocolGuid because of "Optional". Add a mandatory instance to force

[edk2-devel] [PATCH V2 1/2] MdePkg/UefiDevicePathLib: Separate the device path lib

2019-12-11 Thread Gao, Zhichao
REF: https://bugzilla.tianocore.org/show_bug.cgi?id=2298 UefiDevicePathLibOptionalDevicePathProtocol's implementation isn't fit its description. It should be implement as blow: Try to find the DevicePathProtocol, if found then use it to implement the interface. Else, use the local interface. It

[edk2-devel] [PATCH V2 2/2] MdePkg/dsc: Add UefiDevicePathLibMandatoryDevicePathProtocol for build

2019-12-11 Thread Gao, Zhichao
REF: https://bugzilla.tianocore.org/show_bug.cgi?id=2298 Add the new instance lib for build. Cc: Michael D Kinney Cc: Liming Gao Cc: Vitaly Cheptsov Signed-off-by: Zhichao Gao --- MdePkg/MdePkg.dsc | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/MdePkg/MdePkg.dsc

Re: [edk2-devel] [PATCH] SimicsOpenBoardPkg: Replace CMOS Hardcoded Addresses

2019-12-11 Thread Nate DeSimone
Reviewed-by: Nate DeSimone -Original Message- From: Agyeman, Prince Sent: Friday, December 6, 2019 9:32 AM To: devel@edk2.groups.io Cc: Desimone, Nathaniel L ; Kubacki, Michael A Subject: [PATCH] SimicsOpenBoardPkg: Replace CMOS Hardcoded Addresses REF:

[edk2-devel] [edk2-platforms] [PATCH] WhiskeylakeOpenBoardPkg: Update PCDs to enable stack sharing

2019-12-11 Thread Agyeman, Prince
REF: https://bugzilla.tianocore.org/show_bug.cgi?id=2409 Updated WhiskeylakeURvp PCDs to enable FSP/BL stack sharing. This fixes the boot failure seen with the latest Coffee Lake (CFL) FSP binary (v 7.0.68.41) Cc: Chasel Chiu Cc: Nate DeSimone Cc: Michael Kubacki Signed-off-by: Prince

Re: [edk2-devel] [PATCH] SimicsOpenBoardPkg: Replace CMOS Hardcoded Addresses

2019-12-11 Thread Kubacki, Michael A
Please include [edk2-platforms] in the subject in the future. Reviewed-by: Michael Kubacki > -Original Message- > From: Agyeman, Prince > Sent: Friday, December 6, 2019 9:32 AM > To: devel@edk2.groups.io > Cc: Desimone, Nathaniel L ; Kubacki, > Michael A > Subject: [PATCH]

Re: [edk2-devel] [edk2][PATCH 1/1] MdePkg/Include: Add DCC and BCM2835 SPCR UART types

2019-12-11 Thread Liming Gao
Reviewed-by: Liming Gao > -Original Message- > From: Ard Biesheuvel > Sent: Wednesday, December 11, 2019 6:01 PM > To: Pete Batard ; Kinney, Michael D > ; Gao, Liming > Cc: edk2-devel-groups-io ; Leif Lindholm > ; Philippe Mathieu-Daudé > > Subject: Re: [edk2][PATCH 1/1]

Re: [edk2-devel] [edk2-staging/EdkRepo] [PATCH] EdkRepo: Support uninstalling packages with '_' in their name.

2019-12-11 Thread Desimone, Ashley E
Reviewed-by: Ashley DeSimone -Original Message- From: Desimone, Nathaniel L Sent: Tuesday, December 10, 2019 1:56 AM To: devel@edk2.groups.io Cc: Desimone, Ashley E ; Pandya, Puja ; Bret Barkelew Subject: [edk2-staging/EdkRepo] [PATCH] EdkRepo: Support uninstalling packages with '_'

Re: [edk2-devel] [edk2-staging/EdkRepo] [PATCH] EdkRepo: Allow user to override python command

2019-12-11 Thread Desimone, Ashley E
Reviewed-by: Ashley DeSimone -Original Message- From: devel@edk2.groups.io [mailto:devel@edk2.groups.io] On Behalf Of Nate DeSimone Sent: Tuesday, December 10, 2019 1:55 AM To: devel@edk2.groups.io Cc: Desimone, Ashley E ; Pandya, Puja ; Bret Barkelew Subject: [edk2-devel]

[edk2-devel] [PATCH] [edk2/BaseTools] edksetup.bat stuck on unicode locale Windows

2019-12-11 Thread Park, Aiden
This issue happens under two conditions. 1. Unicode language environment in Windows 2. Call 'edksetup.bat forcerebuild' with python subprocess.call() Steps to reproduce C:\edk2>python Python 2.7.12 (v2.7.12:d33e0cf91556, Jun 27 2016, 15:24:40) Type help, copyright, credits or license

[edk2-devel] [TLS][HTTPS] TLS Server support

2019-12-11 Thread davysouza94
Hi everyone, I'm working on a HTTPS server on EFI using EDK II, but I found the following code on *\edk2\CryptoPkg\Library\TlsLib\TlsConfig.c: * if ( ! IsServer) { // // Set TLS to work in Client mode. // SSL_set_connect_state (TlsConn->Ssl) ; } else { // // Set TLS to work in Server mode. // 

[edk2-devel] [edk2-platforms][PATCH 1/1] Platform/RPi: Inhibit serial output for all runtime drivers

2019-12-11 Thread Pete Batard
From: Ard Biesheuvel This is an improvement of e9db04631b63574b090aeab769cc47dcb75a29f7 where we inhibit serial output of MMIO mapped UARTs to all runtime drivers rather than just RTC, as other drivers may crash the OS just the same. Also add it to the Pi 4 platform where it was missing

Re: [edk2-devel] [edk2-platforms][PATCH 0/5] Add initial Raspberry Pi 4 platform

2019-12-11 Thread Ard Biesheuvel
On Wed, 11 Dec 2019 at 12:26, Pete Batard wrote: > > This series adds basic support to build the Raspberry Pi 4 platform. > > It requires the earlier series, that add binary files for the platform, > to have been applied to edk2-non-osi. > > For the introduction of the platform, USB support is

Re: [edk2-devel] [edk2-non-osi][PATCH 2/2] Platforms/RPi4: Add Trusted Firmware binaries

2019-12-11 Thread Pete Batard
Hi Leif, On 2019.12.11 14:14, Leif Lindholm wrote: On Wed, Dec 11, 2019 at 11:24:00 +, Pete Batard wrote: Similar to what is the case with the Raspberry Pi 3, the Raspberry Pi 4 UEFI firmware requires the provision of a Trusted Firmware binary (TF-A). The binary is built for a dtb base

[edk2-devel] [Patch 1/1] BaseTools: Fix build failure when multiple build targets given

2019-12-11 Thread Bob Feng
BZ: https://bugzilla.tianocore.org/show_bug.cgi?id=2371 This patch is to fix a regression issue that build fails if multiple build targets given. Two changes cause this regression issue. One is AutoGen object __hash__ function only hash file path and arch, missing ToolChain and build target.

Re: [edk2-devel] [edk2-discuss] Design discussion for SEV-ES

2019-12-11 Thread Lendacky, Thomas
On 12/10/19 9:04 PM, Ni, Ray wrote: > Can you please have your slides ready in >

Re: [edk2-devel] Debugging aarch64 edk2 built with gcc-8.x 64-bit arm compiler

2019-12-11 Thread Ard Biesheuvel
On Tue, 10 Dec 2019 at 18:25, Vladimir Olovyannikov wrote: > > > -Original Message- > > From: Ard Biesheuvel > > Sent: Tuesday, December 10, 2019 9:13 AM > > To: Vladimir Olovyannikov ; Sami > > Mujawar > > Cc: edk2-devel-groups-io > > Subject: Re: Debugging aarch64 edk2 built with

Re: [edk2-devel] [edk2-non-osi][PATCH 2/2] Platforms/RPi4: Add Trusted Firmware binaries

2019-12-11 Thread Leif Lindholm
On Wed, Dec 11, 2019 at 11:24:00 +, Pete Batard wrote: > Similar to what is the case with the Raspberry Pi 3, the Raspberry Pi > 4 UEFI firmware requires the provision of a Trusted Firmware binary > (TF-A). > > The binary is built for a dtb base address of 0x0002 and a UEFI > payload base

Re: [edk2-devel] [edk2-non-osi][PATCH 0/2] Add prerequisite binary files for RPi4 platform

2019-12-11 Thread Ard Biesheuvel
On Wed, 11 Dec 2019 at 12:24, Pete Batard wrote: > > This series of patches adds the binaries we need available to introduce > Raspberry Pi 4 support. > > Similar to the Pi 3, we must have TF-A and Device Tree binaries we can embed. > > Pete Batard (2): > Platforms/RPi4: Add Device Tree >

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

2019-12-11 Thread Ni, Ray
Jeff, Tom from AMD booked the meeting for SEV discussion months ago. I am afraid there is no time for this discussion. Let's try to resolve it in mails. Firstly, let me rephase the problem and your proposed solutions here (subjective + verb + objective). Sunny's input is also included. Hope

Re: [edk2-devel] [edk2-staging/UEFI_PCI_ENHANCE-2 PATCH V6] MdePkg/Protocols: New PCI Express Platform Protocol, EFI encodings

2019-12-11 Thread Javeed, Ashraf
This change is pushed in to the following EDK Repo for review:- https://github.com/ashrafj/edk2-staging/commit/560b6c1dd5b7eae8c49bd43e83335c7d13fef4a3 Please review and provide your feedback. Thanks Ashraf > -Original Message- > From: devel@edk2.groups.io On Behalf Of Javeed, > Ashraf

Re: [edk2-devel] [edk2-platforms][PATCH 2/5] Platform/RPi: Don't describe MMIO regions as memory

2019-12-11 Thread Philippe Mathieu-Daudé
On 12/11/19 12:25 PM, Pete Batard wrote: From: Ard Biesheuvel When using ACPI OpRegions to poke device registers, Linux will use the UEFI memory map to decide which memory attributes to use, and so they should not be described as cacheable memory. Since MMIO regions that don't require an OS

Re: [edk2-devel] [edk2-platforms][PATCH 1/5] Platform/RPi: Fix overlap of SoC registers and RAM

2019-12-11 Thread Philippe Mathieu-Daudé
On 12/11/19 12:25 PM, Pete Batard wrote: From: Ard Biesheuvel Having RAM and SoC register regions overlap is problematic for MMIO, since, at the very least, we don't want these regions to be declared as cacheable. Signed-off-by: Pete Batard ---

[edk2-devel] [edk2-platforms][PATCH 2/5] Platform/RPi: Don't describe MMIO regions as memory

2019-12-11 Thread Pete Batard
From: Ard Biesheuvel When using ACPI OpRegions to poke device registers, Linux will use the UEFI memory map to decide which memory attributes to use, and so they should not be described as cacheable memory. Since MMIO regions that don't require an OS virtual mapping at runtime don't really

[edk2-devel] [edk2-platforms][PATCH 0/5] Add initial Raspberry Pi 4 platform

2019-12-11 Thread Pete Batard
This series adds basic support to build the Raspberry Pi 4 platform. It requires the earlier series, that add binary files for the platform, to have been applied to edk2-non-osi. For the introduction of the platform, USB support is not yet enabled which means that user I/O has to be carried out

[edk2-devel] [edk2-platforms][PATCH 5/5] Platform/RPi4: Add base platform files

2019-12-11 Thread Pete Batard
From: Andrei Warkentin This enables building the initial RPi4 platform firmware. Note that PCIe and xHCI are missing at this stage and that this version of the firmware uses miniUART for serial I/O. PCIe and xHCI support will be added in a later patch series as well as the ability to switch

[edk2-devel] [edk2-platforms][PATCH 3/5] Platform/RPi4: Add initial ACPI tables

2019-12-11 Thread Pete Batard
From: Samer El-Haj-Mahmoud For this initial commit, we duplicate the RPi3 ones. Signed-off-by: Pete Batard --- Platform/RaspberryPi/RPi4/AcpiTables/AcpiTables.h | 76 +++ Platform/RaspberryPi/RPi4/AcpiTables/AcpiTables.inf | 40 ++ Platform/RaspberryPi/RPi4/AcpiTables/Csrt.aslc | 326

[edk2-devel] [edk2-platforms][PATCH 4/5] Platform/RPi4: Update ACPI tables for the new platform

2019-12-11 Thread Pete Batard
From: Andrei Warkentin Update CSRT, DSDT, GTDT, MADT, SDHC and serial tables for the new base addresses and switch ACPI to GIC. We use ACPI 5.1 for MADT because older versions of the Linux kernel can be finicky when it comes to checking the size of the GICC entries the table: depending on the

[edk2-devel] [edk2-platforms][PATCH 1/5] Platform/RPi: Fix overlap of SoC registers and RAM

2019-12-11 Thread Pete Batard
From: Ard Biesheuvel Having RAM and SoC register regions overlap is problematic for MMIO, since, at the very least, we don't want these regions to be declared as cacheable. Signed-off-by: Pete Batard --- Platform/RaspberryPi/Library/PlatformLib/RaspberryPiMem.c | 36 +--- 1

[edk2-devel] [edk2-non-osi][PATCH 2/2] Platforms/RPi4: Add Trusted Firmware binaries

2019-12-11 Thread Pete Batard
Similar to what is the case with the Raspberry Pi 3, the Raspberry Pi 4 UEFI firmware requires the provision of a Trusted Firmware binary (TF-A). The binary is built for a dtb base address of 0x0002 and a UEFI payload base address of 0x0003. Binaries are built using a custom version of

[edk2-devel] [edk2-non-osi][PATCH 0/2] Add prerequisite binary files for RPi4 platform

2019-12-11 Thread Pete Batard
This series of patches adds the binaries we need available to introduce Raspberry Pi 4 support. Similar to the Pi 3, we must have TF-A and Device Tree binaries we can embed. Pete Batard (2): Platforms/RPi4: Add Device Tree Platforms/RPi4: Add Trusted Firmware binaries

Re: [edk2-devel] [PATCH v1 1/1] BaseTools: Build ASL files before C files

2019-12-11 Thread PierreGondois
Hello Bob and Liming, I am coming back to you after re-reading Bob's answer: > I meet another problem related to .asl file. In one of platforms build > process, there is a step which need to compile a .asl file a .h file firstly > and that .h file will be included by a module’s source code. >

[edk2-devel] [edk2-non-osi][PATCH 1/2] Platforms/RPi4: Add Device Tree

2019-12-11 Thread Pete Batard
Similar to what is the case with the Raspberry Pi 3, the Raspberry Pi 4 UEFI firmware requires a binary Device Tree for the Broadcom 2711 SoC. This patch adds the most up to date Device Tree binary, as published with commit 601d36df3aa541560e4cf9b571105d20db2b4b7c from

[edk2-devel] [edk2-staging/UEFI_PCI_ENHANCE-2 PATCH V6] MdePkg/Protocols: New PCI Express Platform Protocol, EFI encodings

2019-12-11 Thread Javeed, Ashraf
BZ: https://bugzilla.tianocore.org/show_bug.cgi?id=1954 Two interfaces added to PCI Express Platform Protocol:- (1) GetDevicePolicy() -> to retrieve device-specific platform policies (2) NotifyDeviceState -> to notify platform about device PCI Express configuration state PCI Express Override

Re: [edk2-devel] [RFC PATCH 2/2] MdeModulePkg/BdsDxe: allow consoles on drivers loaded via Driver####

2019-12-11 Thread Ard Biesheuvel
On Mon, 9 Dec 2019 at 09:42, Ard Biesheuvel wrote: > > On Mon, 9 Dec 2019 at 03:12, Ni, Ray wrote: > > > > > Exactly. This flow is identical to how option ROMs are processed if > > > they are discovered before EndOfDxe signalling completes (which is why > > > the Juno platform was broken without

Re: [edk2-devel] [edk2][PATCH 1/1] MdePkg/Include: Add DCC and BCM2835 SPCR UART types

2019-12-11 Thread Ard Biesheuvel
(add MdePkg maintainers) On Tue, 10 Dec 2019 at 19:23, Pete Batard wrote: > > As per the Microsoft Debug Port Table 2 (DBG2) documentation, that > can be found online, we are missing 2 serial interface types for > Arm DCC and Bcm2835 (the latter being used with the Raspberry Pi). > > These same

[edk2-devel] [PATCH v2] Maintainers.txt: Update email address and role

2019-12-11 Thread Maciej Rabeda
Created new email account that will not append legal disclaimers to my responses/patches. Switching to NetworkPkg maintainer. Cc: Jiaxin Wu Cc: Siyuan Fu Signed-off-by: Maciej Rabeda --- Maintainers.txt | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/Maintainers.txt

Re: [edk2-discuss] [edk2-devel] [PATCH] Support skipping automatic BM enumeration

2019-12-11 Thread Wang, Sunny (HPS SW)
Hi all, Since I can't attend the design meeting (EMEA), I would like to add some use cases and a suggestion for your guys' reference before this topic being discussed: We had similar needs (use cases) as Ashish like the following, so making this code change would be definitely good for

Re: [edk2-devel] [PATCH v2 103/105] .mailmap: Add an entry for Yonghong Zhu

2019-12-11 Thread Zhu, Yonghong
Reviewed-by: Yonghong Zhu Best Regards, Zhu Yonghong -Original Message- From: Philippe Mathieu-Daude [mailto:phi...@redhat.com] Sent: Friday, December 6, 2019 7:27 PM To: devel@edk2.groups.io Cc: Philippe Mathieu-Daude ; Zhu, Yonghong Subject: [PATCH v2 103/105] .mailmap: Add an