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

cmd/link: incorrect relocations during external linking on linux/386 with musl libc #14478

Closed
minux opened this issue Feb 23, 2016 · 8 comments
Closed
Labels
FrozenDueToAge NeedsInvestigation Someone must examine and confirm this is a valid issue and not a duplicate of an existing one. WaitingForInfo Issue is not actionable because of missing required information, which needs to be provided.
Milestone

Comments

@minux
Copy link
Member

minux commented Feb 23, 2016

See #14476 (comment)

Tentatively labeled Go1.6.1, but it probably doesn't meet the line.

@minux minux added this to the Go1.6.1 milestone Feb 23, 2016
@bradfitz bradfitz modified the milestones: Go1.6.1, Go1.6.2 Apr 7, 2016
@bradfitz bradfitz changed the title cmd/link: external linking broken on linux/386 with mucl libc cmd/link: external linking broken on linux/386 with musl libc Apr 7, 2016
@rsc
Copy link
Contributor

rsc commented Apr 7, 2016

Is this a tracking bug for the other being in 1.6.1 or a separate bug? If it's a separate bug maybe it can wait until 1.6.3.

@rsc rsc added the Release-OK label Apr 7, 2016
@minux
Copy link
Member Author

minux commented Apr 7, 2016 via email

@adg adg modified the milestones: Go1.6.3, Go1.6.2 Apr 19, 2016
@adg adg removed the Release-OK label Apr 19, 2016
@rsc
Copy link
Contributor

rsc commented May 23, 2016

Given there's no fix, it seems like this should be at least investigated for Go 1.7. It won't be in Go 1.6.3 (which probably won't happen).

@rsc rsc modified the milestones: Go1.7, Go1.6.3 May 23, 2016
@quentinmit quentinmit added the NeedsInvestigation Someone must examine and confirm this is a valid issue and not a duplicate of an existing one. label May 26, 2016
@rsc rsc modified the milestones: Go1.7Maybe, Go1.7 May 27, 2016
@ianlancetaylor
Copy link
Contributor

This would be nice to fix but it doesn't seem to be happening. Dropping to 1.8.

@ianlancetaylor ianlancetaylor modified the milestones: Go1.8, Go1.7Maybe Jun 9, 2016
@rsc
Copy link
Contributor

rsc commented Oct 25, 2016

The relevant part of Minux's comment from #14476:

An externally linked hello world program couldn't even start:
(gdb) r
Starting program: /root/go.git/src/hello

Program received signal SIGSEGV, Segmentation fault.
0xb7f98656 in do_relocs (dso=dso@entry=0xb7ffd660 , rel=0x80000910,
rel_size=145784, stride=2)
at src/ldso/dynlink.c:407
407 src/ldso/dynlink.c: No such file or directory.
(gdb) bt
#0 0xb7f98656 in do_relocs (dso=dso@entry=0xb7ffd660 ,
rel=0x80000910, rel_size=145784,
stride=2) at src/ldso/dynlink.c:407
#1 0xb7f98ea7 in reloc_all (p=p@entry=0xb7ffd660 ) at
src/ldso/dynlink.c:1115
#2 0xb7f9a5e3 in __dls3 (sp=0xbffffd60) at src/ldso/dynlink.c:1598
#3 0xb7f99e5c in __dls2 (base=0xb7f7b000 "\177ELF\001\001\001",
sp=0xbffffd60)
at src/ldso/dynlink.c:1389
#4 0xb7f97ec5 in _dlstart_c (sp=0xbffffd60, dynv=0xb7ffcef8) at
src/ldso/dlstart.c:147
#5 0xb7f97dbb in _dlstart () from /lib/ld-musl-i386.so.1

Something is wrong with the relocations.

@rsc rsc modified the milestones: Go1.8Maybe, Go1.8 Oct 25, 2016
@rsc rsc changed the title cmd/link: external linking broken on linux/386 with musl libc cmd/link: incorrect relocations during external linking on linux/386 with musl libc Oct 25, 2016
@rsc
Copy link
Contributor

rsc commented Oct 25, 2016

This doesn't seem like it is affecting many people. Moving to Go1.8Maybe.

@rsc rsc modified the milestones: Go1.9, Go1.8Maybe Nov 2, 2016
@bradfitz bradfitz added this to the Go1.10 milestone Jun 7, 2017
@bradfitz bradfitz removed this from the Go1.9 milestone Jun 7, 2017
@ianlancetaylor
Copy link
Contributor

Is this still a problem?

@ianlancetaylor ianlancetaylor added the WaitingForInfo Issue is not actionable because of missing required information, which needs to be provided. label Jun 7, 2017
@gopherbot
Copy link

Timed out in state WaitingForInfo. Closing.

(I am just a bot, though. Please speak up if this is a mistake or you have the requested information.)

@golang golang locked and limited conversation to collaborators Jul 7, 2018
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
FrozenDueToAge NeedsInvestigation Someone must examine and confirm this is a valid issue and not a duplicate of an existing one. WaitingForInfo Issue is not actionable because of missing required information, which needs to be provided.
Projects
None yet
Development

No branches or pull requests

7 participants