702764 - Library loading times need to be adjusted on birch

Status: RESOLVED FIXED

(Core Graveyard :: Widget: Android, defect)

Description

[(dormant account)]

We should use RUSAGE_THREAD in getrusage. This will allow us to capture cpu time spent on library loading and compare it to overall cpu time.

Comment 1

[Mike Hommey [:glandium]]

Why birch only?

Comment 2

[(dormant account)]

(In reply to Mike Hommey [:glandium] from comment #1)
> Why birch only?

birch is my justification. This should happen on mc too.

Comment 3

[(dormant account)]

Note we also need another metric to see how long the process was alive before we started loading libraries, this will notify us of lag introduced by native ui.

Comment 4

[Doug Turner (:dougt)]

put it on m-c.  we can merge it in.

Comment 5

[Mike Hommey [:glandium]]

Attachment: part 1 - Use RUSAGE_THREAD for library loading time consumption reporting

Comment 6

[Mike Hommey [:glandium]]

Attachment: part 2 - Make nsAppStartup::GetStartupInfo more flexible

Comment 7

[Mike Hommey [:glandium]]

Attachment: part 3 - Add linker initialization and library loading timestamps for nsAppStartup::GetStartupInfo

Pushed to try:
https://tbpl.mozilla.org/?noignore=1&tree=Try&rev=d21392a69d35

Comment 8

[Mike Hommey [:glandium]]

Another try that doesn't fail due to the switch to mobile/xul tree:
https://tbpl.mozilla.org/?noignore=1&tree=Try&rev=09e355059af0

Comment 9

[(dormant account)]

Comment on attachment 575222 [details] [diff] [review]
part 2 - Make nsAppStartup::GetStartupInfo more flexible

RecordOnce is more expensive, but I understand why you do it

Comment 10

[Mike Hommey [:glandium]]

http://hg.mozilla.org/integration/mozilla-inbound/rev/80ddfdb7e575
http://hg.mozilla.org/integration/mozilla-inbound/rev/f9f4594a621b
http://hg.mozilla.org/integration/mozilla-inbound/rev/c744ccd70348

Comment 11

[Ed Morley [:emorley]]

https://hg.mozilla.org/mozilla-central/rev/80ddfdb7e575
https://hg.mozilla.org/mozilla-central/rev/f9f4594a621b
https://hg.mozilla.org/mozilla-central/rev/c744ccd70348