Closed Bug 1724129 Opened 3 years ago Closed 2 years ago

Categories

(Core :: Graphics: WebRender, defect)

defect

Tracking

()

RESOLVED FIXED
Tracking Status
firefox99 --- fixed

People

(Reporter: mayankleoboy1, Assigned: gw)

References

(Blocks 2 open bugs)

Details

Attachments

(1 file)

Attached file abut:supprt
Blocks: wr-perf

:gw, can you comment to this bug?

Flags: needinfo?(gwatson)

This seems to struggle even on a high-end ryzen system with a discrete radeon GPU (though it's not a particularly high-end GPU).

From a quick look at the page, the vertex, primitive, and glyph counts look low.

The render target count is high (21 targets per frame), but not so high that I'd expect it to cause that kind of slow down on my GPU.

However, the GPU profile counters show a very long time in each render target clear, draw and clip-mask creation. This makes me suspect that each of the render targets is really large.

It will need further investigation, but my guess is that we're not selecting an appropriate scale factor for the render targets, and drawing each of them at a huge size when they don't need to be.

Assignee: nobody → gwatson
Flags: needinfo?(gwatson)

bug1749380 might have fixed this. The original Codepen link 404s, it is available on archive.org but the preview pane does not function. I found this fork with the same source code that does work. It is very laggy in v97 stable but seems fine in the latest Nightly (I think I see some hitching but it might be in my head).

I confirmed the results from comment 6, thanks for checking - it's much faster in nightly. By enabling gfx.webrender.debug.render-targets, it's possible to see how many render targets and what size they are. In release version we allocate a lot more targets, and they are often 4k x 4k or even 8k x 8k. In nightly, we allocate fewer targets and they are all 2k x 2x or less, which makes sense given the observed performance differences.

Status: NEW → RESOLVED
Closed: 2 years ago
Depends on: 1749380
Resolution: --- → FIXED
Type: task → defect

Agree with this resolution. The demo is quite smooth now.

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

Attachment

General

Creator:
Created:
Updated:
Size: