Closed Bug 1804233 Opened 2 years ago Closed 1 year ago

Avoid sync wait of remote texture in RenderThread

Categories

(Core :: Graphics, enhancement, P1)

enhancement

Tracking

()

RESOLVED FIXED
Webcompat Priority P1

People

(Reporter: sotaro, Assigned: sotaro)

References

(Blocks 1 open bug)

Details

(Whiteboard: [sp3])

Attachments

(1 obsolete file)

After Bug 1776885 fix, in async mode, remote texture could cause a sync wait in RenderThread when WebGL task is very heavy. It is better to be avoided.

Assignee: nobody → sotaro.ikeda.g
Depends on: 1776885
Blocks: 1803734
Blocks: 1801437
Blocks: 1804447
See Also: → 1768308
See Also: → 1820005

Does this work also help with avoiding the waits on the content process main thread? Or is it only targeted at eliminating the Renderer thread wait?

pref webgl.out-of-process.async-present.force-sync = false already avoids the wait on the content process main thread. But it works only with Threadsafe GL case. And if it works, a wait could happen on Renderer thread in a case that oop WebGL rendering is not done.

This issue is impacting our Speedometer 3 score by preventing Accelerated Canvas2D from running. Setting to P2/S2.

Severity: -- → S2
Priority: -- → P2
Webcompat Priority: --- → P1
Priority: P2 → P1
Whiteboard: [sp3]
Blocks: 1820005

Will this also remove other webgl/canvas related sync ipc like readpixels, getlinkresult, getCompileResult ?

(In reply to Mayank Bansal from comment #5)

Will this also remove other webgl/canvas related sync ipc like readpixels, getlinkresult, getCompileResult ?

It does not affect to them. The change affect only to ClientWebGLContext::GetFrontBuffer().

Depends on: 1823352
Depends on: 1824309
Depends on: 1825194
Blocks: 1826134
No longer blocks: 1826134
Depends on: 1826204
Depends on: 1826280
Depends on: 1826687
Depends on: 1827578
Depends on: 1828088
Attachment #9322650 - Attachment is obsolete: true
Depends on: 1829052
Blocks: 1828904
No longer blocks: 1804447
No longer blocks: 1803734
Depends on: 1803734
Depends on: 1831548
Depends on: 1831811
See Also: 1820005
Depends on: 1832480
Depends on: 1839314
Depends on: 1839264
Depends on: 1841214
Depends on: 1841380
Status: NEW → RESOLVED
Closed: 1 year ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: