<div dir="ltr"><br><div class="gmail_extra"><br><div class="gmail_quote">On Mon, Jun 20, 2016 at 4:46 PM, Chad Versace <span dir="ltr"><<a href="mailto:chad.versace@intel.com" target="_blank">chad.versace@intel.com</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 Sat 11 Jun 2016, Jason Ekstrand wrote:<br>
> ---<br>
>  src/intel/isl/isl.h               | 3 +++<br>
>  src/intel/isl/isl_surface_state.c | 9 +++++++++<br>
>  2 files changed, 12 insertions(+)<br>
><br>
> diff --git a/src/intel/isl/isl.h b/src/intel/isl/isl.h<br>
> index a987482..4dd4a2f 100644<br>
> --- a/src/intel/isl/isl.h<br>
> +++ b/src/intel/isl/isl.h<br>
> @@ -844,6 +844,9 @@ struct isl_surf_fill_state_info {<br>
>      * Valid values depend on hardware generation.<br>
>      */<br>
>     union isl_color_value clear_color;<br>
> +<br>
> +   /* Intra-tile offset */<br>
> +   uint16_t x_offset, y_offset;<br>
>  };<br>
><br>
>  struct isl_buffer_fill_state_info {<br>
> diff --git a/src/intel/isl/isl_surface_state.c b/src/intel/isl/isl_surface_state.c<br>
> index 9bfc55f..65e4b8e 100644<br>
> --- a/src/intel/isl/isl_surface_state.c<br>
> +++ b/src/intel/isl/isl_surface_state.c<br>
> @@ -401,6 +401,15 @@ isl_genX(surf_fill_state_s)(const struct isl_device *dev, void *state,<br>
>     s.MOCS = info->mocs;<br>
>  #endif<br>
><br>
> +#if GEN_GEN > 4 || GEN_IS_G4X<br>
> +   const unsigned x_div = 4;<br>
> +   const unsigned y_div = GEN_GEN >= 8 ? 4 : 2;<br>
> +   assert(info->x_offset % x_div == 0);<br>
> +   assert(info->y_offset % y_div == 0);<br>
> +   s.XOffset = info->x_offset / x_div;<br>
> +   s.YOffset = info->y_offset / y_div;<br>
> +#endif<br>
<br>
</div></div>Perhaps add...<br>
<br>
   #else<br>
      assert(info->x_offset == 0);<br>
      assert(info->y_offset == 0);<br>
   #endif<br></blockquote><div><br></div><div>I added exactly that code as I was rebasing in the G4X fixes in the gen4 patch.<br></div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<br>
Or maybe not. You've thought about it more than I have.<br>
<br>
Either way, patch 39 is<br>
Reviewed-by: Chad Versace <<a href="mailto:chad.versace@intel.com">chad.versace@intel.com</a>><br>
</blockquote></div><br></div></div>