diff options
author | Android Build Coastguard Worker <android-build-coastguard-worker@google.com> | 2021-06-30 01:06:26 +0000 |
---|---|---|
committer | Android Build Coastguard Worker <android-build-coastguard-worker@google.com> | 2021-06-30 01:06:26 +0000 |
commit | 774970790e41fa0ebbcc110a8f0a8465123830d7 (patch) | |
tree | 89d5a41d8a9c2eb0a6d03437e5784749968817ef | |
parent | f6d777facedfa9c92fee540b47737eed20b50996 (diff) | |
parent | 57caf0895f596a617fd1d9b56068e7fc08e70476 (diff) | |
download | skia-android12-d1-s1-release.tar.gz |
Snap for 7506386 from 57caf0895f596a617fd1d9b56068e7fc08e70476 to sc-d1-releaseandroid-12.0.0_r7android-12.0.0_r6android-12.0.0_r5android-12.0.0_r4android12-d1-s4-releaseandroid12-d1-s3-releaseandroid12-d1-s2-releaseandroid12-d1-s1-release
Change-Id: I58dbe70eae82839338373e6780d6fc33f032b248
-rw-r--r-- | src/gpu/vk/GrVkCaps.h | 12 | ||||
-rw-r--r-- | src/gpu/vk/GrVkDescriptorSetManager.cpp | 5 |
2 files changed, 15 insertions, 2 deletions
diff --git a/src/gpu/vk/GrVkCaps.h b/src/gpu/vk/GrVkCaps.h index 44c26c244f..df9c0fb585 100644 --- a/src/gpu/vk/GrVkCaps.h +++ b/src/gpu/vk/GrVkCaps.h @@ -147,6 +147,18 @@ public: // Returns true if it supports ycbcr conversion for samplers bool supportsYcbcrConversion() const { return fSupportsYcbcrConversion; } + // Returns the number of descriptor slots used by immutable ycbcr VkImages. + // + // TODO: We should update this to return a count for a specific format or external format. We + // can use vkGetPhysicalDeviceImageFormatProperties2 with a + // VkSamplerYcbcrConversionImageFormatProperties to query this. However, right now that call + // does not support external android formats which is where the majority of ycbcr images are + // coming from. So for now we stay safe and always return 3 here which is the max value that the + // count could be for any format. + uint32_t ycbcrCombinedImageSamplerDescriptorCount() const { + return 3; + } + // Returns true if the device supports protected memory. bool supportsProtectedMemory() const { return fSupportsProtectedMemory; } diff --git a/src/gpu/vk/GrVkDescriptorSetManager.cpp b/src/gpu/vk/GrVkDescriptorSetManager.cpp index ae99342a19..14cb1b09ee 100644 --- a/src/gpu/vk/GrVkDescriptorSetManager.cpp +++ b/src/gpu/vk/GrVkDescriptorSetManager.cpp @@ -78,6 +78,7 @@ static bool get_layout_and_desc_count(GrVkGpu* gpu, uint32_t numBindings = visibilities.count(); std::unique_ptr<VkDescriptorSetLayoutBinding[]> dsSamplerBindings( new VkDescriptorSetLayoutBinding[numBindings]); + *descCountPerSet = 0; for (uint32_t i = 0; i < numBindings; ++i) { uint32_t visibility = visibilities[i]; dsSamplerBindings[i].binding = i; @@ -86,8 +87,10 @@ static bool get_layout_and_desc_count(GrVkGpu* gpu, dsSamplerBindings[i].stageFlags = visibility_to_vk_stage_flags(visibility); if (VK_DESCRIPTOR_TYPE_COMBINED_IMAGE_SAMPLER == type) { if (immutableSamplers[i]) { + (*descCountPerSet) += gpu->vkCaps().ycbcrCombinedImageSamplerDescriptorCount(); dsSamplerBindings[i].pImmutableSamplers = immutableSamplers[i]->samplerPtr(); } else { + (*descCountPerSet)++; dsSamplerBindings[i].pImmutableSamplers = nullptr; } } @@ -117,8 +120,6 @@ static bool get_layout_and_desc_count(GrVkGpu* gpu, if (result != VK_SUCCESS) { return false; } - - *descCountPerSet = visibilities.count(); } else if (type == VK_DESCRIPTOR_TYPE_UNIFORM_BUFFER) { static constexpr int kUniformDescPerSet = 1; SkASSERT(kUniformDescPerSet == visibilities.count()); |