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

2015-07-29 Thread Ard Biesheuvel
Now that GenFw honors the ELF section alignment when placing the PE/COFF sections in the output, the start of the PE/COFF version of .data will be aligned to the alignment of .text if its alignment is higher than the default. So duplicate this behavior in the ELF output, this will make the memory

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

2015-07-29 Thread Ard Biesheuvel
All AutoGen.obj files consist of global GUID definitions, fixed and patchable PCDs and other data that is essentially read-only at runtime but has not been declared as such for various reasons. By moving these contents to .text we achieve two things: - global GUIDs and other data items which must

[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

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

2015-07-29 Thread Ard Biesheuvel
The NOP padding in the GCC linker scripts ensures that all empty regions in the ELF binary are filled with x86 NOP instructions. There is no upside to doing this: if the CPU ends up executing these instructions, we have little hope of resuming normal execution of the program anyway. And having

[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

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

2015-07-29 Thread Ard Biesheuvel
The .rodata ELF section contains constant non-executable data that should never be modified by the program itself. Since the risk of inadvertent modification is typically higher than the risk of inadvertent execution, it makes sense to put this data in the R-X .text section rather than in the RW-

[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 ---

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

2015-07-29 Thread Ard Biesheuvel
Third attempt at unifying the various GCC linker scripts for different architectures, GCC versions and minimum alignments. Changes since v2: - for easier bisection, factor out the differences between the original and the unified linker scripts for X86 before making the switch (patches #1 - #4

[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 ---

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

2015-07-29 Thread Ard Biesheuvel
Move the .got contents to the PE/COFF .text section. This should be a no-op, since we typically don't generate position independent code (i.e., using -fPIC). But since the GOT contains variable addresses that are updated at relocation time only, its contents are best kept in .text to prevent them

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

2015-07-29 Thread Ard Biesheuvel
Remove obsolete ARM and AARCH64 platforms so the maintainers can focus on the ones that are still supported, which are: - TC2 (ArmVExpress-CTA15-A7.dsc) - Foundation model and Fast model emulators (ArmVExpress-FVP-AArch64.dsc) - Juno (ArmJunoPkg/ArmJuno.dsc) - Cortex-A15 MPcore RTSM

Re: [edk2] [PATCH] EmbeddedPkg: Added Marvell Yukon Ethernet support

2015-07-29 Thread Jordan Justen
On 2015-07-28 10:51:38, Leif Lindholm wrote: Hi Jordan, Sorry, (metaphorically) snowed under for a few days. On Fri, Jul 17, 2015 at 02:44:10PM -0700, Jordan Justen wrote: Does this depend on ArmPkg / EmbeddedPkg, or is this just a generic PCI based driver? It could be nice

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

2015-07-29 Thread Jordan Justen
On 2015-07-29 08:11:59, Ard Biesheuvel wrote: 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

Re: [edk2] efi acpidump port

2015-07-29 Thread Smith, Jonathan D
Password: edk2efiacpidump -Original Message- From: edk2-devel [mailto:edk2-devel-boun...@lists.01.org] On Behalf Of Andrew Fish Sent: Wednesday, July 29, 2015 1:50 PM To: Smith, Jonathan D Cc: edk2-devel@lists.01.org Subject: Re: [edk2] efi acpidump port On Jul 29, 2015, at 1:43 PM,

Re: [edk2] efi acpidump port

2015-07-29 Thread Andrew Fish
On Jul 29, 2015, at 1:43 PM, Smith, Jonathan D jonathan.d.sm...@intel.com wrote: Attaching a zip of the files Looks like you need to rename the inf, or add a password to the Zip. cat 0_Warning.txt BLOCKED FILE ALERTA file has been blocked due to the 'Intel Email File Filtering Policy'

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

2015-07-29 Thread Jordan Justen
Subject prefix: BaseTools/X86|IA32 = BaseTools IA32/X64 What about 1 more step? :) This change starts to make use of the -z common-page-size and --defsym=PECOFF_HEADER_SIZE params, but the description says 'move to unified script'. So, how about first modifying the gcc*-ld-script files to use

Re: [edk2] posting to the new list without subscription?

2015-07-29 Thread Jordan Justen
On 2015-07-28 12:59:46, Laszlo Ersek wrote: Hi Jordan, what are the rules for posting to the new list without being subscribed? On the old list, I think we dropped all such emails. Currently we are rejecting them on the new list. Paolo posted a few absolutely great comments recently, and

Re: [edk2] efi acpidump port

2015-07-29 Thread Jordan Justen
On 2015-07-29 16:27:38, Andrew Fish wrote: On Jul 29, 2015, at 4:11 PM, Jordan Justen jordan.l.jus...@intel.com wrote: On 2015-07-29 13:43:43, Smith, Jonathan D wrote: Attaching a zip of the files -Original Message- From: edk2-devel [mailto:edk2-devel-boun...@lists.01.org

Re: [edk2] efi acpidump port

2015-07-29 Thread Jordan Justen
On 2015-07-29 13:43:43, Smith, Jonathan D wrote: Attaching a zip of the files -Original Message- From: edk2-devel [mailto:edk2-devel-boun...@lists.01.org] On Behalf Of Smith, Jonathan D Sent: Wednesday, July 29, 2015 1:37 PM To: 'Andrew Fish'; 'edk2-devel@lists.01.org' Subject:

Re: [edk2] efi acpidump port

2015-07-29 Thread Andrew Fish
On Jul 29, 2015, at 4:11 PM, Jordan Justen jordan.l.jus...@intel.com wrote: On 2015-07-29 13:43:43, Smith, Jonathan D wrote: Attaching a zip of the files -Original Message- From: edk2-devel [mailto:edk2-devel-boun...@lists.01.org mailto:edk2-devel-boun...@lists.01.org] On

[edk2] [Patch] MdeModulePkg: Enhance PciBusDxe to handle high 32bit of MEM64 BAR returns 0

2015-07-29 Thread Ruiyu Ni
According to the PCI spec, when software writes all-one to BAR for size probing, the value read back should be 0b1...10...0 after masking the BAR type bits. But in real world, it's possible that certain device returns 0b0...01...10...0 for MEM64 BAR size probing: some bits in the high 32bit may be

Re: [edk2] [Patch 0/2] Update License Header to use BSD License

2015-07-29 Thread Ni, Ruiyu
Reviewed-by: Ruiyu Ni ruiyu...@intel.com -Original Message- From: Dong, Eric Sent: Wednesday, July 29, 2015 4:59 PM To: Ni, Ruiyu ruiyu...@intel.com; Gao, Liming liming@intel.com; edk2-devel@lists.01.org Subject: [Patch 0/2] Update License Header to use BSD License Some code

Re: [edk2] [Patch 0/3] Add UiApp/LegacyBootMaintUiLib used for New BdsDxe code.

2015-07-29 Thread Ni, Ruiyu
Reviewed-by: Ruiyu Ni ruiyu...@intel.com Thanks a lot for filling the gap after new BDS driver is added. -Original Message- From: Dong, Eric Sent: Wednesday, July 29, 2015 4:26 PM To: Ni, Ruiyu ruiyu...@intel.com; Gao, Liming liming@intel.com; edk2-devel@lists.01.org Subject:

Re: [edk2] [patch 2/2] MdeModulePkg/PciBus: Simplify an complex if statement to pass VS2015

2015-07-29 Thread Ni, Ruiyu
Reviewed-by: Ruiyu Ni ruiyu...@intel.com -Original Message- From: Tian, Feng Sent: Wednesday, July 29, 2015 11:26 AM To: Zeng, Star star.z...@intel.com; Ni, Ruiyu ruiyu...@intel.com Cc: edk2-devel@lists.01.org; Kinney, Michael D michael.d.kin...@intel.com Subject: [patch 2/2]

Re: [edk2] [PATCH v2 4/6] ArmPkg: cache detected revision in ArmGicArchLib

2015-07-29 Thread Ryan Harkin
This list moves too fast for me to keep up with changes before they are committed... :-( But still... On 26 July 2015 at 10:40, Ard Biesheuvel ard.biesheu...@linaro.org wrote: Instead of inferring the GIC revision from the CPU id registers and the presence/availability of the system register

Re: [edk2] [PATCH v2 4/6] ArmPkg: cache detected revision in ArmGicArchLib

2015-07-29 Thread Ard Biesheuvel
On 29 July 2015 at 11:27, Ryan Harkin ryan.har...@linaro.org wrote: This list moves too fast for me to keep up with changes before they are committed... :-( But still... Well, these have been on the list for a while, but I never cc'ed you the first time around. But thanks for taking a look

Re: [edk2] [Patch 0/3] Add UiApp/LegacyBootMaintUiLib used for New BdsDxe code.

2015-07-29 Thread David Woodhouse
On Wed, 2015-07-29 at 16:26 +0800, Eric Dong wrote: After new BdsDxe driver checked in at MdeModulePkg/Universal/BdsDxe, it does not have an UI part code to cowork with this new bds code. Now this UiApp and LegacyBootMaintUiLib is develop to meet this requirement. We split the

[edk2] [Patch 0/3] Add UiApp/LegacyBootMaintUiLib used for New BdsDxe code.

2015-07-29 Thread Eric Dong
After new BdsDxe driver checked in at MdeModulePkg/Universal/BdsDxe, it does not have an UI part code to cowork with this new bds code. Now this UiApp and LegacyBootMaintUiLib is develop to meet this requirement. We split the legacyBootMaintUiLib from UiApp is to make the UiApp are legacy

[edk2] [Patch 2/2] Update copyright info, use BDS license.

2015-07-29 Thread Eric Dong
Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Eric Dong eric.d...@intel.com --- .../Include/Guid/HiiBootMaintenanceFormset.h | 28 ++ 1 file changed, 12 insertions(+), 16 deletions(-) diff --git

[edk2] [Patch 0/2] Update License Header to use BSD License

2015-07-29 Thread Eric Dong
Some code error used intel license, this patch update it to use BSD license. Eric Dong (2): Update copyright info, use BDS license. Update copyright info, use BDS license. .../LegacyBootMaintUiLib/LegacyBootMaintUi.c | 19 +++--- .../LegacyBootMaintUiLib/LegacyBootMaintUi.h

Re: [edk2] [PATCH 38/58] UefiCpuPkg: CpuDxe: optionally save MTRR settings to AcpiNVS memory block

2015-07-29 Thread Laszlo Ersek
On 07/28/15 23:41, Yao, Jiewen wrote: HI Laszlo I like the diagram in GMANE. Good work! Thanks! Will you consider the option to merge CpuS3DataDxe into CpuMpDxe? Then we can reduce the driver number. Well, that's a hard question. The code that I'm gradually creating / porting under

Re: [edk2] [Patch] MdeModulePkg: Enhance PciBusDxe to handle high 32bit of MEM64 BAR returns 0

2015-07-29 Thread Dong, Eric
Reviewed-by: Eric Dong eric.d...@intel.com -Original Message- From: Ni, Ruiyu Sent: Thursday, July 30, 2015 11:11 AM To: edk2-devel@lists.01.org Cc: Ni, Ruiyu; Dong, Eric Subject: [Patch] MdeModulePkg: Enhance PciBusDxe to handle high 32bit of MEM64 BAR returns 0 According to the PCI

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

2015-07-29 Thread Gao, Liming
Jordan: I have verified 4K aligned image build. Test-by: Liming Gao liming.gao@intel Thanks Liming -Original Message- From: Justen, Jordan L Sent: Thursday, July 30, 2015 5:16 AM To: Ard Biesheuvel; edk2-devel@lists.01.org; Liu, Yingke D; Gao, Liming Cc: ler...@redhat.com;