vulkan: Fix descriptor set creation with zero bindings.
authorBas Nieuwenhuizen <bas@basnieuwenhuizen.nl>
Sun, 28 Mar 2021 20:46:19 +0000 (22:46 +0200)
committerMarge Bot <eric+marge@anholt.net>
Mon, 29 Mar 2021 23:32:50 +0000 (23:32 +0000)
commit83c92a48b77e6ce6f4dd2875886d7eff10ab426f
tree28b83d07724e72089f9c17034832861aaf13bb70
parent80b90a0f2b83291da69f7eef38395e61e919ab97
vulkan: Fix descriptor set creation with zero bindings.

MAX2(count * struct size, 1) results in 1 for count=0, not the size of a struct.

Since this MAX only seems to exist so we can keep using NULL for error reporting,
just refactor to return a VkResult.

Fixes: ad241b15a9e ("vk: consolidate dynamic descriptor binding sorting")
Closes: https://gitlab.freedesktop.org/mesa/mesa/-/issues/4522
Reviewed-By: Mike Blumenkrantz <michael.blumenkrantz@gmail.com>
Acked-by: Dave Airlie <airlied@redhat.com>
Reviewed-by: Alejandro PiƱeiro <apinheiro@igalia.com>
Part-of: <https://gitlab.freedesktop.org/mesa/mesa/-/merge_requests/9880>
src/amd/vulkan/radv_descriptor_set.c
src/broadcom/vulkan/v3dv_descriptor_set.c
src/freedreno/vulkan/tu_descriptor_set.c
src/gallium/frontends/lavapipe/lvp_descriptor_set.c
src/vulkan/util/vk_descriptors.c
src/vulkan/util/vk_descriptors.h