Frame skipping on Windows Intel 4400
Categories
(Core :: Graphics: WebRender, defect, P2)
Tracking
()
People
(Reporter: jrmuizel, Assigned: kvark)
References
(Blocks 1 open bug)
Details
(Keywords: leave-open)
Attachments
(1 file, 1 obsolete file)
(deleted),
text/x-phabricator-request
|
Details |
I see some frame skipping when scrolling on youtube.com. It's not at all clear what the cause is. More profiling to come...
Reporter | ||
Comment 1•5 years ago
|
||
The frame skips are caused by texture upload. https://perfht.ml/2okzO92
Reporter | ||
Updated•5 years ago
|
Reporter | ||
Updated•5 years ago
|
Updated•5 years ago
|
Assignee | ||
Comment 3•5 years ago
|
||
The profile Jeff used definitely shows us waiting on texture updates, caused by the slow path in Angle as a workaround for Intel Broadwell/Skylake. See https://bugzilla.mozilla.org/show_bug.cgi?id=1562462 for the history of why we use this slow path.
Testing a try with this workaround disabled shows close to be ideal scrolling. There are still smallest hitches, but it's hard to pinpoint the cause - might simply be the fact that the memory bandwidth is consumed by other threads/processes, and that slows the renderer momentarily.
The first step I suggest to do is restricting the workaround to only affect specific texture formats, e.g. 128-bit formats we use for vertex textures. Adding ni? for Glenn to check if that would be sufficient with respect to bug 1562462. I'm working on this in https://github.com/mozilla/angle/pull/21
The next step could be extending the GPU scattering code path (that we have optionally supported for GPU cache specifically) to support textures. This would allow us to avoid the slow path on the affected platforms entirely.
Comment 4•5 years ago
|
||
It's hard to say conclusively, but yes, I think restricting the workaround in the way you've suggested should be sufficient.
Assignee | ||
Updated•5 years ago
|
Assignee | ||
Comment 5•5 years ago
|
||
We believe the change should land and bake a bit in Nightly before
we'd consider upstreaming it to Angle proper.
Updated•5 years ago
|
Comment 6•5 years ago
|
||
Updated•5 years ago
|
Comment 8•5 years ago
|
||
bugherder |
Reporter | ||
Updated•5 years ago
|
Updated•5 years ago
|
Assignee | ||
Comment 10•5 years ago
|
||
Since we haven't gotten any feedback on the patch, sounds like indeed that would be a useful thing to upstream, assuming that wouldn't be too difficult. Having this to bake a bit more is a good option as well.
Comment 11•5 years ago
|
||
Description
•