<div dir="ltr"><div class="gmail_extra"><div class="gmail_quote">On Fri, May 26, 2017 at 11:42 AM, Ilia Mirkin <span dir="ltr"><<a href="mailto:imirkin@alum.mit.edu" target="_blank">imirkin@alum.mit.edu</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 Tue, May 16, 2017 at 6:45 PM, Jason Ekstrand <<a href="mailto:jason@jlekstrand.net">jason@jlekstrand.net</a>> wrote:<br>
> @@ -118,55 +42,25 @@ copy_miptrees(struct brw_context *brw,<br>
>                int dst_x, int dst_y, int dst_z, unsigned dst_level,<br>
>                int src_width, int src_height)<br>
>  {<br>
> -   unsigned bw, bh;<br>
> -<br>
> -   if (brw->gen >= 6) {<br>
> -      brw_blorp_copy_miptrees(brw,<br>
> -                              src_mt, src_level, src_z,<br>
> -                              dst_mt, dst_level, dst_z,<br>
> -                              src_x, src_y, dst_x, dst_y,<br>
> -                              src_width, src_height);<br>
> -      return;<br>
> +   if (brw->gen < 5) {<br>
> +      /* On gen4-5, try BLT first.<br>
> +       *<br>
> +       * Gen4-5 have a single ring for both 3D and BLT operations, so there's<br>
> +       * no inter-ring synchronization issues like on Gen6+.  It is apparently<br>
> +       * faster than using the 3D pipeline.  Original Gen4 also has to rebase<br>
> +       * and copy miptree slices in order to render to unaligned locations.<br>
> +       */<br>
> +      if (intel_miptree_copy(brw, src_mt, src_level, src_z, src_x, src_y,<br>
> +                             dst_mt, dst_level, dst_z, dst_x, dst_y,<br>
> +                             src_width, src_height))<br>
> +         return;<br>
>     }<br>
<br>
</div></div>Just curious -- was that meant to be<br>
<br>
if (brw->gen <= 5) ?<br>
</blockquote><div><br></div><div>It was... I'll send out a fix-up. <br></div></div></div></div>