anv: add a comment describing has_relocs field
authorLionel Landwerlin <lionel.g.landwerlin@intel.com>
Sun, 6 Dec 2020 20:23:17 +0000 (22:23 +0200)
committerMarge Bot <eric+marge@anholt.net>
Tue, 9 Feb 2021 21:08:52 +0000 (21:08 +0000)
Signed-off-by: Lionel Landwerlin <lionel.g.landwerlin@intel.com>
Reviewed-by: Jason Ekstrand <jason@jlekstrand.net>
Part-of: <https://gitlab.freedesktop.org/mesa/mesa/-/merge_requests/8934>

src/intel/vulkan/anv_batch_chain.c

index fd19793..5ea24bb 100644 (file)
@@ -1110,6 +1110,11 @@ struct anv_execbuf {
    /* Allocated length of the 'objects' and 'bos' arrays */
    uint32_t                                  array_length;
 
+   /* Indicates whether any of the command buffers have relocations. This
+    * doesn't not necessarily mean we'll need the kernel to process them. It
+    * might be that a previous execbuf has already placed things in the VMA
+    * and we can make i915 skip the relocations.
+    */
    bool                                      has_relocs;
 
    const VkAllocationCallbacks *             alloc;