[Mesa-dev] [PATCH] anv: Accurately advertise dynamic descriptor limits

[Eduardo Lima Mitev]

I would add a comment, at least in the commit log, explaining why these
limits are MAX_DYNAMIC_BUFFERS/2. It is not immediately evident.

In any case, patch is:

Reviewed-by: Eduardo Lima Mitev <elima at igalia.com>

On 03/04/2017 08:05 PM, Jason Ekstrand wrote:
> Cc: "17.0 13.0" <mesa-stable at lists.freedesktop.org>
> ---
>  src/intel/vulkan/anv_device.c | 4 ++--
>  1 file changed, 2 insertions(+), 2 deletions(-)
> 
> diff --git a/src/intel/vulkan/anv_device.c b/src/intel/vulkan/anv_device.c
> index fbcbd40..3fc51ac 100644
> --- a/src/intel/vulkan/anv_device.c
> +++ b/src/intel/vulkan/anv_device.c
> @@ -565,9 +565,9 @@ void anv_GetPhysicalDeviceProperties(
>        .maxPerStageResources                     = 128,
>        .maxDescriptorSetSamplers                 = 256,
>        .maxDescriptorSetUniformBuffers           = 256,
> -      .maxDescriptorSetUniformBuffersDynamic    = 256,
> +      .maxDescriptorSetUniformBuffersDynamic    = MAX_DYNAMIC_BUFFERS / 2,
>        .maxDescriptorSetStorageBuffers           = 256,
> -      .maxDescriptorSetStorageBuffersDynamic    = 256,
> +      .maxDescriptorSetStorageBuffersDynamic    = MAX_DYNAMIC_BUFFERS / 2,
>        .maxDescriptorSetSampledImages            = 256,
>        .maxDescriptorSetStorageImages            = 256,
>        .maxDescriptorSetInputAttachments         = 256,
>