Closed Bug 1787706 Opened 2 years ago Closed 2 years ago

Increase the blob tile size to 512x512

Categories

(Core :: Graphics: WebRender, enhancement, P3)

enhancement

Tracking

()

RESOLVED FIXED
106 Branch
Tracking Status
firefox106 --- fixed

People

(Reporter: nical, Assigned: nical)

References

(Blocks 2 open bugs)

Details

Attachments

(1 file)

It improves blob rasterization and upload time in almost all test cases I've come across, especially the challenging ones. The only downside is that we lose invalidation granularity

It improves rasterization and upload times by a lot in almost all of the test cases I can find. The only drawback is that our invalidation granulatiry is the tile so invalidation gets coarser as we increase the tile size.

512 is a bit special because it is the limit above which a different texture upload path is taken, so there will be more risk of performance side effects if/when we decide to make tiles even larger.

Pushed by nsilva@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/2f9e4f4b4131
Increase the blob tile size to 512. r=jrmuizel
Flags: needinfo?(nical.bugzilla)
Flags: needinfo?(nical.bugzilla)
Pushed by nsilva@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/7223a50e6cbf
Increase the blob tile size to 512. r=jrmuizel

Backed out changeset 7223a50e6cbf (Bug 1787706) for causing reftest failures on 1524353.html and marker-path-021.svg.
Backout link
Push with failures <--> R3 and Wr6
Failure Log R3
Failure Log Wr6
Also R2 Failure Log
Also R25 Failure Log

Flags: needinfo?(nical.bugzilla)
Pushed by nsilva@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/c8b33bd2b4bf
Increase the blob tile size to 512. r=jrmuizel
Flags: needinfo?(nical.bugzilla)
Pushed by nsilva@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/105858693e9a
Increase the blob tile size to 512. r=jrmuizel
Status: NEW → RESOLVED
Closed: 2 years ago
Resolution: --- → FIXED
Target Milestone: --- → 106 Branch
Regressions: 1789612
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: