Skip to content

Instantly share code, notes, and snippets.

@DethRaid
Last active April 16, 2024 20:48
Show Gist options
  • Star 22 You must be signed in to star a gist
  • Fork 1 You must be signed in to fork a gist
  • Save DethRaid/0171f3cfcce51950ee4ef96c64f59617 to your computer and use it in GitHub Desktop.
Save DethRaid/0171f3cfcce51950ee4ef96c64f59617 to your computer and use it in GitHub Desktop.

How do you descriptor set?

Descriptor sets have vexed me at every step of development. They're new and different and they have a lot of rules which aren't all that obvious. This document will hopefully lay out everything in one convenient place that I - and also you - can refer to

First, let's talk about what we're trying to do

Use Case

Most renderers need some way for shaders to access resources like textures, buffers, etc. For the Vulkan API, this way is the almighty descriptor set. Descriptors, as I understand them, are essentially a pointer to a resource. You update your descriptor sets with your resources, then you bind the descriptor sets to your command buffer, then shaders involved in subsequent drawcalls can look at the descriptors to know what resources they should actually read from. I'm not entirely sure why there's this indirection - and in fact, on AMD GPUs descriptor sets are actually just pointers - but the indirection exists, and we all have to find a way to deal with it

So, how do we update descriptor sets with our resources? How do we bind descriptor sets to a command buffer? Where do the descriptor sets live?

Where do descriptor sets live?

Before getting into updating and binding descriptor sets, we're going to talk about where they live. Every developer I've talked to has had a different answer for this question. My answer makes sense for me, but you may want to do something different. Such is Vulkan.

My renderer has a concept of a Material. My Materials have a VkPipeline, and a bunch of VkDescriptorSets that represent all the resources needed by the Material. This includes things the artists care about, like textures and artist-tunable material parameters, along with things like the camera matrices, per-frame information like the current time and player's position, etc. The idea is that I can bind each Material's descriptors atomically, then issue drawcalls for everything using that Material, then bind the next Material's descriptors, all without worrying about only updating some of the descriptors. This is somewhat inefficient and I'm binding more than I need to, but for now it's fine

I tried having my resource own their own descriptor sets. This initially made sense to me, until I realized that descriptor sets were tied to the pipeline layout they're created from. Having one descriptor per resource didn't work, but having one descriptor per resource per pipeline made more sense

Descriptor Indexing Guide

Note: This guide is written for Vulkan 1.2, where descriptor indexing is a core feature. If you're using an older version of Vulkan, you need to enable the descriptor indexing extension and use the EXT versions of all the types and constants mentioned here

Creating your VkDevice

You need to enable descriptor indexing features through the VkPhysicalDeviceDescriptorIndexingFeatures struct. This struct should be in the pNext chain of your VkDeviceCreateInfo. You should only enable the features that you'll actually need. Nova enables dynamic indexing of sampled image arrays, variable descriptor counts, and partially bound descriptors. This allows me to have a single large array of textures to use for texture streaming

VkPhysicalDeviceDescriptorIndexingFeatures descriptor_indexing_features{};
descriptor_indexing_features.sType = VK_STRUCTURE_TYPE_PHYSICAL_DEVICE_DESCRIPTOR_INDEXING_FEATURES;

// Enable non-uniform indexing
descriptor_indexing_features.shaderSampledImageArrayNonUniformIndexing = VK_TRUE;
descriptor_indexing_features.runtimeDescriptorArray = VK_TRUE;
descriptor_indexing_features.descriptorBindingVariableDescriptorCount = VK_TRUE;
descriptor_indexing_features.descriptorBindingPartiallyBound = VK_TRUE;

VkDeviceCreateInfo device_create_info = ...;
device_create_info.pNext = &descriptor_indexing_features;

VkDevice device;
vkCreateDevice(phys_device, &device_create_info, nullptr, &device);

Creating the descriptor set layout

Unbounded (variable size) descriptor arrays

Nova uses an unbounded descriptor array for its texture resources. This allows Nova to stream in textures and add them to the array as needed. Materials use indexes to refer to which texture in the array they use. The Chunk Stories blog has a wonderful explanation of why this is a good strategy, I agree with all the points they make

Vulkan has a restriction that an unbounded array must be the last descriptor in the set. This is kinda annoying, but it's something that I can easily check with shader reflection so no worries

You have to tell Vulkan which descriptors will be a variable size array when you create your VkDescriptorSetLayout. To do this, you use the VkDescriptorSetLayoutBindingFlagsCreateInfo struct. This struct should be in the pNext chain of your VkDescriptorSetLayoutCreateInfo struct. VkDescriptorSetLayoutBindingFlagsCreateInfo has an array of flags for each binding in the descriptor set. Any descriptors which can be an unbounded array must have the flag VK_DESCRIPTOR_BINDING_VARIABLE_DESCRIPTOR_COUNT_BIT at their index in the flags array. For example, if binding 3 is an unbounded array, descriptor, then VkDescriptorSetLayoutBindingFlagsCreateInfo.pBindingFlags[2] must have VK_DESCRIPTOR_BINDING_VARIABLE_DESCRIPTOR_COUNT_BIT

When one of the descriptors is an unbounded array, the descriptorCount member of the VkDescriptorSetLayoutBinding struct is an upper bound on the size of the array. This means that the array isn't truly unbounded, but near as I can tell most people say "unbounded", so for the same of convention I use that word as well

Partially populated (bound) descriptors

We enabled partially populated descriptors when creating our VkDevice, but we also have to tell Vulkan which descriptors we plan to partially populate. Doing this is simple: add the VK_DESCRIPTOR_BINDING_PARTIALLY_BOUND_BIT flag to your VkDescriptorSetLayoutBindingFlagsCreateInfo for any bindings for a partially bound array. Nova makes heavy use of partially bound descriptor arrays - All of my unbounded descriptor arrays are partially sized

Example code

VkDescriptorBindingFlags flags[3];
flags[0] = 0;
flags[1] = 0;
flags[2] = VK_DESCRIPTOR_BINDING_VARIABLE_DESCRIPTOR_COUNT_BIT | VK_DESCRIPTOR_BINDING_PARTIALLY_BOUND_BIT;

VkDescriptorSetLayoutBindingFlagsCreateInfo binding_flags{};
binding_flags.sType = VK_STRUCTURE_TYPE_DESCRIPTOR_SET_LAYOUT_BINDING_FLAGS_CREATE_INFO;
binding_flags.bindingCount = 3;
binding_flags.pBindingFlags = flags;

// When you make the VkDescriptorSetLayoutBinding for your variable length array, the `descriptorCount` member is the
// maximum number of descriptors in that array. I've omitted the code here for brevity, and because it's not different
// for varaible count arrays vs normal descriptor arrays

VkDescriptorSetLayoutCreateInfo create_info = ...;
create_info.pNext = &binding_flags;

VkDescriptorSetLayout layout;
vkCreateDescriptorSetLayout(device, &create_info, nullptr, &layout);

Possible problems

One problem I ran into, which caused me a great deal of grief, is that I got the descriptorCount for my unbounded array by performing shader reflection on some SPIR-V that I had compiled from HLSL. Turns out, SPIR-V Cross will report an array size of 1 for a HLSL array without a size

My HLSL:

[[vk::binding(1, 1)]]
Texture2D ui_textures[] : register(t0);

Giving the array a size resolved my issue

Creating the descriptor sets

When you create an descriptor set for an unbounded array with a variable size, you must use the VkDescriptorSetVariableDescriptorCountAllocateInfo struct to tell Vulkan the descriptor counts of each set with a variable descriptor count. This struct should be in the pNext chain of your VkDescriptorSetAllocateInfo.

The spec is worded strangely for this struct. Essentially it seems to be the maximum number of descriptors in each of your unbounded arrays. Since am unbounded array must be the last binding in a descriptor set, there can only be one unbounded array per descriptor set. This means you only need to specify one count per descriptor set - which is exactly what VkDescriptorSetVariableDescriptorCountAllocateInfo has you do

You don't need to worry about descriptor counts for sets that don't have any unbounded arrays - or if you do, the spec doesn't make that clear - but you absolutely do need to worry about descriptor counts for sets that do have unbounded arrays. If you're like me and you get your descriptor sizes from shader reflection, you'll need to propagate that information to where you create your descriptor sets. I simply pass down an array of the descriptor counts of variable size descriptors in each descriptor set as an array of ints, this strategy will probably work for you

uint32_t counts[1];
counts[0] = 32; // Set 0 has a variable count descriptor with a maximum of 32 elements

VkDescriptorSetVariableDescriptorCountAllocateInfo set_counts = {};
set_counts.sType = VK_STRUCTURE_TYPE_DESCRIPTOR_SET_VARIABLE_DESCRIPTOR_COUNT_ALLOCATE_INFO;
set_counts.descriptorSetCount = 1;
set_counts.pDescriptorCounts = counts;

VkDescriptorSetAllocateInfo alloc_info = ...;
alloc_info.pNext = &set_counts;

VkDescriptorSet set;
vkAllocateDescriptorSets(device, &alloc_info, &set);

Binding resources to unbounded array descriptors

Binding resources to an unbounded array descriptor works just like binding resources to a regular array descriptor. You create a VkDescriptorSetWrite struct with the appropriate VkDescriptorSet and descriptor binding. The descriptorCount member should be the number of elements in your array. Here's an example for binding textures:

VkDescriptorImageInfo image_infos[2];
image_infos[0].imageView = image_view_1;
image_infos[0].imageLayout = VK_IMAGE_LAYOUT_SHADER_READ_ONLY_OPTIMAL;
image_infos[1].imageView = image_view_2;
image_infos[1].imageLayout = VK_IMAGE_LAYOUT_SHADER_READ_ONLY_OPTIMAL;

VkWriteDescriptorSet write{};
vk_write.sType = VK_STRUCTURE_TYPE_WRITE_DESCRIPTOR_SET;
vk_write.dstSet = descriptor_set;
vk_write.dstBinding = 2;    // We're updating binding 2, which is the one with the VK_DESCRIPTOR_BINDING_VARIABLE_DESCRIPTOR_COUNT_BIT and VK_DESCRIPTOR_BINDING_PARTIALLY_BOUND_BIT flags from the above example
vk_write.descriptorCount = 2;   // Write two textures to this descriptor
vk_write.dstArrayElement = 0;   // Start at array index 0
vk_write.pImageInfos = image_infos;

vkUpdateDescriptorSets(device, 1, &write, 0, nullptr);
@rubbyzhang
Copy link

how to use in shader(glsl)~

@DethRaid
Copy link
Author

Just declare an array of textures - layout(set = 0, binding = 2) uniform texture2D all_textures[]; for the code in this gist

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment