[edk2] [patch] MdeModulePkg:Fix the issue FindQuestionFromProgress in SetupBrowserDxe is broken

2015-07-30 Thread Dandan Bi
If the storage of the question is EFI_HII_VARSTORE_BUFFER/EFI_HII_VARSTORE_EFI_VARIABLE_BUFFER, in SetupBrowserDxe the configuration stings contain uppercase,but HiiDataBaseDxe generates the ConfigResp string in lowercase,they mismatch,so FindQuestionFromProgress function is broken. Now convert

Re: [edk2] [PATCH v2 0/4] FFS/FV aligment optimization (was: [RFC] small C model and LLVM/clang support for AARCH64)

2015-07-30 Thread Gao, Liming
Ard: This way may not work. I need to confirm it. I think you can provide the two Ffs Rules. One is with Fixed and 4K, another is no. They have the different Rule names. Then, apply the different one in the different tool chains. The example is like below. Could you try it? !If $(TOOL_CHAI

Re: [edk2] [Patch] Remove the useless code to fix build failure caused by error depend on IntelFrameworkModulePkg.

2015-07-30 Thread Gao, Liming
Reviewed-by: Liming Gao -Original Message- From: Dong, Eric Sent: Friday, July 31, 2015 9:32 AM To: Ni, Ruiyu; Gao, Liming; edk2-devel@lists.01.org Subject: [Patch] Remove the useless code to fix build failure caused by error depend on IntelFrameworkModulePkg. Contributed-under: TianoC

[edk2] [Patch] Remove the useless code to fix build failure caused by error depend on IntelFrameworkModulePkg.

2015-07-30 Thread Eric Dong
Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Eric Dong --- .../Application/UiApp/BootMaint/BootOption.c | 58 -- .../Application/UiApp/BootMaint/UpdatePage.c | 29 --- MdeModulePkg/Application/UiApp/Ui.h| 2 - MdeM

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

2015-07-30 Thread Jordan Justen
On 2015-07-30 15:35:58, Yao, Jiewen wrote: > Hello > Do we consider below 2 options? > -- DriversPkg/Vendor//Bus/ ? > or > -- DriversPkg/Bus//Vendor/ ? > > Another option is that we add , like > GOP/UNDI/RAID/SIO. Then we can put UNDI driver together, no matter > it is PCI based or USB based. The

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

2015-07-30 Thread Dong, Eric
Daryl and Ard, Sorry for the incorrect description. This file is new created and changed from an old existed file, so the old copyright is error kept. I have update the copyright info. Because this is new added file, so I only kept the 2015 copyright. Thanks, Eric -Original Message-

Re: [edk2] [PATCH 2/4] ArmVirtPkg: use 'auto' alignment and FIXED placement for XIP modules

2015-07-30 Thread Laszlo Ersek
On 07/28/15 18:42, Ard Biesheuvel wrote: > Now that GenFw correctly propagates the minimum alignment of the ELF > input sections to the PE/COFF binary, we can simply select 'auto' > alignment in the FDF Rule section instead of tweaking it by hand. > > Also add the FIXED FFS attribute to the module

Re: [edk2] [PATCH v2 0/4] FFS/FV aligment optimization

2015-07-30 Thread Laszlo Ersek
On 07/31/15 01:49, Ard Biesheuvel wrote: > On 31 July 2015 at 01:17, Laszlo Ersek wrote: >> On 07/27/15 15:52, Ard Biesheuvel wrote: >>> On 27 July 2015 at 15:34, Liu, Yingke D wrote: Reviewed-by: Yingke Liu >>> >>> Thank you >>> >>> Committed as SVN r18077 ... r18080 >>> >>> I do have

Re: [edk2] [PATCH v2 0/4] FFS/FV aligment optimization (was: [RFC] small C model and LLVM/clang support for AARCH64)

2015-07-30 Thread Ard Biesheuvel
On 31 July 2015 at 01:17, Laszlo Ersek wrote: > On 07/27/15 15:52, Ard Biesheuvel wrote: >> On 27 July 2015 at 15:34, Liu, Yingke D wrote: >>> Reviewed-by: Yingke Liu >>> >> >> Thank you >> >> Committed as SVN r18077 ... r18080 >> >> I do have another question related to the use of FIXED in a [R

Re: [edk2] [PATCH v2 0/4] FFS/FV aligment optimization

2015-07-30 Thread Laszlo Ersek
On 07/31/15 01:31, Andrew Fish wrote: > >> On Jul 30, 2015, at 4:17 PM, Laszlo Ersek wrote: >> >> On 07/27/15 15:52, Ard Biesheuvel wrote: >>> On 27 July 2015 at 15:34, Liu, Yingke D wrote: Reviewed-by: Yingke Liu >>> >>> Thank you >>> >>> Committed as SVN r18077 ... r18080 >>> >>> I

Re: [edk2] [PATCH v2 0/4] FFS/FV aligment optimization (was: [RFC] small C model and LLVM/clang support for AARCH64)

2015-07-30 Thread Andrew Fish
> On Jul 30, 2015, at 4:17 PM, Laszlo Ersek wrote: > > On 07/27/15 15:52, Ard Biesheuvel wrote: >> On 27 July 2015 at 15:34, Liu, Yingke D wrote: >>> Reviewed-by: Yingke Liu >>> >> >> Thank you >> >> Committed as SVN r18077 ... r18080 >> >> I do have another question related to the use of

Re: [edk2] [PATCH v2 0/4] FFS/FV aligment optimization (was: [RFC] small C model and LLVM/clang support for AARCH64)

2015-07-30 Thread Laszlo Ersek
On 07/27/15 15:52, Ard Biesheuvel wrote: > On 27 July 2015 at 15:34, Liu, Yingke D wrote: >> Reviewed-by: Yingke Liu >> > > Thank you > > Committed as SVN r18077 ... r18080 > > I do have another question related to the use of FIXED in a [Rule] section: > since the increased alignment on AARCH6

Re: [edk2] today's US CERT UEFI advisory

2015-07-30 Thread Andrew Fish
> On Jul 30, 2015, at 3:42 PM, Blibbet wrote: > > [...] Sorry I was thinking more about routine maintenance (like the string n cleanup recently), or refactoring of the code disabling the exploit mechanism with out knowledge that the exploit exists. I guess an older branch ca

Re: [edk2] more code sharing joy between OvmfPkg and ArmVirtPkg, re SMBIOS

2015-07-30 Thread Andrew Fish
> On Jul 30, 2015, at 3:35 PM, Laszlo Ersek wrote: > > On 07/30/15 20:54, Jordan Justen wrote: >> On 2015-07-30 10:09:34, Laszlo Ersek wrote: >>> (Sigh, I left off the list address. This should be discussed publicly. >>> Resending.) >>> >>> Clearly, the SMBIOS patches I posted and got committed

Re: [edk2] today's US CERT UEFI advisory

2015-07-30 Thread Blibbet
[...] >>> Sorry I was thinking more about routine maintenance (like the string n >>> cleanup recently), or refactoring of the code disabling the exploit >>> mechanism with out knowledge that the exploit exists. I guess an older >>> branch can get patched, but the commit history in master is not go

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

2015-07-30 Thread Yao, Jiewen
Hello Do we consider below 2 options? -- DriversPkg/Vendor//Bus/ ? or -- DriversPkg/Bus//Vendor/ ? Another option is that we add , like GOP/UNDI/RAID/SIO. Then we can put UNDI driver together, no matter it is PCI based or USB based. Thank you Yao Jiewen -Original Message- From: edk2-dev

Re: [edk2] more code sharing joy between OvmfPkg and ArmVirtPkg, re SMBIOS

2015-07-30 Thread Laszlo Ersek
On 07/30/15 20:54, Jordan Justen wrote: > On 2015-07-30 10:09:34, Laszlo Ersek wrote: >> (Sigh, I left off the list address. This should be discussed publicly. >> Resending.) >> >> Clearly, the SMBIOS patches I posted and got committed last time are not >> good enough. That's because the SMBIOS 3.0

Re: [edk2] [PATCH 11/58] OvmfPkg: implement EFI_SMM_CONTROL2_PROTOCOL with a DXE_RUNTIME_DRIVER

2015-07-30 Thread Paolo Bonzini
On 28/07/2015 20:44, Laszlo Ersek wrote: > I have a significant update for this patch. On S3 resume, the APMC_EN > bit (and other bits) are cleared in SMI_EN (which is necessary, see qemu > commit be66680e). For the trigger method to work right after S3 resume, > the APMC_EN bit must be set again

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

2015-07-30 Thread Jordan Justen
On 2015-07-30 11:51:56, Kinney, Michael D wrote: > Jordan, > > I agree with the concept of having package(s) for device driver content. > > OptionRomPkg is really only intended to provide good examples for > writing drivers, and we want to restrict what goes in there to a > good example for each

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

2015-07-30 Thread Yao, Jiewen
Thanks for the info. That makes sense. It is pity that there is no X64 version and SMP not work. We can make improvement step by step. Thank you Yao Jiewen -Original Message- From: edk2-devel [mailto:edk2-devel-boun...@lists.01.org] On Behalf Of Laszlo Ersek Sent: Wednesday, July 29, 2

[edk2] [patch] Add Dual-FSP support (MemoryInitUpd/SiliconInitUpd)

2015-07-30 Thread jiewen yao
Add FspUpdSignatureCheck() API in FspSecPlatformLib, so that FspSecCore can check if UPD data is valid in FSP API. Add Set/GetFspMemoryInitUpdDataPointer() and Set/GetFspSiliconInitUpdDataPointer() API in FspCommonLib, so that core can set this UdpDataPointer and platform code may get UpdDataPoi

Re: [edk2] today's US CERT UEFI advisory

2015-07-30 Thread Laszlo Ersek
On 07/30/15 21:02, Andrew Fish wrote: >> On Jul 30, 2015, at 11:43 AM, Laszlo Ersek > > wrote: >> On 07/30/15 20:09, Andrew Fish wrote: >>> I’m not sure that the commit messages is the best way to communicate the >>> info, and something could get fixed prior CVE going o

Re: [edk2] [PATCH 11/58] OvmfPkg: implement EFI_SMM_CONTROL2_PROTOCOL with a DXE_RUNTIME_DRIVER

2015-07-30 Thread Laszlo Ersek
On 07/30/15 20:50, Paolo Bonzini wrote: > > > On 28/07/2015 20:44, Laszlo Ersek wrote: >> I have a significant update for this patch. On S3 resume, the APMC_EN >> bit (and other bits) are cleared in SMI_EN (which is necessary, see qemu >> commit be66680e). For the trigger method to work right aft

Re: [edk2] today's US CERT UEFI advisory

2015-07-30 Thread Andrew Fish
> On Jul 30, 2015, at 11:43 AM, Laszlo Ersek wrote: > > On 07/30/15 20:09, Andrew Fish wrote: >> >>> On Jul 30, 2015, at 9:58 AM, Laszlo Ersek >> >>> >> wrote: >>> >>> On 07/30/15 17:49, Blibbet wrote: FYI, in

Re: [edk2] more code sharing joy between OvmfPkg and ArmVirtPkg, re SMBIOS

2015-07-30 Thread Jordan Justen
On 2015-07-30 10:09:34, Laszlo Ersek wrote: > (Sigh, I left off the list address. This should be discussed publicly. > Resending.) > > Clearly, the SMBIOS patches I posted and got committed last time are not > good enough. That's because the SMBIOS 3.0 entry point is structurally > different from

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

2015-07-30 Thread Kinney, Michael D
Jordan, I agree with the concept of having package(s) for device driver content. OptionRomPkg is really only intended to provide good examples for writing drivers, and we want to restrict what goes in there to a good example for each driver type. So I do not think OptionRomPkg is a good landin

Re: [edk2] today's US CERT UEFI advisory

2015-07-30 Thread Laszlo Ersek
On 07/30/15 20:09, Andrew Fish wrote: > >> On Jul 30, 2015, at 9:58 AM, Laszlo Ersek > > wrote: >> >> On 07/30/15 17:49, Blibbet wrote: >>> FYI, in case any OEM's missed today's US-CERT UEFI vulnerability notice: >>> >>> http://firmwaresecurity.com/2015/07/30/us-cert-bios

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

2015-07-30 Thread Daryl McDaniel
The copyright information was incorrectly updated. The correct copyright is: Copyright (c) 2011 - 2015, Intel Corporation. All rights reserved. Previous copyrights must be maintained. This patch should be rejected and a new one submitted with the corrected copyright notice and subject li

Re: [edk2] today's US CERT UEFI advisory

2015-07-30 Thread Andrew Fish
> On Jul 30, 2015, at 9:58 AM, Laszlo Ersek wrote: > > On 07/30/15 17:49, Blibbet wrote: >> FYI, in case any OEM's missed today's US-CERT UEFI vulnerability notice: >> >> http://firmwaresecurity.com/2015/07/30/us-cert-bios-vulnerability-note-vu577140/ >> >> Remember that any TianoCore-based bu

Re: [edk2] today's US CERT UEFI advisory

2015-07-30 Thread Blibbet
I agree that OEMs/IBVs/TianoCore Security Advisories can improve their security information. I *DEFINATELY* agree that bugs should include CVEs in them. I also think UEFI Forum needs to define OVAL definitions for UEFI. Right now, firmware is not defined in SCAP, and it's just luck if the right te

Re: [edk2] today's US CERT UEFI advisory

2015-07-30 Thread Bill Jacobs (billjac)
Bravo! -Original Message- From: edk2-devel [mailto:edk2-devel-boun...@lists.01.org] On Behalf Of Laszlo Ersek Sent: Thursday, July 30, 2015 9:58 AM To: Blibbet; edk2-de...@ml01.01.org Cc: Peter Jones Subject: Re: [edk2] today's US CERT UEFI advisory On 07/30/15 17:49, Blibbet wrote: > FY

Re: [edk2] more code sharing joy between OvmfPkg and ArmVirtPkg, re SMBIOS

2015-07-30 Thread Laszlo Ersek
(resending this one too) On 07/30/15 19:09, Laszlo Ersek wrote: > (Sigh, I left off the list address. This should be discussed publicly. > Resending.) > > Clearly, the SMBIOS patches I posted and got committed last time are not > good enough. That's because the SMBIOS 3.0 entry point is structura

[edk2] more code sharing joy between OvmfPkg and ArmVirtPkg, re SMBIOS

2015-07-30 Thread Laszlo Ersek
(Sigh, I left off the list address. This should be discussed publicly. Resending.) Clearly, the SMBIOS patches I posted and got committed last time are not good enough. That's because the SMBIOS 3.0 entry point is structurally different from the prior versions (because why not). Therefore, now tha

Re: [edk2] [PATCH] Vlv2DeviceRefCodePkg: don't redefine struct typedefs

2015-07-30 Thread Bruce Cran
On 7/27/2015 11:39 PM, He, Tim wrote: Yes, these 2 typedefs is being defined twice, we need to remove the second definition, I will check in the code patch. Thanks. Just to check, how long is there normally between accepting the patch and committing it? I don't see any of my recent changesets

Re: [edk2] today's US CERT UEFI advisory

2015-07-30 Thread Laszlo Ersek
On 07/30/15 17:49, Blibbet wrote: > FYI, in case any OEM's missed today's US-CERT UEFI vulnerability notice: > > http://firmwaresecurity.com/2015/07/30/us-cert-bios-vulnerability-note-vu577140/ > > Remember that any TianoCore-based bugs may be in your platorm: > > https://twitter.com/XenoKovah/s

[edk2] today's US CERT UEFI advisory

2015-07-30 Thread Blibbet
FYI, in case any OEM's missed today's US-CERT UEFI vulnerability notice: http://firmwaresecurity.com/2015/07/30/us-cert-bios-vulnerability-note-vu577140/ Remember that any TianoCore-based bugs may be in your platorm: https://twitter.com/XenoKovah/status/623483244890189824 Can anyone clarify if

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

2015-07-30 Thread Leif Lindholm
Hi Jordan, On Wed, Jul 29, 2015 at 02:59:04PM -0700, Jordan Justen wrote: > > > But, the name 'open platform' also sounds strange, assuming this a > > > plain PCI bus driver. Couldn't it live in a 'pci drivers' package? > > > > > > Personally, I think we should rename OptionRomPkg to DriversPkg,

Re: [edk2] efi acpidump port

2015-07-30 Thread Laszlo Ersek
On 07/30/15 01:27, Andrew Fish wrote: > >> On Jul 29, 2015, at 4:11 PM, Jordan Justen 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] [PATCH v3 12/12] BaseTools/X86|IA32: move to unified GCC linker script

2015-07-30 Thread Gao, Liming
Yes. -Original Message- From: edk2-devel [mailto:edk2-devel-boun...@lists.01.org] On Behalf Of Ard Biesheuvel Sent: Thursday, July 30, 2015 10:26 PM To: Gao, Liming Cc: Justen, Jordan L; edk2-devel@lists.01.org; ler...@redhat.com; Liu, Yingke D; leif.lindh...@linaro.org Subject: Re: [ed

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

2015-07-30 Thread Ard Biesheuvel
On 30 July 2015 at 02:59, Gao, Liming wrote: > Jordan: > I have verified 4K aligned image build. > > Test-by: Liming Gao > Thanks Liming Just to be clear, I assume you added something like this """ diff --git a/OvmfPkg/OvmfPkgX64.dsc b/OvmfPkg/OvmfPkgX64.dsc index a86a7f57143b..90dc29a5157d

Re: [edk2] [PATCH 2/2] CryptoPkg/OpensslLib: Undefine NO_BUILTIN_VA_FUNCS to fix varargs breakage

2015-07-30 Thread Laszlo Ersek
On 07/30/15 13:37, David Woodhouse wrote: > On Tue, 2015-07-28 at 20:26 +0200, Laszlo Ersek wrote: >> >> series (up to and including 3/2) >> Tested-by: Laszlo Ersek > > If you did this by pulling my tree, rather than manually applying > patches — which I'm fairly sure you did — then you also test

[edk2] [PATCH v4 13/13] BaseTools IA32/X64: Use GccBase.lds instead of gcc*-ld-script

2015-07-30 Thread Ard Biesheuvel
These scripts all now have the same contents, so we only need to use GccBase.lds. Therefore we can delete gcc-4K-align-ld-script, gcc4.4-ld-script and gcc4.9-ld-script. Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Ard Biesheuvel Reviewed-by: Jordan Justen Tested-by: Lim

[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 Reviewed-by: Jordan Justen Tested-by: Liming Gao -

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

2015-07-30 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 l

[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 Reviewed-by: Jordan J

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

2015-07-30 Thread Ard Biesheuvel
Fourth attempt at unifying the various GCC linker scripts for different architectures, GCC versions and minimum alignments. Changes since v3: - added patch #5 which updates the various IA32/X86 linker scripts to take their PE/COFF header size and section alignment from the command line, before

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

2015-07-30 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 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 Reviewed-by: Jordan Justen Reviewed-by: Leif Lindholm Tested-by:

[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 Reviewed-by: Jordan Justen Reviewed-by: Leif Lindholm Tested-by: Leif Lindholm --- BaseTools/Conf/tools_def.template |

[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 Acked-by: Laszlo Erse

[edk2] [PATCH v4 04/13] BaseTools IA32/X64: move .got contents to the PE/COFF .text section

2015-07-30 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 f

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

2015-07-30 Thread Ard Biesheuvel
This unifies all GCC linker scripts into a single parametrised GCC linker script that can be used for all GCC versions and architectures. The two parameters that can be set on the linker command line are: - PECOFF_HEADER_SIZE, this is a build time property of GenFw, but its value is different be

[edk2] [PATCH v4 05/13] BaseTools IA32/X64: get header size and alignment from ld commandline

2015-07-30 Thread Ard Biesheuvel
Instead of hardcoding the values for the PE/COFF header size and the section alignment, set them on the linker command line. This factors out these values from the various linker scripts, which will allow us to unify them in a subsequent patch. Contributed-under: TianoCore Contribution Agreement 1

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

2015-07-30 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 v4 01/13] BaseTools IA32/X64: remove NOP padding from X86/IA32 GCC linker scripts

2015-07-30 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 NOP

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

2015-07-30 Thread Leif Lindholm
On Wed, Jul 29, 2015 at 05:11:59PM +0200, 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 Contr

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

2015-07-30 Thread Leif Lindholm
On Wed, Jul 29, 2015 at 05:12:01PM +0200, Ard Biesheuvel wrote: > 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 Bi

Re: [edk2] [PATCH v3 08/12] BaseTools/AARCH64: move to unified GCC linker script

2015-07-30 Thread Leif Lindholm
On Wed, Jul 29, 2015 at 05:11:58PM +0200, Ard Biesheuvel wrote: > 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 Reviewed-by: Leif Lindholm Also tested on ARM and AAR

Re: [edk2] [PATCH 2/2] CryptoPkg/OpensslLib: Undefine NO_BUILTIN_VA_FUNCS to fix varargs breakage

2015-07-30 Thread David Woodhouse
On Tue, 2015-07-28 at 20:26 +0200, Laszlo Ersek wrote: > > series (up to and including 3/2) > Tested-by: Laszlo Ersek If you did this by pulling my tree, rather than manually applying patches — which I'm fairly sure you did — then you also tested the previous series of 6 API cleanups. I'll inclu

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

2015-07-30 Thread David Woodhouse
On Wed, 2015-07-29 at 14:38 -0700, Jordan Justen wrote: > 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 o

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

2015-07-30 Thread Laszlo Ersek
On 07/29/15 23:38, Jordan Justen wrote: > 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. > >>

Re: [edk2] [PATCH v2] MdeModulePkg PeiCore: Add PCD to specify PEIM Shadow

2015-07-30 Thread Zeng, Star
Reviewed-by: Star Zeng -Original Message- From: edk2-devel [mailto:edk2-devel-boun...@lists.01.org] On Behalf Of Liming Gao Sent: Thursday, July 30, 2015 5:40 PM To: edk2-devel@lists.01.org Subject: [edk2] [PATCH v2] MdeModulePkg PeiCore: Add PCD to specify PEIM Shadow v2 changelog: Che

Re: [edk2] [PATCH 0/4] StdLib: Add ARM SoftFloat & AArch64 supports

2015-07-30 Thread Laszlo Ersek
On 07/30/15 04:21, Daryl McDaniel wrote: > Now that we have received legal approval of the LLVM / University of > Illinois license, I can say that this set of patches looks OK to me. > 0. [PATCH 0/4] StdLib: Add ARM SoftFloat & AArch64 supports > 1. [PATCH 1/4] StdLib: Added BaseStackLib

[edk2] [PATCH v2] MdeModulePkg PeiCore: Add PCD to specify PEIM Shadow

2015-07-30 Thread Liming Gao
v2 changelog: Check CurrentPeimHandle to check the matched PeimHandle. Add check point to ShadowPeiCore based on PCD. v1 changelog: PeiCore LoadImage always shadow itself and PEIM on normal boot after the physical memory is installed. On the emulator platform, the shadow may be not necessary. To s

Re: [edk2] [Patch] UiApp: Move reset menu from frontpage to BMM page.

2015-07-30 Thread Ni, Ruiyu
Reviewed-by: Ruiyu Ni > -Original Message- > From: Dong, Eric > Sent: Thursday, July 30, 2015 4:56 PM > To: Ni, Ruiyu ; edk2-devel@lists.01.org > Subject: [Patch] UiApp: Move reset menu from frontpage to BMM page. > > Contributed-under: TianoCore Contribution Agreement 1.0 > Signed-off-b

[edk2] [Patch] UiApp: Move reset menu from frontpage to BMM page.

2015-07-30 Thread Eric Dong
Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Eric Dong --- MdeModulePkg/Application/UiApp/BootMaint/Bm.vfr | 8 MdeModulePkg/Application/UiApp/BootMaint/BootMaint.c | 4 MdeModulePkg/Application/UiApp/BootMaint/FormGuid.h | 1 + MdeModulePkg/Applicati

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

2015-07-30 Thread Ard Biesheuvel
On 29 July 2015 at 10:59, Eric Dong wrote: > Contributed-under: TianoCore Contribution Agreement 1.0 > Signed-off-by: Eric Dong The license is called 'BSD' not 'BDS' Could you fix up the commit titles please? -- Ard. > --- > .../Include/Guid/HiiBootMaintenanceFormset.h | 28 > +++

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

2015-07-30 Thread Ard Biesheuvel
On 29 July 2015 at 23:02, Jordan Justen wrote: > 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. >> >