Re: [edk2] [PATCH] BaseTools: Fix GCC49 build failure

2015-08-08 Thread Ard Biesheuvel
option to satisfy this requirement and avoid a GCC44 build failure. Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Scott Duplichan sc...@notabs.org For consistency, we should probably invert the order for all versions, but it can wait until later. Reviewed-by: Ard

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

2015-08-08 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 ard.biesheu

[edk2] [PATCH 0/5] secure boot support for ARM FVP

2015-08-08 Thread Ard Biesheuvel
be disabled on the build command line to use the Intel BDS instead. Patch #5 enables the Secure Boot build of ArmVExpress-FVP, by introducing a define 'SECURE_BOOT_ENABLE' which defaults to FALSE but can be enabled on the build command line. Ard Biesheuvel (5): ArmPlatformPkg/PlatformIntelBdsLib

[edk2] [PATCH 5/5] ArmPlatformPkg/ArmVExpressPkg: enable UEFI Secure Boot

2015-08-08 Thread Ard Biesheuvel
. The FVP Fast model is recommended in this case, since the certificate store is kept in NOR flash. Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Ard Biesheuvel ard.biesheu...@linaro.org --- ArmPlatformPkg/ArmVExpressPkg/ArmVExpress-FVP-AArch64.dsc | 12

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

2015-08-08 Thread Ard Biesheuvel
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 Contribution Agreement 1.0 Signed-off-by: Ard Biesheuvel ard.biesheu...@linaro.org

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

2015-08-08 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 ard.biesheu

[edk2] [PATCH 4/5] ArmPlatformPkg/ArmVExpressPkg: add support for the Intel BDS

2015-08-08 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 ard.biesheu...@linaro.org --- ArmPlatformPkg/ArmVExpressPkg

Re: [edk2] [RFC PATCH 0/4] unify GCC command line options

2015-08-13 Thread Ard Biesheuvel
On 13 August 2015 at 08:27, Ard Biesheuvel ard.biesheu...@linaro.org wrote: On 12 August 2015 at 23:48, David Woodhouse dw...@infradead.org wrote: On Wed, 2015-08-12 at 09:08 +0200, Ard Biesheuvel wrote: Is there any reason these are kept out of sync? Are UNIXGCC and CYGGCC known to be widely

[edk2] [PATCH v2] BaseTools IA32/X64: prevent .eh_frame sections from being generated

2015-08-12 Thread Ard Biesheuvel
called .debug_frame instead of into .eh_frame. Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Ard Biesheuvel ard.biesheu...@linaro.org Reviewed-by: Laszlo Ersek ler...@redhat.com Build-tested-by: Laszlo Ersek ler...@redhat.com --- v2: give IA32 the same treatment as X86 add

Re: [edk2] [PATCH v2] BaseTools IA32/X64: prevent .eh_frame sections from being generated

2015-08-13 Thread Ard Biesheuvel
On 13 August 2015 at 07:44, Gao, Liming liming@intel.com wrote: Reviewed-by: Liming Gao liming@intel.com Thanks Committed as SVN r18217 -- Ard. -Original Message- From: Ard Biesheuvel [mailto:ard.biesheu...@linaro.org] Sent: Thursday, August 13, 2015 1:19 PM To: edk2

Re: [edk2] [PATCH v2 00/16] unify GCC command line options

2015-08-18 Thread Ard Biesheuvel
On 17 August 2015 at 21:16, David Woodhouse dw...@infradead.org wrote: See http://www.infradead.org/rpr.html X-SRS-Rewrite: SMTP reverse-path rewritten from dw...@infradead.org by twosheds.infradead.org See http://www.infradead.org/rpr.html On 2015-08-17 11:25:41, David Woodhouse wrote:

Re: [edk2] [PATCH v2 00/16] unify GCC command line options

2015-08-18 Thread Ard Biesheuvel
On 17 August 2015 at 20:53, Jordan Justen jordan.l.jus...@intel.com wrote: On 2015-08-17 11:25:56, Ard Biesheuvel wrote: On 17 August 2015 at 20:22, Jordan Justen jordan.l.jus...@intel.com wrote: Can't you use an elf-based GCC4.9 with the GCC49 toolchain instead? I'm not sure it makes

Re: [edk2] [PATCH 00/15] unify GCC command line options

2015-08-17 Thread Ard Biesheuvel
On 16 August 2015 at 23:48, Scott Duplichan sc...@notabs.org wrote: ]Ard Biesheuvel [mailto:ard.biesheu...@linaro.org] wrote: . . . ] Thanks for this much needed tool chain definition consolidation. I ran ] a build test with and without the patch. The build test uses GCC44-49

Re: [edk2] [PATCH] ArmPkg: Bug fix for UncachedMemoryAllocationLib

2015-08-17 Thread Ard Biesheuvel
leif.lindh...@linaro.org Cc: Ard Biesheuvel ard.biesheu...@linaro.org --- .../UncachedMemoryAllocationLib/UncachedMemoryAllocationLib.c | 7 --- 1 file changed, 4 insertions(+), 3 deletions(-) Reviewed-by: Ard Biesheuvel ard.biesheu...@linaro.org diff --git a/ArmPkg/Library

[edk2] [PATCH v2 07/16] BaseTools GCC: use leading underscore for symbol names where appropriate

2015-08-17 Thread Ard Biesheuvel
-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Ard Biesheuvel ard.biesheu...@linaro.org --- BaseTools/Conf/tools_def.template | 22 +++- 1 file changed, 12 insertions(+), 10 deletions(-) diff --git a/BaseTools/Conf/tools_def.template b/BaseTools/Conf/tools_def.template

[edk2] [PATCH v2 08/16] BaseTools GCC: refactor tools_def internal GCC defines for [AS]DLINK

2015-08-17 Thread Ard Biesheuvel
Contribution Agreement 1.0 Signed-off-by: Ard Biesheuvel ard.biesheu...@linaro.org --- BaseTools/Conf/tools_def.template | 30 +++- 1 file changed, 16 insertions(+), 14 deletions(-) diff --git a/BaseTools/Conf/tools_def.template b/BaseTools/Conf/tools_def.template index 1304f514cd37

[edk2] [PATCH v2 06/16] BaseTools GCC: unify all IA32 and X64 CC flags for ELF based GCC

2015-08-17 Thread Ard Biesheuvel
GCC4x and ELFGCC are both ELF based GCC toolchains, so there is no justification for allowing the command line option to deviate. So align them. Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Ard Biesheuvel ard.biesheu...@linaro.org --- BaseTools/Conf/tools_def.template

[edk2] [PATCH v2 04/16] BaseTools GCC: merge warning flags for all GCC versions

2015-08-17 Thread Ard Biesheuvel
The warning flags -Wno-address -Wno-unused-but-set-variable are added for version 4.6 and up, but since they are happily accepted by version 4.4 and 4.5, add them there as well. Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Ard Biesheuvel ard.biesheu...@linaro.org

[edk2] [PATCH v2 14/16] BaseTools GCC: unify ASM flags for all GCC versions

2015-08-17 Thread Ard Biesheuvel
Use the same GCC options for assembling regardless of the exact GCC version. Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Ard Biesheuvel ard.biesheu...@linaro.org --- BaseTools/Conf/tools_def.template | 67 1 file changed, 27 insertions(+), 40

[edk2] [PATCH v2 15/16] BaseTools GCC: align ELFGCC with GCC4x toolchains

2015-08-17 Thread Ard Biesheuvel
This aligns the remaining configuration options for ELFGCC with the other ELF based toolchains. Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Ard Biesheuvel ard.biesheu...@linaro.org --- BaseTools/Conf/tools_def.template | 18 +- 1 file changed, 9

[edk2] [PATCH v2 16/16] OvmfPkg/X64: enable 4 KB alignment for DXE_RUNTIME modules

2015-08-17 Thread Ard Biesheuvel
This enables 4 KB section alignment for DXE_RUNTIME modules, for ELF based toolchains and for the UNIXGCC PE/COFF toolchain. Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Ard Biesheuvel ard.biesheu...@linaro.org --- OvmfPkg/OvmfPkgX64.dsc | 7 +++ 1 file changed, 7

[edk2] [PATCH v2 11/16] BaseTools GCC: unify all IA32 and X64 [AS]DLINK[2] flags for ELF based GCC

2015-08-17 Thread Ard Biesheuvel
GCC4x and ELFGCC are both ELF based GCC toolchains, so there is no justification for allowing the command line options to deviate. So align them. Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Ard Biesheuvel ard.biesheu...@linaro.org --- BaseTools/Conf/tools_def.template

[edk2] [PATCH v2 02/16] BaseTools/GenFv: use PE/COFF virtual section size if raw size is larger

2015-08-17 Thread Ard Biesheuvel
of the two. Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Ard Biesheuvel ard.biesheu...@linaro.org --- BaseTools/Source/C/GenFv/GenFvInternalLib.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/BaseTools/Source/C/GenFv/GenFvInternalLib.c b/BaseTools/Source/C

[edk2] [PATCH v2 10/16] BaseTools GCC: remove GCC 4.9 specific linker alignment override

2015-08-17 Thread Ard Biesheuvel
a need for special treatment of GCC 4.9, and the linker command line override can be removed. Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Ard Biesheuvel ard.biesheu...@linaro.org --- BaseTools/Conf/tools_def.template | 8 1 file changed, 4 insertions(+), 4

[edk2] [PATCH v2 12/16] BaseTools GCC: unify ARM and AARCH64 GCC compiler flags

2015-08-17 Thread Ard Biesheuvel
1.0 Signed-off-by: Ard Biesheuvel ard.biesheu...@linaro.org --- BaseTools/Conf/tools_def.template | 39 1 file changed, 16 insertions(+), 23 deletions(-) diff --git a/BaseTools/Conf/tools_def.template b/BaseTools/Conf/tools_def.template index b92bd4c16a0e..8161660ca839

[edk2] [PATCH v2 05/16] BaseTools GCC: refactor tools_def internal GCC defines for CC flags

2015-08-17 Thread Ard Biesheuvel
not modify the options that are ultimately passed to GCC, although it does deduplicate ARM and AARCH64 since those toolchains were passing both GCC_ALL_CC_FLAGS and GCC44_ALL_CC_FLAGS, which overlap to a great extent. Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Ard Biesheuvel

[edk2] [PATCH v2 09/16] BaseTools GCC: don't set .data address explicitly

2015-08-17 Thread Ard Biesheuvel
, but advance the location pointer by the same amount. This way, the linker will be free to choose the start address, but will ultimately end up using the exact same value. Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Ard Biesheuvel ard.biesheu...@linaro.org --- BaseTools/Scripts

[edk2] [PATCH v2 13/16] BaseTools GCC: unify ARM and AARCH64 DLINK flags for all GCC versions

2015-08-17 Thread Ard Biesheuvel
Use the same GCC options for linking regardless of the exact GCC version. Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Ard Biesheuvel ard.biesheu...@linaro.org --- BaseTools/Conf/tools_def.template | 63 +++- 1 file changed, 23 insertions(+), 40

[edk2] [PATCH v2 00/16] unify GCC command line options

2015-08-17 Thread Ard Biesheuvel
Branch can be found here https://git.linaro.org/people/ard.biesheuvel/uefi-next.git/shortlog/refs/heads/unify-gcc-v2 Ard Biesheuvel (15): BaseTools/GenFv: use PE/COFF virtual section size if raw size is larger BaseTools GCC: remove unused definition of GCC_WINDRES_FLAGS BaseTools GCC: merge

[edk2] [PATCH v2 03/16] BaseTools GCC: remove unused definition of GCC_WINDRES_FLAGS

2015-08-17 Thread Ard Biesheuvel
The definition of GCC_WINDRES_FLAGS is never referenced again so remove it. Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Ard Biesheuvel ard.biesheu...@linaro.org --- BaseTools/Conf/tools_def.template | 1 - 1 file changed, 1 deletion(-) diff --git a/BaseTools/Conf

Re: [edk2] [PATCH v2 16/16] OvmfPkg/X64: enable 4 KB alignment for DXE_RUNTIME modules

2015-08-17 Thread Ard Biesheuvel
[mailto:edk2-devel-boun...@lists.01.org] On Behalf Of Ard Biesheuvel Sent: Monday, August 17, 2015 10:25 PM To: edk2-devel@lists.01.org; Liu, Yingke D Cc: wp...@windriver.com; sc...@notabs.org; Ard Biesheuvel; Justen, Jordan L; Gao, Liming; dw...@infradead.org Subject: [edk2] [PATCH v2 16/16

Re: [edk2] [PATCH 13/15] ArmVirtPkg: Link separated VarCheckUefiLib NULL class library instance

2015-08-18 Thread Ard Biesheuvel
On 17 August 2015 at 10:24, Star Zeng star.z...@intel.com wrote: Cc: Laszlo Ersek ler...@redhat.com Cc: Ard Biesheuvel ard.biesheu...@linaro.org Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Star Zeng star.z...@intel.com Reviewed-by: Ard Biesheuvel ard.biesheu

Re: [edk2] [PATCH 14/15] ArmPlatformPkg: Link separated VarCheckUefiLib NULL class library instance

2015-08-18 Thread Ard Biesheuvel
On 17 August 2015 at 10:24, Star Zeng star.z...@intel.com wrote: Cc: Leif Lindholm leif.lindh...@linaro.org Cc: Ard Biesheuvel ard.biesheu...@linaro.org Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Star Zeng star.z...@intel.com Reviewed-by: Ard Biesheuvel ard.biesheu

Re: [edk2] [PATCH 3/3] ArmPlatformPkg: PL061: support multiple controller

2015-08-18 Thread Ard Biesheuvel
On 18 August 2015 at 14:30, Haojian Zhuang haojian.zhu...@linaro.org wrote: On Tue, 2015-08-18 at 11:43 +0200, Ard Biesheuvel wrote: On 18 August 2015 at 11:31, Haojian Zhuang haojian.zhu...@linaro.org wrote: On Tue, 2015-08-18 at 11:22 +0200, Ard Biesheuvel wrote: On 18 August 2015 at 11:04

Re: [edk2] [PATCH 07/15] ArmPlatformPkg: Add VarCheckLib library mapping

2015-08-18 Thread Ard Biesheuvel
On 17 August 2015 at 10:24, Star Zeng star.z...@intel.com wrote: Since Variable driver has been updated to consume the separated VarCheckLib. Cc: Leif Lindholm leif.lindh...@linaro.org Cc: Ard Biesheuvel ard.biesheu...@linaro.org Contributed-under: TianoCore Contribution Agreement 1.0 Signed

Re: [edk2] [PATCH 06/15] ArmVirtPkg: Add VarCheckLib library mapping

2015-08-18 Thread Ard Biesheuvel
On 17 August 2015 at 10:24, Star Zeng star.z...@intel.com wrote: Since Variable driver has been updated to consume the separated VarCheckLib. Cc: Laszlo Ersek ler...@redhat.com Cc: Ard Biesheuvel ard.biesheu...@linaro.org Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off

Re: [edk2] [PATCH 0/5] secure boot support for ARM FVP

2015-08-18 Thread Ard Biesheuvel
On 8 August 2015 at 14:00, Ard Biesheuvel ard.biesheu...@linaro.org wrote: This series adds support for using the Intel BDS with ArmVExpress-FVP, and for building it with UEFI Secure Boot enabled. Note that the former is a prerequisite of the latter, since the ARM BDS has no GUI for enrolling

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

2015-08-18 Thread Ard Biesheuvel
On 18 August 2015 at 14:12, Leif Lindholm leif.lindh...@linaro.org wrote: On Tue, Aug 18, 2015 at 01:15:54PM +0200, Ard Biesheuvel wrote: On 12 August 2015 at 08:45, Ard Biesheuvel ard.biesheu...@linaro.org wrote: Instead of omitting some drivers that are known to break the Foundation model

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

2015-08-18 Thread Ard Biesheuvel
On 12 August 2015 at 08:45, Ard Biesheuvel ard.biesheu...@linaro.org wrote: Instead of omitting some drivers that are known to break the Foundation model when ARM_FOUNDATION_FVP is defined, fix those drivers so that they simply fail to load without interfering with the boot. This way, we can

Re: [edk2] [PATCH 0/2] AARCH64 tiny code model support

2015-08-18 Thread Ard Biesheuvel
On 10 August 2015 at 12:27, Ard Biesheuvel ard.biesheu...@linaro.org wrote: The AARCH64 GCC builds use the GCC large code model by default, simply because it is the code model that requires the least amount of hacking to produce code that supports the PE/COFF conversion applied by Tianocore

Re: [edk2] [PATCH v2 00/16] unify GCC command line options

2015-08-17 Thread Ard Biesheuvel
On 17 August 2015 at 19:53, Jordan Justen jordan.l.jus...@intel.com wrote: On 2015-08-17 07:24:57, Ard Biesheuvel wrote: This got a bit out of hand after I noticed the ELFGCC and UNIXGCC toolchains that needed some tlc as well. Anyway, this series aims to refactor the toolchains definitions

[edk2] [PATCH] ShellBinPkg: Arm/AArch64 Shell binary update.

2015-08-19 Thread Ard Biesheuvel
The binaries of ShellBinPkg are generated with ShellPkg project 18222. Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Ard Biesheuvel ard.biesheu...@linaro.org --- These are built with the tiny code model, note the code size reduction for AARCH64. Branch: https

Re: [edk2] [PATCH v2 00/16] unify GCC command line options

2015-08-19 Thread Ard Biesheuvel
On 19 August 2015 at 09:53, Ard Biesheuvel ard.biesheu...@linaro.org wrote: On 18 August 2015 at 22:29, Ard Biesheuvel ard.biesheu...@linaro.org wrote: On 18 August 2015 at 22:03, Ard Biesheuvel ard.biesheu...@linaro.org wrote: On 18 August 2015 at 19:35, David Woodhouse dw...@infradead.org

Re: [edk2] [PATCH v2 00/16] unify GCC command line options

2015-08-18 Thread Ard Biesheuvel
On 18 August 2015 at 19:35, David Woodhouse dw...@infradead.org wrote: On Tue, 2015-08-18 at 17:52 +0200, Ard Biesheuvel wrote: On 18 August 2015 at 17:19, Jordan Justen jordan.l.jus...@intel.com wrote: Last time I checked, GCC44 ~ GCC49 all produced images roughly in the same ball park size

Re: [edk2] [PATCH v2] ArmPkg: remove ARMv6 support code

2015-08-19 Thread Ard Biesheuvel
On 19 August 2015 at 12:47, Leif Lindholm leif.lindh...@linaro.org wrote: On Wed, Aug 19, 2015 at 11:51:46AM +0200, Ard Biesheuvel wrote: No platforms use the ARMv6 (ARM11) support code anymore. In fact, the only reference to it in ArmPkg.dsc was commented out by Andrew in SVN r11298 (2011-02

Re: [edk2] [PATCH 3/3] ArmPlatformPkg: PL061: support multiple controller

2015-08-18 Thread Ard Biesheuvel
On 18 August 2015 at 11:04, Haojian Zhuang haojian.zhu...@linaro.org wrote: Support multiple PL061 controllers. Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Haojian Zhuang haojian.zhu...@linaro.org --- ArmPlatformPkg/Drivers/PL061GpioDxe/PL061Gpio.c| 107

Re: [edk2] [PATCH 3/3] ArmPlatformPkg: PL061: support multiple controller

2015-08-18 Thread Ard Biesheuvel
On 18 August 2015 at 11:31, Haojian Zhuang haojian.zhu...@linaro.org wrote: On Tue, 2015-08-18 at 11:22 +0200, Ard Biesheuvel wrote: On 18 August 2015 at 11:04, Haojian Zhuang haojian.zhu...@linaro.org wrote: Support multiple PL061 controllers. Contributed-under: TianoCore Contribution

Re: [edk2] [PATCH v2 00/16] unify GCC command line options

2015-08-24 Thread Ard Biesheuvel
On 24 August 2015 at 19:20, Bill Paul wp...@windriver.com wrote: Of all the gin joints in all the towns in all the world, Ard Biesheuvel had to walk into mine at 10:06:10 on Monday 24 August 2015 and say: On 24 August 2015 at 19:02, Bill Paul wp...@windriver.com wrote: Of all the gin joints

Re: [edk2] [PATCH v2 00/16] unify GCC command line options

2015-08-24 Thread Ard Biesheuvel
On 19 August 2015 at 00:27, Laszlo Ersek ler...@redhat.com wrote: On 08/18/15 22:04, Paolo Bonzini wrote: On 18/08/2015 08:52, Ard Biesheuvel wrote: Personally, I would not mind deprecating GCC44, but the biggest question I would have is what toolchains do the latest UDK releases claim

Re: [edk2] [PATCH v2 1/2] ShellPkg: force use of AARCH64 small model when building DEBUG shell

2015-08-20 Thread Ard Biesheuvel
On 19 August 2015 at 19:03, Carsey, Jaben jaben.car...@intel.com wrote: Reviewed-by: Jaben Carsey jaben.car...@intel.com Thanks. Series committed as SVN r18241 .. r18242 -Original Message- From: edk2-devel [mailto:edk2-devel-boun...@lists.01.org] On Behalf Of Ard Biesheuvel Sent

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

2015-08-18 Thread Ard Biesheuvel
the PrimeCell ID registers. This is currently not required, since the Foundation model deals gracefully with the reads to the unpopulated range, but this may change in future versions - Added Leif's R-b - Rebased onto latest upstream Ard Biesheuvel (3): ArmPlatformPkg/PL180MciDxe: check PrimeCell ID

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

2015-08-18 Thread Ard Biesheuvel
On 18 August 2015 at 15:50, Leif Lindholm leif.lindh...@linaro.org wrote: On Tue, Aug 18, 2015 at 02:14:53PM +0200, Ard Biesheuvel wrote: On 18 August 2015 at 14:12, Leif Lindholm leif.lindh...@linaro.org wrote: On Tue, Aug 18, 2015 at 01:15:54PM +0200, Ard Biesheuvel wrote: On 12 August

[edk2] [PATCH v2 1/3] ArmPlatformPkg/PL180MciDxe: check PrimeCell ID before initializing

2015-08-18 Thread Ard Biesheuvel
To deal gracefully with the absence of the PL180 hardware on the Foundation model, check the PrimeCell ID before proceeding with the installation. Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Ard Biesheuvel ard.biesheu...@linaro.org Reviewed-by: Leif Lindholm leif.lindh

[edk2] [PATCH v2 3/3] ArmPlatformPkg/FVP: unify support for Foundation and Base models

2015-08-18 Thread Ard Biesheuvel
Contribution Agreement 1.0 Signed-off-by: Ard Biesheuvel ard.biesheu...@linaro.org Reviewed-by: Leif Lindholm leif.lindh...@linaro.org --- ArmPlatformPkg/ArmVExpressPkg/ArmVExpress-FVP-AArch64.dsc | 13 - ArmPlatformPkg/ArmVExpressPkg/ArmVExpress-FVP-AArch64.fdf | 4 2 files changed, 17

[edk2] [PATCH v2 2/3] ArmPlatformPkg/LcdGraphicsOutputDxe: check PrimeCell ID before initializing

2015-08-18 Thread Ard Biesheuvel
To deal gracefully with the absence of the PL111 hardware on the Foundation model, check the PrimeCell ID before proceeding with the installation. Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Ard Biesheuvel ard.biesheu...@linaro.org Reviewed-by: Leif Lindholm leif.lindh

[edk2] [PATCH v2 1/2] ShellPkg: force use of AARCH64 small model when building DEBUG shell

2015-08-19 Thread Ard Biesheuvel
-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Ard Biesheuvel ard.biesheu...@linaro.org Reviewed-by: Leif Lindholm leif.lindh...@linaro.org Tested-by: Leif Lindholm leif.lindh...@linaro.org --- ShellPkg/Application/Shell/Shell.inf | 6 ++ 1 file changed, 6 insertions(+) diff --git

Re: [edk2] [PATCH v2 00/16] unify GCC command line options

2015-08-18 Thread Ard Biesheuvel
On 18 August 2015 at 22:03, Ard Biesheuvel ard.biesheu...@linaro.org wrote: On 18 August 2015 at 19:35, David Woodhouse dw...@infradead.org wrote: On Tue, 2015-08-18 at 17:52 +0200, Ard Biesheuvel wrote: On 18 August 2015 at 17:19, Jordan Justen jordan.l.jus...@intel.com wrote: Last time I

Re: [edk2] [PATCH v2 00/16] unify GCC command line options

2015-08-18 Thread Ard Biesheuvel
On 18 August 2015 at 17:19, Jordan Justen jordan.l.jus...@intel.com wrote: On 2015-08-18 03:57:51, Ard Biesheuvel wrote: On 17 August 2015 at 20:53, Jordan Justen jordan.l.jus...@intel.com wrote: On 2015-08-17 11:25:56, Ard Biesheuvel wrote: MinGW generates PE/COFF not ELF, so much

Re: [edk2] [PATCH v2 00/16] unify GCC command line options

2015-08-19 Thread Ard Biesheuvel
On 18 August 2015 at 22:29, Ard Biesheuvel ard.biesheu...@linaro.org wrote: On 18 August 2015 at 22:03, Ard Biesheuvel ard.biesheu...@linaro.org wrote: On 18 August 2015 at 19:35, David Woodhouse dw...@infradead.org wrote: On Tue, 2015-08-18 at 17:52 +0200, Ard Biesheuvel wrote: On 18 August

Re: [edk2] [PATCH v2 00/16] unify GCC command line options

2015-08-17 Thread Ard Biesheuvel
On 17 August 2015 at 20:37, Scott Duplichan sc...@notabs.org wrote: Ard Biesheuvel [mailto:ard.biesheu...@linaro.org] wrote: ]Sent: Monday, August 17, 2015 09:25 AM ]To: edk2-de...@ml01.01.org; yingke.d@intel.com ]Cc: wp...@windriver.com; sc...@notabs.org; Ard Biesheuvel ard.biesheu

Re: [edk2] [PATCH v2 00/16] unify GCC command line options

2015-08-17 Thread Ard Biesheuvel
On 17 August 2015 at 20:22, Jordan Justen jordan.l.jus...@intel.com wrote: On 2015-08-17 11:10:57, Bill Paul wrote: Of all the gin joints in all the towns in all the world, David Woodhouse had to walk into mine at 11:00:23 on Monday 17 August 2015 and say: On Mon, 2015-08-17 at 10:53 -0700,

Re: [edk2] [PATCH 00/15] unify GCC command line options

2015-08-16 Thread Ard Biesheuvel
On 15 August 2015 at 22:43, Scott Duplichan sc...@notabs.org wrote: Ard Biesheuvel [mailto:ard.biesheu...@linaro.org] wrote: ]Sent: Friday, August 14, 2015 02:15 PM ]To: edk2-de...@ml01.01.org; yingke.d@intel.com ]Cc: wp...@windriver.com; sc...@notabs.org; Ard Biesheuvel ard.biesheu

[edk2] [PATCH] ArmPkg: remove ARMv6 support code

2015-08-17 Thread Ard Biesheuvel
No platforms use the ARMv6 (ARM11) support code anymore. In fact, the only reference to it in ArmPkg.dsc was commented out by Andrew in SVN r11298 (2011-02-03) so it may well be broken. So remove it. Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Ard Biesheuvel ard.biesheu

[edk2] [PATCH 2/2] BaseTools GCC: update mingw-gcc-build.py to GCC 4.9.3

2015-08-17 Thread Ard Biesheuvel
This updates mingw-gcc-build.py to the newest version of GCC currently supported by the EDK2 build system, which is 4.9.3. At the same time, binutils is updated to version 2.24.51.0.2. Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Ard Biesheuvel ard.biesheu...@linaro.org

Re: [edk2] [PATCH] ArmPkg/CpuDxe: Disable interrupt before restoring context

2015-08-24 Thread Ard Biesheuvel
On 23 August 2015 at 17:59, Ard Biesheuvel ard.biesheu...@linaro.org wrote: On 23 August 2015 at 15:39, Heyi Guo heyi@linaro.org wrote: On 08/17/2015 05:52 PM, Ard Biesheuvel wrote: On 13 August 2015 at 05:10, Heyi Guoheyi@linaro.org wrote: Interrupt must be disabled before we

Re: [edk2] Some pages on new TianoCore wiki not being indexed (no results on Google)?

2015-08-24 Thread Ard Biesheuvel
(+ Leif) On 24 August 2015 at 06:27, Bruce Cran br...@cran.org.uk wrote: It seems indexing of the new wiki at http://tianocore.org is rather poor: I was looking for some information about the BeagleBoard/BeagleBone, Those instructions are slightly outdated after some recent cleanup work (I was

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

2015-08-24 Thread Ard Biesheuvel
On 18 August 2015 at 16:10, Ard Biesheuvel ard.biesheu...@linaro.org wrote: Instead of omitting some drivers that are known to break the Foundation model when ARM_FOUNDATION_FVP is defined, fix those drivers so that they simply fail to load without interfering with the boot. Changes since v1

[edk2] [PATCH v3 06/12] BaseTools/GCC: align start of .data to .text alignment

2015-07-29 Thread Ard Biesheuvel
layout of the PE/COFF binary match the layout of the ELF version more closely. Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Ard Biesheuvel ard.biesheu...@linaro.org --- BaseTools/Scripts/GccBase.lds | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git

[edk2] [PATCH v3 07/12] BaseTools/GCC: move AutoGen.obj contents to .text section

2015-07-29 Thread Ard Biesheuvel
, which reduces wasted FV space if the PE/COFF section alignment is large. Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Ard Biesheuvel ard.biesheu...@linaro.org --- BaseTools/Scripts/GccBase.lds | 7 +++ 1 file changed, 7 insertions(+) diff --git a/BaseTools/Scripts

[edk2] [PATCH v3 10/12] ArmVirtPkg: move to unified GCC linker script

2015-07-29 Thread Ard Biesheuvel
Move to the parametrised generic GCC linker script and set 64 KB alignment, instead of using the AARCH64 specific incremental linker script for 64 KB alignment which is about to be removed. Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Ard Biesheuvel ard.biesheu

[edk2] [PATCH v3 01/12] BaseTools/GCC: remove NOP padding from X86/IA32 GCC linker scripts

2015-07-29 Thread Ard Biesheuvel
NOP slides in memory only makes it easier for attackers to launch exploits. So remove them. Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Ard Biesheuvel ard.biesheu...@linaro.org --- BaseTools/Scripts/gcc4.4-ld-script | 2 +- BaseTools/Scripts/gcc4.9-ld-script | 2 +- 2

[edk2] [PATCH v3 09/12] ArmPlatformPkg/ArmVExpressPkg: move to unified GCC linker script

2015-07-29 Thread Ard Biesheuvel
Move to the parametrised generic GCC linker script and set 64 KB alignment, instead of using the AARCH64 specific incremental linker script for 64 KB alignment which is about to be removed. Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Ard Biesheuvel ard.biesheu

[edk2] [PATCH v3 02/12] BaseTools/GCC: move .rodata to PE/COFF .text section

2015-07-29 Thread Ard Biesheuvel
- .data section. So move it there. Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Ard Biesheuvel ard.biesheu...@linaro.org --- BaseTools/Scripts/gcc-4K-align-ld-script | 2 +- BaseTools/Scripts/gcc4.4-ld-script | 2 +- BaseTools/Scripts/gcc4.9-ld-script | 2 +- 3

[edk2] [PATCH v3 03/12] BaseTools/GCC: drop redundant alignment from linker script

2015-07-29 Thread Ard Biesheuvel
There is no need to pad out the end of a section of the start of the following section is aligned to the same value. So drop the redundant ALIGN() statements. Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Ard Biesheuvel ard.biesheu...@linaro.org --- BaseTools/Scripts/gcc

[edk2] [PATCH v3 00/12] BaseTools: unify all GCC linker scripts

2015-07-29 Thread Ard Biesheuvel
-scripts-v3 Ard Biesheuvel (12): BaseTools/GCC: remove NOP padding from X86/IA32 GCC linker scripts BaseTools/GCC: move .rodata to PE/COFF .text section BaseTools/GCC: drop redundant alignment from linker script BaseTools/GCC: move .got contents to the PE/COFF .text section BaseTools/GCC

[edk2] [PATCH v3 11/12] BaseTools/AARCH64: remove incremental linker script for 64K alignment

2015-07-29 Thread Ard Biesheuvel
Now that we moved all users to the unified GCC linker script, remove the old 64 KB incremental linker script for AARCH64 since it is now unused. Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Ard Biesheuvel ard.biesheu...@linaro.org --- BaseTools/Scripts/gcc-aarch64-64K

[edk2] [PATCH v3 04/12] BaseTools/GCC: move .got contents to the PE/COFF .text section

2015-07-29 Thread Ard Biesheuvel
from being overwritten inadvertently. Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Ard Biesheuvel ard.biesheu...@linaro.org --- BaseTools/Scripts/gcc-4K-align-ld-script | 5 + BaseTools/Scripts/gcc4.4-ld-script | 5 + BaseTools/Scripts/gcc4.9-ld-script

[edk2] [PATCH v2] ArmPlatformPkg: remove obsolete ARM and AARCH64 platforms

2015-07-29 Thread Ard Biesheuvel
-RTSM-A15_MPCore) Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Ard Biesheuvel ard.biesheu...@linaro.org --- Branch is here: https://git.linaro.org/people/ard.biesheuvel/uefi-next.git/shortlog/refs/heads/remove-obsolete-platforms ArmPlatformPkg/ArmRealViewEbPkg

Re: [edk2] [PATCH] ArmPkg: Move FDT offset higher in RAM

2015-07-28 Thread Ard Biesheuvel
On 28 July 2015 at 11:41, Ryan Harkin ryan.har...@linaro.org wrote: On 28 July 2015 at 10:26, Ard Biesheuvel ard.biesheu...@linaro.org wrote: On 28 July 2015 at 11:01, Ryan Harkin ryan.har...@linaro.org wrote: [+ Tixy as he's interested in making sure UEFI follows the Linux requirements

Re: [edk2] [PATCH] ArmPkg: Move FDT offset higher in RAM

2015-07-28 Thread Ard Biesheuvel
On 28 July 2015 at 11:01, Ryan Harkin ryan.har...@linaro.org wrote: [+ Tixy as he's interested in making sure UEFI follows the Linux requirements] On 28 July 2015 at 07:39, Ard Biesheuvel ard.biesheu...@linaro.org wrote: On 27 July 2015 at 22:42, Ryan Harkin ryan.har...@linaro.org wrote

[edk2] [PATCH v4 03/13] BaseTools IA32/X64: drop redundant alignment from linker script

2015-07-30 Thread Ard Biesheuvel
There is no need to pad out the end of a section of the start of the following section is aligned to the same value. So drop the redundant ALIGN() statements. Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Ard Biesheuvel ard.biesheu...@linaro.org Reviewed-by: Jordan Justen

[edk2] [PATCH v4 00/13] BaseTools: unify all GCC linker scripts

2015-07-30 Thread Ard Biesheuvel
(COMMONPAGESIZE) in linker scripts, and is otherwise unused internally by the linker. Tested with ArmVirtQemu/AARCH64 and Ovmf/X64 Branch is here https://git.linaro.org/people/ard.biesheuvel/uefi-next.git/shortlog/refs/heads/unify-gcc-ld-scripts-v4 (now correctly based on the GitHub repo) Ard

[edk2] [PATCH v4 07/13] BaseTools GCC: align start of .data to .text alignment

2015-07-30 Thread Ard Biesheuvel
layout of the PE/COFF binary match the layout of the ELF version more closely. Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Ard Biesheuvel ard.biesheu...@linaro.org Reviewed-by: Jordan Justen jordan.l.jus...@intel.com Tested-by: Liming Gao liming@intel.com Tested

[edk2] [PATCH v4 10/13] ArmPlatformPkg/ArmVExpressPkg: move to unified GCC linker script

2015-07-30 Thread Ard Biesheuvel
Move to the parametrised generic GCC linker script and set 64 KB alignment, instead of using the AARCH64 specific incremental linker script for 64 KB alignment which is about to be removed. Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Ard Biesheuvel ard.biesheu

Re: [edk2] [PATCH v3 12/12] BaseTools/X86|IA32: move to unified GCC linker script

2015-07-30 Thread Ard Biesheuvel
Biesheuvel; edk2-devel@lists.01.org; Liu, Yingke D; Gao, Liming Cc: ler...@redhat.com; leif.lindh...@linaro.org; Ard Biesheuvel Subject: Re: [PATCH v3 12/12] BaseTools/X86|IA32: move to unified GCC linker script Subject prefix: BaseTools/X86|IA32 = BaseTools IA32/X64 What about 1 more step

[edk2] [PATCH v4 09/13] BaseTools AARCH64: move to unified GCC linker script

2015-07-30 Thread Ard Biesheuvel
Drop the GCC AARCH64 specific linker script and use the new unified one instead. Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Ard Biesheuvel ard.biesheu...@linaro.org Reviewed-by: Jordan Justen jordan.l.jus...@intel.com Reviewed-by: Leif Lindholm leif.lindh...@linaro.org

[edk2] [PATCH v4 08/13] BaseTools GCC: move AutoGen.obj contents to .text section

2015-07-30 Thread Ard Biesheuvel
, which reduces wasted FV space if the PE/COFF section alignment is large. Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Ard Biesheuvel ard.biesheu...@linaro.org Reviewed-by: Jordan Justen jordan.l.jus...@intel.com Tested-by: Liming Gao liming@intel.com Tested-by: Leif

[edk2] [PATCH v4 12/13] BaseTools AARCH64: remove incremental linker script for 64K alignment

2015-07-30 Thread Ard Biesheuvel
Now that we moved all users to the unified GCC linker script, remove the old 64 KB incremental linker script for AARCH64 since it is now unused. Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Ard Biesheuvel ard.biesheu...@linaro.org Reviewed-by: Jordan Justen jordan.l.jus

[edk2] [PATCH v4 01/13] BaseTools IA32/X64: remove NOP padding from X86/IA32 GCC linker scripts

2015-07-30 Thread Ard Biesheuvel
NOP slides in memory only makes it easier for attackers to launch exploits. So remove them. Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Ard Biesheuvel ard.biesheu...@linaro.org Reviewed-by: Jordan Justen jordan.l.jus...@intel.com Tested-by: Liming Gao liming@intel.com

[edk2] [PATCH v4 02/13] BaseTools IA32/X64: move .rodata to PE/COFF .text section

2015-07-30 Thread Ard Biesheuvel
- .data section. So move it there. Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Ard Biesheuvel ard.biesheu...@linaro.org Reviewed-by: Jordan Justen jordan.l.jus...@intel.com Tested-by: Liming Gao liming@intel.com --- BaseTools/Scripts/gcc-4K-align-ld-script | 2

[edk2] [PATCH v4 06/13] BaseTools GCC: add unified GCC linker script for all archs and versions

2015-07-30 Thread Ard Biesheuvel
-off-by: Ard Biesheuvel ard.biesheu...@linaro.org Reviewed-by: Jordan Justen jordan.l.jus...@intel.com Tested-by: Liming Gao liming@intel.com Tested-by: Leif Lindholm leif.lindh...@linaro.org --- BaseTools/Scripts/GccBase.lds | 63 1 file changed, 63 insertions(+) diff

[edk2] [PATCH v4 11/13] ArmVirtPkg: move to unified GCC linker script

2015-07-30 Thread Ard Biesheuvel
Move to the parametrised generic GCC linker script and set 64 KB alignment, instead of using the AARCH64 specific incremental linker script for 64 KB alignment which is about to be removed. Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Ard Biesheuvel ard.biesheu

Re: [edk2] [PATCH 1/2] ArmPkg/ArmGicLibArchLib: common check for GICv3

2015-07-31 Thread Ard Biesheuvel
On 31 July 2015 at 14:21, Ryan Harkin ryan.har...@linaro.org wrote: On 31 July 2015 at 13:13, Ard Biesheuvel ard.biesheu...@linaro.org wrote: On 31 July 2015 at 14:06, Ryan Harkin ryan.har...@linaro.org wrote: Make Arm and Aarch64 both use the same code, conditionally compiled, to check

[edk2] [PATCH] ArmVirtPkg/ArmVirtQemu: drop ARM BDS and make Intel BDS the default

2015-08-04 Thread Ard Biesheuvel
of integration with the QEMU command line that the Intel BDS has when running under ArmVirtPkg or OvmfPkg. Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Ard Biesheuvel ard.biesheu...@linaro.org --- Note that this supersedes 'ArmVirtPkg: align ARM BDS build with LinuxLoader

Re: [edk2] [PATCH] ArmVirtPkg: align ARM BDS build with LinuxLoader changes

2015-08-04 Thread Ard Biesheuvel
On 4 August 2015 at 11:21, Sharma Bhupesh bhupesh.sha...@freescale.com wrote: -Original Message- From: Laszlo Ersek [mailto:ler...@redhat.com] Sent: Tuesday, August 04, 2015 2:36 PM On 08/04/15 10:38, Sharma Bhupesh wrote: -Original Message- From: Ard Biesheuvel

Re: [edk2] [PATCH] ArmVirtPkg: align ARM BDS build with LinuxLoader changes

2015-08-04 Thread Ard Biesheuvel
On 4 August 2015 at 10:54, Laszlo Ersek ler...@redhat.com wrote: On 08/04/15 09:57, Ard Biesheuvel wrote: LinuxLoader has been split off from the ARM BDS into a separate EFI application. Because we never included this application into the ArmVirtPkg platforms, its ARM BDS builds have

Re: [edk2] [PATCH] ArmVirtPkg: align ARM BDS build with LinuxLoader changes

2015-08-04 Thread Ard Biesheuvel
Message- From: edk2-devel [mailto:edk2-devel-boun...@lists.01.org] On Behalf Of Ard Biesheuvel Sent: Tuesday, August 04, 2015 1:27 PM To: edk2-devel@lists.01.org; leif.lindh...@linaro.org; ler...@redhat.com Cc: ryan.har...@linaro.org; Ard Biesheuvel Subject: [edk2] [PATCH] ArmVirtPkg: align ARM

[edk2] [PATCH] ArmVirtPkg/ArmVirtQemu: add LinuxLoader UEFI app to ARM build

2015-08-04 Thread Ard Biesheuvel
:zImage -c console=ttyAMA0 Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Ard Biesheuvel ard.biesheu...@linaro.org --- ArmVirtPkg/ArmVirt.dsc.inc | 6 +++--- ArmVirtPkg/ArmVirtQemu.dsc | 9 + ArmVirtPkg/ArmVirtQemu.fdf | 5 + 3 files changed, 17 insertions(+), 3

Re: [edk2] [PATCH] ArmVirtPkg/ArmVirtQemu: drop ARM BDS and make Intel BDS the default

2015-08-04 Thread Ard Biesheuvel
On 4 August 2015 at 16:50, Laszlo Ersek ler...@redhat.com wrote: On 08/04/15 16:34, Ard Biesheuvel wrote: On 4 August 2015 at 15:35, Laszlo Ersek ler...@redhat.com wrote: [...] I propose the following: - The Xen hit should be removed as a separate patch, as it has never been useful

[edk2] [PATCH 3/3] ArmVirtPkg/ArmVirtQemu: add LinuxLoader UEFI app to ARM build

2015-08-04 Thread Ard Biesheuvel
:zImage -c console=ttyAMA0 Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Ard Biesheuvel ard.biesheu...@linaro.org Reviewed-by: Laszlo Ersek ler...@redhat.com --- ArmVirtPkg/ArmVirt.dsc.inc | 6 +++--- ArmVirtPkg/ArmVirtQemu.dsc | 9 + ArmVirtPkg/ArmVirtQemu.fdf | 5

  1   2   3   4   5   6   7   8   9   10   >