i965: Don't allow rendering to non-GL_RED/RG/RGBA integer textures.
authorEric Anholt <eric@anholt.net>
Wed, 25 Jan 2012 00:32:36 +0000 (16:32 -0800)
committerEric Anholt <eric@anholt.net>
Fri, 27 Jan 2012 19:46:10 +0000 (11:46 -0800)
Fixes piglit EXT_texture_integer/fbo-blending.

src/mesa/drivers/dri/i965/brw_wm_surface_state.c

index bdce057366e4d34506284e136d22a0c471afe204..51d3a46245f1de792a30483acf5d10d1eb30739c 100644 (file)
@@ -563,6 +563,17 @@ brw_render_target_supported(struct intel_context *intel,
    struct brw_context *brw = brw_context(&intel->ctx);
    gl_format format = rb->Format;
 
+   /* Many integer formats are promoted to RGBA (like XRGB8888 is), which means
+    * we would consider them renderable even though we don't have surface
+    * support for their alpha behavior and don't have the blending unit
+    * available to fake it like we do for XRGB8888.  Force them to being
+    * unsupported.
+    */
+   if ((rb->_BaseFormat != GL_RGBA &&
+       rb->_BaseFormat != GL_RG &&
+       rb->_BaseFormat != GL_RED) && _mesa_is_format_integer_color(format))
+      return false;
+
    return brw->format_supported_as_render_target[format];
 }