<div dir="ltr"><div class="gmail_extra"><div class="gmail_quote">On Thu, May 11, 2017 at 12:42 AM, Chris Wilson <span dir="ltr"><<a href="mailto:chris@chris-wilson.co.uk" target="_blank">chris@chris-wilson.co.uk</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 Wed, May 10, 2017 at 04:08:55PM -0700, Jason Ekstrand wrote:<br>
> We use the look-up table mechanism for relocations so this isn't the<br>
> value we want.  The correct value is filled out at execbuf2 time by<br>
> anv_cmd_buffer_process_relocs.<br>
> ---<br>
>  src/intel/vulkan/anv_batch_<wbr>chain.c | 2 +-<br>
>  1 file changed, 1 insertion(+), 1 deletion(-)<br>
><br>
> diff --git a/src/intel/vulkan/anv_batch_<wbr>chain.c b/src/intel/vulkan/anv_batch_<wbr>chain.c<br>
> index cc4f92e..5811503 100644<br>
> --- a/src/intel/vulkan/anv_batch_<wbr>chain.c<br>
> +++ b/src/intel/vulkan/anv_batch_<wbr>chain.c<br>
> @@ -165,7 +165,7 @@ anv_reloc_list_add(struct anv_reloc_list *list,<br>
<br>
</span>Just before this is a comment talking about using LUT - you already are.<br>
It's relevant as the invalid target_handle for LUT is -1 and 0 for !LUT.<span class="HOEnZb"><font color="#888888"><br>
</font></span></blockquote></div><br></div><div class="gmail_extra">Right.  So now the kernel will fail to execbuf if we ever don't fill the value out rather than just doing the wrong thing.  Would you like a comment about that in the commit message or something?<br></div></div>