-
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/testshared: segfaults on linux/arm64 Fedora 29 #31823
Comments
Odd. The builder isn't reporting any problems. Can you reproduce the problem |
Yeah, it looks odd. Possibly something wrong with the binaries/runtime. Build/test log looks sane to me, although failing. This unfortunately flew under my radar for some time :(. For the record what are the versions of binutils, glibc and gcc on builders? To rule out that angle. |
CC @bradfitz: see question about tools versions on builders. Thanks. |
Actually, never mind, I can use gomote to find the tool versions. We're using gcc 4.9.2, binutils 2.25, glibc 2.19-18. So quite a bit older than the tools you are using. We are running Debian jessie. |
Thanks for sending the log output. Unfortunately I agree that it all looks normal. My first guess would of course be that the newer tools are causing a problem somehow, but I don't know why that would be. I think we'll need some way to reproduce the problem ourselves. |
I can add a Fedora builder, but in the meantime (still commuting) I just updated our debian-sid builder. It hadn't been updated in quite some time, and it gcc was pretty old. From 7.3.0 to 8.3.0:
Just deployed now. |
Change https://golang.org/cl/175737 mentions this issue: |
@ianlancetaylor Just did quick poke on it with gdb and the crash seems to happen during executable startup prior main function is invoked.
Intriguing is that the shared pie tests pass just fine. To note I will continue digging in to this next week as I will be mostly offline for next 7 days. |
@bradfitz Note that this bug is for arm64. I can't tell whether you updated the arm64 builder or not. |
Updates golang/go#31823 Change-Id: I96d83cf7a9994c8ed6d0ea58af038577d48ae737 Reviewed-on: https://go-review.googlesource.com/c/build/+/175737 Reviewed-by: Ian Lance Taylor <iant@golang.org>
I just did But sadly that didn't catch it. We do have enough arm64 capacity that we could run a Fedora arm64 Docker container on it, assuming that'd be enough and it's not a kernel issue. That's a bit more tedious so I likely won't get to it right away. |
I also tried Fedora 30 (also amd64) on a whim but it passed fine. I didn't try Fedora 29 on amd64, though. |
More digging in, this is actually dupe of #28334 and it has been/is binutils/gold issue https://sourceware.org/bugzilla/show_bug.cgi?id=23870 fixed on the trunk. |
Hi, Got the same issue on arm64, seems that it's caused (one of the reasons I think) by 'default pie' not being enabled, the issue could be reproduced by clang (6.0.0-1ubuntu2) and gcc without "--enable-default-pie" in my environment. I have a preliminary fix which introduces '-extldflags=-pie' in gc-toolchain's ld part under a few conditions (arm64, building linkshared, etc.), before finalizing the fix I'd like to ask for the expert's input if adding -extldflags is an option? Thanks. |
@shawn-xdji This issue was closed several months ago, so this is not the best place to discuss possible changes. Please open a new issue or send a message to golang-dev. Thanks. |
What version of Go are you using (
go version
)?Does this issue reproduce with the latest release?
Does reproduce on master, Go 1.12 and Go 1.11 on Fedora 29.
What operating system and processor architecture are you using (
go env
)?linux/arm64
What did you do?
I have built master branch with go1.11.6 on linux/arm64 host(Fedora 29) and
This is with
What did you expect to see?
Tests passing
What did you see instead?
The text was updated successfully, but these errors were encountered: