Closed Bug 1945683 Opened 1 year ago Closed 5 months ago

Enable layer compositor on Windows in Nightly

Categories

(Core :: Graphics: WebRender, task)

task

Tracking

()

RESOLVED FIXED
144 Branch
Tracking Status
firefox144 --- fixed

People

(Reporter: gw, Assigned: sotaro)

References

(Blocks 1 open bug, Regressed 1 open bug)

Details

(Keywords: perf-alert)

Attachments

(1 file, 1 obsolete file)

No description provided.
Assignee: nobody → gwatson
Status: NEW → ASSIGNED

Sheriffs - this patch enables a very significant change to how we composite all pages on Windows. It passes all tests in CI, and seems OK in general browsing. However, given the scale of the change, I am expecting some regressions to occur when enabled in nightly. If / when this occurs, please just back this patch out (it's deliberately a very small patch to just enable a preference so it's easy to back out) as any regressions are likely to (a) affect a lot of users and (b) take me a while to investigate and resolve.

Pushed by gwatson@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/9d8b4c0b99ed Enable layer compositor on Windows r=gfx-reviewers,lsalzman

Backed out for causing btime failures.

Flags: needinfo?(gwatson)

Patch to fix this is in 1946269 - will reland this once that passes try and lands.

Flags: needinfo?(gwatson)
See Also: → 1946487
Regressions: 1946657
Regressions: 1946766

(In reply to Serban Stanca [:SerbanS] from comment #4)

Backed out for causing btime failures.

Perfherder has detected a talos performance change from push e96db7f2fc84bf31d8081316be884a96496fecb6.

Improvements:

Ratio Test Platform Options Absolute values (old vs new)
86% displaylist_mutate windows11-64-shippable-qr e10s fission stylo webrender 9,969.93 -> 1,349.22
14% rasterflood_gradient windows11-64-shippable-qr e10s fission stylo webrender 1,201.08 -> 1,370.33
11% rasterflood_svg windows11-64-shippable-qr e10s fission stylo webrender 10,012.49 -> 8,871.36

Details of the alert can be found in the alert summary, including links to graphs and comparisons for each of the affected tests.

If you need the profiling jobs you can trigger them yourself from treeherder job view or ask a sheriff to do that for you.

You can run these tests on try with ./mach try perf --alert 43708

For more information on performance sheriffing please see our FAQ.

Keywords: perf-alert

(In reply to Serban Stanca [:SerbanS] from comment #4)

Backed out for causing btime failures.

Perfherder has detected a browsertime performance change from push e96db7f2fc84bf31d8081316be884a96496fecb6.

Regressions:

Ratio Test Platform Options Absolute values (old vs new) Performance Profiles
11% speedometer Vanilla-ES2015-TodoMVC/DeletingItems/Async windows11-64-shippable-qr fission webrender 0.20 -> 0.22 Before/After
5% speedometer Vanilla-ES2015-TodoMVC/DeletingItems/Sync windows11-64-shippable-qr fission webrender 3.90 -> 4.09 Before/After

Improvements:

Ratio Test Platform Options Absolute values (old vs new) Performance Profiles
79% motionmark-htmlsuite-1-3 powerUsage_gpu windows11-64-shippable-qr fission webrender 125.83 -> 26.20
26% motionmark-htmlsuite-1-3 cpuTime windows11-64-shippable-qr fission webrender 963.94 -> 714.08
17% speedometer Preact-TodoMVC/CompletingAllItems/Async windows11-64-shippable-qr fission webrender 2.98 -> 2.47 Before/After
14% speedometer VanillaJS-TodoMVC/Adding100Items/Async windows11-64-shippable-qr fission webrender 2.32 -> 2.00 Before/After
11% speedometer Preact-TodoMVC/CompletingAllItems windows11-64-shippable-qr fission webrender 3.88 -> 3.47 Before/After
... ... ... ... ... ...
2% speedometer AngularJS-TodoMVC windows11-64-shippable-qr fission webrender 106.11 -> 103.85 Before/After

As author of one of the patches included in that push, we need your help to address this regression.
Details of the alert can be found in the alert summary, including links to graphs and comparisons for each of the affected tests. Please follow our guide to handling regression bugs and let us know your plans within 3 business days, or the patch(es) may be backed out in accordance with our regression policy.

If you need the profiling jobs you can trigger them yourself from treeherder job view or ask a sheriff to do that for you.

You can run these tests on try with ./mach try perf --alert 43703

For more information on performance sheriffing please see our FAQ.

Regressions: 1947292

(In reply to Pulsebot from comment #3)

Pushed by gwatson@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/9d8b4c0b99ed
Enable layer compositor on Windows r=gfx-reviewers,lsalzman

Perfherder has detected a browsertime performance change from push 9d8b4c0b99ed4e5f39825c3c8890979ca64efdd8.

Improvements:

Ratio Test Platform Options Absolute values (old vs new) Performance Profiles
8% speedometer Preact-TodoMVC/CompletingAllItems/Sync windows11-64-shippable-qr fission webrender 0.95 -> 0.88 Before/After
4% speedometer VanillaJS-TodoMVC/DeletingAllItems windows11-64-shippable-qr fission webrender 3.58 -> 3.43 Before/After
4% speedometer VanillaJS-TodoMVC/DeletingAllItems/Sync windows11-64-shippable-qr fission webrender 3.35 -> 3.21 Before/After

Details of the alert can be found in the alert summary, including links to graphs and comparisons for each of the affected tests.

If you need the profiling jobs you can trigger them yourself from treeherder job view or ask a sheriff to do that for you.

You can run these tests on try with ./mach try perf --alert 43668

For more information on performance sheriffing please see our FAQ.

Regressions: 1947708
Regressions: 1948182
Regressions: 1948462
Regressions: 1948882
Regressions: 1949078
Regressions: 1949080
Depends on: 1949102
Regressions: 1949116
Depends on: 1949341
Regressions: 1950271
Regressions: 1950825
Depends on: 1954416
Depends on: 1954901
Depends on: 1955950
Depends on: 1956297
Depends on: 1958604
No longer depends on: 1956297
No longer depends on: 1958604
Blocks: 1959009
Summary: Enable layer compositor on Windows → Enable layer compositor on Windows in Nightly
Assignee: gwatson → sotaro.ikeda.g
Depends on: 1968814
Depends on: 1970043
Depends on: 1970550
Depends on: 1970775
Depends on: 1970776
Depends on: 1970778
Depends on: 1970782
Depends on: 1970784
Depends on: 1971071
Status: ASSIGNED → RESOLVED
Closed: 8 months ago
Resolution: --- → FIXED
Target Milestone: --- → 141 Branch
Depends on: 1971681
Regressions: 1971766
Pushed by agoloman@mozilla.com: https://github.com/mozilla-firefox/firefox/commit/782799050648 https://hg.mozilla.org/integration/autoland/rev/03f3e71928f5 Revert "Bug 1945683 - Enable layer compositor on Windows in Nightly r=gfx-reviewers,lsalzman" for causing performance regression on Speedometer 3 on Windows 11.
Status: RESOLVED → REOPENED
Resolution: FIXED → ---
Status: REOPENED → RESOLVED
Closed: 8 months ago8 months ago
Resolution: --- → FIXED
Depends on: 1971766
Attachment #9491631 - Attachment is obsolete: true
Attachment #9491631 - Attachment is obsolete: false
Attachment #9463648 - Attachment is obsolete: true
Regressions: 1971661
Pushed by chorotan@mozilla.com: https://github.com/mozilla-firefox/firefox/commit/baa374a65199 https://hg.mozilla.org/integration/autoland/rev/017a5fb444d4 Revert "Bug 1945683 - Enable layer compositor on Windows in Nightly r=gfx-reviewers,lsalzman" for causing reftest failures
Status: RESOLVED → REOPENED
Resolution: FIXED → ---
Target Milestone: 141 Branch → ---
Status: REOPENED → RESOLVED
Closed: 8 months ago8 months ago
Resolution: --- → FIXED
Target Milestone: --- → 141 Branch
Pushed by abutkovits@mozilla.com: https://github.com/mozilla-firefox/firefox/commit/397feb42dc22 https://hg.mozilla.org/integration/autoland/rev/1cfeac9e52c0 Revert "Bug 1945683 - Enable layer compositor on Windows in Nightly r=gfx-reviewers,lsalzman" for causing performance regression in a critical test (speedometer3).

Backed out for causing performance regression in a critical test (speedometer3). (requested by sparky)

Backout link: https://hg-edge.mozilla.org/integration/autoland/rev/1cfeac9e52c09a98fb1a38c4b381c49aeded2ac3

Status: RESOLVED → REOPENED
Flags: needinfo?(sotaro.ikeda.g)
Resolution: FIXED → ---
Target Milestone: 141 Branch → ---
Flags: needinfo?(sotaro.ikeda.g)

(In reply to Pulsebot from comment #13)

Pushed by agoloman@mozilla.com:
https://github.com/mozilla-firefox/firefox/commit/782799050648
https://hg.mozilla.org/integration/autoland/rev/03f3e71928f5
Revert "Bug 1945683 - Enable layer compositor on Windows in Nightly
r=gfx-reviewers,lsalzman" for causing performance regression on Speedometer
3 on Windows 11.

(In reply to agoloman from comment #14)

Backout merged into central: https://hg-edge.mozilla.org/mozilla-central/rev/03f3e71928f5db04a68e19d0af6f92d3519d50c3

Perfherder has detected a browsertime performance change from push 03f3e71928f5db04a68e19d0af6f92d3519d50c3.

If you have any questions, please reach out to a performance sheriff. Alternatively, you can find help on Slack by joining #perf-help, and on Matrix you can find help by joining #perftest.

Improvements:

Ratio Test Platform Options Absolute values (old vs new) Performance Profiles
10% speedometer3 TodoMVC-Vue/CompletingAllItems/Sync windows11-64-24h2-shippable fission webrender 1.72 -> 1.55 Before/After
7% speedometer3 TodoMVC-Preact-Complex-DOM/DeletingAllItems/Sync windows11-64-24h2-shippable fission webrender 1.11 -> 1.03 Before/After
7% speedometer3 TodoMVC-Svelte-Complex-DOM/DeletingAllItems/Sync windows11-64-24h2-shippable fission webrender 1.27 -> 1.19 Before/After
6% speedometer3 Charts-observable-plot/Dotted/Async windows11-64-24h2-shippable fission webrender 4.89 -> 4.59 Before/After
6% speedometer3 TodoMVC-Lit-Complex-DOM/DeletingAllItems/Sync windows11-64-24h2-shippable fission webrender 1.89 -> 1.78 Before/After
... ... ... ... ... ...
2% speedometer3 windows11-64-24h2-shippable fission webrender 21.45 -> 21.89 Before/After

Details of the alert can be found in the alert summary, including links to graphs and comparisons for each of the affected tests.

If you need the profiling jobs you can trigger them yourself from treeherder job view or ask a performance sheriff to do that for you.

You can run all of these tests on try with ./mach try perf --alert 45548

The following documentation link provides more information about this command.

Depends on: 1972743
Regressions: 1972993
Depends on: 1973147
Depends on: 1973150

(In reply to Cristian Tuns from comment #24)

Backout merged to central: https://hg.mozilla.org/mozilla-central/rev/1cfeac9e52c0

Perfherder has detected a talos performance change from push 1cfeac9e52c09a98fb1a38c4b381c49aeded2ac3.

If you have any questions, please reach out to a performance sheriff. Alternatively, you can find help on Slack by joining #perf-help, and on Matrix you can find help by joining #perftest.

Improvements:

Ratio Test Platform Options Absolute values (old vs new)
35% tp5o_scroll_paint_skip windows11-64-24h2-shippable e10s fission stylo webrender 0.70 -> 0.45
27% tp5o_scroll windows11-64-24h2-shippable e10s fission stylo webrender 1.43 -> 1.05
18% tscrollx windows11-64-24h2-shippable e10s fission stylo webrender 0.67 -> 0.55
10% tart windows11-64-24h2-shippable e10s fission stylo webrender 1.96 -> 1.77
5% displaylist_mutate windows11-64-24h2-shippable e10s fission stylo webrender 1,387.47 -> 1,312.22

Details of the alert can be found in the alert summary, including links to graphs and comparisons for each of the affected tests.

If you need the profiling jobs you can trigger them yourself from treeherder job view or ask a performance sheriff to do that for you.

You can run all of these tests on try with ./mach try perf --alert 45653

The following documentation link provides more information about this command.

(In reply to Pulsebot from comment #13)

Pushed by agoloman@mozilla.com:
https://github.com/mozilla-firefox/firefox/commit/782799050648
https://hg.mozilla.org/integration/autoland/rev/03f3e71928f5
Revert "Bug 1945683 - Enable layer compositor on Windows in Nightly
r=gfx-reviewers,lsalzman" for causing performance regression on Speedometer
3 on Windows 11.

Perfherder has detected a browsertime performance change from push 03f3e71928f5db04a68e19d0af6f92d3519d50c3. As author of one of the patches included in that push, we need your help to address this regression.

Please acknowledge, and begin investigating this alert within 3 business days, or the patch(es) may be backed out in accordance with our regression policy. Our guide to handling regression bugs has information about how you can proceed with this investigation.

If you have any questions or need any help with the investigation, please reach out to a performance sheriff. Alternatively, you can find help on Slack by joining #perf-help, and on Matrix you can find help by joining #perftest.

Regressions:

Ratio Test Platform Options Absolute values (old vs new)
1408% facebook-nav.groups FirstVisualChange windows11-64-24h2-shippable cold fission webrender 56.39 -> 850.48
1153% facebook-nav.groups PerceptualSpeedIndex windows11-64-24h2-shippable cold fission webrender 68.04 -> 852.42
665% imgur FirstVisualChange windows11-64-24h2-shippable fission warm webrender 109.78 -> 839.35
530% facebook-nav.groups SpeedIndex windows11-64-24h2-shippable cold fission webrender 136.71 -> 861.23
258% imgur PerceptualSpeedIndex windows11-64-24h2-shippable fission warm webrender 272.54 -> 975.18
254% imgur ContentfulSpeedIndex windows11-64-24h2-shippable fission warm webrender 277.47 -> 980.98
208% microsoft SpeedIndex windows11-64-24h2-shippable fission warm webrender 449.64 -> 1,384.55
117% microsoft PerceptualSpeedIndex windows11-64-24h2-shippable fission warm webrender 1,170.76 -> 2,538.38
96% instagram ContentfulSpeedIndex windows11-64-24h2-shippable bytecode-cached cold fission webrender 318.43 -> 624.86
96% instagram ContentfulSpeedIndex windows11-64-24h2-shippable cold fission webrender 314.75 -> 616.91
40% outlook ContentfulSpeedIndex windows11-64-24h2-shippable cold fission webrender 423.68 -> 594.20
18% facebook ContentfulSpeedIndex windows11-64-24h2-shippable cold fission webrender 396.80 -> 466.79
10% microsoft SpeedIndex windows11-64-24h2-shippable cold fission webrender 592.10 -> 650.97
9% microsoft PerceptualSpeedIndex windows11-64-24h2-shippable cold fission webrender 1,314.52 -> 1,428.00
5% microsoft LastVisualChange windows11-64-24h2-shippable cold fission webrender 6,040.06 -> 6,335.42

Improvements:

Ratio Test Platform Options Absolute values (old vs new)
84% reddit-billgates-post-1.comments LastVisualChange windows11-64-24h2-shippable cold fission webrender 370.05 -> 58.99
84% reddit-billgates-post-1.billg SpeedIndex windows11-64-24h2-shippable cold fission webrender 389.56 -> 62.79
84% reddit-billgates-post-1.comments PerceptualSpeedIndex windows11-64-24h2-shippable cold fission webrender 132.74 -> 21.56
84% reddit-billgates-post-1.billg FirstVisualChange windows11-64-24h2-shippable cold fission webrender 181.58 -> 29.50
84% reddit-billgates-post-1.billg PerceptualSpeedIndex windows11-64-24h2-shippable cold fission webrender 381.97 -> 62.26
... ... ... ... ...
2% speedometer3 TodoMVC-jQuery/CompletingAllItems/total windows11-64-24h2-shippable fission webrender 69.17 -> 67.78

Details of the alert can be found in the alert summary, including links to graphs and comparisons for each of the affected tests.

If you need the profiling jobs you can trigger them yourself from treeherder job view or ask a performance sheriff to do that for you.

You can run all of these tests on try with ./mach try perf --alert 45548

The following documentation link provides more information about this command.

(In reply to Cristian Tuns from comment #24)

Backout merged to central: https://hg.mozilla.org/mozilla-central/rev/1cfeac9e52c0

Perfherder has detected a browsertime performance change from push 1cfeac9e52c09a98fb1a38c4b381c49aeded2ac3. As author of one of the patches included in that push, we need your help to address this regression.

Please acknowledge, and begin investigating this alert within 3 business days, or the patch(es) may be backed out in accordance with our regression policy. Our guide to handling regression bugs has information about how you can proceed with this investigation.

If you have any questions or need any help with the investigation, please reach out to a performance sheriff. Alternatively, you can find help on Slack by joining #perf-help, and on Matrix you can find help by joining #perftest.

Regressions:

Ratio Test Platform Options Absolute values (old vs new) Performance Profiles
5% welcome loadtime windows11-64-24h2-shippable cold fission webrender 34.38 -> 36.07

Improvements:

Ratio Test Platform Options Absolute values (old vs new) Performance Profiles
29% pinterest powerUsage_gpu windows11-64-24h2-shippable cold fission webrender 209.52 -> 148.37 Before/After
25% buzzfeed powerUsage_gpu windows11-64-24h2-shippable cold fission webrender 221.59 -> 167.22
24% buzzfeed powerUsage_gpu windows11-64-24h2-shippable fission warm webrender 186.29 -> 140.78
19% speedometer Preact-TodoMVC/DeletingItems/Sync windows11-64-24h2-shippable fission webrender 1.01 -> 0.82
13% speedometer Preact-TodoMVC/DeletingItems windows11-64-24h2-shippable fission webrender 2.09 -> 1.83
... ... ... ... ... ...
2% speedometer3 Perf-Dashboard/SelectingPoints/Sync windows11-64-24h2-shippable fission webrender 29.35 -> 28.76 Before/After

Details of the alert can be found in the alert summary, including links to graphs and comparisons for each of the affected tests.

If you need the profiling jobs you can trigger them yourself from treeherder job view or ask a performance sheriff to do that for you.

You can run all of these tests on try with ./mach try perf --alert 45652

The following documentation link provides more information about this command.

(In reply to Cristina Horotan [:chorotan] from comment #19)

Backout merged to central https://hg.mozilla.org/mozilla-central/rev/017a5fb444d4f5e1eb1772cd89db2244afa8cce4

Perfherder has detected a browsertime performance change from push 017a5fb444d4f5e1eb1772cd89db2244afa8cce4.

If you have any questions, please reach out to a performance sheriff. Alternatively, you can find help on Slack by joining #perf-help, and on Matrix you can find help by joining #perftest.

Improvements:

Ratio Test Platform Options Absolute values (old vs new) Performance Profiles
13% yahoo-mail SpeedIndex windows11-64-24h2-shippable cold fission webrender 159.87 -> 138.81 Before/After
8% yahoo-mail FirstVisualChange windows11-64-24h2-shippable cold fission webrender 138.67 -> 127.41 Before/After
7% yahoo-mail ContentfulSpeedIndex windows11-64-24h2-shippable cold fission webrender 184.28 -> 171.16 Before/After
6% yahoo-mail SpeedIndex windows11-64-24h2-shippable cold fission webrender 165.02 -> 154.38 Before/After
5% yahoo-mail PerceptualSpeedIndex windows11-64-24h2-shippable cold fission webrender 214.96 -> 204.54 Before/After
... ... ... ... ... ...
2% yahoo-mail largestContentfulPaint windows11-64-24h2-shippable cold fission webrender 487.53 -> 477.60 Before/After

Details of the alert can be found in the alert summary, including links to graphs and comparisons for each of the affected tests.

If you need the profiling jobs you can trigger them yourself from treeherder job view or ask a performance sheriff to do that for you.

You can run all of these tests on try with ./mach try perf --alert 45752

The following documentation link provides more information about this command.

Depends on: 1976135
Depends on: 1979238
Depends on: 1980885
Depends on: 1982661
Depends on: 1983735
Depends on: 1984709
Depends on: 1984916
Depends on: 1985197
Status: REOPENED → RESOLVED
Closed: 8 months ago5 months ago
Resolution: --- → FIXED
Target Milestone: --- → 144 Branch

:sotaro, anything you want to mention here in a release note? (Process info)
We could include it in the nightly only release notes.

Flags: needinfo?(sotaro.ikeda.g)

Release Note Request (optional, but appreciated)
[Why is this notable]: Change WebRender compositing more similar to chromium. It could affect to compositing performance.
[Affects Firefox for Android]: No
[Suggested wording]:
[Links (documentation, blog post, etc)]:

Flags: needinfo?(sotaro.ikeda.g)

Will take a look

relnote-firefox: --- → ?

Thanks, added to the Fx144 nightly release notes, please allow 30 minutes for the site to update.

Regressions: 1987978
Regressions: 1988379

Removed from the Nightly release notes as this has been there for 3 cycles now. Feel free to nominate again once the feature is enabled for Release.

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

Attachment

General

Created:
Updated:
Size: