Closed Bug 1585404 Opened 5 years ago Closed 5 years ago

Frame skipping on Windows Intel 4400

Categories

(Core :: Graphics: WebRender, defect, P2)

defect

Tracking

()

RESOLVED FIXED

People

(Reporter: jrmuizel, Assigned: kvark)

References

(Blocks 1 open bug)

Details

(Keywords: leave-open)

Attachments

(1 file, 1 obsolete file)

I see some frame skipping when scrolling on youtube.com. It's not at all clear what the cause is. More profiling to come...

Blocks: wr-70
Priority: -- → P2

The frame skips are caused by texture upload. https://perfht.ml/2okzO92

Blocks: wr-71
No longer blocks: wr-70
Blocks: wr-72
No longer blocks: wr-71
Flags: needinfo?(dmalyshau)

kvark and Jeff to discuss this bug

Flags: needinfo?(jmuizelaar)

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.

Flags: needinfo?(dmalyshau) → needinfo?(gwatson)

It's hard to say conclusively, but yes, I think restricting the workaround in the way you've suggested should be sufficient.

Flags: needinfo?(gwatson)
Keywords: leave-open

We believe the change should land and bake a bit in Nightly before
we'd consider upstreaming it to Angle proper.

Assignee: nobody → dmalyshau
Pushed by jgilbert@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/165b21fe5674 ANGLE fix for frame skipping on Windows Intel 4400. r=kvark
Attachment #9113815 - Attachment is obsolete: true
Blocks: wr-wild, wr-perf
No longer blocks: wr-72
Flags: needinfo?(jmuizelaar)
No longer blocks: wr-wild
Status: NEW → RESOLVED
Closed: 5 years ago
Resolution: --- → FIXED

Are we ready to upstream this?

Flags: needinfo?(dmalyshau)

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.

Flags: needinfo?(dmalyshau)
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: