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: add eabi tag to arm binary #5640

Open
gopherbot opened this Issue Jun 5, 2013 · 9 comments

Comments

Projects
None yet
4 participants
@gopherbot

gopherbot commented Jun 5, 2013

by namsgorf:

What steps will reproduce the problem?

1. Run "readelf -A -- /path/to/go/binary"

What is the expected output?

Something like the following, which is the result of running the same command on
/bin/bash instead:

Attribute Section: aeabi
File Attributes
  Tag_CPU_name: "7-A"
  Tag_CPU_arch: v7
  Tag_CPU_arch_profile: Application
  Tag_ARM_ISA_use: Yes
  Tag_THUMB_ISA_use: Thumb-2
  Tag_FP_arch: VFPv3-D16
  Tag_ABI_PCS_wchar_t: 4
  Tag_ABI_FP_rounding: Needed
  Tag_ABI_FP_denormal: Needed
  Tag_ABI_FP_exceptions: Needed
  Tag_ABI_FP_number_model: IEEE 754
  Tag_ABI_align_needed: 8-byte
  Tag_ABI_align_preserved: 8-byte, except leaf SP
  Tag_ABI_enum_size: int
  Tag_ABI_HardFP_use: SP and DP
  Tag_ABI_VFP_args: VFP registers
  Tag_CPU_unaligned_access: v6

What do you see instead?

No output; zero exit status.

Which compiler are you using (5g, 6g, 8g, gccgo)?

N/A

Which operating system are you using?

Ubuntu Saucy

Which version are you using?  (run 'go version')

go version go1.1 linux/arm

Please provide any additional information below.

See https://bugs.launchpad.net/ubuntu/+source/golang/+bug/1187722 for a golang build
problem this caused.

The ARM Architecture ABI r2.09 Addenda specifies the "eabi" attribute tags. In
this case, a Tag_ABI_VFP_args setting of VFP would allow tools to detect the binary as
armhf. I think this could be achieved with:

    .eabi_attribute 28, 1

But note that this bug is a request to add all useful attributes in general, rather than
the specific issue that prompted this report.

I'm trying to find out if is a requirement that ARM executables specify this information
if they want to link with system libraries that do. Nevertheless, it would be useful as
a wishlist item for the go binaries to provide this information so that distribution
tooling can make use of it.
@bradfitz

This comment has been minimized.

Member

bradfitz commented Jun 18, 2013

Comment 2:

Labels changed: removed priority-triage.

Status changed to Accepted.

@davecheney

This comment has been minimized.

Contributor

davecheney commented Jun 27, 2013

Comment 3:

Adding myself as CC as I am responsible for the bug on the Canonical side of the house.
@davecheney

This comment has been minimized.

Contributor

davecheney commented Jun 30, 2013

Comment 4:

Started, https://golang.org/cl/10171043

Owner changed to @minux.

Status changed to Started.

@rsc

This comment has been minimized.

Contributor

rsc commented Jul 30, 2013

Comment 5:

Labels changed: added go1.2maybe.

@rsc

This comment has been minimized.

Contributor

rsc commented Jul 30, 2013

Comment 6:

Labels changed: added feature.

@rsc

This comment has been minimized.

Contributor

rsc commented Sep 9, 2013

Comment 7:

Labels changed: added go1.3maybe, removed go1.2maybe.

@rsc

This comment has been minimized.

Contributor

rsc commented Nov 27, 2013

Comment 8:

Labels changed: removed feature.

@rsc

This comment has been minimized.

Contributor

rsc commented Dec 4, 2013

Comment 9:

Labels changed: added release-none, removed go1.3maybe.

@rsc

This comment has been minimized.

Contributor

rsc commented Dec 4, 2013

Comment 10:

Labels changed: added repo-main.

@rsc rsc added this to the Unplanned milestone Apr 10, 2015

@rsc rsc removed release-none labels Apr 10, 2015

@rsc rsc changed the title from cmd/5l: add eabi tag to binary to cmd/link: add eabi tag to arm binary Jun 8, 2015

@bradfitz bradfitz removed the Started label Jan 6, 2017

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment