<div dir="ltr">On 3 June 2013 09:02, Eric Anholt <span dir="ltr"><<a href="mailto:eric@anholt.net" target="_blank">eric@anholt.net</a>></span> wrote:<br><div class="gmail_extra"><div class="gmail_quote"><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">

<div><div>Paul Berry <<a href="mailto:stereotype441@gmail.com" target="_blank">stereotype441@gmail.com</a>> writes:<br>
<br>
> The meta path for CopyTexSubImage operates by reading the source image<br>
> to a temporary buffer using glReadPixels and then uploading it to the<br>
> texture using TexSubImage.  This patch ensures that when the texture<br>
> format is MESA_FORMAT_Z32_FLOAT_X24S8, we use the proper image type<br>
> for the temporary buffer (GL_FLOAT_32_UNSIGNED_INT_24_8_REV).<br>
><br>
> Fixes some failures with piglit tests "copyteximage<br>
> {1D,2D,CUBE,RECT,2D_ARRAY}".<br>
> ---<br>
>  src/mesa/drivers/common/meta.c | 5 ++++-<br>
>  1 file changed, 4 insertions(+), 1 deletion(-)<br>
><br>
> diff --git a/src/mesa/drivers/common/meta.c b/src/mesa/drivers/common/meta.c<br>
> index ca5f5a1..637ec70 100644<br>
> --- a/src/mesa/drivers/common/meta.c<br>
> +++ b/src/mesa/drivers/common/meta.c<br>
> @@ -3767,7 +3767,10 @@ get_temp_image_type(struct gl_context *ctx, gl_format format)<br>
>     case GL_DEPTH_COMPONENT:<br>
>        return GL_UNSIGNED_INT;<br>
>     case GL_DEPTH_STENCIL:<br>
> -      return GL_UNSIGNED_INT_24_8;<br>
> +      if (format == MESA_FORMAT_Z32_FLOAT_X24S8)<br>
> +         return GL_FLOAT_32_UNSIGNED_INT_24_8_REV;<br>
> +      else<br>
> +         return GL_UNSIGNED_INT_24_8;<br>
>     default:<br>
>        _mesa_problem(ctx, "Unexpected format %d in get_temp_image_type()",<br>
>                   baseFormat);<br>
> --<br>
> 1.8.3<br>
<br>
</div></div>I think the version of this patch I sent out on Thursday is better.<br>
</blockquote></div><br></div><div class="gmail_extra">Do you mean "[PATCH 2/2] meta: Fix temporary image type for float depth/stencil."?  I'm ok going with your version instead.  It looks like it got a positive review on Thrusday--when are you planning to push it?<br>

</div></div>