Fix AArch64 build failure

This commit is contained in:
Daiki Ueno 2020-09-05 10:46:10 +02:00
parent c77e79a71e
commit 2bb137e19d
2 changed files with 62 additions and 0 deletions

58
nss-aarch64-gcc.patch Normal file
View File

@ -0,0 +1,58 @@
# HG changeset patch
# User Daiki Ueno <dueno@redhat.com>
# Date 1599294537 -7200
# Sat Sep 05 10:28:57 2020 +0200
# Node ID 1294add76fecdfc301c9cab3ed68ed1681979780
# Parent e03296e73ba666329bd9c1257038353bc9074466
Bug 1659256, add gcc version check on AArch64 optimization, r?rrelyea
Reviewers: rrelyea
Bug #: 1659256
Differential Revision: https://phabricator.services.mozilla.com/D87174
diff -r e03296e73ba6 -r 1294add76fec lib/freebl/Makefile
--- a/lib/freebl/Makefile Sat Sep 05 08:53:40 2020 +0200
+++ b/lib/freebl/Makefile Sat Sep 05 10:28:57 2020 +0200
@@ -119,11 +119,24 @@
DEFINES += -DNSS_X86
endif
endif
-ifdef NS_USE_GCC
ifeq ($(CPU_ARCH),aarch64)
- DEFINES += -DUSE_HW_AES -DUSE_HW_SHA1 -DUSE_HW_SHA2
- EXTRA_SRCS += aes-armv8.c gcm-aarch64.c sha1-armv8.c sha256-armv8.c
-endif
+ ifdef CC_IS_CLANG
+ DEFINES += -DUSE_HW_AES -DUSE_HW_SHA1 -DUSE_HW_SHA2
+ EXTRA_SRCS += aes-armv8.c gcm-aarch64.c sha1-armv8.c sha256-armv8.c
+ else ifeq (1,$(CC_IS_GCC))
+ # GCC versions older than 4.9 don't support ARM AES. The check
+ # is done in two parts, first allows "major.minor" == "4.9",
+ # and then rejects any major versions prior to 5. Note that
+ # there has been no GCC 4.10, as it is renamed to GCC 5.
+ ifneq (,$(filter 4.9,$(word 1,$(GCC_VERSION)).$(word 2,$(GCC_VERSION))))
+ DEFINES += -DUSE_HW_AES -DUSE_HW_SHA1 -DUSE_HW_SHA2
+ EXTRA_SRCS += aes-armv8.c gcm-aarch64.c sha1-armv8.c sha256-armv8.c
+ endif
+ ifeq (,$(filter 0 1 2 3 4,$(word 1,$(GCC_VERSION))))
+ DEFINES += -DUSE_HW_AES -DUSE_HW_SHA1 -DUSE_HW_SHA2
+ EXTRA_SRCS += aes-armv8.c gcm-aarch64.c sha1-armv8.c sha256-armv8.c
+ endif
+ endif
endif
ifeq ($(CPU_ARCH),arm)
ifndef NSS_DISABLE_ARM32_NEON
@@ -133,7 +146,10 @@
DEFINES += -DUSE_HW_AES -DUSE_HW_SHA1 -DUSE_HW_SHA2
EXTRA_SRCS += aes-armv8.c sha1-armv8.c sha256-armv8.c
else ifeq (1,$(CC_IS_GCC))
- # Old compiler doesn't support ARM AES.
+ # GCC versions older than 4.9 don't support ARM AES. The check
+ # is done in two parts, first allows "major.minor" == "4.9",
+ # and then rejects any major versions prior to 5. Note that
+ # there has been no GCC 4.10, as it is renamed to GCC 5.
ifneq (,$(filter 4.9,$(word 1,$(GCC_VERSION)).$(word 2,$(GCC_VERSION))))
DEFINES += -DUSE_HW_AES -DUSE_HW_SHA1 -DUSE_HW_SHA2
EXTRA_SRCS += aes-armv8.c sha1-armv8.c sha256-armv8.c

View File

@ -111,6 +111,10 @@ Patch4: iquote.patch
Patch11: nss-disable-legacydb.patch Patch11: nss-disable-legacydb.patch
%endif %endif
Patch12: nss-signtool-format.patch Patch12: nss-signtool-format.patch
# Upstream: https://bugzilla.mozilla.org/show_bug.cgi?id=1659256
# Also fixes the regression in bug 1652032:
# https://phabricator.services.mozilla.com/D87174#2817994
Patch13: nss-aarch64-gcc.patch
%if 0%{?fedora} < 34 %if 0%{?fedora} < 34
%if 0%{?rhel} < 9 %if 0%{?rhel} < 9
Patch20: nss-gcm-param-default-pkcs11v2.patch Patch20: nss-gcm-param-default-pkcs11v2.patch