Re: [edk2] [PATCH 1/2] MdeModulePkg/Tcp4Dxe: Fix unconditional window shrinking

2017-03-28 Thread Tian, Feng
Adding network module owners for further review. Thanks Feng -Original Message- From: edk2-devel [mailto:edk2-devel-boun...@lists.01.org] On Behalf Of ate...@kraftway.ru Sent: Tuesday, March 28, 2017 3:20 PM To: edk2-devel@lists.01.org Subject: [edk2] [PATCH 1/2] MdeModulePkg/Tcp4Dxe: Fi

Re: [edk2] [PATCH 00/12] OvmfPkg: Enable variable access in PEI

2017-03-28 Thread Jordan Justen
On 2017-03-28 02:22:37, Laszlo Ersek wrote: > > > InstallProtocolInterface: [EfiLoadedImageProtocol] 7E9EEC10 > > SmmInstallProtocolInterface: [EfiLoadedImageProtocol] 7FFDD47C > > Loading SMM driver at 0x0007FF5F000 EntryPoint=0x0007FF60034 > > FvbServicesSmm.efi > > ASSERT .../OvmfPkg/QemuFlash

[edk2] [PATCH] ShellPkg/setvar: Support data format in Shell 2.2 spec

2017-03-28 Thread Ruiyu Ni
From: Chen A Chen Shell 2.2 spec defines =0x/=0X, =H/=h, =S, =L and =P for hex number, hex array, ascii string, unicode string and device path data. The patch adds such support. Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Chen A Chen Cc: Ruiyu Ni Cc: Michael D Kinney

[edk2] [PATCH 08/10] UefiCpuPkg/MtrrLib: Use a better algorithm to calculate MTRR

2017-03-28 Thread Ruiyu Ni
The new algorithm finds out the more optimal MTRR solution for current memory type settings. Compare against the original algorithm, the new one guarantees to find the correct MTRR solution, but doesn't guarantee to find the most optimal MTRR solution. Contributed-under: TianoCore Contribution Agr

[edk2] [PATCH 10/10] UefiCpuPkg/MtrrLib: All functions use definitions in Msr.h

2017-03-28 Thread Ruiyu Ni
Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Ruiyu Ni Cc: Jeff Fan --- UefiCpuPkg/Library/MtrrLib/MtrrLib.c | 55 +--- 1 file changed, 33 insertions(+), 22 deletions(-) diff --git a/UefiCpuPkg/Library/MtrrLib/MtrrLib.c b/UefiCpuPkg/Libr

[edk2] [PATCH 00/10] Use a better algorithm to calculate MTRR

2017-03-28 Thread Ruiyu Ni
The new algorithm finds out the more optimal MTRR solution for current memory type settings. Compare against the original algorithm, the new one guarantees to find the correct MTRR solution, but doesn't guarantee to find the most optimal MTRR solution. Ruiyu Ni (10): UefiCpuPkg/MtrrLib: Correct

[edk2] [PATCH 09/10] UefiCpuPkg/MtrrLib: Refine MtrrGetMemoryAttributeByAddressWorker

2017-03-28 Thread Ruiyu Ni
Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Ruiyu Ni Cc: Jeff Fan --- UefiCpuPkg/Library/MtrrLib/MtrrLib.c | 237 --- 1 file changed, 80 insertions(+), 157 deletions(-) diff --git a/UefiCpuPkg/Library/MtrrLib/MtrrLib.c b/UefiCpuPkg/Lib

[edk2] [PATCH 07/10] UefiCpuPkg/MtrrLib: MtrrLibInitializeMtrrMask() uses definitions in CpuId.h

2017-03-28 Thread Ruiyu Ni
Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Ruiyu Ni Cc: Jeff Fan --- UefiCpuPkg/Library/MtrrLib/MtrrLib.c | 20 +--- 1 file changed, 9 insertions(+), 11 deletions(-) diff --git a/UefiCpuPkg/Library/MtrrLib/MtrrLib.c b/UefiCpuPkg/Library/MtrrLib/MtrrL

[edk2] [PATCH 05/10] UefiCpuPkg/MtrrLib: Add MtrrLib prefix to ProgramFixedMtrr

2017-03-28 Thread Ruiyu Ni
Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Ruiyu Ni Cc: Jeff Fan --- UefiCpuPkg/Library/MtrrLib/MtrrLib.c | 22 +++--- 1 file changed, 11 insertions(+), 11 deletions(-) diff --git a/UefiCpuPkg/Library/MtrrLib/MtrrLib.c b/UefiCpuPkg/Library/MtrrLib/Mt

[edk2] [PATCH 06/10] UefiCpuPkg/MtrrLib: Add MtrrLib prefix to several internal functions

2017-03-28 Thread Ruiyu Ni
Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Ruiyu Ni Cc: Jeff Fan --- UefiCpuPkg/Library/MtrrLib/MtrrLib.c | 30 +++--- 1 file changed, 15 insertions(+), 15 deletions(-) diff --git a/UefiCpuPkg/Library/MtrrLib/MtrrLib.c b/UefiCpuPkg/Library/Mt

[edk2] [PATCH 01/10] UefiCpuPkg/MtrrLib: Correct typo in comments and remove TABs

2017-03-28 Thread Ruiyu Ni
Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Ruiyu Ni Cc: Jeff Fan --- UefiCpuPkg/Include/Library/MtrrLib.h | 14 +++--- UefiCpuPkg/Library/MtrrLib/MtrrLib.c | 4 ++-- 2 files changed, 9 insertions(+), 9 deletions(-) diff --git a/UefiCpuPkg/Include/Library/Mtr

[edk2] [PATCH 02/10] UefiCpuPkg/MtrrLib: Add CacheInvalid enum type to MtrrLib.h

2017-03-28 Thread Ruiyu Ni
Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Ruiyu Ni Cc: Jeff Fan --- UefiCpuPkg/Include/Library/MtrrLib.h | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/UefiCpuPkg/Include/Library/MtrrLib.h b/UefiCpuPkg/Include/Library/MtrrLib.h index a769279..

[edk2] [PATCH 04/10] UefiCpuPkg/MtrrLib: GetVariableMtrrCountWorker uses definitions in Msr.h

2017-03-28 Thread Ruiyu Ni
Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Ruiyu Ni Cc: Jeff Fan --- UefiCpuPkg/Library/MtrrLib/MtrrLib.c | 8 1 file changed, 4 insertions(+), 4 deletions(-) diff --git a/UefiCpuPkg/Library/MtrrLib/MtrrLib.c b/UefiCpuPkg/Library/MtrrLib/MtrrLib.c index 380

[edk2] [PATCH 03/10] UefiCpuPkg/MtrrLib: IsMtrrSupported uses definitions in Msr.h

2017-03-28 Thread Ruiyu Ni
Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Ruiyu Ni Cc: Jeff Fan --- UefiCpuPkg/Library/MtrrLib/MtrrLib.c | 20 +++- 1 file changed, 11 insertions(+), 9 deletions(-) diff --git a/UefiCpuPkg/Library/MtrrLib/MtrrLib.c b/UefiCpuPkg/Library/MtrrLib/MtrrL

Re: [edk2] [Patch] MdeModulePkg/DxeHttpLib: Fix the incorrect return status if URI port is invalid

2017-03-28 Thread Zhang, Lubo
Reviewed-by: Zhang Lubo -Original Message- From: Wu, Jiaxin Sent: Monday, March 27, 2017 2:48 PM To: edk2-devel@lists.01.org Cc: Zhang, Lubo ; Ye, Ting ; Fu, Siyuan ; Wu, Jiaxin Subject: [Patch] MdeModulePkg/DxeHttpLib: Fix the incorrect return status if URI port is invalid Cc: Zhang

Re: [edk2] Hypervisor with UEFI

2017-03-28 Thread Blibbet
> There are some web pages described Hypervisor with UEFI Firmware. Are there > some Hypervisor sample implementations in current EDKII source code base? Alex's SimpleVisor is a UEFI hypervisor. https://firmwaresecurity.com/2016/08/31/alex-working-on-uefisor-simplevisor-for-uefi/ https://firmware

Re: [edk2] [PATCH 1/3] MdeModulePkg/Include: Add IOMMU protocol definition.

2017-03-28 Thread Yao, Jiewen
Agree. That is a good idea. I will add that in V2 patch. Thank you Yao Jiewen From: Kinney, Michael D Sent: Wednesday, March 29, 2017 7:03 AM To: Ard Biesheuvel ; Yao, Jiewen ; Kinney, Michael D Cc: Ni, Ruiyu ; Leo Duran ; edk2-devel@lists.01.org; Brijesh Singh Subject: RE: [edk2] [PATCH 1/3

Re: [edk2] [PATCH 1/3] MdeModulePkg/Include: Add IOMMU protocol definition.

2017-03-28 Thread Kinney, Michael D
Ard, I agree. And I think the IOMMU protocol should also support flexible double buffer operations so the extra copies by the host CPU can be avoided if the logical DMA address can directly map to the original buffer. Mike > -Original Message- > From: edk2-devel [mailto:edk2-devel-boun.

Re: [edk2] [PATCH 1/3] MdeModulePkg/Include: Add IOMMU protocol definition.

2017-03-28 Thread Ard Biesheuvel
On 25 March 2017 at 09:28, Jiewen Yao wrote: > This protocol is to abstract IOMMU access. > > Cc: Ruiyu Ni > Cc: Leo Duran > Cc: Brijesh Singh > Contributed-under: TianoCore Contribution Agreement 1.0 > Signed-off-by: Jiewen Yao Hello all, It would be very useful for a IOMMU protocol such as

Re: [edk2] [edk2-staging][PATCH 2/2] ShellPkg: Add acpiview tool to dump ACPI tables

2017-03-28 Thread Jeff Westfahl
Hi Sami, Just another note, acpiview doesn't currently build on Windows, at least not with Visual Studio 2015. This isn't a problem for me as I don't usually build the shell on Windows, I just happened to notice it. Regards, Jeff Westfahl On Fri, 17 Mar 2017, Sami Mujawar wrote: Hi Jeff,

Re: [edk2] [PATCH v2] EmbeddedPkg: add DT platform driver to select between DT and ACPI

2017-03-28 Thread Ard Biesheuvel
On 28 March 2017 at 19:00, Laszlo Ersek wrote: > On 03/28/17 19:30, Ard Biesheuvel wrote: >> On 28 March 2017 at 18:26, Laszlo Ersek wrote: > > +#define DT_ACPI_VARIABLE_NAME L"DtAcpiPref" > >>> Can you replace this instance of "... DtAcpiPref ..." with "... %a ..." >>> and DT_ACPI_VARIABLE

Re: [edk2] [PATCH v2] EmbeddedPkg: add DT platform driver to select between DT and ACPI

2017-03-28 Thread Laszlo Ersek
On 03/28/17 19:30, Ard Biesheuvel wrote: > On 28 March 2017 at 18:26, Laszlo Ersek wrote: +#define DT_ACPI_VARIABLE_NAME L"DtAcpiPref" >> Can you replace this instance of "... DtAcpiPref ..." with "... %a ..." >> and DT_ACPI_VARIABLE_NAME? No need to resubmit. >> > > That should be %s, I

Re: [edk2] [RFC v2] GitBook documentation process

2017-03-28 Thread Kinney, Michael D
Hello, I have added draft versions of the EDK II DEC, INF, DSC, FDF, UNI, and Template Specifications based on this RFC. These are now hosted in the Tianocore-docs organization on GitHub and the Tianocore-docs organization on Gitbook. https://github.com/tianocore-docs https://www.gitbook.com

Re: [edk2] [PATCH v2] EmbeddedPkg: add DT platform driver to select between DT and ACPI

2017-03-28 Thread Ard Biesheuvel
On 28 March 2017 at 18:26, Laszlo Ersek wrote: > On 03/28/17 18:16, Ard Biesheuvel wrote: >> (use correct address for Marcin -- please take into account when replying) >> >> On 28 March 2017 at 17:15, Ard Biesheuvel wrote: >>> As a follow up to the changes proposed by Laszlo to make ACPI and DT >

Re: [edk2] [PATCH v2] EmbeddedPkg: add DT platform driver to select between DT and ACPI

2017-03-28 Thread Laszlo Ersek
On 03/28/17 18:16, Ard Biesheuvel wrote: > (use correct address for Marcin -- please take into account when replying) > > On 28 March 2017 at 17:15, Ard Biesheuvel wrote: >> As a follow up to the changes proposed by Laszlo to make ACPI and DT >> mutually exclusive on ArmVirtQemu, this patch propo

Re: [edk2] [PATCH v2] EmbeddedPkg: add DT platform driver to select between DT and ACPI

2017-03-28 Thread Ard Biesheuvel
On 28 March 2017 at 17:16, Ard Biesheuvel wrote: > (use correct address for Marcin -- please take into account when replying) > > On 28 March 2017 at 17:15, Ard Biesheuvel wrote: >> As a follow up to the changes proposed by Laszlo to make ACPI and DT >> mutually exclusive on ArmVirtQemu, this pat

Re: [edk2] [PATCH v2] EmbeddedPkg: add DT platform driver to select between DT and ACPI

2017-03-28 Thread Ard Biesheuvel
(use correct address for Marcin -- please take into account when replying) On 28 March 2017 at 17:15, Ard Biesheuvel wrote: > As a follow up to the changes proposed by Laszlo to make ACPI and DT > mutually exclusive on ArmVirtQemu, this patch proposed a DT platform > DXE driver that either instal

[edk2] [PATCH v2] EmbeddedPkg: add DT platform driver to select between DT and ACPI

2017-03-28 Thread Ard Biesheuvel
As a follow up to the changes proposed by Laszlo to make ACPI and DT mutually exclusive on ArmVirtQemu, this patch proposed a DT platform DXE driver that either installs the NULL protocol PlatformHasAcpiGuid, or installs the FV embedded DTB binary as a configuration table under the appropriate GUID

Re: [edk2] Reset from post BDS pahse to PEI phase

2017-03-28 Thread Andrew Fish
> On Mar 28, 2017, at 5:23 AM, Amit kumar wrote: > > > Hi , > I have been facing some problem with blockIO reads even after reconnect and > update map. when i try to do dblk on some of my blocks it returns with > EFI_INVALID_PARAMETER. > Is there a way to reinitialize the EFI platform from p

Re: [edk2] [PATCH v4 00/12] ArmVirtPkg: don't forward the DT to the OS if QEMU provides ACPI

2017-03-28 Thread Laszlo Ersek
On 03/28/17 15:57, Ard Biesheuvel wrote: > On 28 March 2017 at 14:50, Laszlo Ersek wrote: >> This is version 4 of the series posted (as v3) previously at >> . >> >> Main changes in this version: >> >> * move gEdkiiPlatformHasAcpiGui

[edk2] [PATCH v4 00/12] ArmVirtPkg: don't forward the DT to the OS if QEMU provides ACPI

2017-03-28 Thread Laszlo Ersek
This is version 4 of the series posted (as v3) previously at . Main changes in this version: * move gEdkiiPlatformHasAcpiGuid from MdeModulePkg to EmbeddedPkg (based on IRC discussion), * move PlatformHasAcpiLib from ArmPkg to E

Re: [edk2] [PATCH v4 00/12] ArmVirtPkg: don't forward the DT to the OS if QEMU provides ACPI

2017-03-28 Thread Ard Biesheuvel
On 28 March 2017 at 14:50, Laszlo Ersek wrote: > This is version 4 of the series posted (as v3) previously at > . > > Main changes in this version: > > * move gEdkiiPlatformHasAcpiGuid from MdeModulePkg to EmbeddedPkg (based > on

[edk2] [PATCH v4 12/12] ArmVirtPkg: remove PURE_ACPI_BOOT_ENABLE and PcdPureAcpiBoot

2017-03-28 Thread Laszlo Ersek
The build flag and the FeaturePCD have no effect any longer, remove them. Cc: Ard Biesheuvel Cc: Leif Lindholm Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Laszlo Ersek Reviewed-by: Ard Biesheuvel --- Notes: v3: - pick up Ard's R-b ArmVirtPkg/ArmVirtPkg.dec

[edk2] [PATCH v4 10/12] ArmVirtPkg/FdtClientDxe: install DT as sysconfig table in protocol notify

2017-03-28 Thread Laszlo Ersek
Replace the dependency on PcdPureAcpiBoot with a Platform Has Device Tree notification callback. Move the sysconfig table installation from the entry point function to the callback. Cc: Ard Biesheuvel Cc: Leif Lindholm Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Laszlo

[edk2] [PATCH v4 08/12] ArmVirtPkg: add XenPlatformHasAcpiDtDxe

2017-03-28 Thread Laszlo Ersek
This driver produces the EDKII Platform Has ACPI and Platform Has Device Tree protocols, exactly matching the current ACPI / DT exposure on Xen, according to ARM vs. AARCH64. At this point it differs from the QEMU driver PlatformHasAcpiDtDxe in that this one always installs the DT. Cc: Ard Biesheu

[edk2] [PATCH v4 09/12] ArmVirtPkg: enable AcpiTableDxe and EFI_ACPI_TABLE_PROTOCOL dynamically

2017-03-28 Thread Laszlo Ersek
In this patch, the ACPI protocol / driver chain is enabled dynamically, when appropriate. This is being done in one larger patch, because ArmVirt.dsc.inc, where AcpiTableDxe is built, is used by all the platform DSCs. No change in behavior should be observable after this patch on any ArmVirtPkg pl

[edk2] [PATCH v4 11/12] ArmVirtPkg/PlatformHasAcpiDtDxe: don't expose DT if QEMU provides ACPI

2017-03-28 Thread Laszlo Ersek
This will let QEMU's "-no-acpi" option exclusively expose DT vs. ACPI to the guest. Showing both is never needed (it is actually detrimental to the adoption of standards, such as SBSA / SBBR). * Without "-no-acpi", the firmware logs (from PlatformHasAcpiDtDxe) > Found FwCfg @ 0x9020008/0x902

[edk2] [PATCH v4 07/12] ArmVirtPkg: add PlatformHasAcpiDtDxe

2017-03-28 Thread Laszlo Ersek
This driver produces the EDKII Platform Has ACPI and Platform Has Device Tree protocols, exactly matching the current ACPI / DT exposure on QEMU, according to ARM vs. AARCH64, and (in the latter case) to PcdPureAcpiBoot. Cc: Ard Biesheuvel Cc: Leif Lindholm Contributed-under: TianoCore Contribut

[edk2] [PATCH v4 02/12] Revert "ArmVirtPkg/FdtClientDxe: install DT configuration table at ReadyToBoot"

2017-03-28 Thread Laszlo Ersek
This reverts commit 18f6d4df9ece8b91b86511bcdd1cf7da478c3627. We realized that DXE drivers that are independent of AcpiPlatformDxe (that is, independent of QEMU's ACPI generation), such as RamDiskDxe and BootGraphicsResourceTableDxe, may produce and/or manipulate ACPI tables, at driver dispatch or

[edk2] [PATCH v4 06/12] EmbeddedPkg: introduce EDKII Platform Has Device Tree GUID

2017-03-28 Thread Laszlo Ersek
The presence of this GUID in the PPI database, and/or in the DXE protocol database (as dictated by the platform's needs in these firmware phases) implies that the platform provides the operating system with a Device Tree-based hardware description. This is not necessarily exclusive with other types

[edk2] [PATCH v4 04/12] EmbeddedPkg: introduce EDKII Platform Has ACPI GUID

2017-03-28 Thread Laszlo Ersek
The presence of this GUID in the PPI database, and/or in the DXE protocol database (as dictated by the platform's needs in these firmware phases) implies that the platform provides the operating system with an ACPI-based hardware description. This is not necessarily exclusive with other types of ha

[edk2] [PATCH v4 05/12] EmbeddedPkg: introduce PlatformHasAcpiLib

2017-03-28 Thread Laszlo Ersek
Add the shorter-term library instance outlined in the previous patch to EmbeddedPkg, so that we can imbue AcpiTableDxe with a protocol dependency on EDKII_PLATFORM_HAS_ACPI_GUID. Cc: Ard Biesheuvel Cc: Leif Lindholm Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Laszlo Er

[edk2] [PATCH v4 01/12] Revert "ArmVirtPkg/FdtClientDxe: make DT table installation !ACPI dependent"

2017-03-28 Thread Laszlo Ersek
This reverts commit 78c41ff519b187d8979cda7074f007a6323f9acd. We realized that DXE drivers that are independent of AcpiPlatformDxe (that is, independent of QEMU's ACPI generation), such as RamDiskDxe and BootGraphicsResourceTableDxe, may produce and/or manipulate ACPI tables, at driver dispatch or

[edk2] [PATCH v4 03/12] ArmVirtPkg/XenAcpiPlatformDxe: don't cast UINT64 to pointer directly

2017-03-28 Thread Laszlo Ersek
Because that breaks the (potential) 32-bit build of the driver. Cc: Ard Biesheuvel Cc: Leif Lindholm Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Laszlo Ersek Reviewed-by: Ard Biesheuvel --- Notes: v3: - modify commit message (s/upcoming/potential/ build) to

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

2017-03-28 Thread Yao, Jiewen
Reviewed-by: jiewen@intel.com > -Original Message- > From: Gao, Liming > Sent: Tuesday, March 28, 2017 2:11 PM > To: edk2-devel@lists.01.org > Cc: Yao, Jiewen > Subject: [Patch] SignedCapsulePkg: Update RecoveryModuleLoadPei to report > the correct FvInfo > > Update logic to install

[edk2] Reset from post BDS pahse to PEI phase

2017-03-28 Thread Amit kumar
Hi , I have been facing some problem with blockIO reads even after reconnect and update map. when i try to do dblk on some of my blocks it returns with EFI_INVALID_PARAMETER. Is there a way to reinitialize the EFI platform from post bds phase or post dxe phase without putting the board in powe

Re: [edk2] [PATCH] EmbeddedPkg: add DT platform driver to select between DT and ACPI

2017-03-28 Thread Laszlo Ersek
On 03/28/17 12:51, Ard Biesheuvel wrote: > On 28 March 2017 at 11:49, Laszlo Ersek wrote: >> On 03/28/17 12:43, Laszlo Ersek wrote: >>> Ard, >>> >>> On 03/27/17 12:07, Ard Biesheuvel wrote: >> >> [snip] >> +/** + The entry point for DtPlatformDxe driver. + + @param[in] ImageH

Re: [edk2] [PATCH] EmbeddedPkg: add DT platform driver to select between DT and ACPI

2017-03-28 Thread Ard Biesheuvel
On 28 March 2017 at 11:49, Laszlo Ersek wrote: > On 03/28/17 12:43, Laszlo Ersek wrote: >> Ard, >> >> On 03/27/17 12:07, Ard Biesheuvel wrote: > > [snip] > >>> +/** >>> + The entry point for DtPlatformDxe driver. >>> + >>> + @param[in] ImageHandle The image handle of the driver. >>> + @para

Re: [edk2] [PATCH] EmbeddedPkg: add DT platform driver to select between DT and ACPI

2017-03-28 Thread Laszlo Ersek
On 03/28/17 12:43, Laszlo Ersek wrote: > Ard, > > On 03/27/17 12:07, Ard Biesheuvel wrote: [snip] >> +/** >> + The entry point for DtPlatformDxe driver. >> + >> + @param[in] ImageHandle The image handle of the driver. >> + @param[in] SystemTable The system table. >> + >> + @retval EF

Re: [edk2] [PATCH] EmbeddedPkg: add DT platform driver to select between DT and ACPI

2017-03-28 Thread Ard Biesheuvel
On 28 March 2017 at 11:43, Laszlo Ersek wrote: > Ard, > > On 03/27/17 12:07, Ard Biesheuvel wrote: >> As a follow up to the changes proposed by Laszlo to make ACPI and DT >> mutually exclusive on ArmVirtQemu, this patch proposes a DT platform >> DXE driver that either installs the NULL protocol Pl

Re: [edk2] [PATCH] EmbeddedPkg: add DT platform driver to select between DT and ACPI

2017-03-28 Thread Laszlo Ersek
Ard, On 03/27/17 12:07, Ard Biesheuvel wrote: > As a follow up to the changes proposed by Laszlo to make ACPI and DT > mutually exclusive on ArmVirtQemu, this patch proposes a DT platform > DXE driver that either installs the NULL protocol PlatformHasAcpiGuid, > or installs the FV embedded DTB bin

[edk2] [Patch] BaseTools: tools_def.txt to append new option in VFRPP flag

2017-03-28 Thread Yonghong Zhu
tools_def.txt to append new option (/FI$(MODULE_NAME)ImgDefs.h) in VFRPP flag. Cc: Liming Gao Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Yonghong Zhu --- BaseTools/Conf/tools_def.template | 172 +++--- 1 file changed, 86 insertions(+),

[edk2] [Patch] BaseTools: Update Pkcs7 and RSA2048 tool with shell=True

2017-03-28 Thread Yonghong Zhu
Pkcs7Sign, Rsa2048Sha256Sign and Rsa2048Sha256GenerateKeys doesn't work on Linux. It needs to be changed with shell=True. Cc: Liming Gao Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Yonghong Zhu --- BaseTools/Source/Python/Pkcs7Sign/Pkcs7Sign.py |

Re: [edk2] [PATCH 00/12] OvmfPkg: Enable variable access in PEI

2017-03-28 Thread Laszlo Ersek
On 03/27/17 23:47, Jordan Justen wrote: > On 2017-03-27 11:03:42, Laszlo Ersek wrote: >> On 03/27/17 10:05, Jordan Justen wrote: >>> web: https://github.com/jljusten/edk2/tree/pei-vars-v1 >>> >>> git: https://github.com/jljusten/edk2.git pei-vars-v1 >>> >>> This series moves flash detection into PE

[edk2] [PATCH 1/3] UefiCpuPkg/PiSmmCpuDxeSmm: Save SMM ranges info into global variables

2017-03-28 Thread Jeff Fan
Cc: Jiewen Yao Cc: Michael Kinney Cc: Feng Tian Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Jeff Fan --- UefiCpuPkg/PiSmmCpuDxeSmm/PiSmmCpuDxeSmm.c | 44 -- UefiCpuPkg/PiSmmCpuDxeSmm/PiSmmCpuDxeSmm.h | 4 ++- 2 files changed, 27 insertion

[edk2] [PATCH 3/3] UefiCpuPkg/PiSmmCpuDxeSmm: Update saved SMM ranges check in SmmProfile

2017-03-28 Thread Jeff Fan
SmmProfile feature required to protect all SMM ranges by structure mProtectionMemRangeTemplate. This update is to add additonal save SMM ranges into mProtectionMemRangeTemplate besides the range specified by mCpuHotPlugData.SmrrBase/mCpuHotPlugData.SmrrSiz. Cc: Jiewen Yao Cc: Michael Kinney Cc:

[edk2] [PATCH 2/3] UefiCpuPkg/PiSmmCpuDxeSmm: Add IsInSmmRanges() to check SMM range

2017-03-28 Thread Jeff Fan
Internal function IsInSmmRanges() is added t check SMM range by saved SMM ranges beside by mCpuHotPlugData.SmrrBase/mCpuHotPlugData.SmrrSiz. Cc: Jiewen Yao Cc: Michael Kinney Cc: Feng Tian Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Jeff Fan --- UefiCpuPkg/PiSmmCpuD

[edk2] [PATCH 0/3] UefiCpuPkg/PiSmmCpuDxeSmm: Check all SMM ranges found

2017-03-28 Thread Jeff Fan
NX/SmmProfile feature required to protect all SMM ranges. This update is to check additonal saved SMM ranges besides the range specified by mCpuHotPlugData.SmrrBase/mCpuHotPlugData.SmrrSiz. Cc: Jiewen Yao Cc: Michael Kinney Cc: Feng Tian Contributed-under: TianoCore Contribution Agreement 1.0 S

[edk2] [PATCH 2/3] UefiCpuPkg/PiSmmCpuDxeSmm: Add IsInSmmRanges() to check SMM range

2017-03-28 Thread Jeff Fan
Internal function IsInSmmRanges() is added t check SMM range by saved SMM ranges beside by mCpuHotPlugData.SmrrBase/mCpuHotPlugData.SmrrSiz. Cc: Jiewen Yao Cc: Michael Kinney Cc: Feng Tian Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Jeff Fan --- UefiCpuPkg/PiSmmCpuD

[edk2] [PATCH 1/3] UefiCpuPkg/PiSmmCpuDxeSmm: Save SMM ranges info into global variables

2017-03-28 Thread Jeff Fan
NX/SmmProfile feature required to protect all SMM ranges. This update is to check all saved SMM ranges by SMM ACCESS protocol besides the range specified by mCpuHotPlugData.SmrrBase/mCpuHotPlugData.SmrrSiz. Cc: Jiewen Yao Cc: Michael Kinney Cc: Feng Tian Contributed-under: TianoCore Contributio

[edk2] [PATCH 3/3] UefiCpuPkg/PiSmmCpuDxeSmm: Update saved SMM ranges check in SmmProfile

2017-03-28 Thread Jeff Fan
SmmProfile feature required to protect all SMM ranges by structure mProtectionMemRangeTemplate. This update is to add additonal save SMM ranges into mProtectionMemRangeTemplate besides the range specified by mCpuHotPlugData.SmrrBase/mCpuHotPlugData.SmrrSiz. Cc: Jiewen Yao Cc: Michael Kinney Cc:

[edk2] [PATCH 0/3] UefiCpuPkg/PiSmmCpuDxeSmm: Check all SMM ranges found

2017-03-28 Thread Jeff Fan
NX/SmmProfile feature required to protect all SMM ranges. This update is to check additonal saved SMM ranges besides the range specified by mCpuHotPlugData.SmrrBase/mCpuHotPlugData.SmrrSiz. Cc: Jiewen Yao Cc: Michael Kinney Cc: Feng Tian Contributed-under: TianoCore Contribution Agreement 1.0 S

Re: [edk2] [PATCH v3 04/12] MdeModulePkg: introduce EDKII Platform Has ACPI GUID

2017-03-28 Thread Laszlo Ersek
Star, Feng, are you okay with this patch? As we discussed under the v2 thread, BaseTools don't currently generate EDKII_PLATFORM_HAS_ACPI_GUID style macros that are usable for initializing static GUID variables, and GUID fields of static structures. Therefore the MdeModulePkg/Include/Guid/Pla

Re: [edk2] [PATCH v2 00/11] Upgrade CryptoPkg to use the OpenSSL 1.1.0xx/stable release

2017-03-28 Thread Ye, Ting
Series Reviewed-by: Ye Ting -Original Message- From: Long, Qin Sent: Thursday, March 23, 2017 9:19 PM To: edk2-devel@lists.01.org Cc: Ye, Ting ; Wu, Jiaxin ; ler...@redhat.com; ard.biesheu...@linaro.org; g...@suse.com; ronald.c...@arm.com; moso@citrix.com; thomas.pal...@hpe.com Su

Re: [edk2] [RFC PATCH v2 04/10] OvmfPkg/BaseMemcryptSevLib: Add SEV helper library

2017-03-28 Thread Laszlo Ersek
On 03/27/17 20:44, Brijesh Singh wrote: > On Mon, Mar 27, 2017 at 5:07 AM, Laszlo Ersek wrote: > >> On 03/27/17 11:19, Laszlo Ersek wrote: >>> On 03/21/17 22:13, Brijesh Singh wrote: >> + Returns a boolean to indicate whether SEV is enabled + + @retval TRUE When SEV is

Re: [edk2] [RFC PATCH v2 09/10] OvmfPkg/QemuFwCfgLib: Add Secure Encrypted Virtualization (SEV) support

2017-03-28 Thread Laszlo Ersek
On 03/27/17 21:24, Brijesh Singh wrote: > On Mon, Mar 27, 2017 at 5:19 AM, Laszlo Ersek wrote: >> - regarding InternalQemuFwCfgDmaBytes(): would it be possible to hook >> the SEV-related tricks into the existent function, rather than >> copy+modify the entire function as InternalQemuFwCfgSevDmaBy

Re: [edk2] [PATCH v2 05/12] ArmPkg: introduce EDKII Platform Has ACPI Protocol, and plug-in library

2017-03-28 Thread Laszlo Ersek
On 03/28/17 07:25, Gao, Liming wrote: > Ersek: > I don't want to block your patch. You can still check in Guid header file > if you think it is necessary. I don't want to check in without formal MdeModulePkg maintainer approval. If I check in a patch without formal approval, that will only lea

[edk2] [PATCH 1/2] MdeModulePkg/Tcp4Dxe: Fix unconditional window shrinking

2017-03-28 Thread ate...@kraftway.ru
Moving Right window edge to the left on sender side without additional checks leads to the situation when sender assumes the receiver shrunk its rcv buffer, when, in fact, it only reduced window size. This is a TCP deadlock situation. Receiver ACKs proper segment, while sender discards it for futur

[edk2] [PATCH 2/2] NetworkPkg/TcpDxe: Fix unconditional window shrinking

2017-03-28 Thread ate...@kraftway.ru
Moving Right window edge to the left on sender side without additional checks leads to the situation when sender assumes the receiver shrunk its rcv buffer, when, in fact, it only reduced window size. This is a TCP deadlock situation. Receiver ACKs proper segment, while sender discards it for futur