Re: [edk2] [PATCH v3 1/5] ArmPlatformPkg/PlatformIntelBdsLib: remove ARM BDS dependency

2015-09-01 Thread Leif Lindholm
On Tue, Sep 01, 2015 at 07:12:33PM +0200, Ard Biesheuvel wrote: > On 1 September 2015 at 18:59, Leif Lindholm wrote: > > On Tue, Sep 01, 2015 at 11:58:20AM +0200, Ard Biesheuvel wrote: > >> The Intel BDS platform library still depends on the ARM BDS specific > >> BdsLib.

Re: [edk2] [PATCH v3 1/5] ArmPlatformPkg/PlatformIntelBdsLib: remove ARM BDS dependency

2015-09-01 Thread Leif Lindholm
On Tue, Sep 01, 2015 at 11:58:20AM +0200, Ard Biesheuvel wrote: > The Intel BDS platform library still depends on the ARM BDS specific > BdsLib. So replace its invocations with GenericBdsLib counterparts, > and fix up where needed, so that we can drop the dependency. > > Contributed-under:

Re: [edk2] [PATCH v3 1/5] ArmPlatformPkg/PlatformIntelBdsLib: remove ARM BDS dependency

2015-09-01 Thread Ard Biesheuvel
On 1 September 2015 at 18:59, Leif Lindholm wrote: > On Tue, Sep 01, 2015 at 11:58:20AM +0200, Ard Biesheuvel wrote: >> The Intel BDS platform library still depends on the ARM BDS specific >> BdsLib. So replace its invocations with GenericBdsLib counterparts, >> and fix

Re: [edk2] [PATCH v2 4/5] ArmPlatformPkg/ArmVExpress-FVP: add support for the Intel BDS

2015-09-01 Thread Ryan Harkin
On 31 August 2015 at 10:28, Ard Biesheuvel wrote: > On 27 August 2015 at 18:39, Ryan Harkin wrote: > > > > > > On 26 August 2015 at 10:59, Ard Biesheuvel > > wrote: > >> > >> This adds support for the Intel BDS, by

Re: [edk2] [PATCH v2 0/3] unify FVP Base and Foundation model support

2015-09-01 Thread Ryan Harkin
On 1 September 2015 at 08:09, Ryan Harkin wrote: > > > On 31 August 2015 at 10:20, Ard Biesheuvel > wrote: > >> On 27 August 2015 at 15:44, Ryan Harkin wrote: >> > >> > >> > On 24 August 2015 at 12:22, Ard Biesheuvel

[edk2] [Patch 0/3] Fix several PI compliance issues

2015-09-01 Thread Jeff Fan
This serial of patches is to fix some PI compliance issues. Jeff Fan (3): UefiCpuPkg/CpuMpPei: Update the old/new BSP state in SwitchBsp() UefiCpuPkg/CpuMpPei: Check Function pointer in PeiStartupAllAPs () UefiCpuPkg/CpuMpPei: Fix CPU Healthy issue in PeiGetProcessorInfo ()

[edk2] [Patch 3/3] UefiCpuPkg/CpuMpPei: Fix CPU Healthy issue in PeiGetProcessorInfo ()

2015-09-01 Thread Jeff Fan
CPU Healthy state maybe changed by software. We should return Healthy state from Healthy bit instead of from CPU BIST hardware information. Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Jeff Fan CC: Feng Tian ---

[edk2] [Patch 2/3] UefiCpuPkg/CpuMpPei: Check Function pointer in PeiStartupAllAPs ()

2015-09-01 Thread Jeff Fan
Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Jeff Fan CC: Feng Tian --- UefiCpuPkg/CpuMpPei/PeiMpServices.c | 4 1 file changed, 4 insertions(+) diff --git a/UefiCpuPkg/CpuMpPei/PeiMpServices.c

Re: [edk2] [Patch 0/3] Fix several PI compliance issues

2015-09-01 Thread Tian, Feng
Looks good to me Reviewed-by: Feng Tian -Original Message- From: edk2-devel [mailto:edk2-devel-boun...@lists.01.org] On Behalf Of Jeff Fan Sent: Tuesday, September 01, 2015 14:44 To: edk2-devel@lists.01.org Subject: [edk2] [Patch 0/3] Fix several PI compliance

Re: [edk2] [patch] CryptoPkg: Replace string wrapper functions with safe string functions

2015-09-01 Thread Dong, Eric
Reviewed-by: Eric Dong -Original Message- From: Long, Qin Sent: Tuesday, September 01, 2015 4:55 PM To: Ye, Ting; Dong, Eric Cc: edk2-devel@lists.01.org Subject: [patch] CryptoPkg: Replace string wrapper functions with safe string functions EDKII core suggest

Re: [edk2] [PATCH v2 4/5] ArmPlatformPkg/ArmVExpress-FVP: add support for the Intel BDS

