diff options
author | Bas Nieuwenhuizen <bas@basnieuwenhuizen.nl> | 2018-09-16 02:50:34 +0200 |
---|---|---|
committer | Bas Nieuwenhuizen <bas@basnieuwenhuizen.nl> | 2018-09-17 20:19:52 +0200 |
commit | d17443a459383fdd7018fa879134f05b5daff1e1 (patch) | |
tree | 6fa16255c4d095804dc04929d3bbb806d9de4fe8 | |
parent | 08103c5f652b7f0a5fa85c6b7067cd08ec19444a (diff) |
radv: Use build ID if available for cache UUID.
To get an useful UUID for systems that have a non-useful mtime
for the binaries.
I started using SHA1 to ensure we get reasonable mixing in the
various possibilities and the various build id lengths.
CC: <mesa-stable@lists.freedesktop.org>
Reviewed-by: Timothy Arceri <tarceri@itsqueeze.com>
-rw-r--r-- | src/amd/vulkan/radv_device.c | 43 |
1 files changed, 35 insertions, 8 deletions
diff --git a/src/amd/vulkan/radv_device.c b/src/amd/vulkan/radv_device.c index af7754bea3e..e92edec930c 100644 --- a/src/amd/vulkan/radv_device.c +++ b/src/amd/vulkan/radv_device.c @@ -45,22 +45,49 @@ #include "sid.h" #include "gfx9d.h" #include "addrlib/gfx9/chip/gfx9_enum.h" +#include "util/build_id.h" #include "util/debug.h" +#include "util/mesa-sha1.h" + +static bool +radv_get_build_id(void *ptr, struct mesa_sha1 *ctx) +{ + uint32_t timestamp; + +#ifdef HAVE_DL_ITERATE_PHDR + const struct build_id_note *note = NULL; + if ((note = build_id_find_nhdr_for_addr(ptr))) { + _mesa_sha1_update(ctx, build_id_data(note), build_id_length(note)); + } else +#endif + if (disk_cache_get_function_timestamp(ptr, ×tamp)) { + if (!timestamp) { + fprintf(stderr, "radv: The provided filesystem timestamp for the cache is bogus!\n"); + } + + _mesa_sha1_update(ctx, ×tamp, sizeof(timestamp)); + } else + return false; + return true; +} static int radv_device_get_cache_uuid(enum radeon_family family, void *uuid) { - uint32_t mesa_timestamp, llvm_timestamp; - uint16_t f = family; + struct mesa_sha1 ctx; + unsigned char sha1[20]; + unsigned ptr_size = sizeof(void*); memset(uuid, 0, VK_UUID_SIZE); - if (!disk_cache_get_function_timestamp(radv_device_get_cache_uuid, &mesa_timestamp) || - !disk_cache_get_function_timestamp(LLVMInitializeAMDGPUTargetInfo, &llvm_timestamp)) + + if (!radv_get_build_id(radv_device_get_cache_uuid, &ctx) || + !radv_get_build_id(LLVMInitializeAMDGPUTargetInfo, &ctx)) return -1; - memcpy(uuid, &mesa_timestamp, 4); - memcpy((char*)uuid + 4, &llvm_timestamp, 4); - memcpy((char*)uuid + 8, &f, 2); - snprintf((char*)uuid + 10, VK_UUID_SIZE - 10, "radv%zd", sizeof(void *)); + _mesa_sha1_update(&ctx, &family, sizeof(family)); + _mesa_sha1_update(&ctx, &ptr_size, sizeof(ptr_size)); + _mesa_sha1_final(&ctx, sha1); + + memcpy(uuid, sha1, VK_UUID_SIZE); return 0; } |