<p dir="ltr"><br>
On Jun 16, 2015 11:15, "Anuj Phogat" <<a href="mailto:anuj.phogat@gmail.com">anuj.phogat@gmail.com</a>> wrote:<br>
><br>
> Without this patch, piglit test fbo_integer_readpixels_sint_uint fails, when<br>
> forced to use the meta pbo path.<br>
><br>
> Signed-off-by: Anuj Phogat <<a href="mailto:anuj.phogat@gmail.com">anuj.phogat@gmail.com</a>><br>
> Cc: <<a href="mailto:mesa-stable@lists.freedesktop.org">mesa-stable@lists.freedesktop.org</a>><br>
> ---<br>
>  src/mesa/drivers/common/meta_tex_subimage.c | 3 +++<br>
>  1 file changed, 3 insertions(+)<br>
><br>
> diff --git a/src/mesa/drivers/common/meta_tex_subimage.c b/src/mesa/drivers/common/meta_tex_subimage.c<br>
> index 00364f8..84cbc50 100644<br>
> --- a/src/mesa/drivers/common/meta_tex_subimage.c<br>
> +++ b/src/mesa/drivers/common/meta_tex_subimage.c<br>
> @@ -283,6 +283,9 @@ _mesa_meta_pbo_GetTexSubImage(struct gl_context *ctx, GLuint dims,<br>
><br>
>        if (_mesa_need_rgb_to_luminance_conversion(rb->Format, format))<br>
>           return false;<br>
> +<br>
> +      if (_mesa_need_signed_unsigned_int_conversion(rb->Format, format, type))<br>
> +         return false;</p>
<p dir="ltr">Hrm... This seems fishy.  Isn't glBlitFramebuffers supposed to handle format conversion with integers?  If so we should probably fix it rather than just skip it for the meta pbo path.</p>
<p dir="ltr">>     }<br>
><br>
>     /* For arrays, use a tall (height * depth) 2D texture but taking into<br>
> --<br>
> 1.9.3<br>
><br>
> _______________________________________________<br>
> mesa-dev mailing list<br>
> <a href="mailto:mesa-dev@lists.freedesktop.org">mesa-dev@lists.freedesktop.org</a><br>
> <a href="http://lists.freedesktop.org/mailman/listinfo/mesa-dev">http://lists.freedesktop.org/mailman/listinfo/mesa-dev</a><br>
</p>