Re: [edk2] [Xen-devel] Windows 8 64bit Guest BSOD when using OVMF to boot and *install* from CDROM(when configured with more than 4G memory)

2015-05-07 Thread Wei Liu
On Thu, May 07, 2015 at 10:29:55AM +0100, Ian Campbell wrote: > On Thu, 2015-05-07 at 09:02 +0200, Laszlo Ersek wrote: > > (Plus, you are cloning a git repo that may or may not be > > identical to the (semi-)official edk2 git repo.) > > FWIW git://xenbits.xen.org/ovmf.git is a direct descendant of

Re: [edk2] [Patch 0/2] MdePkg: Add URI_DEVICE_PATH.

2015-05-07 Thread Tian, Feng
Looks good to me Reviewed-by: Feng Tian -Original Message- From: Ruiyu Ni [mailto:ruiyu...@intel.com] Sent: Tuesday, April 28, 2015 13:38 To: edk2-devel@lists.sourceforge.net Subject: [edk2] [Patch 0/2] MdePkg: Add URI_DEVICE_PATH. The patch adds URI device path node definition and Fro

Re: [edk2] [Patch] Fix Ecc check failure in UefiPxeBcDxe.inf

2015-05-07 Thread Ye, Ting
Reviewed-by: Ye Ting -Original Message- From: jiaxinwu [mailto:jiaxin...@intel.com] Sent: Friday, May 8, 2015 10:43 AM To: edk2-devel@lists.sourceforge.net Subject: [Patch] Fix Ecc check failure in UefiPxeBcDxe.inf Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by:

[edk2] [PATCH 1/2] MdePkg: Add RO memory and resource HOB attribute definitions.

2015-05-07 Thread Star Zeng
Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Star Zeng Reviewed-by: Liming Gao --- MdePkg/Include/Pi/PiHob.h | 17 + MdePkg/Include/Uefi/UefiSpec.h | 4 2 files changed, 21 insertions(+) diff --git a/MdePkg/Include/Pi/PiHob.h b/MdePkg/Include

[edk2] [PATCH 0/2] Add read only memory support.

2015-05-07 Thread Star Zeng
The patches follow UEFI 2.5 and PI 1.4 Spec to add read only memory support. Star Zeng (2): MdePkg: Add RO memory and resource HOB attribute definitions. MdeModulePkg DxeCore: Add read only memory support. MdeModulePkg/Core/Dxe/Gcd/Gcd.c | 4 MdePkg/Include/Pi/PiHob.h | 17 ++

[edk2] [PATCH 2/2] MdeModulePkg DxeCore: Add read only memory support.

2015-05-07 Thread Star Zeng
Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Star Zeng Reviewed-by: Liming Gao --- MdeModulePkg/Core/Dxe/Gcd/Gcd.c | 4 1 file changed, 4 insertions(+) diff --git a/MdeModulePkg/Core/Dxe/Gcd/Gcd.c b/MdeModulePkg/Core/Dxe/Gcd/Gcd.c index bce01a0..fdd2d40 100644 ---

[edk2] [PATCH V2 0/2] Add Reset2 ppi support

2015-05-07 Thread Star Zeng
The patches follow PI Spec 1.4 to add Reset2 ppi support. Update reference spec from PI1.3 to PI1.4 in file header of MdePkg.dec. Star Zeng (2): MdePkg: Add Reset2 ppi definition. MdeModulePkg: Add Reset2 ppi support in PEI CORE. MdeModulePkg/Core/Pei/PeiMain.h | 27

[edk2] [PATCH V2 2/2] MdeModulePkg: Add Reset2 ppi support in PEI CORE.

2015-05-07 Thread Star Zeng
Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Star Zeng Reviewed-by: Liming Gao --- MdeModulePkg/Core/Pei/PeiMain.h | 27 ++- MdeModulePkg/Core/Pei/PeiMain.inf | 3 ++- MdeModulePkg/Core/Pei/PeiMain/PeiMain.c | 3 ++- MdeModulePkg/Core/Pei

[edk2] [PATCH V2 1/2] MdePkg: Add Reset2 ppi definition.

2015-05-07 Thread Star Zeng
Update reference spec from PI1.3 to PI1.4 in file header of MdePkg.dec. Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Star Zeng Reviewed-by: Liming Gao --- MdePkg/Include/Pi/PiPeiCis.h | 27 - MdePkg/Include/Ppi/Reset2.h | 38 ++

Re: [edk2] [PATCH] Treat ASCII DEL (0x7f) as backspace

2015-05-07 Thread Roy Franz
On Thu, May 7, 2015 at 7:03 PM, Kinney, Michael D wrote: > Roy, > > Did you try the command line at the UEFI Shell prompt through a serial > terminal? There is line editing available, where backspace and delete keys > perform different actions. I think your change prevents a user from ever >

[edk2] [PATCH] MdePkg: Move gEfiSmbios3TableGuid to UEFI2.5 section in MdePkg.dec.

2015-05-07 Thread Star Zeng
Also enhance the comments in MdePkg/Include/Guid/Smbios.h. Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Star Zeng Reviewed-by: Liming Gao --- MdePkg/Include/Guid/SmBios.h | 6 +++--- MdePkg/MdePkg.dec| 8 2 files changed, 7 insertions(+), 7 deletio

Re: [edk2] [PATCH] Treat ASCII DEL (0x7f) as backspace

2015-05-07 Thread Kinney, Michael D
Roy, Did you try the command line at the UEFI Shell prompt through a serial terminal? There is line editing available, where backspace and delete keys perform different actions. I think your change prevents a user from ever getting the delete key behavior. Same would be true for the UEFI She

[edk2] [PATCH 2/3] MdeModulePkg: Update implementation of PeiAllocatePages() to allow EfiReservedMemoryType.

2015-05-07 Thread Star Zeng
Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Star Zeng Reviewed-by: Liming Gao --- MdeModulePkg/Core/Pei/Memory/MemoryServices.c | 5 +++-- 1 file changed, 3 insertions(+), 2 deletions(-) diff --git a/MdeModulePkg/Core/Pei/Memory/MemoryServices.c b/MdeModulePkg/Core/P

[edk2] [PATCH 1/3] MdePkg: Update comments of EFI_PEI_ALLOCATE_PAGES to allow EfiReservedMemoryType.

2015-05-07 Thread Star Zeng
Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Star Zeng Reviewed-by: Liming Gao --- MdePkg/Include/Pi/PiPeiCis.h | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/MdePkg/Include/Pi/PiPeiCis.h b/MdePkg/Include/Pi/PiPeiCis.h index f5f07de..c792b89 100

[edk2] [PATCH 3/3] MdePkg: Update PeiMemoryAllocationLib to allocate EfiReservedMemoryType memory directly.

2015-05-07 Thread Star Zeng
The old implementation was to allocate EfiBootServicesData memory first, and then update the MemoryType of memory allocation hob to EfiReservedMemoryType. Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Star Zeng Reviewed-by: Liming Gao --- .../PeiMemoryAllocationLib/Memo

[edk2] [PATCH 0/3] Allow for the PEI phase memory allocator to allocate memory with the reserved memory type.

2015-05-07 Thread Star Zeng
The patches follow PI Spec 1.4 to allow for the PEI phase memory allocator to allocate memory with the reserved memory type. Star Zeng (3): MdePkg: Update comments of EFI_PEI_ALLOCATE_PAGES to allow EfiReservedMemoryType. MdeModulePkg: Update implementation of PeiAllocatePages() t

Re: [edk2] [Patch] MdeModulePkg: Update FirmwarePerformanceDataTableDxe for LegacyBoot

2015-05-07 Thread Ni, Ruiyu
Reviewed-by: Ruyu Ni -Original Message- From: Liming Gao [mailto:liming@intel.com] Sent: Tuesday, May 5, 2015 2:14 PM To: edk2-devel@lists.sourceforge.net Subject: [edk2] [Patch] MdeModulePkg: Update FirmwarePerformanceDataTableDxe for LegacyBoot Change the code to listen EFI_SW_DX

Re: [edk2] [Patch 0/4] Add PI1.4 CapsulePpi

2015-05-07 Thread Zeng, Star
Reviewed-by: Star Zeng -Original Message- From: Liming Gao [mailto:liming@intel.com] Sent: Tuesday, May 5, 2015 2:14 PM To: edk2-devel@lists.sourceforge.net Subject: [edk2] [Patch 0/4] Add PI1.4 CapsulePpi Add PI1.4 CapsulePpi definition, remove CapsulePpi definition in MdeModulePkg

[edk2] [PATCH 0/2] Add Reset2 ppi support

2015-05-07 Thread Star Zeng
The patches follow PI Spec 1.4 to add Reset2 ppi support. Star Zeng (2): MdePkg: Add Reset2 ppi definition. MdeModulePkg: Add Reset2 ppi support in PEI CORE. MdeModulePkg/Core/Pei/PeiMain.h | 27 ++- MdeModulePkg/Core/Pei/PeiMain.inf | 3 ++- MdeModulePkg/Core/

