<div dir="ltr"><div class="gmail_extra"><div class="gmail_quote">On Tue, May 16, 2017 at 11:16 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 Tue, May 16, 2017 at 03:44:59PM -0700, Jason Ekstrand wrote:<br>
> The guts of blorp and ISL don't understand i965's partial miptrees.<br>
> Instead, we need to subtract off first_level before we hand anything off<br>
> to blorp.<br>
> ---<br>
>  src/mesa/drivers/dri/i965/brw_<wbr>blorp.c | 3 +++<br>
>  1 file changed, 3 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 ebc4612..64aa4c9 100644<br>
> --- a/src/mesa/drivers/dri/i965/<wbr>brw_blorp.c<br>
> +++ b/src/mesa/drivers/dri/i965/<wbr>brw_blorp.c<br>
> @@ -271,6 +271,9 @@ blorp_surf_for_miptree(struct brw_context *brw,<br>
>     }<br>
>     assert((surf->aux_usage == ISL_AUX_USAGE_NONE) ==<br>
>            (surf->aux_addr.buffer == NULL));<br>
<br>
</span>Here it would be nice to have an assert:<br>
<br>
      assert(*level >= mt->first_level);<span class=""><br></span></blockquote><div><br></div><div>I'm happy to add that explicitly here.  However it is already checked above by check_level_layer.<br></div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><span class="">
> +<br>
> +   /* ISL wants real levels, not offset ones. */<br>
> +   *level -= mt->first_level;<br>
>  }<br>
><br>
>  static enum isl_format<br>
> --<br>
> 2.5.0.400.gff86faf<br>
><br>
</span>> ______________________________<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>