Closed Bug 1258691 Opened 9 years ago Closed 5 years ago

WebGL basic compositor does sync readback

Categories

(Core :: Graphics: CanvasWebGL, defect, P3)

45 Branch
x86_64
Windows 7
defect

Tracking

()

RESOLVED WONTFIX

People

(Reporter: alorken, Unassigned)

Details

(Whiteboard: gfx-noted)

Attachments

(3 files, 1 obsolete file)

I develop a game, and in 45.0.1 version I have performance problem. In "Call Graph" retreived over "Gecko Profiler"/"Cleopatra.io" I see what after every frame function PressShell::Paint works 25-30msec. But in old version 40.0.3 function PressShell::Paint appears not after every frame and works 1ms. In nightly build (48.0a1) seems all right too. What do I need to do to get everything working in the 45th version?
Attached image preview
Attached file profile log version 45
Attached file profile log version 40
Attachment #8733353 - Attachment description: preview of problem and profile logs → external links for preview of problem and profile logs
Comment on attachment 8733353 [details] external links for preview of problem and profile logs attachments are added separately
Attachment #8733353 - Attachment is obsolete: true
Component: General → Canvas: WebGL
Product: Firefox → Core
Does this ring any bells for you, BenWa?
Flags: needinfo?(bgirard)
Whiteboard: gfx-noted
Looks like you're just using Basic compositor. Perhaps something regressed from 40 -> 45 causing a readback to be required. If you can run mozregression to pin-point the change where the performance got worse that would be very useful.
Flags: needinfo?(bgirard)
Status: UNCONFIRMED → NEW
Ever confirmed: true
Summary: WebGL performance, PresShell::Paint problem → WebGL basic compositor does sync readback
"Refresh Firefox" solved this problem. I had to try this function first. It seems it does not depend on the version of the browser.

There's not really a path to fixing this. Readback has to happen at some point, or at least a blit+fence+wait.

Status: NEW → RESOLVED
Closed: 5 years ago
Resolution: --- → WONTFIX
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Creator:
Created:
Updated:
Size: