From 07a599ff3e32e2a429a25ccb3fae40ff756a8a58 Mon Sep 17 00:00:00 2001 From: Zan Dobersek Date: Tue, 16 Sep 2025 15:48:41 +0200 Subject: [PATCH] tu/drm: avoid has_set_iova-specific util_vma_heap freeing in tu_bo_init After the refactoring, tu_bo_init() is not allocating iova anymore so it should also not free the util_vma_heap allocation for the has_set_iova case. Signed-off-by: Zan Dobersek Fixes: 93a80f4bb96 ("tu/drm: Split out iova allocation and BO allocation") Part-of: --- src/freedreno/vulkan/tu_knl_drm_msm.cc | 2 -- 1 file changed, 2 deletions(-) diff --git a/src/freedreno/vulkan/tu_knl_drm_msm.cc b/src/freedreno/vulkan/tu_knl_drm_msm.cc index 8c69dc6d37a..a5c74bb34be 100644 --- a/src/freedreno/vulkan/tu_knl_drm_msm.cc +++ b/src/freedreno/vulkan/tu_knl_drm_msm.cc @@ -754,8 +754,6 @@ tu_bo_init(struct tu_device *dev, result = tu_bo_add_to_bo_list(dev, gem_handle, flags, iova, &idx); if (result != VK_SUCCESS) { mtx_unlock(&dev->bo_mutex); - if (dev->physical_device->has_set_iova) - util_vma_heap_free(&dev->vma, iova, size); tu_gem_close(dev, gem_handle); return result; }