[v5,1/1] package/libopenssl: set no-asm with generic architectures
diff mbox series

Message ID 20191019122759.529-1-fontaine.fabrice@gmail.com
State Accepted
Headers show
Series
  • [v5,1/1] package/libopenssl: set no-asm with generic architectures
Related show

Commit Message

Fabrice Fontaine Oct. 19, 2019, 12:27 p.m. UTC
Use no-asm when building with generic architectures such as gcc or
linux-generic32, see https://github.com/openssl/openssl/issues/9839

This will fix a static build failure on x86_64 due to the removal of
x86/x86_64 BSAES and AES_ASM support by
https://github.com/openssl/openssl/commit/87bea6550ae0dda7c40937cff2e86cc2b0b09491

Fixes:
 - http://autobuild.buildroot.org/results/e4f04bb13ec1b82b73db645bea4933e52bca4185

Signed-off-by: Fabrice Fontaine <fontaine.fabrice@gmail.com>
---
Changes v4 -> v5 (after review of Thomas Petazzoni):
 - Reword commit message to remove statically

Changes v3 -> v4 (after review of Thomas Petazzoni):
 - Fix copy/paste error and remove the second assignation

Changes v2 -> v3 (after review of Thomas Petazzoni):
 - Set no-asm with linux-generic32

Changes v1 -> v2 (after review of Thomas Petazzoni):
 - Set no-asm with all static builds and not only x86_64

 package/libopenssl/libopenssl.mk | 8 ++++++--
 1 file changed, 6 insertions(+), 2 deletions(-)

Comments

Thomas Petazzoni Oct. 27, 2019, 9:31 a.m. UTC | #1
On Sat, 19 Oct 2019 14:27:59 +0200
Fabrice Fontaine <fontaine.fabrice@gmail.com> wrote:

> Use no-asm when building with generic architectures such as gcc or
> linux-generic32, see https://github.com/openssl/openssl/issues/9839
> 
> This will fix a static build failure on x86_64 due to the removal of
> x86/x86_64 BSAES and AES_ASM support by
> https://github.com/openssl/openssl/commit/87bea6550ae0dda7c40937cff2e86cc2b0b09491
> 
> Fixes:
>  - http://autobuild.buildroot.org/results/e4f04bb13ec1b82b73db645bea4933e52bca4185
> 
> Signed-off-by: Fabrice Fontaine <fontaine.fabrice@gmail.com>
> ---
> Changes v4 -> v5 (after review of Thomas Petazzoni):
>  - Reword commit message to remove statically

Applied to master, thanks.

Thomas
Peter Korsgaard Oct. 31, 2019, 2:14 p.m. UTC | #2
>>>>> "Fabrice" == Fabrice Fontaine <fontaine.fabrice@gmail.com> writes:

 > Use no-asm when building with generic architectures such as gcc or
 > linux-generic32, see https://github.com/openssl/openssl/issues/9839

 > This will fix a static build failure on x86_64 due to the removal of
 > x86/x86_64 BSAES and AES_ASM support by
 > https://github.com/openssl/openssl/commit/87bea6550ae0dda7c40937cff2e86cc2b0b09491

 > Fixes:
 >  - http://autobuild.buildroot.org/results/e4f04bb13ec1b82b73db645bea4933e52bca4185

 > Signed-off-by: Fabrice Fontaine <fontaine.fabrice@gmail.com>
 > ---
 > Changes v4 -> v5 (after review of Thomas Petazzoni):
 >  - Reword commit message to remove statically

 > Changes v3 -> v4 (after review of Thomas Petazzoni):
 >  - Fix copy/paste error and remove the second assignation

 > Changes v2 -> v3 (after review of Thomas Petazzoni):
 >  - Set no-asm with linux-generic32

 > Changes v1 -> v2 (after review of Thomas Petazzoni):
 >  - Set no-asm with all static builds and not only x86_64

 >  package/libopenssl/libopenssl.mk | 8 ++++++--
 >  1 file changed, 6 insertions(+), 2 deletions(-)

Committed to 2019.02.x and 2019.08.x, thanks.

Patch
diff mbox series

diff --git a/package/libopenssl/libopenssl.mk b/package/libopenssl/libopenssl.mk
index 4a430bcb4d..a1bbf9a900 100644
--- a/package/libopenssl/libopenssl.mk
+++ b/package/libopenssl/libopenssl.mk
@@ -12,7 +12,9 @@  LIBOPENSSL_LICENSE_FILES = LICENSE
 LIBOPENSSL_INSTALL_STAGING = YES
 LIBOPENSSL_DEPENDENCIES = zlib
 HOST_LIBOPENSSL_DEPENDENCIES = host-zlib
-LIBOPENSSL_TARGET_ARCH = linux-generic32
+# no-asm is needed with generic architectures such as linux-generic32, see
+# https://github.com/openssl/openssl/issues/9839
+LIBOPENSSL_TARGET_ARCH = linux-generic32 no-asm
 LIBOPENSSL_CFLAGS = $(TARGET_CFLAGS)
 LIBOPENSSL_PROVIDES = openssl
 
@@ -55,7 +57,9 @@  endif
 
 ifeq ($(BR2_STATIC_LIBS),y)
 # Use "gcc" minimalistic target to disable DSO
-LIBOPENSSL_TARGET_ARCH = gcc
+# no-asm is needed with generic architectures such as gcc, see
+# https://github.com/openssl/openssl/issues/9839
+LIBOPENSSL_TARGET_ARCH = gcc no-asm
 else
 # Some architectures are optimized in OpenSSL
 # Doesn't work for thumb-only (Cortex-M?)