1540135 - Add a startup mochitest tracking mainthread I/O

Status: RESOLVED FIXED

(Firefox :: General, enhancement, P1)

Description

[Florian Quèze [:florian]]

We know main thread I/O is terrible for (cold) startup performance. Despite working on removing existing main thread I/O syscalls during startup, we regularly see regressions (eg. bug 1527704, bug 1538250, and even the older bug 1527879).

Comment 1

[Florian Quèze [:florian]]

Attachment: Bug 1540135 - Add a startup mochitest tracking mainthread I/O, r=felipe.

Comment 2

[Florian Quèze [:florian]]

https://treeherder.mozilla.org/#/jobs?repo=try&revision=fb38eeb912e2f73ea114670da89bc923f8df3abd is green :-)

Comment 3

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/eb8757f9586f
Add a startup mochitest tracking mainthread I/O, r=Felipe.

Comment 4

[Florian Quèze [:florian]]

Attachment: Fix for asan builds

Comment 5

[Mike Conley (:mconley) (:⚙️) (PTO: May 24 - June 2nd)]

Comment on attachment 9063311 [details] [diff] [review]
Fix for asan builds

Looks okay - hopefully this fixes the build!

Comment 6

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/5add0a9fd0fb
follow up to fix browser_startup_mainthreadio.js on asan builds where the crash reporter is disabled, r=mconley CLOSED TREE

Comment 7

[Natalia Csoregi [:nataliaCs]]

https://hg.mozilla.org/mozilla-central/rev/eb8757f9586f
https://hg.mozilla.org/mozilla-central/rev/5add0a9fd0fb

Comment 8

[Alexandru Michis [:malexandru]]

Backed out changeset 5add0a9fd0fb (Bug 1540135) for causing leaks

Backout link: https://hg.mozilla.org/integration/autoland/rev/7602b3fa01c780c9159329c9f99202b7c26462bb

Push with failures: https://treeherder.mozilla.org/#/jobs?repo=autoland&resultStatus=success%2Ctestfailed%2Cbusted%2Cexception%2Cretry%2Cusercancel%2Crunnable&group_state=expanded&searchStr=browser-chrome&revision=5add0a9fd0fb6f0f50df29e04b52a16b2975c373

Log link: https://treeherder.mozilla.org/logviewer.html#?job_id=245283117&repo=autoland

Comment 9

[Pulsebot]

Backout by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/7602b3fa01c7
Backed out changeset 5add0a9fd0fb for causing leaks

Comment 10

[Alexandru Michis [:malexandru]]

Backed out eb8757f9586f (Bug 1540135) for causing leaks

Backout link: https://hg.mozilla.org/integration/autoland/rev/5ea2c049391367f5840bdf0f74d4e820733dae45

Push with failures: https://treeherder.mozilla.org/#/jobs?repo=autoland&resultStatus=success%2Ctestfailed%2Cbusted%2Cexception%2Cretry%2Cusercancel%2Crunnable&group_state=expanded&searchStr=browser-chrome&revision=72723e7257b1f9825a5f1a6478c0dbdea9202c97

Log link: https://treeherder.mozilla.org/logviewer.html#?job_id=245184064&repo=autoland

Comment 11

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/7785e3f08f85
Add a startup mochitest tracking mainthread I/O, r=Felipe.

Comment 12

[Oana Pop-Rus]

https://hg.mozilla.org/mozilla-central/rev/7785e3f08f85