Don't let D3D11 double-buffering ride the trains yet
Categories
(Core :: Graphics, defect, P2)
Tracking
()
Tracking | Status | |
---|---|---|
firefox67.0.1 | --- | unaffected |
firefox68 | --- | fixed |
firefox69 | --- | fixed |
People
(Reporter: kats, Assigned: kats, NeedInfo)
References
Details
Attachments
(1 file)
(deleted),
text/x-phabricator-request
|
jcristau
:
approval-mozilla-beta+
|
Details |
Bug 1547775 turned on D3D11 double-buffering, but it seems to have caused some regressions that need investigation. Until those issues are sorted out we shouldn't let it ride the trains. See https://bugzilla.mozilla.org/show_bug.cgi?id=1555098#c6
Assignee | ||
Updated•5 years ago
|
Assignee | ||
Updated•5 years ago
|
Assignee | ||
Comment 1•5 years ago
|
||
Assignee | ||
Comment 2•5 years ago
|
||
Comment on attachment 9068955 [details]
Bug 1555956 - Disable double buffering on non-Nightly. r?jrmuizel
Beta/Release Uplift Approval Request
- User impact if declined: Users may be affected by bug 1555098 and/or bug 1554610, both of which are regressions (direct or indirect) from bug 1547775 which turned on D3D11 double-buffering.
- Is this code covered by automated tests?: Unknown
- Has the fix been verified in Nightly?: No
- Needs manual test from QE?: No
- If yes, steps to reproduce:
- List of other uplifts needed: None
- Risk to taking this patch: Low
- Why is the change risky/not risky? (and alternatives if risky): This is a low-risk patch to prevent the double-buffering from riding the trains as it produces regressions that are not yet understood.
- String changes made/needed: none
Comment 4•5 years ago
|
||
bugherder |
Comment 5•5 years ago
|
||
Comment on attachment 9068955 [details]
Bug 1555956 - Disable double buffering on non-Nightly. r?jrmuizel
disable gfx.direct3d11.use-double-buffering for 68.0b7
Comment 6•5 years ago
|
||
bugherder uplift |
Comment 7•5 years ago
|
||
Considering the very large perf win (and power win) double buffering gives us (i.e. scrolling goes from being awful to being good on my P52), can we figure out how many D3D11 users this causes us to lose and whether it's the right tradeoff? Possibly just disabling double buffering for those users where it causes issues?
Comment 8•5 years ago
|
||
This is actually only an issue when Advanced Layers is blacklisted for some reason, Advanced Layers already dealt with this situation.
Assignee | ||
Comment 9•5 years ago
|
||
Thanks for the patch. Any thoughts on bug 1554610?
Comment 10•5 years ago
|
||
(In reply to Kartikaya Gupta (email:kats@mozilla.com) from comment #9)
Thanks for the patch. Any thoughts on bug 1554610?
Not without a machine where I can reproduce it :-(. Although I don't think it's bad enough that it should block double buffering.
Comment 11•5 years ago
|
||
(In reply to Kartikaya Gupta (email:kats@mozilla.com) from comment #9)
Thanks for the patch. Any thoughts on bug 1554610?
My best guess here is that this is a debug feature in VirtualBox that is accidentally leaking in somewhere, they might put some kind of debugging info for their graphics driver development for double buffering in that single pixel. The easiest solution in my mind would be to disable double buffering on VirtualBox, or for that matter, perhaps blacklist it by default altogether, I'm not certain they win much by it.
Assignee | ||
Comment 12•5 years ago
|
||
(In reply to Bas Schouten (:bas.schouten) from comment #11)
The easiest solution in my mind would be to disable double buffering on VirtualBox, or for that matter, perhaps blacklist it by default altogether, I'm not certain they win much by it.
Could you write that patch?
Description
•