Skip to content
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

crypto/ecdsa, crypto/tls, crypto/x509: many crypto tests are failing on s390x #70771

Closed
srinivas-pokala opened this issue Dec 11, 2024 · 5 comments
Labels
arch-s390x Issues solely affecting the s390x architecture. NeedsFix The path to resolution is known, but the work has not been done.
Milestone

Comments

@srinivas-pokala
Copy link
Contributor

Go version

go version devel go1.24-a9922d096f Wed Dec 11 04:00:12 2024 +0000 linux/s390x

Output of go env in your module/workspace:

AR='ar'
CC='gcc'
CGO_CFLAGS='-O2 -g'
CGO_CPPFLAGS=''
CGO_CXXFLAGS='-O2 -g'
CGO_ENABLED='1'
CGO_FFLAGS='-O2 -g'
CGO_LDFLAGS='-O2 -g'
CXX='g++'
GCCGO='gccgo'
GO111MODULE=''
GOARCH='s390x'
GOAUTH='netrc'
GOBIN='/root/go/bin'
GOCACHE='/root/.cache/go-build'
GODEBUG=''
GOENV='/root/.config/go/env'
GOEXE=''
GOEXPERIMENT=''
GOFIPS140='off'
GOFLAGS=''
GOHOSTARCH='s390x'
GOHOSTOS='linux'
GOINSECURE=''
GOMOD='/root/go_compiler/dev/go_src/src/go.mod'
GOMODCACHE='/root/go/pkg/mod'
GONOPROXY=''
GONOSUMDB=''
GOOS='linux'
GOPATH='/root/go'
GOPRIVATE=''
GOPROXY='https://proxy.golang.org,direct'
GOROOT='/root/go_compiler/dev/go_src'
GOSUMDB='sum.golang.org'
GOTELEMETRY='local'
GOTELEMETRYDIR='/root/.config/go/telemetry'
GOTMPDIR=''
GOTOOLCHAIN='auto'
GOTOOLDIR='/root/go_compiler/dev/go_src/pkg/tool/linux_s390x'
GOVCS=''
GOVERSION='devel go1.24-a9922d096f Wed Dec 11 04:00:12 2024 +0000'
GOWORK=''
PKG_CONFIG='pkg-config'

What did you do?

cd src/crypto/ecdsa/
go test -v
cd src/crypto/tls
go test -v
cd src/crypto/x509
go test -v

What did you see happen?

ECDSA:
--- FAIL: TestSignAndVerify (0.15s)
--- FAIL: TestSignAndVerifyASN1 (0.09s)
--- FAIL: TestVectors (0.46s)
--- FAIL: TestZeroHashSignature (0.07s)
--- FAIL: TestRFC6979 (0.05s)

TLS:
--- FAIL: TestBogoSuite (0.66s)
--- FAIL: TestFIPSServerSignatureAndHash (0.52s)
--- FAIL: TestHandshakeClientECDHEECDSAAES (0.00s)
--- FAIL: TestHandshakeClientECDHEECDSAAESGCM (0.00s)
--- FAIL: TestHandshakeClientAES256GCMSHA384 (0.00s)
--- FAIL: TestHandshakeClientECDHEECDSAAES128CBCSHA256 (0.00s)
--- FAIL: TestHandshakeClientECDHEECDSAChaCha20 (0.00s)
--- FAIL: TestHandshakeClientECDSATLS13 (0.00s)
--- FAIL: TestHandshakeClientCertRSA (0.00s)
--- FAIL: TestHandshakeClientCertECDSA (0.00s)
--- FAIL: TestHandshakeServerECDHEECDSAAES (0.00s)
--- FAIL: TestClientAuth (0.00s)

x509:
--- FAIL: TestECDSA (0.00s)
--- FAIL: TestCreateCertificateRequest (0.06s)

What did you expect to see?

All tests pass

@srinivas-pokala srinivas-pokala added the arch-s390x Issues solely affecting the s390x architecture. label Dec 11, 2024
@srinivas-pokala
Copy link
Contributor Author

#70660

@srinivas-pokala
Copy link
Contributor Author

/cc @FiloSottile

@cagedmantis cagedmantis added the NeedsInvestigation Someone must examine and confirm this is a valid issue and not a duplicate of an existing one. label Dec 11, 2024
@cagedmantis cagedmantis changed the title crypto/ecdsa, crypto/tls, crypto/x509: Lot of crypto tests are failing on s390x crypto/ecdsa, crypto/tls, crypto/x509: many crypto tests are failing on s390x Dec 11, 2024
@cagedmantis
Copy link
Contributor

@gopherbot
Copy link
Contributor

Change https://go.dev/cl/636015 mentions this issue: crypto/ecdsa: fix s390x assembly with P-521

@dmitshur dmitshur added NeedsFix The path to resolution is known, but the work has not been done. and removed NeedsInvestigation Someone must examine and confirm this is a valid issue and not a duplicate of an existing one. labels Jan 8, 2025
wyf9661 pushed a commit to wyf9661/go that referenced this issue Jan 21, 2025
I had incorrectly assumed that the blocksize was always the same as the
curve field size. This is true of P-256 and P-384, but not P-521.

Fixes golang#70660
Fixes golang#70771

Change-Id: Idb6b510fcd3dd42d9b1e6cf42c1bb92e0ce8bd07
Reviewed-on: https://go-review.googlesource.com/c/go/+/636015
Run-TryBot: Filippo Valsorda <[email protected]>
Reviewed-by: Carlos Amedee <[email protected]>
LUCI-TryBot-Result: Go LUCI <[email protected]>
Auto-Submit: Filippo Valsorda <[email protected]>
TryBot-Result: Gopher Robot <[email protected]>
Reviewed-by: Roland Shoemaker <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
arch-s390x Issues solely affecting the s390x architecture. NeedsFix The path to resolution is known, but the work has not been done.
Projects
None yet
Development

No branches or pull requests

5 participants