mesa: Loosen glBlitFramebuffer restrictions on depthstencil buffers (v2)
authorChad Versace <chad.versace@linux.intel.com>
Tue, 17 Jan 2012 20:01:34 +0000 (12:01 -0800)
committerChad Versace <chad.versace@linux.intel.com>
Fri, 20 Jan 2012 18:35:49 +0000 (10:35 -0800)
commitf74d8aacbf2f6d140381e272d17c31162a3481b3
treea44d59966a1cf79b0ea7faab285671e050f315c6
parentab69d584f923101fab05560b8e9ff97cf3cc2c5f
mesa: Loosen glBlitFramebuffer restrictions on depthstencil buffers (v2)

This loosens the format validation in glBlitFramebuffer. When blitting
depth bits, don't require an exact match between the depth formats; only
require that the two formats have the same number of depth bits and the
same depth datatype (float vs uint). Ditto for stencil.

Between S8_Z24 buffers, the EXT_framebuffer_blit spec allows
glBlitFramebuffer to blit the depth and stencil bits separately. So I see
no reason to prevent blitting the depth bits between X8_Z24 and S8_Z24 or
the stencil bits between S8 and S8_Z24. However, we of course don't want
to allow blitting from Z32 to Z32_FLOAT.

Fixes Piglit fbo/fbo-blit-d24s8 on Intel drivers with separate stencil
enabled.

The problem was that, on Intel drivers with separate stencil, the default
framebuffer has separate depth and stencil buffers with formats X8_Z24 and
S8. The test attempts to blit the depth bits from a S8_Z24 buffer into the
default framebuffer.

v2: Check that depth datatypes match.

Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=44665
Note: This is a candidate for the 8.0 branch.
Reported-by: Xunx Fang <xunx.fang@intel.com>
Reviewed-by: Ian Romanick <ian.d.romanick@intel.com>
Signed-off-by: Chad Versace <chad.versace@linux.intel.com>
src/mesa/main/fbobject.c