From e26962f756a15f0b6b56822b677b7a984acb9029 Mon Sep 17 00:00:00 2001 From: Eric Anholt Date: Wed, 4 Dec 2019 12:21:50 -0800 Subject: [PATCH] turnip: Sanity check that we're adding valid BOs to the list. I tripped over this during CS enabling when my program BO wasn't set up. Easier to debug this way than the kernel telling us a 0 handle is invalid. Reviewed-by: Jonathan Marek --- src/freedreno/vulkan/tu_cmd_buffer.c | 2 ++ 1 file changed, 2 insertions(+) diff --git a/src/freedreno/vulkan/tu_cmd_buffer.c b/src/freedreno/vulkan/tu_cmd_buffer.c index 47f95e4b00a..bb9d18604b8 100644 --- a/src/freedreno/vulkan/tu_cmd_buffer.c +++ b/src/freedreno/vulkan/tu_cmd_buffer.c @@ -62,6 +62,8 @@ static uint32_t tu_bo_list_add_info(struct tu_bo_list *list, const struct drm_msm_gem_submit_bo *bo_info) { + assert(bo_info->handle != 0); + for (uint32_t i = 0; i < list->count; ++i) { if (list->bo_infos[i].handle == bo_info->handle) { assert(list->bo_infos[i].presumed == bo_info->presumed); -- 2.30.2