<p dir="ltr"><br>
On Jun 16, 2015 11:15 AM, "Anuj Phogat" <<a href="mailto:anuj.phogat@gmail.com">anuj.phogat@gmail.com</a>> wrote:<br>
><br>
> Without this patch, arb_color_buffer_float-readpixels test fails, when<br>
> forced to use 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 | 10 ++++++----<br>
>  1 file changed, 6 insertions(+), 4 deletions(-)<br>
><br>
> diff --git a/src/mesa/drivers/common/meta_tex_subimage.c b/src/mesa/drivers/common/meta_tex_subimage.c<br>
> index d2474f5..00364f8 100644<br>
> --- a/src/mesa/drivers/common/meta_tex_subimage.c<br>
> +++ b/src/mesa/drivers/common/meta_tex_subimage.c<br>
> @@ -273,12 +273,14 @@ _mesa_meta_pbo_GetTexSubImage(struct gl_context *ctx, GLuint dims,<br>
>         format == GL_COLOR_INDEX)<br>
>        return false;<br>
><br>
> -   if (ctx->_ImageTransferState)<br>
> -      return false;<br>
> -<br>
> -<br>
> +   /* Don't use meta path for readpixels in below conditions. */</p>
<p dir="ltr">A more descriptive comment would be nice.</p>
<p dir="ltr">>     if (!tex_image) {<br>
>        rb = ctx->ReadBuffer->_ColorReadBuffer;<br>
> +<br>
> +      if (_mesa_get_readpixels_transfer_ops(ctx, rb->Format, format,<br>
> +                                            type, GL_FALSE))<br>
> +         return false;<br>
> +<br>
>        if (_mesa_need_rgb_to_luminance_conversion(rb->Format, format))<br>
>           return false;<br>
>     }<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>