diff options
author | Sergey Sharybin <sergey.vfx@gmail.com> | 2015-08-26 13:10:24 +0300 |
---|---|---|
committer | Sergey Sharybin <sergey.vfx@gmail.com> | 2015-08-26 13:10:24 +0300 |
commit | 6ca12d157fbfdf1a71a06a87fb5f258ae98ebe02 (patch) | |
tree | 28f7fb0a5c0341d1023da267ab4660f72675cdd5 /intern/opensubdiv | |
parent | bcc0d2fb1d66d5191276dcbe17621d50ae0fab35 (diff) |
Fix T45909: Garbage output in Viewport with OpenSubdiv device set to GLSL Compute
This isn't a Blender issue and the same bug happens with official OpenSubdiv
examples. For until it's either worked around from OpenSubdiv side or fixed
in the driver we'll force disable GLSL Compute for AMD hardware.
Diffstat (limited to 'intern/opensubdiv')
-rw-r--r-- | intern/opensubdiv/opensubdiv_utils_capi.cc | 20 |
1 files changed, 19 insertions, 1 deletions
diff --git a/intern/opensubdiv/opensubdiv_utils_capi.cc b/intern/opensubdiv/opensubdiv_utils_capi.cc index 5a2d017ed26..a3aff5327db 100644 --- a/intern/opensubdiv/opensubdiv_utils_capi.cc +++ b/intern/opensubdiv/opensubdiv_utils_capi.cc @@ -26,6 +26,7 @@ #include "opensubdiv_capi.h" +#include <cstring> #include <GL/glew.h> #ifdef _MSC_VER @@ -71,7 +72,24 @@ int openSubdiv_getAvailableEvaluators(void) #endif /* OPENSUBDIV_HAS_GLSL_TRANSFORM_FEEDBACK */ #ifdef OPENSUBDIV_HAS_GLSL_COMPUTE - flags |= OPENSUBDIV_EVALUATOR_GLSL_COMPUTE; + static bool vendor_checked = false; + static bool disable_glsl_compute = false; + /* Force disable GLSL Compute on AMD hardware because it has really + * hard time evaluating required shaders. + */ + if (!vendor_checked) { + const char *vendor = (const char *)glGetString(GL_VENDOR); + const char *renderer = (const char *)glGetString(GL_RENDERER); + if (strstr(vendor, "ATI") || + strstr(renderer, "Mesa DRI R") || + (strstr(renderer, "Gallium ") && strstr(renderer, " on ATI "))) + { + disable_glsl_compute = true; + } + } + if (!disable_glsl_compute) { + flags |= OPENSUBDIV_EVALUATOR_GLSL_COMPUTE; + } #endif /* OPENSUBDIV_HAS_GLSL_COMPUTE */ return flags; |