Re: [edk2] [patch] Clean up unused data type - BOOL.

2015-08-14 Thread Zhang, Chao B
It is good to me. Reviewed-by: Chao Zhang chao.b.zh...@intel.com Thanks Best regards Chao Zhang -Original Message- From: Yao, Jiewen Sent: Friday, August 14, 2015 2:08 PM To: edk2-devel@lists.01.org Cc: Yao, Jiewen; Zhang, Chao B Subject: [patch] Clean up unused data type - BOOL.

[edk2] [patch] Clean up unused data type - BOOL.

2015-08-14 Thread jiewen yao
Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Yao, Jiewen jiewen@intel.com CC: Zhang, Chao B chao.b.zh...@intel.com --- MdePkg/Include/IndustryStandard/Tpm20.h | 1 - 1 file changed, 1 deletion(-) diff --git a/MdePkg/Include/IndustryStandard/Tpm20.h

Re: [edk2] [Patch] NetworkPkg: Fix hang issue after system reconnected when IPSec has set up

2015-08-14 Thread Ye, Ting
Reviewed-by: Ye Ting ting...@intel.com -Original Message- From: Wu, Jiaxin Sent: Thursday, August 13, 2015 10:10 PM To: edk2-devel@lists.01.org Cc: Ye, Ting Subject: [Patch] NetworkPkg: Fix hang issue after system reconnected when IPSec has set up IpSecStop() is incompetent to send

Re: [edk2] [PATCH 0/2] Add HttpBoot support to OvmfPkg

2015-08-14 Thread David Woodhouse
On Fri, 2015-08-14 at 17:03 +0800, Gary Ching-Pang Lin wrote: I've tested the HttpBoot implementation with a simple test environment: Is this tested with IPv6? -- David WoodhouseOpen Source Technology Centre david.woodho...@intel.com

Re: [edk2] GetFirstGuidHob availability across phases

2015-08-14 Thread Andrew Fish
On Aug 14, 2015, at 3:57 AM, Michael Zimmermann sigmaepsilo...@gmail.com wrote: I registered a data Hob from the ARM PrePI(2ndstage) phase like this: BuildGuidDataHob (gLKApiAddrGuid, mLKApiAddress, sizeof mLKApiAddress); Unfortunately this hob is not available in later phase

Re: [edk2] [PATCH 0/2] Add HttpBoot support to OvmfPkg

2015-08-14 Thread Blibbet
On 08/14/2015 02:03 AM, Gary Ching-Pang Lin wrote: Although the current HttpBoot implementation is incomplete, it would be easier to develop and test the software stack with a virtual machine. THANKS for adding to OVMF! Besides 1219 UEFI.Next Feature - UEFI TLS API, what else is missing?

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

2015-08-14 Thread Ard Biesheuvel
As a first step towards disentangling the arguments passed to the various flavors of GCC we support, refactor the CC flags so that we distinguish more clearly between toolchains that generate PE/COFF directly (for IA32 and X86 only) and toolchains that generate ELF only. Contributed-under:

[edk2] [PATCH 01/15] BaseTools: Fix GCC49 build failure

2015-08-14 Thread Ard Biesheuvel
From: Scott Duplichan sc...@notabs.org Some gnu linkers used with GCC44, such as GNU ld 2.19.1, require a --defsym= command line option to precede the --script= option in order for the definition to be available for use by the script. Move the --defsym= command line option to satisfy this

[edk2] [PATCH 03/15] BaseTools: remove unused definition of GCC_WINDRES_FLAGS

2015-08-14 Thread Ard Biesheuvel
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/tools_def.template b/BaseTools/Conf/tools_def.template index

[edk2] [PATCH 12/15] BaseTools: unify ARM and AARCH64 DLINK flags for all GCC versions

2015-08-14 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 07/15] BaseTools: use leading underscore for symbol names where appropriate

2015-08-14 Thread Ard Biesheuvel
The MS ABI uses leading underscores to decorate symbol names when generating 32-bit code only. Due to a bug in GCC prior to version 4.3, it used the same decoration for 64-bit code but this has been fixed since. So uses the leading underscore for IA32 only, and remove it for X64.

[edk2] [PATCH 09/15] BaseTools: remove GCC 4.9 specific linker alignment override

2015-08-14 Thread Ard Biesheuvel
If any version of GCC emits any object whose actual alignment requirement exceeds 32 bytes, this actual alignment value will automatically become the PE/COFF section alignment value after PE/COFF conversion now that GenFw propagates the alignment of the ELF input sections. So there is no longer a

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

2015-08-14 Thread Ard Biesheuvel
When copying the relocted sections into the FFS file, we need to take care that we don't overrun the end of the file. Since, unlike the virtual size, the PE/COFF raw section size must be a multiple of file alignment, which means its size may exceed the virtual size. So use the minimum of the two.

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

2015-08-14 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 as well, add them there as well. Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Ard Biesheuvel ard.biesheu...@linaro.org

[edk2] [PATCH 13/15] BaseTools: unify ASM flags for all GCC versions

2015-08-14 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 14/15] BaseTools: align ELFGCC with GCC4x toolchains

2015-08-14 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 05/15] BaseTools GCC: refactor tools_def internal GCC defines for CC flags

2015-08-14 Thread Ard Biesheuvel
As a first step towards disentangling the arguments passed to the various flavors of GCC we support, refactor the CC flags so that we distinguish more clearly between toolchains that generate PE/COFF directly (for IA32 and X86 only) and toolchains that generate ELF only. Note that this does not

Re: [edk2] Shell 2.0 and 2.1

2015-08-14 Thread Daniel Samuelraj
*From:* Daniel Samuelraj [mailto:daniel.samuel...@avagotech.com] *Sent:* Friday, August 14, 2015 12:40 PM *To:* 'uswg'; edk2-de...@lists.sourceforge.net *Subject:* Shell 2.0 and 2.1 Hi, I assume that Shell 2.0 apps are expected to run fine in Shell 2.1. Can you please confirm? Should shell

[edk2] Help debugging PEIM on Minnowboard Max

2015-08-14 Thread Eric Wittmayer
I'm writing a PEIM for a USB3 device but having trouble even getting the UsbBusPie and XhciPei modules to load during boot. I thought getting the existing Usb Peims to load would be a good first step. I looked at the DEPEX for both of the above modules and tried removing

[edk2] [patch] Fix typo in BaseCrypto2HashInit() which causes sanity check incorrect.

2015-08-14 Thread jiewen yao
Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Yao, Jiewen jiewen@intel.com Cc: Zhang, Chao B chao.b.zh...@intel.com --- SecurityPkg/Hash2DxeCrypto/Hash2DxeCrypto.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git

[edk2] OVMF: iasl Control Method should be made Serialized

2015-08-14 Thread Bruce Cran
With iasl version 20150410-64 building OvmfX64 (using OvmfPkg/build.sh -a X64 -t GCC49 -b RELEASE) results in a couple of warnings about methods that should be serialized: iasl -p/home/bcran/workspace/edk2/Build/OvmfX64/RELEASE_GCC49/X64/OvmfPkg/AcpiTables/AcpiTables/OUTPUT/./Dsdt.aml

[edk2] [patch] SecurityPkg: Fixed build error due to FixedAtBuild PcdTcg2HashAlgorithmBitmap

2015-08-14 Thread El-Haj-Mahmoud, Samer
PcdTcg2HashAlgorithmBitmap is declared in a section that allows it to be Fixed or PatchableAtBuild, but there is code that sets it. This breaks the build on some platforms. Changed it to be PcdsDynamic and PcdsDynamicEx only. Contributed-under: TianoCore Contribution Agreement 1.0

Re: [edk2] [Patch 2/3] OvmfPkg: use new BDS and UiApp in MdeModulePkg

2015-08-14 Thread Laszlo Ersek
On 08/14/15 10:28, Ni, Ruiyu wrote: Laszlo, Where can I read your first 17 remarks? I didn't find it in my mail folder. Strange; your email address ruiyu...@intel.com was the only one in the To: field. (The list and Jordan were Cc'd.) In any case, here's the link into the archive:

Re: [edk2] [PATCH 2/2] OvmfPkg: Add HttpBoot support

2015-08-14 Thread Laszlo Ersek
Hi Gary, On 08/14/15 11:03, Gary Ching-Pang Lin wrote: This commit introdues a new build option to OvmfPkg: HTTP_BOOT_ENABLE. When HttpBoot is enabled, a new Network boot option will show in the boot manager menu with the device path like this:

Re: [edk2] [PATCH 0/2] Add HttpBoot support to OvmfPkg

2015-08-14 Thread Gary Ching-Pang Lin
On Fri, Aug 14, 2015 at 10:27:31AM +0100, David Woodhouse wrote: On Fri, 2015-08-14 at 17:03 +0800, Gary Ching-Pang Lin wrote: I've tested the HttpBoot implementation with a simple test environment: Is this tested with IPv6? No, IPv6 is not implemented in HttpBootDxe, so I only tested