[Mesa-dev] [PATCH 2/2] radv: ignore inline uniform blocks in radv_CmdPushDescriptorSetKHR()
joseph.kucia at gmail.com
Tue Jun 11 10:05:11 UTC 2019
On Tue, Jun 11, 2019 at 11:57 AM Samuel Iglesias Gonsálvez
<siglesias at igalia.com> wrote:
> According to the Vulkan spec, uniform blocks are not allowed to be
> updated through vkCmdPushDescriptorSetKHR().
> There are these spec quotes from "13.2.1. Descriptor Set Layout"
> that are relevant for this case:
> "VK_DESCRIPTOR_SET_LAYOUT_CREATE_PUSH_DESCRIPTOR_BIT_KHR specifies
> that descriptor sets must not be allocated using this layout, and
> descriptors are instead pushed by vkCmdPushDescriptorSetKHR."
> "If flags contains
> VK_DESCRIPTOR_SET_LAYOUT_CREATE_PUSH_DESCRIPTOR_BIT_KHR, then all
> elements of pBindings must not have a descriptorType of
> There is no explicit mention in vkCmdPushDescriptorSetKHR() to forbid
> this case but it is implied in the creation of the descriptor set
> layout as aforementioned.
> Signed-off-by: Samuel Iglesias Gonsálvez <siglesias at igalia.com>
> My only doubt is I did well in the case of
> radv_meta_push_descriptor_set(), let me know if you prefer to change
> it to false.
Perhaps I'm missing something, but what is the point to add the
additional checks for invalid usage? A correct program must not use
inline uniform blocks with push descriptors.
More information about the mesa-dev