Closed Bug 1773712 Opened 2 years ago Closed 2 years ago

Enable accelerated Canvas2D on macOS nightly

Categories

(Core :: Graphics: Canvas2D, enhancement)

Unspecified
macOS
enhancement

Tracking

()

RESOLVED FIXED
105 Branch
Tracking Status
firefox105 --- fixed

People

(Reporter: lsalzman, Assigned: lsalzman)

References

(Blocks 1 open bug, Regressed 2 open bugs)

Details

(Keywords: perf-alert)

Attachments

(1 file, 1 obsolete file)

No description provided.
Pushed by lsalzman@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/5c16ac03eca7
Enable accelerated Canvas2D on macOS nightly. r=jrmuizel
Pushed by csabou@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/767a824e1cc8
Fix license lint failure. a=lint-fix
Status: NEW → RESOLVED
Closed: 2 years ago
Resolution: --- → FIXED
Target Milestone: --- → 103 Branch
See Also: → 1769254
Regressions: 1773814
Regressions: 1773818

Backed out for causing crashes in Bug 1769254

Status: RESOLVED → REOPENED
Flags: needinfo?(lsalzman)
Resolution: FIXED → ---
Target Milestone: 103 Branch → ---
Regressions: 1774037
Attachment #9280793 - Attachment is obsolete: true

(In reply to Pulsebot from comment #2)

Pushed by lsalzman@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/5c16ac03eca7
Enable accelerated Canvas2D on macOS nightly. r=jrmuizel

== Change summary for alert #34439 (as of Mon, 13 Jun 2022 12:00:51 GMT) ==

Regressions:

Ratio Test Platform Options Absolute values (old vs new)
21% google-docs SpeedIndex macosx1015-64-shippable-qr fission warm webrender 712.08 -> 861.67
17% google-docs SpeedIndex macosx1015-64-shippable-qr cold fission webrender 1,100.00 -> 1,291.00
6% google-docs PerceptualSpeedIndex macosx1015-64-shippable-qr cold fission webrender 1,318.38 -> 1,394.67
6% google-docs FirstVisualChange macosx1015-64-shippable-qr cold fission webrender 720.00 -> 760.00
3% google-docs ContentfulSpeedIndex macosx1015-64-shippable-qr cold fission webrender 1,359.00 -> 1,398.17

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

(In reply to Cristian Tuns from comment #6)

Backed out for causing crashes in Bug 1769254

== Change summary for alert #34484 (as of Wed, 15 Jun 2022 00:16:06 GMT) ==

Improvements:

Ratio Test Platform Options Absolute values (old vs new)
18% google-docs SpeedIndex macosx1015-64-shippable-qr fission warm webrender 862.17 -> 709.25
15% google-docs SpeedIndex macosx1015-64-shippable-qr cold fission webrender 1,291.42 -> 1,099.08
12% google-docs loadtime macosx1015-64-shippable-qr cold fission webrender 920.67 -> 813.25
5% google-docs PerceptualSpeedIndex macosx1015-64-shippable-qr cold fission webrender 1,395.58 -> 1,320.08
5% google-docs FirstVisualChange macosx1015-64-shippable-qr cold fission webrender 760.00 -> 720.00
3% google-docs ContentfulSpeedIndex macosx1015-64-shippable-qr cold fission webrender 1,399.83 -> 1,357.17

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

Blocks: gpu-canvas
Flags: needinfo?(lsalzman)
Pushed by lsalzman@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/07e7637819ec
Enable accelerated Canvas2D on macOS nightly. r=jrmuizel
Status: REOPENED → RESOLVED
Closed: 2 years ago2 years ago
Resolution: --- → FIXED
Target Milestone: --- → 105 Branch

Backed out for causing reftest failures on 1304353-text-global-composite-op-1.html

Failure line: REFTEST TEST-UNEXPECTED-FAIL | layout/reftests/canvas/1304353-text-global-composite-op-1.html == layout/reftests/canvas/1304353-text-global-composite-op-1-ref.html | image comparison, max difference: 1, number of differing pixels: 34

Push with failures

Failure log

Backout link

Flags: needinfo?(lsalzman)
Status: RESOLVED → REOPENED
Resolution: FIXED → ---
Target Milestone: 105 Branch → ---
Flags: needinfo?(lsalzman)
Pushed by lsalzman@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/da8fba53f728
Enable accelerated Canvas2D on macOS nightly. r=jrmuizel
Status: REOPENED → RESOLVED
Closed: 2 years ago2 years ago
Resolution: --- → FIXED
Target Milestone: --- → 105 Branch
Regressions: 1783141
Regressions: 1783182
No longer regressions: 1783182
Regressions: 1769254
Regressions: 1783717

Given the high crash rate in bug 1769254, I think we should probably back this change out for now.

Flags: needinfo?(lsalzman)

I put in a stop-gap patch to reduce the crashiness in bug 1769254. We are actively investigating what's going on.

Flags: needinfo?(lsalzman)
Regressions: 1784910
Regressions: 1862925
Regressions: 1874314
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: