From 7dcac3d55bc667659687a2ff14b0de28d0ede8ae Mon Sep 17 00:00:00 2001 From: Olivia Lee Date: Mon, 14 Jul 2025 11:45:47 -0700 Subject: [PATCH] panvk: add error checking for dump/trace mmap call Fixes: c257bf51427 ("panvk: Conditionally register an host address when tracking user memory") Signed-off-by: Olivia Lee Reviewed-by: Mary Guillemard Reviewed-by: Erik Faye-Lund Part-of: --- src/panfrost/vulkan/panvk_device_memory.c | 10 +++++++--- 1 file changed, 7 insertions(+), 3 deletions(-) diff --git a/src/panfrost/vulkan/panvk_device_memory.c b/src/panfrost/vulkan/panvk_device_memory.c index ee65b74f0b4..10edf39245b 100644 --- a/src/panfrost/vulkan/panvk_device_memory.c +++ b/src/panfrost/vulkan/panvk_device_memory.c @@ -145,9 +145,13 @@ panvk_AllocateMemory(VkDevice _device, if (device->debug.decode_ctx) { if (instance->debug_flags & (PANVK_DEBUG_DUMP | PANVK_DEBUG_TRACE)) { - mem->debug.host_mapping = - pan_kmod_bo_mmap(mem->bo, 0, pan_kmod_bo_size(mem->bo), - PROT_READ | PROT_WRITE, MAP_SHARED, NULL); + void *cpu = pan_kmod_bo_mmap(mem->bo, 0, pan_kmod_bo_size(mem->bo), + PROT_READ | PROT_WRITE, MAP_SHARED, NULL); + if (cpu != MAP_FAILED) + mem->debug.host_mapping = cpu; + else + vk_logw(VK_LOG_OBJS(_device), + "failed to map VkMemory for dump or trace.\n"); } pandecode_inject_mmap(device->debug.decode_ctx, mem->addr.dev,