<div dir="ltr"><div class="gmail_extra"><div class="gmail_quote">On Mon, Oct 23, 2017 at 11:07 PM, Pohjolainen, Topi <span dir="ltr"><<a href="mailto:topi.pohjolainen@gmail.com" target="_blank">topi.pohjolainen@gmail.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><span class="">On Mon, Oct 23, 2017 at 05:23:08PM -0700, Jason Ekstrand wrote:<br>
> ---<br>
>  src/mesa/drivers/dri/i965/brw_<wbr>blorp.c | 15 +++++++--------<br>
>  1 file changed, 7 insertions(+), 8 deletions(-)<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 f7d128d..05204a9 100644<br>
> --- a/src/mesa/drivers/dri/i965/<wbr>brw_blorp.c<br>
> +++ b/src/mesa/drivers/dri/i965/<wbr>brw_blorp.c<br>
> @@ -329,6 +329,8 @@ brw_blorp_blit_miptrees(struct brw_context *brw,<br>
>     intel_miptree_prepare_access(<wbr>brw, src_mt, src_level, 1, src_layer, 1,<br>
>                                  src_aux_usage, src_clear_supported);<br>
><br>
> +   enum isl_format dst_isl_format =<br>
> +      brw_blorp_to_isl_format(brw, dst_format, true);<br>
>     enum isl_aux_usage dst_aux_usage =<br>
>        intel_miptree_render_aux_<wbr>usage(brw, dst_mt, encode_srgb, false);<br>
>     const bool dst_clear_supported = dst_aux_usage != ISL_AUX_USAGE_NONE;<br>
> @@ -352,10 +354,9 @@ brw_blorp_blit_miptrees(struct brw_context *brw,<br>
>     struct blorp_batch batch;<br>
>     blorp_batch_init(&brw->blorp, &batch, brw, 0);<br>
>     blorp_blit(&batch, &src_surf, src_level, src_layer,<br>
> -              brw_blorp_to_isl_format(brw, src_format, false), src_isl_swizzle,<br>
> +              src_isl_format, src_isl_swizzle,<br>
<br>
</span>This is functional change, isn't it? It effectively switches from<br>
brw_blorp_to_isl_format() to brw_isl_format_for_mesa_<wbr>format().<br><div><div class="h5"></div></div></blockquote><div><br></div><div>Good catch!  We also need to change the declaration of src_isl_format to use brw_blorp_to_isl_format which, while it looks like a functional change, isn't because the only difference is for render or depth and we explicitly disable HiZ usage.  Would you rather that be a precursor patch?<br></div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div><div class="h5">
>                &dst_surf, dst_level, dst_layer,<br>
> -              brw_blorp_to_isl_format(brw, dst_format, true),<br>
> -              ISL_SWIZZLE_IDENTITY,<br>
> +              dst_isl_format, ISL_SWIZZLE_IDENTITY,<br>
>                src_x0, src_y0, src_x1, src_y1,<br>
>                dst_x0, dst_y0, dst_x1, dst_y1,<br>
>                filter, mirror_x, mirror_y);<br>
> @@ -1158,6 +1159,7 @@ do_single_blorp_clear(struct brw_context *brw, struct gl_framebuffer *fb,<br>
>     mesa_format format = irb->Base.Base.Format;<br>
>     if (!encode_srgb && _mesa_get_format_color_<wbr>encoding(format) == GL_SRGB)<br>
>        format = _mesa_get_srgb_format_linear(<wbr>format);<br>
> +   enum isl_format isl_format = brw->mesa_to_isl_render_<wbr>format[format];<br>
><br>
>     x0 = fb->_Xmin;<br>
>     x1 = fb->_Xmax;<br>
> @@ -1256,8 +1258,7 @@ do_single_blorp_clear(struct brw_context *brw, struct gl_framebuffer *fb,<br>
><br>
>        struct blorp_batch batch;<br>
>        blorp_batch_init(&brw->blorp, &batch, brw, 0);<br>
> -      blorp_fast_clear(&batch, &surf,<br>
> -                       brw->mesa_to_isl_render_<wbr>format[format],<br>
> +      blorp_fast_clear(&batch, &surf, isl_format,<br>
>                         level, irb->mt_layer, num_layers,<br>
>                         x0, y0, x1, y1);<br>
>        blorp_batch_finish(&batch);<br>
> @@ -1290,9 +1291,7 @@ do_single_blorp_clear(struct brw_context *brw, struct gl_framebuffer *fb,<br>
><br>
>        struct blorp_batch batch;<br>
>        blorp_batch_init(&brw->blorp, &batch, brw, 0);<br>
> -      blorp_clear(&batch, &surf,<br>
> -                  brw->mesa_to_isl_render_<wbr>format[format],<br>
> -                  ISL_SWIZZLE_IDENTITY,<br>
> +      blorp_clear(&batch, &surf, isl_format, ISL_SWIZZLE_IDENTITY,<br>
>                    level, irb->mt_layer, num_layers,<br>
>                    x0, y0, x1, y1,<br>
>                    clear_color, color_write_disable);<br>
> --<br>
> 2.5.0.400.gff86faf<br>
><br>
</div></div>> ______________________________<wbr>_________________<br>
> mesa-dev mailing list<br>
> <a href="mailto:mesa-dev@lists.freedesktop.org">mesa-dev@lists.freedesktop.org</a><br>
> <a href="https://lists.freedesktop.org/mailman/listinfo/mesa-dev" rel="noreferrer" target="_blank">https://lists.freedesktop.org/<wbr>mailman/listinfo/mesa-dev</a><br>
</blockquote></div><br></div></div>