Revert "anv: Disable snooping for allocator pools again"
authorKristian Høgsberg Kristensen <kristian.h.kristensen@intel.com>
Wed, 17 Feb 2016 07:19:27 +0000 (23:19 -0800)
committerKristian Høgsberg Kristensen <kristian.h.kristensen@intel.com>
Wed, 17 Feb 2016 07:23:49 +0000 (23:23 -0800)
This reverts commit c136672c593399e831db0da8b8cc6d8a5d73c1d3.

We still have the intermittent missing flush for VkEvent in certain
vulkancts cases:

  piglit.deqp-vk.api.command_buffers.execute_large_primary
  piglit.deqp-vk.api.command_buffers.submit_count_non_zero,

Let's reenable the snooping until we figure out the root cause.

src/vulkan/anv_allocator.c

index 1f858bbdff4fa886f283dc44fca3b89049944665..a7ae975656b314fcf982117d42d3a003a5b9aa64 100644 (file)
@@ -441,6 +441,7 @@ anv_block_pool_grow(struct anv_block_pool *pool, struct anv_block_state *state)
       goto fail;
    cleanup->gem_handle = gem_handle;
 
+#if 0
    /* Regular objects are created I915_CACHING_CACHED on LLC platforms and
     * I915_CACHING_NONE on non-LLC platforms. However, userptr objects are
     * always created as I915_CACHING_CACHED, which on non-LLC means
@@ -452,6 +453,7 @@ anv_block_pool_grow(struct anv_block_pool *pool, struct anv_block_state *state)
       anv_gem_set_domain(pool->device, gem_handle,
                          I915_GEM_DOMAIN_GTT, I915_GEM_DOMAIN_GTT);
    }
+#endif
 
    /* Now that we successfull allocated everything, we can write the new
     * values back into pool. */