#63622 closed defect (fixed)
openssl3: buildbot failures related to AVX-512 instruction set
Reported by: | mascguy (Christopher Nielsen) | Owned by: | mascguy (Christopher Nielsen) |
---|---|---|---|
Priority: | Normal | Milestone: | |
Component: | ports | Version: | |
Keywords: | avx | Cc: | kencu (Ken), cjones051073 (Chris Jones) |
Port: | openssl3 |
Description
This port is failing to build on various buildbots, with errors related to use of the AVX-512 instruction set:
crypto/bn/libcrypto-lib-rsaz-avx512.o crypto/bn/rsaz-avx512.s crypto/bn/rsaz-avx512.s:36:2: error: instruction requires: AVX-512 ISA AVX-512 VL ISA vpxord %ymm0,%ymm0,%ymm0 ^ crypto/bn/rsaz-avx512.s:37:2: error: instruction requires: AVX-512 ISA AVX-512 VL ISA vmovdqa64 %ymm0,%ymm1 ^ crypto/bn/rsaz-avx512.s:38:2: error: instruction requires: AVX-512 ISA AVX-512 VL ISA vmovdqa64 %ymm0,%ymm16 ^ crypto/bn/rsaz-avx512.s:39:2: error: instruction requires: AVX-512 ISA AVX-512 VL ISA vmovdqa64 %ymm0,%ymm17 ^
This is occurring with Xcode Clang, and is fixable with an appropriate blacklist. A commit to follow shortly.
Change History (9)
comment:1 Changed 3 years ago by Christopher Nielsen <mascguy@…>
Resolution: | → fixed |
---|---|
Status: | assigned → closed |
comment:2 follow-up: 4 Changed 3 years ago by ryandesign (Ryan Carsten Schmidt)
which means that newer MacPorts clangs won't be able to have openssl3 in their dependency chain.
comment:3 Changed 3 years ago by mascguy (Christopher Nielsen)
I'm reasonably certain that AVX-512 instruction use can be disabled, if necessary. But my immediate goal was simply to eliminate build failures; didn't have time to look at anything more involved.
comment:4 Changed 3 years ago by mascguy (Christopher Nielsen)
Resolution: | fixed |
---|---|
Status: | closed → reopened |
Replying to ryandesign:
which means that newer MacPorts clangs won't be able to have openssl3 in their dependency chain.
Reopening, to cover the work necessary to eliminate blacklisting of Xcode Clangs. Hopefully that's doable, by disabling use of the AVX* instruction set.
comment:5 follow-up: 6 Changed 3 years ago by cjones051073 (Chris Jones)
This is really an upstream issue. See
https://github.com/openssl/openssl/issues/16670
in short we should see with future openssl 3 releases if the black-listing can just be removed.
comment:6 Changed 3 years ago by mascguy (Christopher Nielsen)
Cc: | kencu cjones051073 added |
---|
Replying to cjones051073:
This is really an upstream issue. See
https://github.com/openssl/openssl/issues/16670
in short we should see with future openssl 3 releases if the black-listing can just be removed.
Great to know!
As for the near-term: @kencu, we could use some guidance here. Any thoughts/suggestions, per Ryan's observation in comment:2?
comment:7 Changed 3 years ago by cjones051073 (Chris Jones)
If you want a shorter term fix there is a suggestion in the above issue as to a change to make to address it https://github.com/openssl/openssl/issues/16670#issuecomment-935184461
comment:8 Changed 3 years ago by Chris Jones <jonesc@…>
Resolution: | → fixed |
---|---|
Status: | reopened → closed |
In 02db4e7a363141c67f384f1f29db50fb2f70e53e/macports-ports (master):