[Mesa-dev] [PATCH] drivers/meta: Accept GL_TEXTURE_3D as target for tex image decompression

[Eduardo Lima Mitev]

Gentle reminder :)

Thanks!

On 10/18/2016 09:55 AM, Eduardo Lima Mitev wrote:
> An assert is currently raised, preventing decompression of a texture image into
> a GL_TEXTURE_3D target. I have not found any spec wording that would explain
> this, or implementation detail that would prevent it. And in any case, the
> driver should not cause a crash upon user input arguments.
> 
> Fixes most failing subcases in CTS tests:
> * GL44-CTS.gtf32.GL3Tests.packed_pixels.packed_pixels_pixelstore
> * GL45-CTS.gtf32.GL3Tests.packed_pixels.packed_pixels_pixelstore
> 
> These tests were crashing the driver before. Now they just fail, but due
> to an unrelated issue affecting 2 out of the 45 test subcases.
> 
> No regressions observed against piglit or CTS-GL.
> ---
>  src/mesa/drivers/common/meta.c | 4 ----
>  1 file changed, 4 deletions(-)
> 
> diff --git a/src/mesa/drivers/common/meta.c b/src/mesa/drivers/common/meta.c
> index 890e98a..0d8c667 100644
> --- a/src/mesa/drivers/common/meta.c
> +++ b/src/mesa/drivers/common/meta.c
> @@ -3026,10 +3026,6 @@ decompress_texture_image(struct gl_context *ctx,
>        assert(!"No compressed 1D textures.");
>        return false;
>  
> -   case GL_TEXTURE_3D:
> -      assert(!"No compressed 3D textures.");
> -      return false;
> -
>     case GL_TEXTURE_CUBE_MAP_ARRAY:
>        faceTarget = GL_TEXTURE_CUBE_MAP_POSITIVE_X + (slice % 6);
>        break;
>