<div dir="ltr"><div class="gmail_extra"><div class="gmail_quote">On Wed, Apr 12, 2017 at 4:04 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"><div class="HOEnZb"><div class="h5">On 1 April 2017 at 00:17, Jason Ekstrand <<a href="mailto:jason@jlekstrand.net">jason@jlekstrand.net</a>> wrote:<br>
> Cc: "13.0 17.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 | 12 ++++++++++++<br>
> 1 file changed, 12 insertions(+)<br>
><br>
> diff --git a/src/intel/vulkan/genX_cmd_<wbr>buffer.c b/src/intel/vulkan/genX_cmd_<wbr>buffer.c<br>
> index 1ce549a..b5297f4 100644<br>
> --- a/src/intel/vulkan/genX_cmd_<wbr>buffer.c<br>
> +++ b/src/intel/vulkan/genX_cmd_<wbr>buffer.c<br>
> @@ -597,6 +597,18 @@ genX(BeginCommandBuffer)(<br>
><br>
> genX(cmd_buffer_emit_state_<wbr>base_address)(cmd_buffer);<br>
><br>
> + /* We sometimes store vertex data in the dynamic state buffer for blorp<br>
> + * operations and our dynamic state stream may re-use data from previous<br>
> + * command buffers. In order to prevent stale cache data, we flush the VF<br>
> + * cache. We could do this on every blorp call but that's not really<br>
> + * needed as all of the data will get written by the CPU prior to the GPU<br>
> + * executing anything. The chances are fairly high that they will use<br>
> + * blorp at least once per primary command buffer so it shouldn't be<br>
> + * wasted.<br>
> + */<br>
> + if (cmd_buffer->level == VK_COMMAND_BUFFER_LEVEL_<wbr>PRIMARY)<br>
> + cmd_buffer->state.pending_<wbr>pipe_bits |= ANV_PIPE_VF_CACHE_INVALIDATE_<wbr>BIT;<br>
> +<br>
> VkResult result = VK_SUCCESS;<br>
> if (cmd_buffer->usage_flags &<br>
> VK_COMMAND_BUFFER_USAGE_<wbr>RENDER_PASS_CONTINUE_BIT) {<br>
</div></div>Friendly poke:<br>
<br>
Jason, patches 1&2 are queued for 17.0, yet 3&4 seems to have not made<br>
it in master.<br>
Are they superseded, rejected or fell through the cracks?<span class="HOEnZb"><font color="#888888"><br></font></span></blockquote><div><br></div><div>Just haven't been reviewed. Lionel? <br></div></div><br></div></div>