<div dir="ltr"><div class="gmail_extra"><div class="gmail_quote">On Mon, Jan 23, 2017 at 4:53 PM, Chad Versace <span dir="ltr"><<a href="mailto:chadversary@chromium.org" target="_blank">chadversary@chromium.org</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div class="HOEnZb"><div class="h5">On Mon 23 Jan 2017, Jason Ekstrand wrote:<br>
> In brw_blorp_copyteximage, we use the format from the render buffer.<br>
> This could be a combined depth/stencil format.  In this case, we handle<br>
> stencil properly but we give blorp the wrong ISL format.  Specifically,<br>
> we would give blorp ISL_FORMAT_R32G32B32A32_FLOAT which is the wrong<br>
> size was causing GPU hangs.<br>
><br>
> Fixes: GL45-CTS.gtf30.GL3Tests.<wbr>packed_depth_stencil.packed_<wbr>depth_stencil_copyteximage<br>
><br>
> Cc: "13.0 17.0" <<a href="mailto:mesa-stable@lists.freedesktop.org">mesa-stable@lists.<wbr>freedesktop.org</a>><br>
> Cc: Kenneth Graunke <<a href="mailto:kenneth@whitecape.org">kenneth@whitecape.org</a>><br>
> ---<br>
>  src/mesa/drivers/dri/i965/brw_<wbr>blorp.c | 2 ++<br>
>  1 file changed, 2 insertions(+)<br>
><br>
> diff --git a/src/mesa/drivers/dri/i965/<wbr>brw_blorp.c b/src/mesa/drivers/dri/i965/<wbr>brw_blorp.c<br>
> index d79f529..3a7cf84 100644<br>
> --- a/src/mesa/drivers/dri/i965/<wbr>brw_blorp.c<br>
> +++ b/src/mesa/drivers/dri/i965/<wbr>brw_blorp.c<br>
> @@ -284,8 +284,10 @@ brw_blorp_to_isl_format(struct brw_context *brw, mesa_format format,<br>
>     case MESA_FORMAT_S_UINT8:<br>
>        return ISL_FORMAT_R8_UINT;<br>
>     case MESA_FORMAT_Z24_UNORM_X8_UINT:<br>
> +   case MESA_FORMAT_Z24_UNORM_S8_UINT:<br>
>        return ISL_FORMAT_R24_UNORM_X8_<wbr>TYPELESS;<br>
>     case MESA_FORMAT_Z_FLOAT32:<br>
> +   case MESA_FORMAT_Z32_FLOAT_S8X24_<wbr>UINT:<br>
>        return ISL_FORMAT_R32_FLOAT;<br>
>     case MESA_FORMAT_Z_UNORM16:<br>
>        return ISL_FORMAT_R16_UNORM;<br>
<br>
</div></div>Locally, this looks correct to me.<br>
<br>
Reviewed-by: Chad Versace <<a href="mailto:chadversary@chromium.org">chadversary@chromium.org</a>><br>
<br>
</blockquote></div><br></div><div class="gmail_extra">Thanks!<br></div></div>