New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
x/crypto/blake2b: supportSSE4 detection wrong, SIGILL on Atom D2700 #18113
Labels
Milestone
Comments
bradfitz
changed the title
Invalid golang.org/x/crypto/blake2b.supportSSE4 detection for Atom D2700
x/crypto/blake2b: supportSSE4 detection wrong for Atom D2700
Nov 30, 2016
bradfitz
changed the title
x/crypto/blake2b: supportSSE4 detection wrong for Atom D2700
x/crypto/blake2b: supportSSE4 detection wrong, SIGILL on Atom D2700
Nov 30, 2016
/cc @agl |
Thanks for the report. Does simply changing the 15 to 19 (as in this fix) work for you? |
CL https://golang.org/cl/33830 mentions this issue. |
Yes, 19-bit solves the problem (it correctly detects SSS4 absence and does not try to use that optimized instructions). |
Thank you! |
c-expert-zigbee
pushed a commit
to c-expert-zigbee/crypto_go
that referenced
this issue
Mar 28, 2022
I missed this in the initial review but this will cause blake2b to SIGILL on, at least, some Atom chips. Fixes golang/go#18113. Change-Id: I0c16319df71f3ce4a38411ad4c4e9d3871fd9533 Reviewed-on: https://go-review.googlesource.com/33830 Run-TryBot: Adam Langley <agl@golang.org> TryBot-Result: Gobot Gobot <gobot@golang.org> Reviewed-by: Brad Fitzpatrick <bradfitz@golang.org>
c-expert-zigbee
added a commit
to c-expert-zigbee/crypto_go
that referenced
this issue
Mar 29, 2022
I missed this in the initial review but this will cause blake2b to SIGILL on, at least, some Atom chips. Fixes golang/go#18113. Change-Id: I0c16319df71f3ce4a38411ad4c4e9d3871fd9533 Reviewed-on: https://go-review.googlesource.com/33830 Run-TryBot: Adam Langley <agl@golang.org> TryBot-Result: Gobot Gobot <gobot@golang.org> Reviewed-by: Brad Fitzpatrick <bradfitz@golang.org>
LewiGoddard
pushed a commit
to LewiGoddard/crypto
that referenced
this issue
Feb 16, 2023
I missed this in the initial review but this will cause blake2b to SIGILL on, at least, some Atom chips. Fixes golang/go#18113. Change-Id: I0c16319df71f3ce4a38411ad4c4e9d3871fd9533 Reviewed-on: https://go-review.googlesource.com/33830 Run-TryBot: Adam Langley <agl@golang.org> TryBot-Result: Gobot Gobot <gobot@golang.org> Reviewed-by: Brad Fitzpatrick <bradfitz@golang.org>
BiiChris
pushed a commit
to BiiChris/crypto
that referenced
this issue
Sep 15, 2023
I missed this in the initial review but this will cause blake2b to SIGILL on, at least, some Atom chips. Fixes golang/go#18113. Change-Id: I0c16319df71f3ce4a38411ad4c4e9d3871fd9533 Reviewed-on: https://go-review.googlesource.com/33830 Run-TryBot: Adam Langley <agl@golang.org> TryBot-Result: Gobot Gobot <gobot@golang.org> Reviewed-by: Brad Fitzpatrick <bradfitz@golang.org>
Sign up for free
to subscribe to this conversation on GitHub.
Already have an account?
Sign in.
I am using Arch Linux x86-64 with 4.8.10-1-arch Linux kernel. I am using some code that runs blake2b, but it fails with the following error:
seems that it is trying to call SSE4.1-related assembler instruction, but host CPU does not support it.
supportSSE4 assembler code has the following line:
but Wikipedia says that SSE4.1 is indicated by 19-bit: https://en.wikipedia.org/wiki/SSE4#SSE4.1
I have never programmed in assembler, so possibly this is false assumption.
The text was updated successfully, but these errors were encountered: