Closed Bug 729883 Opened 10 years ago Closed 9 years ago

Avoid linker warnings when starting up on Android

Categories

(Core :: mozglue, defect)

ARM
Android
defect
Not set
normal

Tracking

()

RESOLVED FIXED
mozilla13

People

(Reporter: glandium, Assigned: glandium)

References

Details

Attachments

(1 file)

The new linker outputs warnings when it sees dynamic entry types it doesn't handle. It was done on purpose to avoid surprises. The entry types that the linker emits warnings for are:
DT_PLTREL, DT_SONAME, DT_SYMBOLIC, DT_RELCOUNT, DT_VERSYM, DT_VERDEF, DT_VERDEFNUM, DT_VERNEED, DT_VERNEEDNUM.

DT_PLTREL can be checked. DT_SONAME could, but it's really not important so we can ignore it. DT_RELCOUNT indicates how many relocations are relative (and thus how many can be skipped when the binary is prelinked), which is useless information for us, and DT_VER* types are for symbol versioning, which is not supported. DT_SYMBOLIC is a flag to modify the way symbols are resolved.

We can make a conscious choice of ignoring them (which, in fact, we currently do), as long as we know why we do.
Attachment #599949 - Flags: review?(taras.mozilla)
Assignee: nobody → mh+mozilla
Comment on attachment 599949 [details] [diff] [review]
Avoid linker warnings when starting up on Android

might be some value in leaving some of these on in debug builds. your call.
Attachment #599949 - Flags: review?(taras.mozilla) → review+
https://hg.mozilla.org/mozilla-central/rev/962c92bbed02
Status: NEW → RESOLVED
Closed: 9 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla13
You need to log in before you can comment on or make changes to this bug.