[edk2] [PATCH 1/2] MdePkg: Add Reset2 ppi definition.

2015-05-07 Thread Star Zeng
Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Star Zeng Reviewed-by: Liming Gao --- MdePkg/Include/Pi/PiPeiCis.h | 27 - MdePkg/Include/Ppi/Reset2.h | 38 MdePkg/Include/Uefi/UefiMultiPhase.h |

[edk2] [PATCH 2/2] MdeModulePkg: Add Reset2 ppi support in PEI CORE.

2015-05-07 Thread Star Zeng
Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Star Zeng Reviewed-by: Liming Gao --- MdeModulePkg/Core/Pei/PeiMain.h | 27 ++- MdeModulePkg/Core/Pei/PeiMain.inf | 3 ++- MdeModulePkg/Core/Pei/PeiMain/PeiMain.c | 3 ++- MdeModulePkg/Core/Pei

[edk2] [PATCH] Treat ASCII DEL (0x7f) as backspace

2015-05-07 Thread Roy Franz
Most Linux terminal emulators use ASCII 0x7f (^?) for backspace, rather than 0x08 (^H) since ^H is used by some programs for other purposes, such as help menus. This results in backspace not working in most Linux environments, particularly in emulated environments such as QEMU where telnet or xter

Re: [edk2] [PATCH] BaseTools: add support for gcc 5.1 tool chain

2015-05-07 Thread Scott Duplichan
Jordan Justen [mailto:jordan.l.jus...@intel.com] wrote: ]Sent: Tuesday, May 05, 2015 01:42 AM ]To: edk2-devel@lists.sourceforge.net; Scott Duplichan ]Subject: Re: [edk2] [PATCH] BaseTools: add support for gcc 5.1 tool chain ] ]With the new GCC versioning scheme, we should name it GCC5. ] ]https:/

Re: [edk2] [Patch] MdePkg: Add EFI_LOAD_OPTION definition.

2015-05-07 Thread Kinney, Michael D
Felix, I agree with your points and agree that the C structure definition from UEFI 2.5 should be added to UefiSpec.h. I would like to see the comment block above the structure state that some of the variable length structure fields are commented out on purpose. Mike -Original Message

Re: [edk2] [Patch] MdePkg: Add EFI_LOAD_OPTION definition.

2015-05-07 Thread Felix Poludov
The EFI_LOAD_OPTION was not defined in the UEFI specification, but it is now (starting from UEFI 2.5). The situation with EFI_LOAD_OPTION was somewhat confusing. The type was mentioned in the specification, but was not explicitly defined. I think having the type explicitly defined has some value.

Re: [edk2] MdeModulePkg: BaseSortLib, UefiBootManagerLib support DXE_RUNTIME_DRIVER.

2015-05-07 Thread El-Haj-Mahmoud, Samer
Reviewed-by: Samer El-Haj-Mahmoud mailto:samer.el-haj-mahm...@hp.com>> From: Ni, Ruiyu [mailto:ruiyu...@intel.com] Sent: Wednesday, May 06, 2015 5:35 AM To: Dong, Eric Cc: edk2-devel@lists.sourceforge.net Subject: [edk2] MdeModulePkg: BaseSortLib, UefiBootManagerLib support DXE_RUNTIME_DRIV

Re: [edk2] [PATCH 1/2] MdePkg: Add UEFI2.5 bluetooth protocol/devicepath definition in MdePkg.

2015-05-07 Thread El-Haj-Mahmoud, Samer
Series Reviewed-by: Samer El-Haj-Mahmoud -Original Message- From: Qiu Shumin [mailto:shumin@intel.com] Sent: Tuesday, May 05, 2015 1:27 AM To: edk2-devel@lists.sourceforge.net Subject: [edk2] [PATCH 1/2] MdePkg: Add UEFI2.5 bluetooth protocol/devicepath definition in MdePkg. Cont

Re: [edk2] [PATCH] MdePkg: Add longer term new CPER memory section definition.

2015-05-07 Thread El-Haj-Mahmoud, Samer
Changes look good Reviewed-by: Samer El-Haj-Mahmoud -Original Message- From: Qiu Shumin [mailto:shumin@intel.com] Sent: Wednesday, May 06, 2015 2:51 AM To: edk2-devel@lists.sourceforge.net; jeff@intel.com Subject: [edk2] [PATCH] MdePkg: Add longer term new CPER memory section

Re: [edk2] [Patch 2/3] MdeModulePkg: Implement UEFI25 HII Config keyword handler protocol.

2015-05-07 Thread El-Haj-Mahmoud, Samer
Eric, I only have one quick feedback: Instead of defining this: CHAR8 gNameSpace[] = "x-UEFI-"; You should use the existing definition for "x-UEFI" in UefiInternalFormRepresentation.h, which is also form the UEFI 2.1+ spec: #define UEFI_CONFIG_LANG "x-UEFI" The rest of the code is very invo

Re: [edk2] [Patch] MdePkg/Include: Add support for flexible array members

2015-05-07 Thread Hauch, Larry
Hi Mike, Looks good! Reviewed-by: lhauch Cheers, Larry From: Kinney, Michael D [mailto:michael.d.kin...@intel.com] Sent: Tuesday, May 05, 2015 6:59 PM To: edk2-devel@lists.sourceforge.net Subject: [edk2] [Patch] MdePkg/Include: Add support for flexible array members Use pragma to disable warni

Re: [edk2] [PATCH v4 2/3] ArmVExpressPkg: avoid the use of ArmVExpressSysConfigLib at runtime

2015-05-07 Thread Olivier Martin
Reviewed-by: Olivier Martin -Original Message- From: Ard Biesheuvel [mailto:ard.biesheu...@linaro.org] Sent: 04 May 2015 07:05 To: Olivier Martin; ler...@redhat.com; edk2-devel@lists.sourceforge.net Cc: roy.fr...@linaro.org; leif.lindh...@linaro.org; Ard Biesheuvel Subject: [PATCH v4 2/3]

Re: [edk2] [PATCH v4 3/3] ArmVExpressPkg: restrict ArmVExpressSysConfigLib to SEC and DXE_DRIVER

2015-05-07 Thread Olivier Martin
Reviewed-by: Olivier Martin -Original Message- From: Ard Biesheuvel [mailto:ard.biesheu...@linaro.org] Sent: 04 May 2015 07:05 To: Olivier Martin; ler...@redhat.com; edk2-devel@lists.sourceforge.net Cc: roy.fr...@linaro.org; leif.lindh...@linaro.org; Ard Biesheuvel Subject: [PATCH v4 3/3]

Re: [edk2] [PATCH 4/5] ArmPlatformPkg: add support for FV embedded device tree blobs

2015-05-07 Thread Olivier Martin
Reviewed-by: Olivier Martin -Original Message- From: Ard Biesheuvel [mailto:ard.biesheu...@linaro.org] Sent: 05 May 2015 15:49 To: Olivier Martin; edk2-devel@lists.sourceforge.net; Ronald Cron Cc: leif.lindh...@linaro.org; roy.fr...@linaro.org; ler...@redhat.com; Ard Biesheuvel Subject:

Re: [edk2] [PATCH 5/5] ArmVExpressPkg: add support for embedding DTBs for AArch64 variants

2015-05-07 Thread Olivier Martin
Reviewed-by: Olivier Martin -Original Message- From: Ard Biesheuvel [mailto:ard.biesheu...@linaro.org] Sent: 05 May 2015 15:49 To: Olivier Martin; edk2-devel@lists.sourceforge.net; Ronald Cron Cc: leif.lindh...@linaro.org; roy.fr...@linaro.org; ler...@redhat.com; Ard Biesheuvel Subject:

Re: [edk2] Replacement EDK2 email list coming soon

2015-05-07 Thread B Cran
On Thursday, May 7, 2015, Olivier Martin wrote: > Announcing Changes Coming Soon: "5. Open Discussions" > > Let's hope in the future community feedbacks will be more taking in > account. Given that people have proposed solutions in recent months and nothing changed, I suspect more of a 'benevol

Re: [edk2] Replacement EDK2 email list coming soon

2015-05-07 Thread Olivier Martin
Announcing Changes Coming Soon: "5. Open Discussions" Let's hope in the future community feedbacks will be more taking in account. I am happy to see this list of new changes but it would have been better to involve the community in the discussion for the chosen solutions. As Jordan said Github a

Re: [edk2] [PATCH v3 2/3] ArmPlatformPkg: enable use of authenticated variables in NorFlashDxe

2015-05-07 Thread Olivier Martin
I prefer this new version. I will push it into our CI. And if it comes back green, I will commit it! Thanks again Ard for the contribution :-) On 07/05/15 08:27, Ard Biesheuvel wrote: > On 7 May 2015 at 08:55, Laszlo Ersek wrote: >> On 05/07/15 08:16, Ard Biesheuvel wrote: >>> The NorFlashDxe u

Re: [edk2] Question about WinDbg Support-- Target with USB->Serial Adapter.

2015-05-07 Thread Brian J. Johnson
> *From:* Saiprasad Chavali [mailto:s...@marvell.com] > *Sent:* Thursday, May 7, 2015 1:18 AM > *To:* edk2-devel@lists.sourceforge.net > ; Tian, Feng > *Subject:* RE: [edk2] Question about WinDbg Support-- Target with > USB->Serial Adapter. > > Hi Tian, > >

Re: [edk2] [PATCH v2 2/3] ArmPlatformPkg: enable use of authenticated variables in NorFlashDxe

2015-05-07 Thread Olivier Martin
On 06/05/15 19:31, Ard Biesheuvel wrote: > On 6 May 2015 at 20:20, Olivier Martin wrote: >> I have to say I generally do not really like the 'APRIORI DXE' (because >> of risks of circular dependency and harder to debug when forgotten) but >> I have not found a better way to do in the current form

Re: [edk2] [Xen-devel] Windows 8 64bit Guest BSOD when using OVMF to boot and *install* from CDROM(when configured with more than 4G memory)

2015-05-07 Thread Laszlo Ersek
On 05/07/15 11:29, Ian Campbell wrote: > On Thu, 2015-05-07 at 09:02 +0200, Laszlo Ersek wrote: >> (Plus, you are cloning a git repo that may or may not be >> identical to the (semi-)official edk2 git repo.) > > FWIW git://xenbits.xen.org/ovmf.git is a direct descendant of the > https://github.com

Re: [edk2] [patch] BaseTool : The PCD value in uninitialized data range should be natural aligned.

2015-05-07 Thread Ard Biesheuvel
On 7 May 2015 at 11:14, Feng, Bob C wrote: > Hi, > > Here is the patch to fix the BaseTools issue that the PCD value in > uninitialized data range is not natural aligned. > > Contributed-under: TianoCore Contribution Agreement 1.0 > Signed-off-by: Bob Feng < bob.c.f...@intel.com > Hello Bob, Th

[edk2] [patch] BaseTool : The PCD value in uninitialized data range should be natural aligned.

2015-05-07 Thread Feng, Bob C
Hi, Here is the patch to fix the BaseTools issue that the PCD value in uninitialized data range is not natural aligned. Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Bob Feng < bob.c.f...@intel.com > Best Regards, Bob Feng alignedUnini

Re: [edk2] Windows 8 64bit Guest BSOD when using OVMF to boot and *install* from CDROM(when configured with more than 4G memory)

2015-05-07 Thread lidonglin
I just git clone ovmf from official edk2 git repo ang git latest qemu 2.3.50. But the issue still exists. Besides, I find windows server 2012 has the same BSOD issue but windows 8.1 works normally. -Original Message- From: Laszlo Ersek [mailto:ler...@redhat.com] Sent: 2015年5月7日 15:03

Re: [edk2] [Patch]Vlv2TbltDevicePkg: Add a setup option to configure fTPM feature

2015-05-07 Thread Wei, David
Please set default setup value of FTPM to be false to avoid violating some country's import law. And change FTPM build option of 32-bit build tip to be TRUE, keep FTPM build option of 64-bit build tip to be FALSE. Thanks, David | SSG BIOS From: He, Tim Sent: Thursday, May 07, 2015 4:13 PM To:

[edk2] [Patch]Vlv2TbltDevicePkg: Add a setup option to configure fTPM feature

2015-05-07 Thread He, Tim
Add a setup option to configure fTPM feature for MinnowBoad Max. Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Tim He mailto:tim...@intel.com>> Best Regards, Tim AddFtpmSetupOption.patch Description: AddFtpmSetupOption.patch ---

Re: [edk2] [PATCH v3 2/3] ArmPlatformPkg: enable use of authenticated variables in NorFlashDxe

2015-05-07 Thread Ard Biesheuvel
On 7 May 2015 at 08:55, Laszlo Ersek wrote: > On 05/07/15 08:16, Ard Biesheuvel wrote: >> The NorFlashDxe uses an explicit 'BEFORE xxx' Depex declaration to >> ensure that it is dispatched before VariableRuntimeDxe, and uses the >> file GUID of the latter as 'xxx' explicitly to accomplish that. >>

Re: [edk2] Windows 8 64bit Guest BSOD when using OVMF to boot and *install* from CDROM(when configured with more than 4G memory)

2015-05-07 Thread Laszlo Ersek
On 05/07/15 08:47, lidonglin wrote: > Dear all: > New issue: > Guest BSOD and restart when using OVMF to boot and install windows8 > 64bit OS(which owns more than 4G memmory)。 > My environment as below: > xen 4.5.0 > qemu 2.2.1 > ovmf git clone from git://xenbits.xen.org/ovmf.git (comm