Mesa (master): radv: align pipeline cache entry and header sizes
GitLab Mirror
gitlab-mirror at kemper.freedesktop.org
Thu Aug 20 11:09:52 UTC 2020
Module: Mesa
Branch: master
Commit: b50ae770144ef6622591c7cc23aa96e45933cf37
URL: http://cgit.freedesktop.org/mesa/mesa/commit/?id=b50ae770144ef6622591c7cc23aa96e45933cf37
Author: Rhys Perry <pendingchaos02 at gmail.com>
Date: Thu Aug 6 14:39:04 2020 +0100
radv: align pipeline cache entry and header sizes
Fixes UBSan error:
../src/amd/vulkan/radv_pipeline_cache.c:603:42: runtime error: member access within misaligned address 0x00000152ebcc for type 'struct cache_entry', which requires 8 byte alignment
Signed-off-by: Rhys Perry <pendingchaos02 at gmail.com>
Reviewed-by: Samuel Pitoiset <samuel.pitoiset at gmail.com>
Part-of: <https://gitlab.freedesktop.org/mesa/mesa/-/merge_requests/6206>
---
src/amd/vulkan/radv_pipeline_cache.c | 5 ++++-
1 file changed, 4 insertions(+), 1 deletion(-)
diff --git a/src/amd/vulkan/radv_pipeline_cache.c b/src/amd/vulkan/radv_pipeline_cache.c
index fe38f301009..a8563496325 100644
--- a/src/amd/vulkan/radv_pipeline_cache.c
+++ b/src/amd/vulkan/radv_pipeline_cache.c
@@ -21,6 +21,7 @@
* IN THE SOFTWARE.
*/
+#include "util/macros.h"
#include "util/mesa-sha1.h"
#include "util/debug.h"
#include "util/disk_cache.h"
@@ -108,6 +109,7 @@ entry_size(struct cache_entry *entry)
for (int i = 0; i < MESA_SHADER_STAGES; ++i)
if (entry->binary_sizes[i])
ret += entry->binary_sizes[i];
+ ret = align(ret, alignof(struct cache_entry));
return ret;
}
@@ -391,6 +393,7 @@ radv_pipeline_cache_insert_shaders(struct radv_device *device,
for (int i = 0; i < MESA_SHADER_STAGES; ++i)
if (variants[i])
size += binaries[i]->total_size;
+ size = align(size, alignof(struct cache_entry));
entry = vk_alloc(&cache->alloc, size, 8,
@@ -580,7 +583,7 @@ VkResult radv_GetPipelineCacheData(
}
void *p = pData, *end = pData + *pDataSize;
header = p;
- header->header_size = sizeof(*header);
+ header->header_size = align(sizeof(*header), alignof(struct cache_entry));
header->header_version = VK_PIPELINE_CACHE_HEADER_VERSION_ONE;
header->vendor_id = ATI_VENDOR_ID;
header->device_id = device->physical_device->rad_info.pci_id;
More information about the mesa-commit
mailing list