From a7bf37e3bd7d79ad41c94cb5222513c3dbd77216 Mon Sep 17 00:00:00 2001 From: Lars-Ivar Hesselberg Simonsen Date: Tue, 21 Oct 2025 14:37:48 +0200 Subject: [PATCH] panvk/v9+: Reduce maxBoundDescriptorSets to 7 Until now, the driver has been using a single set for internal state while leaving maxBoundDescriptorSets to the remaining 15. This gives us no room for optimizations of driver sets, which might become an issue in the future. To remedy this, we therefore reduce maxBoundDescriptorSets to 7. This aligns with the proprietary driver and gives us the space to optimize the driver sets. We might increase this in the future if we see that we don't need all the driver sets we now reserve. Reviewed-by: John Anthony Reviewed-by: Yiwei Zhang Reviewed-by: Olivia Lee Reviewed-by: Boris Brezillon Acked-by: Erik Faye-Lund Part-of: --- src/panfrost/vulkan/panvk_descriptor_set_layout.h | 5 ++--- 1 file changed, 2 insertions(+), 3 deletions(-) diff --git a/src/panfrost/vulkan/panvk_descriptor_set_layout.h b/src/panfrost/vulkan/panvk_descriptor_set_layout.h index 431a0a123ae..cf1687c0cbe 100644 --- a/src/panfrost/vulkan/panvk_descriptor_set_layout.h +++ b/src/panfrost/vulkan/panvk_descriptor_set_layout.h @@ -57,9 +57,8 @@ #else /* Valhall has native support for descriptor sets, and allows a maximum - * of 16 sets, but we reserve one for our internal use, so we have 15 - * left. */ -#define MAX_SETS 15 + * of 16 sets, but we reserve 9 for our internal use, so we have 7 left. */ +#define MAX_SETS 7 /* Hardware limit is 2^24 each of buffer, texture, and sampler descriptors. We * use the same hardware descriptors for multiple kinds of vulkan descriptors,