anv: Stop setting domains to RENDER on EXEC_OBJECT_WRITE
The reason we were doing this was to ensure that the kernel did the appropriate cross-ring synchronization and flushing. However, the kernel only looks at EXEC_OBJECT_WRITE to determine whether or not to insert a fence. It only cares about the domain for determining whether or not it needs to clflush the BO before using it for scanout but the domain automatically gets set to RENDER internally by the kernel if EXEC_OBJECT_WRITE is set. Reviewed-by: Chris Wilson <chris@chris-wilson.co.uk>
This commit is contained in:
parent
6c7b7aa3d8
commit
781263486f
|
@ -148,9 +148,6 @@ anv_reloc_list_add(struct anv_reloc_list *list,
|
||||||
struct drm_i915_gem_relocation_entry *entry;
|
struct drm_i915_gem_relocation_entry *entry;
|
||||||
int index;
|
int index;
|
||||||
|
|
||||||
const uint32_t domain =
|
|
||||||
(target_bo->flags & EXEC_OBJECT_WRITE) ? I915_GEM_DOMAIN_RENDER : 0;
|
|
||||||
|
|
||||||
VkResult result = anv_reloc_list_grow(list, alloc, 1);
|
VkResult result = anv_reloc_list_grow(list, alloc, 1);
|
||||||
if (result != VK_SUCCESS)
|
if (result != VK_SUCCESS)
|
||||||
return result;
|
return result;
|
||||||
|
@ -163,8 +160,8 @@ anv_reloc_list_add(struct anv_reloc_list *list,
|
||||||
entry->delta = delta;
|
entry->delta = delta;
|
||||||
entry->offset = offset;
|
entry->offset = offset;
|
||||||
entry->presumed_offset = target_bo->offset;
|
entry->presumed_offset = target_bo->offset;
|
||||||
entry->read_domains = domain;
|
entry->read_domains = 0;
|
||||||
entry->write_domain = domain;
|
entry->write_domain = 0;
|
||||||
VG(VALGRIND_CHECK_MEM_IS_DEFINED(entry, sizeof(*entry)));
|
VG(VALGRIND_CHECK_MEM_IS_DEFINED(entry, sizeof(*entry)));
|
||||||
|
|
||||||
return VK_SUCCESS;
|
return VK_SUCCESS;
|
||||||
|
|
Loading…
Reference in New Issue