Revert "Revert "radv: fallback to an in-memory cache when no pipline cache is provided""
I tested this 10 times with ./deqp-vk --deqp-case=dEQP-VK.texture.filtering.3d.formats.r4g4b4a4* and one full run of CTS, seems the issue is gone. Also reduces CTS runtime by 30% or so. Reviewed-by: Timothy Arceri <tarceri@itsqueeze.com>
This commit is contained in:
@@ -153,7 +153,8 @@ radv_pipeline_compile(struct radv_pipeline *pipeline,
|
||||
|
||||
if (pipeline->gs_copy_shader) {
|
||||
pipeline->gs_copy_shader =
|
||||
radv_pipeline_cache_insert_shader(cache,
|
||||
radv_pipeline_cache_insert_shader(pipeline->device,
|
||||
cache,
|
||||
gs_copy_sha1,
|
||||
pipeline->gs_copy_shader,
|
||||
gs_copy_code,
|
||||
@@ -166,8 +167,10 @@ radv_pipeline_compile(struct radv_pipeline *pipeline,
|
||||
ralloc_free(nir);
|
||||
|
||||
if (variant)
|
||||
variant = radv_pipeline_cache_insert_shader(cache, sha1, variant,
|
||||
code, code_size);
|
||||
variant = radv_pipeline_cache_insert_shader(pipeline->device,
|
||||
cache, sha1,
|
||||
variant, code,
|
||||
code_size);
|
||||
|
||||
if (code)
|
||||
free(code);
|
||||
@@ -290,11 +293,11 @@ radv_tess_pipeline_compile(struct radv_pipeline *pipeline,
|
||||
ralloc_free(tcs_nir);
|
||||
|
||||
if (tes_variant)
|
||||
tes_variant = radv_pipeline_cache_insert_shader(cache, tes_sha1, tes_variant,
|
||||
tes_variant = radv_pipeline_cache_insert_shader(pipeline->device, cache, tes_sha1, tes_variant,
|
||||
tes_code, tes_code_size);
|
||||
|
||||
if (tcs_variant)
|
||||
tcs_variant = radv_pipeline_cache_insert_shader(cache, tcs_sha1, tcs_variant,
|
||||
tcs_variant = radv_pipeline_cache_insert_shader(pipeline->device, cache, tcs_sha1, tcs_variant,
|
||||
tcs_code, tcs_code_size);
|
||||
|
||||
if (tes_code)
|
||||
|
@@ -162,6 +162,8 @@ radv_create_shader_variant_from_pipeline_cache(struct radv_device *device,
|
||||
|
||||
if (cache)
|
||||
entry = radv_pipeline_cache_search(cache, sha1);
|
||||
else
|
||||
entry = radv_pipeline_cache_search(device->mem_cache, sha1);
|
||||
|
||||
if (!entry)
|
||||
return NULL;
|
||||
@@ -262,13 +264,14 @@ radv_pipeline_cache_add_entry(struct radv_pipeline_cache *cache,
|
||||
}
|
||||
|
||||
struct radv_shader_variant *
|
||||
radv_pipeline_cache_insert_shader(struct radv_pipeline_cache *cache,
|
||||
radv_pipeline_cache_insert_shader(struct radv_device *device,
|
||||
struct radv_pipeline_cache *cache,
|
||||
const unsigned char *sha1,
|
||||
struct radv_shader_variant *variant,
|
||||
const void *code, unsigned code_size)
|
||||
{
|
||||
if (!cache)
|
||||
return variant;
|
||||
cache = device->mem_cache;
|
||||
|
||||
pthread_mutex_lock(&cache->mutex);
|
||||
struct cache_entry *entry = radv_pipeline_cache_search_unlocked(cache, sha1);
|
||||
|
@@ -327,7 +327,8 @@ radv_create_shader_variant_from_pipeline_cache(struct radv_device *device,
|
||||
const unsigned char *sha1);
|
||||
|
||||
struct radv_shader_variant *
|
||||
radv_pipeline_cache_insert_shader(struct radv_pipeline_cache *cache,
|
||||
radv_pipeline_cache_insert_shader(struct radv_device *device,
|
||||
struct radv_pipeline_cache *cache,
|
||||
const unsigned char *sha1,
|
||||
struct radv_shader_variant *variant,
|
||||
const void *code, unsigned code_size);
|
||||
|
Reference in New Issue
Block a user