[edk2] OVMF: hang when booting Linux via GRUB after driver allocates 2GB above 4GB

2016-04-15 Thread Bruce Cran
I've been trying to debug a problem I've come across when booting Linux (CentOS 7) under OVMF (it happens with master, UDK2015 and UDK2014.SP1): if a driver allocates a couple of GB memory above 4GB, then the boot process hangs: e.g. on a VM with plenty of RAM (16GB): EFI_PHYSICAL_ADDRESS

[edk2] [PATCH] [VlvTbltDevPkg]: Add SMM_DEBUG_ENABLE to control the usage of source level debug of SMM code.

2016-04-15 Thread Wei, David
Signed-off-by: Tim He --- Vlv2TbltDevicePkg/PlatformPkg.fdf | 4 +--- Vlv2TbltDevicePkg/PlatformPkgConfig.dsc | 8 Vlv2TbltDevicePkg/PlatformPkgIA32.dsc | 21 ++--- Vlv2TbltDevicePkg/PlatformPkgX64.dsc| 7 +++ 4 files changed, 26

[edk2] [PATCH] [BaseTools]/Build: Consider only build-specified architectures

2016-04-15 Thread Thomas Palmer
When building for any specific architecture, the build script today is loading DSC sections for other architectures not in the build. The build process should disregard DSC sections that are not relevant to the build. This fixes scenario whereby a build occurs in a source tree that was been

Re: [edk2] GenFds fails with MWS.

2016-04-15 Thread Marvin Häuser
PS: I think it might work on Windows because GenFds checks the first character of \ or /. Though Windows paths start with the drive letter. :) Von: Marvin Häuser Gesendet: Freitag, 15. April 2016 22:51 An:

Re: [edk2] GenFds fails with MWS.

2016-04-15 Thread Marvin Häuser
Hey Yonghong, I have investigated the issue a little more. For some reason, it works fine on Windows, though is broken in WSL. After outputing some of the steps done to the ActivePlatform path, I figured out it was this block that causes the issue: if os.path.normcase

Re: [edk2] [PATCH] SecuritPkg: DxeImageVerificationLib: Fix wrong verification logic in DBX & DBT

2016-04-15 Thread Mangefeste, Tony
Yes, this process needs refinement, it's on my list of things to worry about. We will have a standard way for handling security issues, and part of the upcoming rollout of our defect management system, procedure will target security bugs. Got the memo, it's a problem we'll work on a fix. >

Re: [edk2] [PATCH] SecuritPkg: DxeImageVerificationLib: Fix wrong verification logic in DBX & DBT

2016-04-15 Thread Peter Jones
On Fri, Apr 15, 2016 at 08:10:50AM -0700, Dick Wilkins wrote: > Peter, > > Yes, we definatly have a broken procedure here. I think that Tony M. needs to > work with the USRT to develop a procedure for handling security issues in the > open-source code. The general flow should be: > > 1- A

Re: [edk2] [PATCH] SecuritPkg: DxeImageVerificationLib: Fix wrong verification logic in DBX & DBT

2016-04-15 Thread Dick Wilkins
Peter, Yes, we definatly have a broken procedure here. I think that Tony M. needs to work with the USRT to develop a procedure for handling security issues in the open-source code. The general flow should be: 1- A security vulnerability is found and is reported to the USRT as well as the

Re: [edk2] [PATCH] SecuritPkg: DxeImageVerificationLib: Fix wrong verification logic in DBX & DBT

2016-04-15 Thread Peter Jones
On Fri, Apr 15, 2016 at 12:40:14AM +, Zhang, Chao B wrote: > Hi all: > Thank you very much for the info. Do you agree to add "[USRT > 0001604]: Bug found in SecuritPkg: DxeImageVerificationLib" into the > log & check in this patch? What's the point of adding our internal tracker to it?

Re: [edk2] HII incompatibility between edk2 and iPXE?

2016-04-15 Thread Michael Brown
On 15/04/16 12:00, Gao, Liming wrote: In UI, the question is shown to user, such as checkbox, oneof, orderlist, password. One HII driver exposes multiple questions. Those questions may refer to one buffer storage. For this usage, the C structure can be used as the buffer storage. Each

Re: [edk2] HII incompatibility between edk2 and iPXE?

2016-04-15 Thread Michael Brown
On 14/04/16 15:24, Andrew Fish wrote: It is hard to argue that using Hii to do simple things is kind of complex and hard to Grok. Not really. Three people on this list (myself, Laszlo, and Eric) all managed to independently make the exact same mistake. Either we're all incompetent, or

Re: [edk2] HII incompatibility between edk2 and iPXE?

2016-04-15 Thread Gao, Liming
Michael: In UI, the question is shown to user, such as checkbox, oneof, orderlist, password. One HII driver exposes multiple questions. Those questions may refer to one buffer storage. For this usage, the C structure can be used as the buffer storage. Each question refers to one field in C

[edk2] [patch 1/2] MdeModulePkg/SetupBrowserDxe: Get default from callback for orderedList

2016-04-15 Thread Dandan Bi
For orderedlist question, the value is stored in a buffer, not in HiiValue. So when need to get default value from callback function for orderedlist, need to pass the buffer. This patch is to enhance this logic. Cc: Liming Gao Cc: Eric Dong

[edk2] [patch 2/2] MdeModulePkg/DriverSampleDxe: Add a sample case

2016-04-15 Thread Dandan Bi
Add the sample case for orderedlist to get standard default value from Callback function. Cc: Liming Gao Cc: Eric Dong Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Dandan Bi ---

[edk2] [patch 0/2] Get default value from CallBack function for OrderedList

2016-04-15 Thread Dandan Bi
For a question, it can retrive default value from Callback() function. For OrderedList question, the value is stored in a buffer, not in HiiValue. So when calling Callback() function to get default value for OrderedList, it need to pass the buffer, not the HiiValue. Patch 1 is to enahnce the

[edk2] [Patch] BaseTools: enhance error handling for DSC file

2016-04-15 Thread Yonghong Zhu
Add logic for DSC file validation for Prebuild init. Add logic to detect error for DSC parser when '{' is missing. Cc: Liming Gao Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Yonghong Zhu ---

Re: [edk2] [PATCH] Fixed build issue when src level debug is ON,

2016-04-15 Thread Laszlo Ersek
On 04/15/16 10:39, Guo, Mang wrote: > and updated Pismmcpudxesmm build path. > > Contributed-under: TianoCore Contribution Agreement 1.0 > Signed-off-by: Mang Guo > Reviewed-by: Tim He > --- > Vlv2TbltDevicePkg/PlatformPkg.fdf | 5 - >

Re: [edk2] [PATCH] Fixed build issue when src level debug is ON,

2016-04-15 Thread Wei, David
Please add a MACRO to control the SMM debug. -Original Message- From: Guo, Mang Sent: Friday, April 15, 2016 4:39 PM To: edk2-devel@lists.01.org Cc: Wei, David ; He, Tim Subject: [PATCH] Fixed build issue when src level debug is ON, and updated

[edk2] [PATCH] Fixed build issue when src level debug is ON,

2016-04-15 Thread Guo, Mang
and updated Pismmcpudxesmm build path. Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Mang Guo Reviewed-by: Tim He --- Vlv2TbltDevicePkg/PlatformPkg.fdf | 5 - Vlv2TbltDevicePkg/PlatformPkgConfig.dsc | 2 +-