2015-09-01 Thread Benjamin Herrenschmidt
On Wed, 2015-09-02 at 01:01 +0200, Ard Biesheuvel wrote: > Excellent question. > > If you are doing a new port, there is no reason to use the old Intel > BDS and you are better off supporting only the new universal BDS. Thanks ! Cheers, Ben. > For the ARM platforms, I think we should move to

Re: [edk2] [PATCH] MdePkg: Add UefiDebugLibDebugPortProtocol to output logs via EFI_DEBUGPORT_PROTOCOL.Write.

2015-09-01 Thread Masamitsu MURASE
Hi MdePkg maintainers, I'll be grateful if you can reply to me. I believe that this kind of feature helps debugging in PC vendors and 3rd party UEFI driver vendors. Regards, Murase --- Masamitsu MURASE https://github.com/masamitsu-murase -- Mail: masamitsu.mur...@gmail.com 村瀬 昌満

Re: [edk2] [PATCH v2 4/5] ArmPlatformPkg/ArmVExpress-FVP: add support for the Intel BDS

2015-09-01 Thread Benjamin Herrenschmidt
On Wed, 2015-08-26 at 11:59 +0200, Ard Biesheuvel wrote: > This adds support for the Intel BDS, by introducing a define > 'USE_ARM_BDS' which defaults to TRUE, and can be overridden on > the build command line. Out of curiosity (still in the process of getting my head around the BDS stuff for my

Re: [edk2] [PATCH] OvmfPkg: AcpiTables: serialize control methods that create named objects

2015-09-01 Thread Laszlo Ersek
Jordan, can you please check this one too? It's trivial. Thanks Laszlo On 08/28/15 11:16, Laszlo Ersek wrote: > Bruce Cran reported the following issue: > > With iasl version 20150410-64 building OvmfX64 (using OvmfPkg/build.sh > -a X64 -t GCC49 -b RELEASE) results in a couple of warnings

Re: [edk2] [PATCH v2 4/5] ArmPlatformPkg/ArmVExpress-FVP: add support for the Intel BDS

2015-09-01 Thread Ard Biesheuvel
On 2 September 2015 at 00:32, Benjamin Herrenschmidt wrote: > On Wed, 2015-08-26 at 11:59 +0200, Ard Biesheuvel wrote: >> This adds support for the Intel BDS, by introducing a define >> 'USE_ARM_BDS' which defaults to TRUE, and can be overridden on >> the build command line. > >

[edk2] [PATCH v2] MdeModulePkg: Fix the issue in DxeCore CoreInitializeMemoryServices()

2015-09-01 Thread Liming Gao
CoreInitializeMemoryServices() will essentially pickup a piece of memory out of *any* memory descriptor HOB completely ignoring the memory allocation HOBs. This change considers memory allocation hobs in this API. v2 adds check the memory range in the resource HOB that includes PHIT range to make

[edk2] [PATCH v3 5/5] ArmPlatformPkg/ArmVExpress-FVP: enable UEFI Secure Boot

2015-09-01 Thread Ard Biesheuvel
This allows the FVP target to be built with UEFI Secure Boot enabled, by passing -D SECURE_BOOT_ENABLE to the build command line. Note that you will need to disable the ARM BDS (-D USE_ARM_BDS=FALSE) or you will not be able to enroll certificates, since the ARM BDS does not provide a GUI to do so.

[edk2] [patch] CryptoPkg: Replace string wrapper functions with safe string functions

2015-09-01 Thread Qin Long
EDKII core suggest retire unsafe string functions. This patch is to replace string wrapper functions with new-added safe string functions for consistence. Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Qin Long --- CryptoPkg/Include/OpenSslSupport.h |

[edk2] [PATCH v3 4/5] ArmPlatformPkg/ArmVExpress-FVP: add support for the Intel BDS

2015-09-01 Thread Ard Biesheuvel
This adds support for the Intel BDS, by introducing a define 'USE_ARM_BDS' which defaults to TRUE, and can be overridden on the build command line. Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Ard Biesheuvel Reviewed-by: Ryan Harkin

[edk2] [PATCH v3 1/5] ArmPlatformPkg/PlatformIntelBdsLib: remove ARM BDS dependency

2015-09-01 Thread Ard Biesheuvel
The Intel BDS platform library still depends on the ARM BDS specific BdsLib. So replace its invocations with GenericBdsLib counterparts, and fix up where needed, so that we can drop the dependency. Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Ard Biesheuvel

[edk2] [PATCH v3 2/5] ArmPlatformPkg/PlatformIntelBdsLib: fix error handling

2015-09-01 Thread Ard Biesheuvel
In InitializeConsolePipe (), we clobber the Status variable in the error handling path that reports Status in its output. Instead, use a NULL check on the LocateProtocol () output argument. Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Ard Biesheuvel

[edk2] [patch] MdeModulePkg:Support orderedList with default value

2015-09-01 Thread Dandan Bi
Our tool can support OrderedList which has default value,but doesn't update the source code in HiiDatabase when parse the ifr data.Now update the code and add test case in DriverSample. Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Dandan Bi ---

Re: [edk2] [PATCH v3 3/5] ArmPlatformPkg/PlatformIntelBdsLib: add splash screen support

2015-09-01 Thread Leif Lindholm
On Tue, Sep 01, 2015 at 11:58:22AM +0200, Ard Biesheuvel wrote: > Add a call to EnableQuietBoot () to BdsPlatformPolicyBehavior(), > so that a splash screen is shown in case one is present under the > correct GUID in the FV, and we have graphics support. > > Contributed-under: TianoCore

Re: [edk2] [PATCH v3 5/5] ArmPlatformPkg/ArmVExpress-FVP: enable UEFI Secure Boot

2015-09-01 Thread Leif Lindholm
On Tue, Sep 01, 2015 at 11:58:24AM +0200, Ard Biesheuvel wrote: > This allows the FVP target to be built with UEFI Secure Boot enabled, > by passing -D SECURE_BOOT_ENABLE to the build command line. Note that > you will need to disable the ARM BDS (-D USE_ARM_BDS=FALSE) or you will I guess this

Re: [edk2] [PATCH v3 4/5] ArmPlatformPkg/ArmVExpress-FVP: add support for the Intel BDS

2015-09-01 Thread Leif Lindholm
On Tue, Sep 01, 2015 at 11:58:23AM +0200, Ard Biesheuvel wrote: > This adds support for the Intel BDS, by introducing a define > 'USE_ARM_BDS' which defaults to TRUE, and can be overridden on > the build command line. > > Contributed-under: TianoCore Contribution Agreement 1.0 > Signed-off-by:

Re: [edk2] [PATCH v3 2/5] ArmPlatformPkg/PlatformIntelBdsLib: fix error handling

2015-09-01 Thread Ard Biesheuvel
On 1 September 2015 at 16:59, Leif Lindholm wrote: > On Tue, Sep 01, 2015 at 11:58:21AM +0200, Ard Biesheuvel wrote: >> In InitializeConsolePipe (), we clobber the Status variable in the >> error handling path that reports Status in its output. Instead, >> use a NULL

Re: [edk2] [PATCH v3 2/5] ArmPlatformPkg/PlatformIntelBdsLib: fix error handling

2015-09-01 Thread Ard Biesheuvel
On 1 September 2015 at 16:59, Leif Lindholm wrote: > On Tue, Sep 01, 2015 at 11:58:21AM +0200, Ard Biesheuvel wrote: >> In InitializeConsolePipe (), we clobber the Status variable in the >> error handling path that reports Status in its output. Instead, >> use a NULL

Re: [edk2] [PATCH v3 2/5] ArmPlatformPkg/PlatformIntelBdsLib: fix error handling

2015-09-01 Thread Leif Lindholm
On Tue, Sep 01, 2015 at 11:58:21AM +0200, Ard Biesheuvel wrote: > In InitializeConsolePipe (), we clobber the Status variable in the > error handling path that reports Status in its output. Instead, > use a NULL check on the LocateProtocol () output argument. > > Contributed-under: TianoCore

Re: [edk2] [PATCH v3 4/5] ArmPlatformPkg/ArmVExpress-FVP: add support for the Intel BDS

2015-09-01 Thread Ryan Harkin
On 1 September 2015 at 10:58, Ard Biesheuvel wrote: > This adds support for the Intel BDS, by introducing a define > 'USE_ARM_BDS' which defaults to TRUE, Ooops, it's FALSE now... Sorry, that's my fault ;-) > and can be overridden on > the build command line. > >

Re: [edk2] [PATCH] ArmPlatformPkg: signal EndOfDxe event in PlatformBsdInit

2015-09-01 Thread Ard Biesheuvel
On 1 September 2015 at 17:11, Laszlo Ersek wrote: > On 09/01/15 16:26, Ard Biesheuvel wrote: >> Like the ArmVirtPkg platforms up until SVN r17713, the ArmPlatformPkg >> platforms built with the Intel BDS fail to signal the end-of-DXE event >> 'gEfiEndOfDxeEventGroupGuid' when

Re: [edk2] [PATCH v3 4/5] ArmPlatformPkg/ArmVExpress-FVP: add support for the Intel BDS

2015-09-01 Thread Ard Biesheuvel
On 1 September 2015 at 17:02, Ryan Harkin wrote: > > > On 1 September 2015 at 10:58, Ard Biesheuvel > wrote: >> >> This adds support for the Intel BDS, by introducing a define >> 'USE_ARM_BDS' which defaults to TRUE, > > > Ooops, it's FALSE

[edk2] [PATCH] ArmPlatformPkg: signal EndOfDxe event in PlatformBsdInit

2015-09-01 Thread Ard Biesheuvel
Like the ArmVirtPkg platforms up until SVN r17713, the ArmPlatformPkg platforms built with the Intel BDS fail to signal the end-of-DXE event 'gEfiEndOfDxeEventGroupGuid' when entering the BDS phase, which results in some loss of functionality, i.e., variable reclaim in the VariableDxe drivers, and