From 6049fa454e4752a527cb8e13779082b27adcfb97 Mon Sep 17 00:00:00 2001 From: Dave Airlie Date: Thu, 12 Oct 2017 14:24:41 +1000 Subject: [PATCH] radv: don't crash if cache is disabled. If you set MESA_GLSL_CACHE_DISABLE, radv crashed. Fixes: fd24be134f (radv: make use of on-disk cache) Signed-off-by: Dave Airlie Reviewed-by: Timothy Arceri --- src/amd/vulkan/radv_pipeline_cache.c | 14 +++++++++----- 1 file changed, 9 insertions(+), 5 deletions(-) diff --git a/src/amd/vulkan/radv_pipeline_cache.c b/src/amd/vulkan/radv_pipeline_cache.c index 51fa3f1b31f..fabff9ed815 100644 --- a/src/amd/vulkan/radv_pipeline_cache.c +++ b/src/amd/vulkan/radv_pipeline_cache.c @@ -167,6 +167,8 @@ radv_create_shader_variant_from_pipeline_cache(struct radv_device *device, entry = radv_pipeline_cache_search(device->mem_cache, sha1); if (!entry) { + if (!device->physical_device->disk_cache) + return NULL; uint8_t disk_sha1[20]; disk_cache_compute_key(device->physical_device->disk_cache, sha1, 20, disk_sha1); @@ -317,11 +319,13 @@ radv_pipeline_cache_insert_shader(struct radv_device *device, * compiled shaders by third parties such as steam, even if the app * implements its own pipeline cache. */ - uint8_t disk_sha1[20]; - disk_cache_compute_key(device->physical_device->disk_cache, sha1, 20, - disk_sha1); - disk_cache_put(device->physical_device->disk_cache, - disk_sha1, entry, entry_size(entry), NULL); + if (device->physical_device->disk_cache) { + uint8_t disk_sha1[20]; + disk_cache_compute_key(device->physical_device->disk_cache, sha1, 20, + disk_sha1); + disk_cache_put(device->physical_device->disk_cache, + disk_sha1, entry, entry_size(entry), NULL); + } entry->variant = variant; p_atomic_inc(&variant->ref_count); -- 2.30.2