radeon: workaround an FBO issue
authorMaciej Cencora <m.cencora@gmail.com>
Thu, 3 Dec 2009 19:21:16 +0000 (20:21 +0100)
committerMaciej Cencora <m.cencora@gmail.com>
Thu, 3 Dec 2009 19:23:22 +0000 (20:23 +0100)
Fixes #21501

src/mesa/drivers/dri/radeon/radeon_fbo.c
src/mesa/drivers/dri/radeon/radeon_mipmap_tree.c

index 7ec641f..fc21069 100644 (file)
@@ -369,6 +369,12 @@ radeon_framebuffer_renderbuffer(GLcontext * ctx,
 }
 
 
+/* TODO: According to EXT_fbo spec internal format of texture image
+ * once set during glTexImage call, should be preserved when
+ * attaching image to renderbuffer. When HW doesn't support
+ * rendering to format of attached image, set framebuffer
+ * completeness accordingly in radeon_validate_framebuffer (issue #79).
+ */
 static GLboolean
 radeon_update_wrapper(GLcontext *ctx, struct radeon_renderbuffer *rrb, 
                     struct gl_texture_image *texImage)
index a9d601a..0415a50 100644 (file)
@@ -421,9 +421,12 @@ static void migrate_image_to_miptree(radeon_mipmap_tree *mt,
 
                radeon_mipmap_level *srclvl = &image->mt->levels[image->mtlevel];
 
+               /* TODO: bring back these assertions once the FBOs are fixed */
+#if 0
                assert(image->mtlevel == level);
                assert(srclvl->size == dstlvl->size);
                assert(srclvl->rowstride == dstlvl->rowstride);
+#endif
 
                radeon_bo_map(image->mt->bo, GL_FALSE);