Commit 5fa3baef authored by Jamie Nicol's avatar Jamie Nicol
Browse files

Bug 1707283 - Disable optimized shaders for all Mali-T devices. r=nical, a=RyanVM

Previously we had disabled optimized shaders on Mali-T devices running
Android versions up to 5.1, due to broken rendering. We have now had a
report of the same issue on Android 7.1. As a precaution, disable
optimized shaders for all Mali-T devices regardless of android version.

Differential Revision: https://phabricator.services.mozilla.com/D113913
parent 4ef1a65d
...@@ -641,12 +641,14 @@ nsresult GfxInfo::GetFeatureStatusImpl( ...@@ -641,12 +641,14 @@ nsresult GfxInfo::GetFeatureStatusImpl(
} }
if (aFeature == FEATURE_WEBRENDER_OPTIMIZED_SHADERS) { if (aFeature == FEATURE_WEBRENDER_OPTIMIZED_SHADERS) {
// Optimized shaders result in completely broken rendering on Mali-T // Optimized shaders result in completely broken rendering on some Mali-T
// devices running android versions up to 5.1. // devices. We have seen this on T6xx, T7xx, and T8xx on android versions
// See bug 1689064 and bug 1707283 for details. // up to 5.1, and on T6xx on versions up to android 7.1. As a precaution
// disable for all Mali-T regardless of version. See bug 1689064 and bug
// 1707283 for details.
const bool isMaliT = const bool isMaliT =
mGLStrings->Renderer().Find("Mali-T", /*ignoreCase*/ true) >= 0; mGLStrings->Renderer().Find("Mali-T", /*ignoreCase*/ true) >= 0;
if (isMaliT && mSDKVersion <= 22) { if (isMaliT) {
*aStatus = nsIGfxInfo::FEATURE_BLOCKED_DEVICE; *aStatus = nsIGfxInfo::FEATURE_BLOCKED_DEVICE;
aFailureId = "FEATURE_FAILURE_BUG_1689064"; aFailureId = "FEATURE_FAILURE_BUG_1689064";
} else { } else {
......
Supports Markdown
0% or .
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment