<div dir="ltr">On Thu, Aug 20, 2015 at 1:52 AM, Tapani Pälli <span dir="ltr"><<a href="mailto:tapani.palli@intel.com" target="_blank">tapani.palli@intel.com</a>></span> wrote:<br><div class="gmail_extra"><div class="gmail_quote"><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">We have to re-validate FBOs rendering to the texture like is done<br>
with TexImage and CopyTexImage.<br>
<br>
Signed-off-by: Tapani Pälli <<a href="mailto:tapani.palli@intel.com">tapani.palli@intel.com</a>><br>
Bugzilla: <a href="https://bugs.freedesktop.org/show_bug.cgi?id=91673" rel="noreferrer" target="_blank">https://bugs.freedesktop.org/show_bug.cgi?id=91673</a><br>
Cc: "10.6" <<a href="mailto:mesa-stable@lists.freedesktop.org">mesa-stable@lists.freedesktop.org</a>><br>
---<br>
 src/mesa/main/texstorage.c | 15 +++++++++++++++<br>
 1 file changed, 15 insertions(+)<br>
<br>
diff --git a/src/mesa/main/texstorage.c b/src/mesa/main/texstorage.c<br>
index db2b63d..51f49b3 100644<br>
--- a/src/mesa/main/texstorage.c<br>
+++ b/src/mesa/main/texstorage.c<br>
@@ -189,6 +189,20 @@ clear_texture_fields(struct gl_context *ctx,<br>
 }<br>
<br>
<br>
+/**<br>
+ * Update/re-validate framebuffer object.<br>
+ */<br>
+static void<br>
+update_fbo_texture(struct gl_context *ctx, struct gl_texture_object *texObj)<br>
+{<br>
+   const unsigned numFaces = _mesa_num_tex_faces(texObj->Target);<br>
+   for (int level = 0; level < ARRAY_SIZE(texObj->Image[0]); level++) {<br>
+      for (unsigned face = 0; face < numFaces; face++)<br>
</blockquote><div><br></div><div>We don't normally declare variable in the loops like that.  Not sure if that'll compile everywhere.<br><br></div><div>-Brian<br><br></div><div><br> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">+         _mesa_update_fbo_texture(ctx, texObj, face, level);<br>
+   }<br>
+}<br>
+<br>
+<br>
 GLboolean<br>
 _mesa_is_legal_tex_storage_format(struct gl_context *ctx, GLenum internalformat)<br>
 {<br>
@@ -446,6 +460,7 @@ _mesa_texture_storage(struct gl_context *ctx, GLuint dims,<br>
<br>
       _mesa_set_texture_view_state(ctx, texObj, target, levels);<br>
<br>
+      update_fbo_texture(ctx, texObj);<br>
    }<br>
 }<br>
<span class="HOEnZb"><font color="#888888"><br>
--<br>
2.1.0<br>
<br>
_______________________________________________<br>
mesa-dev mailing list<br>
<a href="mailto:mesa-dev@lists.freedesktop.org">mesa-dev@lists.freedesktop.org</a><br>
<a href="http://lists.freedesktop.org/mailman/listinfo/mesa-dev" rel="noreferrer" target="_blank">http://lists.freedesktop.org/mailman/listinfo/mesa-dev</a><br>
</font></span></blockquote></div><br></div></div>