Closed Bug 1867581 Opened 1 year ago Closed 1 year ago

When run through profileserver.py, speedometer3 hangs at 57/610 test case and hits 404

Categories

(Core :: Performance, defect)

defect

Tracking

()

RESOLVED FIXED
122 Branch
Tracking Status
firefox122 --- fixed

People

(Reporter: sergesanspaille, Assigned: mleclair)

References

(Regressed 1 open bug)

Details

Attachments

(2 files)

Since bug 1862309 , speedometer3 is part of the PGO training set, and it has already improved the speedometer3 score to do so.

However, when running the training, the test actually hangs at 56/610, hits a 404 and then timeouts, moving to next test. It would be great to have it run until the end.

Steps to reproduce

$ mach build
$ mach python ./build/pgo/profileserver.py -b $PWD/obj-x86_64-pc-linux-gnu/dist/bin/firefox-bin

And keep an eye on the window that pops up.

See Also: → 1862309

I see a 404 on macOS also, although interestingly I see it at 57/610 (React-Stockcharts-SVG).

The command line for macOS is slightly different (use the binary under dist/Nightly.app not dist/bin):

$ mach python ./build/pgo/profileserver.py -b $PWD/obj-aarch64-apple-darwin23.1.0/dist/Nightly.app/Contents/MacOS/firefox-bin
OS: Unspecified → All
Hardware: Unspecified → All

This is because some of the files ( I think some) has to be ran as root. I have a fix, but it is failing in ci

Assignee: nobody → mleclair

Here's another try push with some logs and running all of the PGO tests.

Pushed by mleclair@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/5e7ca7b9c60d Fixed PGO training set not working for SP3 since SP3 wasn't running as root r=bas
Status: NEW → RESOLVED
Closed: 1 year ago
Resolution: --- → FIXED
Target Milestone: --- → 122 Branch

Thanks a lot Marc for tracking this down. Did you run some benchmark to compare the impact of running the full speedometer3 suite on the quality of the profiled binary?

Regressions: 1870079
Regressions: 1870279

Looks like this caused a few small improvements on DevTools tests, not sure why though

== Change summary for alert #40647 (as of Mon, 11 Dec 2023 23:46:24 GMT) ==

Improvements:

Ratio Test Platform Options Absolute values (old vs new)
3% damp custom.jsdebugger.open-large-minified-file.DAMP macosx1015-64-shippable-qr e10s fission stylo webrender 1,333.64 -> 1,298.78
2% damp source-map.eachMapping.DAMP windows10-64-shippable-qr e10s fission stylo webrender 257.47 -> 251.19
2% damp custom.jsdebugger.pretty-print.reload-and-pause.DAMP macosx1015-64-shippable-qr e10s fission stylo webrender 7,930.54 -> 7,765.64
2% damp custom.pretty-print.jsdebugger.reload.DAMP macosx1015-64-shippable-qr e10s fission stylo webrender 7,768.59 -> 7,608.71
2% damp custom.jsdebugger.pretty-print.DAMP macosx1015-64-shippable-qr e10s fission stylo webrender 1,777.32 -> 1,741.13

For up to date results, see: https://treeherder.mozilla.org/perfherder/alerts?id=40647

You need to log in before you can comment on or make changes to this bug.

Attachment

General

Creator:
Created:
Updated:
Size: