Re: [edk2] [PATCH] CryptoPkg/OpensslLib AARCH64: clear XIP CC flags

2017-07-15 Thread Ard Biesheuvel
On 15 July 2017 at 13:09, Long, Qin wrote: > Reviewed-by: Long Qin > Thanks all. Pushed as e38eb2595b86 (with the commit log updated to clarify the performance concern) ___ edk2-devel mailing list edk2-devel@lists.01.org https://lists.01.org/mailman/l

Re: [edk2] [PATCH] CryptoPkg/OpensslLib AARCH64: clear XIP CC flags

2017-07-15 Thread Long, Qin
Reviewed-by: Long Qin Best Regards & Thanks, LONG, Qin -Original Message- From: Ard Biesheuvel [mailto:ard.biesheu...@linaro.org] Sent: Saturday, July 15, 2017 1:19 AM To: edk2-devel@lists.01.org; Long, Qin Cc: Ye, Ting ; leif.lindh...@linaro.org; ler...@redhat.com; Ard Biesheuvel S

Re: [edk2] [PATCH] CryptoPkg/OpensslLib AARCH64: clear XIP CC flags

2017-07-14 Thread Laszlo Ersek
On 07/14/17 20:41, Ard Biesheuvel wrote: > On 14 July 2017 at 19:22, Laszlo Ersek wrote: >> On 07/14/17 19:19, Ard Biesheuvel wrote: >>> Commit 0df6c8c157af ("BaseTools/tools_def AARCH64: avoid SIMD registers >>> in XIP code") updated the compiler flags used by AARCH64 when building >>> modules (i

Re: [edk2] [PATCH] CryptoPkg/OpensslLib AARCH64: clear XIP CC flags

2017-07-14 Thread Ard Biesheuvel
On 14 July 2017 at 19:22, Laszlo Ersek wrote: > On 07/14/17 19:19, Ard Biesheuvel wrote: >> Commit 0df6c8c157af ("BaseTools/tools_def AARCH64: avoid SIMD registers >> in XIP code") updated the compiler flags used by AARCH64 when building >> modules (including BASE libraries) that may execute befor

Re: [edk2] [PATCH] CryptoPkg/OpensslLib AARCH64: clear XIP CC flags

2017-07-14 Thread Laszlo Ersek
On 07/14/17 19:19, Ard Biesheuvel wrote: > Commit 0df6c8c157af ("BaseTools/tools_def AARCH64: avoid SIMD registers > in XIP code") updated the compiler flags used by AARCH64 when building > modules (including BASE libraries) that may execute before the MMU is > enabled. > > This broke the build fo

Re: [edk2] [PATCH] CryptoPkg/OpensslLib AARCH64: clear XIP CC flags

2017-07-14 Thread Leif Lindholm
On Fri, Jul 14, 2017 at 06:19:13PM +0100, Ard Biesheuvel wrote: > Commit 0df6c8c157af ("BaseTools/tools_def AARCH64: avoid SIMD registers > in XIP code") updated the compiler flags used by AARCH64 when building > modules (including BASE libraries) that may execute before the MMU is > enabled. > >

[edk2] [PATCH] CryptoPkg/OpensslLib AARCH64: clear XIP CC flags

2017-07-14 Thread Ard Biesheuvel
Commit 0df6c8c157af ("BaseTools/tools_def AARCH64: avoid SIMD registers in XIP code") updated the compiler flags used by AARCH64 when building modules (including BASE libraries) that may execute before the MMU is enabled. This broke the build for OpensslLib/OpensslLibCrypto because the SIMD regist