commit 6ebf677cfd80e0df8cc6042734823cf53ce1b941
author Jason Ekstrand <jason@jlekstrand.net> Thu Nov 21 06:00:14 2019 -0600
committer Jason Ekstrand <jason@jlekstrand.net> Fri Dec 06 19:58:07 2019 +0000
tree 1a7f59eed61b8ba8c5be6ea84c5cfeeb1be2e423
parent 778b51f491cfe56da463195e1392293379b9fe26

anv: Always add in EXEC_OBJECT_WRITE when specified in extra_flags

Otherwise, we're trusting in the execbuf_add_bo which sets
EXEC_OBJECT_WRITE to to always be the first one that gets called.  This
is likely true for fences but it seems somewhat fragile.

Reviewed-by: Lionel Landwerlin <lionel.g.landwerlin@intel.com>

src/intel/vulkan/anv_batch_chain.c

diff --git a/src/intel/vulkan/anv_batch_chain.c b/src/intel/vulkan/anv_batch_chain.c
index f58299b..67bd4df 100644
--- a/src/intel/vulkan/anv_batch_chain.c
+++ b/src/intel/vulkan/anv_batch_chain.c
@@ -1136,6 +1136,11 @@
       obj->rsvd2 = 0;
    }
 
+   if (extra_flags & EXEC_OBJECT_WRITE) {
+      obj->flags |= EXEC_OBJECT_WRITE;
+      obj->flags &= ~EXEC_OBJECT_ASYNC;
+   }
+
    if (relocs != NULL) {
       assert(obj->relocation_count == 0);