Mesa (master): i965/tex: Don' t create read-write textures with non-renderable formats

Jason Ekstrand jekstrand at kemper.freedesktop.org
Wed Jan 28 11:33:58 UTC 2015


Module: Mesa
Branch: master
Commit: 5e8468e6dadb06da061ad51e2c40e85bc1c6c3e4
URL:    http://cgit.freedesktop.org/mesa/mesa/commit/?id=5e8468e6dadb06da061ad51e2c40e85bc1c6c3e4

Author: Jason Ekstrand <jason.ekstrand at intel.com>
Date:   Tue Jan 27 14:18:57 2015 -0800

i965/tex: Don't create read-write textures with non-renderable formats

I haven't actually seen this bug in the wild, but it's possible that
someone could ask to do a S3TC PBO download or something.  This protects us
from accidentally creating a render target with a compressed or otherwise
non-renderable format.

Reviewed-by: Kenneth Graunke <kenneth at whitecape.org>

---

 src/mesa/drivers/dri/i965/intel_tex.c |    5 +++++
 1 file changed, 5 insertions(+)

diff --git a/src/mesa/drivers/dri/i965/intel_tex.c b/src/mesa/drivers/dri/i965/intel_tex.c
index 184702f..2d3009a 100644
--- a/src/mesa/drivers/dri/i965/intel_tex.c
+++ b/src/mesa/drivers/dri/i965/intel_tex.c
@@ -323,6 +323,11 @@ intel_set_texture_storage_for_buffer_object(struct gl_context *ctx,
          perf_debug("Bad PBO alignment; fallback to CPU mapping\n");
          return false;
       }
+
+      if (!brw->format_supported_as_render_target[image->TexFormat]) {
+         perf_debug("Non-renderable PBO format; fallback to CPU mapping\n");
+         return false;
+      }
    }
 
    assert(intel_texobj->mt == NULL);




More information about the mesa-commit mailing list