Closed Bug 704898 Opened 8 years ago Closed 8 years ago

Enable elfhack

Categories

(Core :: Widget: Android, defect)

ARM
Android
defect
Not set

Tracking

()

RESOLVED FIXED

People

(Reporter: glandium, Assigned: dougt)

References

Details

(Whiteboard: [mobilestartupshrink])

Attachments

(1 file)

Bug 701996 broke elfhack, and elfhack was disabled as a result. Bug 703305 fixed elfhack, so we should re-enable it. Especially considering this:

11-23 19:57:42.208 E/GeckoLibLoad( 9085): Loaded libs in 1268ms total, 742ms user, 183ms system, 13 faults
11-23 19:57:42.208 E/GeckoLibLoad( 9085): Spent 273ms on relocations, 3ms on constructors

With elfhack, relocations time goes near 0, and constructors time stays under 30ms (iirc). It also makes the mount of data to uncompress smaller, which means faster decompression.

A nightly with elfhack enabled, on the same device, loads libraries under 900ms.
Attached patch patch v.1Splinter Review
Assignee: nobody → doug.turner
Attachment #576638 - Flags: review?
Attachment #576638 - Flags: review? → review+
http://hg.mozilla.org/projects/birch/rev/ea259c780d05
Status: NEW → RESOLVED
Closed: 8 years ago
Resolution: --- → FIXED
Better with elfhack :)
E/GeckoLibLoad(25781): Loaded libs in 910ms total, 472ms user, 152ms system, 2 faults
E/GeckoLibLoad(25781): Spent 2ms on relocations, 14ms on constructors
Status: RESOLVED → REOPENED
Resolution: FIXED → ---
I didn't mean to reopen the bug. Sorry.
Status: REOPENED → RESOLVED
Closed: 8 years ago8 years ago
Resolution: --- → FIXED
Blocks: 713622
You need to log in before you can comment on or make changes to this bug.