[Mesa-dev] [PATCH 2/4] mesa: Disallow CopyTexSubImage on stencil formats in ES.

[Brian Paul]

On 04/26/2016 04:25 AM, Kenneth Graunke wrote:
> Fixes
> - ES31-CTS.gtf.GL31Tests.texture_stencil8.texture_stencil8
> - ES31-CTS.gtf.GL31Tests.texture_stencil8.texture_stencil8_multisample
>
> Signed-off-by: Kenneth Graunke <kenneth at whitecape.org>
> ---
>   src/mesa/main/teximage.c | 9 +++++++++
>   1 file changed, 9 insertions(+)
>
> diff --git a/src/mesa/main/teximage.c b/src/mesa/main/teximage.c
> index 6ac6fb1..76b6584 100644
> --- a/src/mesa/main/teximage.c
> +++ b/src/mesa/main/teximage.c
> @@ -2530,6 +2530,15 @@ copytexsubimage_error_check(struct gl_context *ctx, GLuint dimensions,
>         }
>      }
>
> +   /* In the ES 3.2 specification's Table 8.13 (Valid CopyTexImage source
> +    * framebuffer/destination texture base internal format combinations),
> +    * all the entries for stencil are left blank (unsupported).
> +    */
> +   if (_mesa_is_gles(ctx) && _mesa_is_stencil_format(texImage->_BaseFormat)) {
> +      _mesa_error(ctx, GL_INVALID_OPERATION, "%s(stencil disallowed)", caller);
> +      return GL_TRUE;
> +   }
> +
>      /* if we get here, the parameters are OK */
>      return GL_FALSE;
>   }
>

For 2&3, Reviewed-by: Brian Paul <brianp at vmware.com>