-
Notifications
You must be signed in to change notification settings - Fork 17.9k
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
misc/cgo: shared libs tests fail on arm64 with segmentation fault [1.10 backport] #24873
Comments
I forgot to mention, this only started happening in 1.10.1, builds were fine until 1.10: https://buildd.debian.org/status/logs.php?pkg=golang-1.10&arch=arm64 |
https://go-review.googlesource.com/c/go/+/104815 fixed this, I never got around to filing a bug. We could use this bug to track the backporting (or not) to 1.10 though... |
Oh yeah, and it's not a change in Go that caused this to start happening, it's a change in glibc (specifically the crt0.o object that all binaries are linked with) |
Adding to 1.10.2 milestone for discussion. @ianlancetaylor |
I'm OK with backporting CL 104815 to 1.10.2, to keep up with glibc changes. |
@mwhudson can you please mail the cherry-pick per the (new) process at https://github.com/golang/go/wiki/MinorReleases? Thank you! |
Change https://golang.org/cl/110035 mentions this issue: |
…when -linkshared is passed As the comment above the code I'm changing says, when building with -buildmode=exe, the default compiler flags produce code incompatible with PIE. But when -linkshared is passed, the default compiler flags are not used so this does not apply. And now I've found a system (linux/arm64 with glibc 2.27) where this combination of flags causes a problem, albeit for reasons I don't really understand, so stop passing -no-pie when -linkshared is passed. Fixes #24873 Change-Id: I412ec7941dc0cb89e6d1b171fc29288aadcb9f20 Reviewed-on: https://go-review.googlesource.com/104815 Run-TryBot: Michael Hudson-Doyle <michael.hudson@canonical.com> TryBot-Result: Gobot Gobot <gobot@golang.org> Reviewed-by: Ian Lance Taylor <iant@golang.org> (cherry picked from commit 2d26a65) Reviewed-on: https://go-review.googlesource.com/110035 Run-TryBot: Andrew Bonventre <andybons@golang.org>
On 24 April 2018 at 10:40, Filippo Valsorda ***@***.***> wrote:
@mwhudson <https://github.com/mwhudson> can you please mail the
cherry-pick per the (new) process at https://github.com/golang/go/
wiki/MinorReleases? Thank you!
Sorry I never got around to this (life being lifey) and thanks for taking
care of it anyway :)
|
What version of Go are you using (
go version
)?go version go1.10.1 linux/arm64
Does this issue reproduce with the latest release?
Yes
What operating system and processor architecture are you using (
go env
)?GOARCH="arm64"
GOHOSTARCH="arm64"
GOHOSTOS="linux"
GOOS="linux"
What did you do?
Building the golang Debian package, misc/cgo/testshared tests fail under arm64 with segmentation fault when trying to execute the test binaries. This is an excerpt from running the tests manually:
Running the binaries under gdb fails immediately without any useful stack trace, seems like a linking issue:
The text was updated successfully, but these errors were encountered: