<div dir="ltr"><div class="gmail_extra"><div class="gmail_quote">On Tue, Mar 28, 2017 at 7:07 AM, Emil Velikov <span dir="ltr"><<a href="mailto:emil.l.velikov@gmail.com" target="_blank">emil.l.velikov@gmail.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Hi all,<br>
<div><div class="h5"><br>
On 15 March 2017 at 18:58, Jason Ekstrand <<a href="mailto:jason@jlekstrand.net">jason@jlekstrand.net</a>> wrote:<br>
> The programming note that says we need to do this still exists in the<br>
> SkyLake PRM and, from looking at the bspec, seems to apply to all<br>
> hardware generations SNB+.  Also, this seems to be the cause of some of<br>
> the GPU hangs we've been seeing in GL with the "Car Chase" benchmark.<br>
><br>
> Reported-by: Topi Pohjolainen <<a href="mailto:topi.pohjolainen@intel.com">topi.pohjolainen@intel.com</a>><br>
> Cc: "17.0 13.0" <<a href="mailto:mesa-stable@lists.freedesktop.org">mesa-stable@lists.<wbr>freedesktop.org</a>><br>
> ---<br>
>  src/intel/vulkan/genX_cmd_<wbr>buffer.c | 3 +--<br>
>  1 file changed, 1 insertion(+), 2 deletions(-)<br>
><br>
> diff --git a/src/intel/vulkan/genX_cmd_<wbr>buffer.c b/src/intel/vulkan/genX_cmd_<wbr>buffer.c<br>
> index 2e95b98..a23fcba 100644<br>
> --- a/src/intel/vulkan/genX_cmd_<wbr>buffer.c<br>
> +++ b/src/intel/vulkan/genX_cmd_<wbr>buffer.c<br>
> @@ -2040,8 +2040,8 @@ flush_pipeline_before_<wbr>pipeline_select(struct anv_cmd_buffer *cmd_buffer,<br>
>      */<br>
>     if (pipeline == GPGPU)<br>
>        anv_batch_emit(&cmd_buffer-><wbr>batch, GENX(3DSTATE_CC_STATE_<wbr>POINTERS), t);<br>
> +#endif<br>
><br>
> -#elif GEN_GEN <= 7<br>
>     /* From "BXML » GT » MI » vol1a GPU Overview » [Instruction]<br>
>      * PIPELINE_SELECT [DevBWR+]":<br>
>      *<br>
> @@ -2067,7 +2067,6 @@ flush_pipeline_before_<wbr>pipeline_select(struct anv_cmd_buffer *cmd_buffer,<br>
>        pc.<wbr>InstructionCacheInvalidateEnab<wbr>le = true;<br>
>        pc.PostSyncOperation                = NoWrite;<br>
>     }<br>
> -#endif<br>
<br>
</div></div>Seems like this patch has fallen through the cracks, right ?<span class="HOEnZb"><font color="#888888"><br></font></span></blockquote><div><br></div><div>Yes it did.  Thanks!  I just pushed it (with a better commit message)<br><br></div><div>--Jason <br></div></div><br></div></div>