<sys/auxv.h> and getauxval() don't exists on many BSDs. FreeBSD recently implemented AT_HWCAP and AT_HWCAP2 via KERN_PROC_AUXV sysctl (getauxval() isn't ready yet) but only for 32bit ARM, on 64bit ARM (i.e., aarch64) userspace is supposed to read ID registers instead. As NSS doesn't use autoconf it'd probably make ifdefs even more ugly. $ cc -v FreeBSD clang version 4.0.0 (tags/RELEASE_400/final 297347) (based on LLVM 4.0.0) Target: aarch64-unknown-freebsd11.1 Thread model: posix InstalledDir: /usr/bin /usr/bin/cc -o FreeBSD11.1_OPT.OBJ/FreeBSD_SINGLE_SHLIB/blinit.o -c -O2 -pipe -I/usr/local/include/nspr -fno-strict-aliasing -fPIC -Wall -Wno-switch -DFREEBSD -DHAVE_STRERROR -DHAVE_BSD_FLOCK -Wall -Qunused-arguments -Wno-parentheses-equality -Wno-array-bounds -Wno-unevaluated-expression -Werror -DXP_UNIX -UDEBUG -DNDEBUG -D_THREAD_SAFE -D_REENTRANT -DNSS_NO_INIT_SUPPORT -DUSE_UTIL_DIRECTLY -DNO_NSPR_10_SUPPORT -DSSL_DISABLE_DEPRECATED_CIPHER_SUITE_NAMES -I./../dist/FreeBSD11.1_OPT.OBJ/include -I./../dist/public/ -I./../dist/private/ -fPIC -Wall -Wno-switch -DFREEBSD -DHAVE_STRERROR -DHAVE_BSD_FLOCK -Wall -Qunused-arguments -Wno-parentheses-equality -Wno-array-bounds -Wno-unevaluated-expression -Werror -DXP_UNIX -UDEBUG -DNDEBUG -D_THREAD_SAFE -D_REENTRANT -DNSS_NO_INIT_SUPPORT -DUSE_UTIL_DIRECTLY -DNO_NSPR_10_SUPPORT -DSSL_DISABLE_DEPRECATED_CIPHER_SUITE_NAMES -I../../dist/FreeBSD11.1_OPT.OBJ/include -I../../dist/public/ -I../../dist/private/ -fPIC -Wall -Wno-switch -DFREEBSD -DHAVE_STRERROR -DHAVE_BSD_FLOCK -Wall -Qunused-arguments -Wno-parentheses-equality -Wno-array-bounds -Wno-unevaluated-expression -Werror -DXP_UNIX -DSHLIB_SUFFIX=\"so\" -DSHLIB_PREFIX=\"lib\" -DSHLIB_VERSION=\"3\" -DSOFTOKEN_SHLIB_VERSION=\"3\" -DRIJNDAEL_INCLUDE_TABLES -UDEBUG -DNDEBUG -D_THREAD_SAFE -D_REENTRANT -DNSS_NO_INIT_SUPPORT -DUSE_UTIL_DIRECTLY -DNO_NSPR_10_SUPPORT -DSSL_DISABLE_DEPRECATED_CIPHER_SUITE_NAMES -DNSS_USE_64 -DFREEBL_LOWHASH -DHAVE_INT128_SUPPORT -DMP_API_COMPATIBLE -I../../../dist/FreeBSD11.1_OPT.OBJ/include -I../../../dist/public/nss -I../../../dist/private/nss -Impi -Iecl -Iverified -fPIC -Wall -Wno-switch -DFREEBSD -DHAVE_STRERROR -DHAVE_BSD_FLOCK -Wall -Qunused-arguments -Wno-parentheses-equality -Wno-array-bounds -Wno-unevaluated-expression -Werror -DXP_UNIX -DSHLIB_SUFFIX=\"so\" -DSHLIB_PREFIX=\"lib\" -DSHLIB_VERSION=\"3\" -DSOFTOKEN_SHLIB_VERSION=\"3\" -DRIJNDAEL_INCLUDE_TABLES -UDEBUG -DNDEBUG -D_THREAD_SAFE -D_REENTRANT -DNSS_NO_INIT_SUPPORT -DUSE_UTIL_DIRECTLY -DNO_NSPR_10_SUPPORT -DSSL_DISABLE_DEPRECATED_CIPHER_SUITE_NAMES -DNSS_USE_64 -DFREEBL_LOWHASH -DHAVE_INT128_SUPPORT -DMP_API_COMPATIBLE -I../../../dist/FreeBSD11.1_OPT.OBJ/include -I../../../dist/public/nss -I../../../dist/private/nss -Impi -Iecl -Iverified -std=gnu99 blinit.c blinit.c:96:10: fatal error: 'sys/auxv.h' file not found #include <sys/auxv.h> ^~~~~~~~~~~~ 1 error generated. blinit.c:134:9: error: 'HWCAP_NEON' macro redefined [-Werror,-Wmacro-redefined] #define HWCAP_NEON (1 << 12) ^ /usr/include/machine/elf.h:134:9: note: previous definition is here #define HWCAP_NEON 0x00001000 ^ blinit.c:139:9: error: 'HWCAP2_AES' macro redefined [-Werror,-Wmacro-redefined] #define HWCAP2_AES (1 << 0) ^ /usr/include/machine/elf.h:148:9: note: previous definition is here #define HWCAP2_AES 0x00000001 ^ blinit.c:140:9: error: 'HWCAP2_PMULL' macro redefined [-Werror,-Wmacro-redefined] #define HWCAP2_PMULL (1 << 1) ^ /usr/include/machine/elf.h:149:9: note: previous definition is here #define HWCAP2_PMULL 0x00000002 ^ blinit.c:141:9: error: 'HWCAP2_SHA1' macro redefined [-Werror,-Wmacro-redefined] #define HWCAP2_SHA1 (1 << 2) ^ /usr/include/machine/elf.h:150:9: note: previous definition is here #define HWCAP2_SHA1 0x00000004 ^ blinit.c:142:9: error: 'HWCAP2_SHA2' macro redefined [-Werror,-Wmacro-redefined] #define HWCAP2_SHA2 (1 << 3) ^ /usr/include/machine/elf.h:151:9: note: previous definition is here #define HWCAP2_SHA2 0x00000008 ^ 5 errors generated. FreeBSD 11.1 aarch64 - https://clbin.com/WjroU (failed) FreeBSD 11.1 armv6 - https://clbin.com/IcVHP (failed) FreeBSD 12.0 aarch64 - https://clbin.com/VgG14 (succeeded) FreeBSD 12.0 armv6 - https://clbin.com/biLA1 (failed) FreeBSD 12.0 armv7 - https://clbin.com/j7oXG (failed)

Hmm, even fallback branch is broken: blinit.c:171:42: error: too few arguments to function call, expected 1, have 0 long hwcaps = getauxval(AT_HWCAP2); ~~~~~~~~~ ^ 1 error generated.

Not on Phabricator because of MFA shenanigans e.g., losing GitHub login. https://hg.mozilla.org/projects/nss-try/rev/951e8ffc461f2dec43a64ed9ad9877ca2c6fcf18

Err, wrong Try link: https://treeherder.mozilla.org/#/jobs?repo=nss-try&revision=951e8ffc461f2dec43a64ed9ad9877ca2c6fcf18

Comment on attachment 8956352 [details] [diff] [review] fix Review of attachment 8956352 [details] [diff] [review]: ----------------------------------------------------------------- Can you fix the clang-format error? Otherwise the patch looks fine.

Comment on attachment 8956352 [details] [diff] [review] fix https://treeherder.mozilla.org/#/jobs?repo=try&revision=64060f14baf1c51c74b7287532c352cb21d8097c (In reply to Franziskus Kiefer [:fkiefer or :franziskus] from comment #4) > Can you fix the clang-format error? Otherwise the patch looks fine. clang-format < 5.0 doesn't digest __has_include. /* clang-format off */ probably is the only option unless someone upgrades clang-format job to use the same version as ASAN job. $ cat a.c #ifndef __has_include #define __has_include(x) 0 #endif int main(void) { #if __has_include(<sys/auxv.h>) return 0; #else return 1; #endif } $ clang-format60 a.c | diff a.c - $ clang-format50 a.c | diff a.c - $ clang-format40 a.c | diff a.c - --- a.c 2018-03-06 00:00:00 +++ - 2018-03-06 00:00:01 @@ -3,7 +3,7 @@ #endif int main(void) { -#if __has_include(<sys/auxv.h>) +#if __has_include(<sys / auxv.h>) return 0; #else return 1; $ clang-format39 a.c | diff a.c - --- a.c 2018-03-06 00:00:00 +++ - 2018-03-06 00:00:01 @@ -3,7 +3,7 @@ #endif int main(void) { -#if __has_include(<sys/auxv.h>) +#if __has_include(<sys / auxv.h>) return 0; #else return 1;

https://treeherder.mozilla.org/#/jobs?repo=nss-try&revision=79dc542bab331d52007a3a2a88225e9e1a8cc269 https://treeherder.mozilla.org/#/jobs?repo=try&revision=b788f3994f6f31b3ecf2d4c824315f6db1aba6e5 Windows (on nss-try) is busted but I don't see the error message, only warnings in an unrelated file.

Comment on attachment 8956375 [details] [diff] [review] fix Review of attachment 8956375 [details] [diff] [review]: ----------------------------------------------------------------- Moving to clang-format 5 is too disruptive right now (also because of this bug https://bugs.llvm.org/show_bug.cgi?id=36611). So I guess disabling it here is the best we can do. Having it in the if though is not so nice. How about doing it around the entire pre-processor block (94-107)? And can you make the patch against NSS? Thanks.

https://treeherder.mozilla.org/#/jobs?repo=nss-try&revision=08071b7d9df574ddce3e6da08f568104a503c51a (In reply to Franziskus Kiefer [:fkiefer or :franziskus] from comment #7) > How about doing it around the entire pre-processor block (94-107)? Like this? > And can you make the patch against NSS? I don't understand. Is there another NSS repository besides https://hg.mozilla.org/projects/nss/ ? Or do you have an issue with patch format? Previous versions of the patch were already against NSS repo and "hg import" accepts git patches just fine.

Can you backport to NSS 3.36 branch? Firefox 60 would likely update bundled NSS to 3.36.1 (or later) during ESR cycle.

We can pick it up if we do a 3.36.1 release at some point. Not sure if that's going to happen though.

On the 3.36 branch for the 3.36.1 release https://hg.mozilla.org/projects/nss/rev/286c245ea50b6e901fbebe26c7cd4cf9587a5f07