rhi: vulkan: Double buffer the descriptor sets backing an srb always

This is almost always the case in a typical Quick or Quick 3D rendering
sequence, since there is almost always a uniform buffer, which is almost
always a QRhiBuffer::Dynamic buffer, thus triggering double (or triple,
whatever max_frames_in_flight is) buffering and using multiple
descriptor sets.

This breaks down e.g. when using a compute pipeline and a
texture for imageLoad/store in the srb. If the texture is rebuilt
between frame N and N+1, then in frame N+1 the srb sees that the
descriptor set needs to be rewritten, but that's wrong since frame
N might still be in flight, using that descriptor set.

From now on, pretend that an srb always needs double buffering,
regardless of what kind of resources it references.

Also drop unused code paths. The updateShaderResources function's
argument was never -1, for example, so no point in keeping that logic.

Pick-to: 6.8
Change-Id: If8abb42fd9a2de5aad59ab6ceab3c8086aebae68
Reviewed-by: Andy Nichols <andy.nichols@qt.io>
(cherry picked from commit 30dc9ed13fcc2691ed656f6f36d419133856c8cd)
Reviewed-by: Qt Cherry-pick Bot <cherrypick_bot@qt-project.org>
This commit is contained in:
Laszlo Agocs 2025-03-04 16:57:14 +01:00 committed by Qt Cherry-pick Bot
parent 7d8b81a491
commit ec386cd2a9
2 changed files with 129 additions and 139 deletions

View File

@ -3524,7 +3524,7 @@ bool QRhiVulkan::ensurePipelineCache(const void *initialData, size_t initialData
return true; return true;
} }
void QRhiVulkan::updateShaderResourceBindings(QRhiShaderResourceBindings *srb, int descSetIdx) void QRhiVulkan::updateShaderResourceBindings(QRhiShaderResourceBindings *srb)
{ {
QVkShaderResourceBindings *srbD = QRHI_RES(QVkShaderResourceBindings, srb); QVkShaderResourceBindings *srbD = QRHI_RES(QVkShaderResourceBindings, srb);
@ -3534,143 +3534,138 @@ void QRhiVulkan::updateShaderResourceBindings(QRhiShaderResourceBindings *srb, i
QVarLengthArray<VkWriteDescriptorSet, 12> writeInfos; QVarLengthArray<VkWriteDescriptorSet, 12> writeInfos;
QVarLengthArray<std::pair<int, int>, 12> infoIndices; QVarLengthArray<std::pair<int, int>, 12> infoIndices;
const bool updateAll = descSetIdx < 0; for (int i = 0, ie = srbD->sortedBindings.size(); i != ie; ++i) {
int frameSlot = updateAll ? 0 : descSetIdx; const QRhiShaderResourceBinding::Data *b = shaderResourceBindingData(srbD->sortedBindings.at(i));
while (frameSlot < (updateAll ? QVK_FRAMES_IN_FLIGHT : descSetIdx + 1)) { QVkShaderResourceBindings::BoundResourceData &bd(srbD->boundResourceData[currentFrameSlot][i]);
for (int i = 0, ie = srbD->sortedBindings.size(); i != ie; ++i) {
const QRhiShaderResourceBinding::Data *b = shaderResourceBindingData(srbD->sortedBindings.at(i));
QVkShaderResourceBindings::BoundResourceData &bd(srbD->boundResourceData[frameSlot][i]);
VkWriteDescriptorSet writeInfo = {}; VkWriteDescriptorSet writeInfo = {};
writeInfo.sType = VK_STRUCTURE_TYPE_WRITE_DESCRIPTOR_SET; writeInfo.sType = VK_STRUCTURE_TYPE_WRITE_DESCRIPTOR_SET;
writeInfo.dstSet = srbD->descSets[frameSlot]; writeInfo.dstSet = srbD->descSets[currentFrameSlot];
writeInfo.dstBinding = uint32_t(b->binding); writeInfo.dstBinding = uint32_t(b->binding);
writeInfo.descriptorCount = 1; writeInfo.descriptorCount = 1;
int bufferInfoIndex = -1; int bufferInfoIndex = -1;
int imageInfoIndex = -1; int imageInfoIndex = -1;
switch (b->type) { switch (b->type) {
case QRhiShaderResourceBinding::UniformBuffer: case QRhiShaderResourceBinding::UniformBuffer:
{ {
writeInfo.descriptorType = b->u.ubuf.hasDynamicOffset ? VK_DESCRIPTOR_TYPE_UNIFORM_BUFFER_DYNAMIC writeInfo.descriptorType = b->u.ubuf.hasDynamicOffset ? VK_DESCRIPTOR_TYPE_UNIFORM_BUFFER_DYNAMIC
: VK_DESCRIPTOR_TYPE_UNIFORM_BUFFER; : VK_DESCRIPTOR_TYPE_UNIFORM_BUFFER;
QRhiBuffer *buf = b->u.ubuf.buf; QRhiBuffer *buf = b->u.ubuf.buf;
QVkBuffer *bufD = QRHI_RES(QVkBuffer, buf); QVkBuffer *bufD = QRHI_RES(QVkBuffer, buf);
bd.ubuf.id = bufD->m_id; bd.ubuf.id = bufD->m_id;
bd.ubuf.generation = bufD->generation; bd.ubuf.generation = bufD->generation;
VkDescriptorBufferInfo bufInfo; VkDescriptorBufferInfo bufInfo;
bufInfo.buffer = bufD->m_type == QRhiBuffer::Dynamic ? bufD->buffers[frameSlot] : bufD->buffers[0]; bufInfo.buffer = bufD->m_type == QRhiBuffer::Dynamic ? bufD->buffers[currentFrameSlot] : bufD->buffers[0];
bufInfo.offset = b->u.ubuf.offset; bufInfo.offset = b->u.ubuf.offset;
bufInfo.range = b->u.ubuf.maybeSize ? b->u.ubuf.maybeSize : bufD->m_size; bufInfo.range = b->u.ubuf.maybeSize ? b->u.ubuf.maybeSize : bufD->m_size;
// be nice and assert when we know the vulkan device would die a horrible death due to non-aligned reads // be nice and assert when we know the vulkan device would die a horrible death due to non-aligned reads
Q_ASSERT(aligned(bufInfo.offset, ubufAlign) == bufInfo.offset); Q_ASSERT(aligned(bufInfo.offset, ubufAlign) == bufInfo.offset);
bufferInfoIndex = bufferInfos.size(); bufferInfoIndex = bufferInfos.size();
bufferInfos.append(bufInfo); bufferInfos.append(bufInfo);
}
break;
case QRhiShaderResourceBinding::SampledTexture:
{
const QRhiShaderResourceBinding::Data::TextureAndOrSamplerData *data = &b->u.stex;
writeInfo.descriptorCount = data->count; // arrays of combined image samplers are supported
writeInfo.descriptorType = VK_DESCRIPTOR_TYPE_COMBINED_IMAGE_SAMPLER;
ArrayOfImageDesc imageInfo(data->count);
for (int elem = 0; elem < data->count; ++elem) {
QVkTexture *texD = QRHI_RES(QVkTexture, data->texSamplers[elem].tex);
QVkSampler *samplerD = QRHI_RES(QVkSampler, data->texSamplers[elem].sampler);
bd.stex.d[elem].texId = texD->m_id;
bd.stex.d[elem].texGeneration = texD->generation;
bd.stex.d[elem].samplerId = samplerD->m_id;
bd.stex.d[elem].samplerGeneration = samplerD->generation;
imageInfo[elem].sampler = samplerD->sampler;
imageInfo[elem].imageView = texD->imageView;
imageInfo[elem].imageLayout = VK_IMAGE_LAYOUT_SHADER_READ_ONLY_OPTIMAL;
} }
break; bd.stex.count = data->count;
case QRhiShaderResourceBinding::SampledTexture: imageInfoIndex = imageInfos.size();
{ imageInfos.append(imageInfo);
const QRhiShaderResourceBinding::Data::TextureAndOrSamplerData *data = &b->u.stex; }
writeInfo.descriptorCount = data->count; // arrays of combined image samplers are supported break;
writeInfo.descriptorType = VK_DESCRIPTOR_TYPE_COMBINED_IMAGE_SAMPLER; case QRhiShaderResourceBinding::Texture:
ArrayOfImageDesc imageInfo(data->count); {
for (int elem = 0; elem < data->count; ++elem) { const QRhiShaderResourceBinding::Data::TextureAndOrSamplerData *data = &b->u.stex;
QVkTexture *texD = QRHI_RES(QVkTexture, data->texSamplers[elem].tex); writeInfo.descriptorCount = data->count; // arrays of (separate) images are supported
QVkSampler *samplerD = QRHI_RES(QVkSampler, data->texSamplers[elem].sampler); writeInfo.descriptorType = VK_DESCRIPTOR_TYPE_SAMPLED_IMAGE;
bd.stex.d[elem].texId = texD->m_id; ArrayOfImageDesc imageInfo(data->count);
bd.stex.d[elem].texGeneration = texD->generation; for (int elem = 0; elem < data->count; ++elem) {
bd.stex.d[elem].samplerId = samplerD->m_id; QVkTexture *texD = QRHI_RES(QVkTexture, data->texSamplers[elem].tex);
bd.stex.d[elem].samplerGeneration = samplerD->generation; bd.stex.d[elem].texId = texD->m_id;
imageInfo[elem].sampler = samplerD->sampler; bd.stex.d[elem].texGeneration = texD->generation;
imageInfo[elem].imageView = texD->imageView; bd.stex.d[elem].samplerId = 0;
imageInfo[elem].imageLayout = VK_IMAGE_LAYOUT_SHADER_READ_ONLY_OPTIMAL; bd.stex.d[elem].samplerGeneration = 0;
} imageInfo[elem].sampler = VK_NULL_HANDLE;
bd.stex.count = data->count; imageInfo[elem].imageView = texD->imageView;
imageInfoIndex = imageInfos.size(); imageInfo[elem].imageLayout = VK_IMAGE_LAYOUT_SHADER_READ_ONLY_OPTIMAL;
imageInfos.append(imageInfo);
} }
break; bd.stex.count = data->count;
case QRhiShaderResourceBinding::Texture: imageInfoIndex = imageInfos.size();
{ imageInfos.append(imageInfo);
const QRhiShaderResourceBinding::Data::TextureAndOrSamplerData *data = &b->u.stex; }
writeInfo.descriptorCount = data->count; // arrays of (separate) images are supported break;
writeInfo.descriptorType = VK_DESCRIPTOR_TYPE_SAMPLED_IMAGE; case QRhiShaderResourceBinding::Sampler:
ArrayOfImageDesc imageInfo(data->count); {
for (int elem = 0; elem < data->count; ++elem) { QVkSampler *samplerD = QRHI_RES(QVkSampler, b->u.stex.texSamplers[0].sampler);
QVkTexture *texD = QRHI_RES(QVkTexture, data->texSamplers[elem].tex); writeInfo.descriptorType = VK_DESCRIPTOR_TYPE_SAMPLER;
bd.stex.d[elem].texId = texD->m_id; bd.stex.d[0].texId = 0;
bd.stex.d[elem].texGeneration = texD->generation; bd.stex.d[0].texGeneration = 0;
bd.stex.d[elem].samplerId = 0; bd.stex.d[0].samplerId = samplerD->m_id;
bd.stex.d[elem].samplerGeneration = 0; bd.stex.d[0].samplerGeneration = samplerD->generation;
imageInfo[elem].sampler = VK_NULL_HANDLE; ArrayOfImageDesc imageInfo(1);
imageInfo[elem].imageView = texD->imageView; imageInfo[0].sampler = samplerD->sampler;
imageInfo[elem].imageLayout = VK_IMAGE_LAYOUT_SHADER_READ_ONLY_OPTIMAL; imageInfo[0].imageView = VK_NULL_HANDLE;
} imageInfo[0].imageLayout = VK_IMAGE_LAYOUT_GENERAL;
bd.stex.count = data->count; imageInfoIndex = imageInfos.size();
imageInfoIndex = imageInfos.size(); imageInfos.append(imageInfo);
imageInfos.append(imageInfo); }
} break;
break; case QRhiShaderResourceBinding::ImageLoad:
case QRhiShaderResourceBinding::Sampler: case QRhiShaderResourceBinding::ImageStore:
{ case QRhiShaderResourceBinding::ImageLoadStore:
QVkSampler *samplerD = QRHI_RES(QVkSampler, b->u.stex.texSamplers[0].sampler); {
writeInfo.descriptorType = VK_DESCRIPTOR_TYPE_SAMPLER; QVkTexture *texD = QRHI_RES(QVkTexture, b->u.simage.tex);
bd.stex.d[0].texId = 0; VkImageView view = texD->perLevelImageViewForLoadStore(b->u.simage.level);
bd.stex.d[0].texGeneration = 0; if (view) {
bd.stex.d[0].samplerId = samplerD->m_id; writeInfo.descriptorType = VK_DESCRIPTOR_TYPE_STORAGE_IMAGE;
bd.stex.d[0].samplerGeneration = samplerD->generation; bd.simage.id = texD->m_id;
bd.simage.generation = texD->generation;
ArrayOfImageDesc imageInfo(1); ArrayOfImageDesc imageInfo(1);
imageInfo[0].sampler = samplerD->sampler; imageInfo[0].sampler = VK_NULL_HANDLE;
imageInfo[0].imageView = VK_NULL_HANDLE; imageInfo[0].imageView = view;
imageInfo[0].imageLayout = VK_IMAGE_LAYOUT_GENERAL; imageInfo[0].imageLayout = VK_IMAGE_LAYOUT_GENERAL;
imageInfoIndex = imageInfos.size(); imageInfoIndex = imageInfos.size();
imageInfos.append(imageInfo); imageInfos.append(imageInfo);
} }
break;
case QRhiShaderResourceBinding::ImageLoad:
case QRhiShaderResourceBinding::ImageStore:
case QRhiShaderResourceBinding::ImageLoadStore:
{
QVkTexture *texD = QRHI_RES(QVkTexture, b->u.simage.tex);
VkImageView view = texD->perLevelImageViewForLoadStore(b->u.simage.level);
if (view) {
writeInfo.descriptorType = VK_DESCRIPTOR_TYPE_STORAGE_IMAGE;
bd.simage.id = texD->m_id;
bd.simage.generation = texD->generation;
ArrayOfImageDesc imageInfo(1);
imageInfo[0].sampler = VK_NULL_HANDLE;
imageInfo[0].imageView = view;
imageInfo[0].imageLayout = VK_IMAGE_LAYOUT_GENERAL;
imageInfoIndex = imageInfos.size();
imageInfos.append(imageInfo);
}
}
break;
case QRhiShaderResourceBinding::BufferLoad:
case QRhiShaderResourceBinding::BufferStore:
case QRhiShaderResourceBinding::BufferLoadStore:
{
QVkBuffer *bufD = QRHI_RES(QVkBuffer, b->u.sbuf.buf);
writeInfo.descriptorType = VK_DESCRIPTOR_TYPE_STORAGE_BUFFER;
bd.sbuf.id = bufD->m_id;
bd.sbuf.generation = bufD->generation;
VkDescriptorBufferInfo bufInfo;
bufInfo.buffer = bufD->m_type == QRhiBuffer::Dynamic ? bufD->buffers[frameSlot] : bufD->buffers[0];
bufInfo.offset = b->u.ubuf.offset;
bufInfo.range = b->u.ubuf.maybeSize ? b->u.ubuf.maybeSize : bufD->m_size;
bufferInfoIndex = bufferInfos.size();
bufferInfos.append(bufInfo);
}
break;
default:
continue;
}
writeInfos.append(writeInfo);
infoIndices.append({ bufferInfoIndex, imageInfoIndex });
} }
++frameSlot; break;
case QRhiShaderResourceBinding::BufferLoad:
case QRhiShaderResourceBinding::BufferStore:
case QRhiShaderResourceBinding::BufferLoadStore:
{
QVkBuffer *bufD = QRHI_RES(QVkBuffer, b->u.sbuf.buf);
writeInfo.descriptorType = VK_DESCRIPTOR_TYPE_STORAGE_BUFFER;
bd.sbuf.id = bufD->m_id;
bd.sbuf.generation = bufD->generation;
VkDescriptorBufferInfo bufInfo;
bufInfo.buffer = bufD->m_type == QRhiBuffer::Dynamic ? bufD->buffers[currentFrameSlot] : bufD->buffers[0];
bufInfo.offset = b->u.ubuf.offset;
bufInfo.range = b->u.ubuf.maybeSize ? b->u.ubuf.maybeSize : bufD->m_size;
bufferInfoIndex = bufferInfos.size();
bufferInfos.append(bufInfo);
}
break;
default:
continue;
}
writeInfos.append(writeInfo);
infoIndices.append({ bufferInfoIndex, imageInfoIndex });
} }
for (int i = 0, writeInfoCount = writeInfos.size(); i < writeInfoCount; ++i) { for (int i = 0, writeInfoCount = writeInfos.size(); i < writeInfoCount; ++i) {
@ -5582,8 +5577,7 @@ void QRhiVulkan::setShaderResources(QRhiCommandBuffer *cb, QRhiShaderResourceBin
} }
QVkShaderResourceBindings *srbD = QRHI_RES(QVkShaderResourceBindings, srb); QVkShaderResourceBindings *srbD = QRHI_RES(QVkShaderResourceBindings, srb);
const int descSetIdx = srbD->hasSlottedResource ? currentFrameSlot : 0; auto &descSetBd(srbD->boundResourceData[currentFrameSlot]);
auto &descSetBd(srbD->boundResourceData[descSetIdx]);
bool rewriteDescSet = false; bool rewriteDescSet = false;
// Do host writes and mark referenced shader resources as in-use. // Do host writes and mark referenced shader resources as in-use.
@ -5720,11 +5714,11 @@ void QRhiVulkan::setShaderResources(QRhiCommandBuffer *cb, QRhiShaderResourceBin
// write descriptor sets, if needed // write descriptor sets, if needed
if (rewriteDescSet) if (rewriteDescSet)
updateShaderResourceBindings(srb, descSetIdx); updateShaderResourceBindings(srb);
// make sure the descriptors for the correct slot will get bound. // make sure the descriptors for the correct slot will get bound.
// also, dynamic offsets always need a bind. // also, dynamic offsets always need a bind.
const bool forceRebind = (srbD->hasSlottedResource && cbD->currentDescSetSlot != descSetIdx) || srbD->hasDynamicOffset; const bool forceRebind = cbD->currentDescSetSlot != currentFrameSlot || srbD->hasDynamicOffset;
const bool srbChanged = gfxPsD ? (cbD->currentGraphicsSrb != srb) : (cbD->currentComputeSrb != srb); const bool srbChanged = gfxPsD ? (cbD->currentGraphicsSrb != srb) : (cbD->currentComputeSrb != srb);
@ -5755,7 +5749,7 @@ void QRhiVulkan::setShaderResources(QRhiCommandBuffer *cb, QRhiShaderResourceBin
df->vkCmdBindDescriptorSets(cbD->activeSecondaryCbStack.last(), df->vkCmdBindDescriptorSets(cbD->activeSecondaryCbStack.last(),
gfxPsD ? VK_PIPELINE_BIND_POINT_GRAPHICS : VK_PIPELINE_BIND_POINT_COMPUTE, gfxPsD ? VK_PIPELINE_BIND_POINT_GRAPHICS : VK_PIPELINE_BIND_POINT_COMPUTE,
gfxPsD ? gfxPsD->layout : compPsD->layout, gfxPsD ? gfxPsD->layout : compPsD->layout,
0, 1, &srbD->descSets[descSetIdx], 0, 1, &srbD->descSets[currentFrameSlot],
uint32_t(dynOfs.size()), uint32_t(dynOfs.size()),
dynOfs.size() ? dynOfs.constData() : nullptr); dynOfs.size() ? dynOfs.constData() : nullptr);
} else { } else {
@ -5764,7 +5758,7 @@ void QRhiVulkan::setShaderResources(QRhiCommandBuffer *cb, QRhiShaderResourceBin
cmd.args.bindDescriptorSet.bindPoint = gfxPsD ? VK_PIPELINE_BIND_POINT_GRAPHICS cmd.args.bindDescriptorSet.bindPoint = gfxPsD ? VK_PIPELINE_BIND_POINT_GRAPHICS
: VK_PIPELINE_BIND_POINT_COMPUTE; : VK_PIPELINE_BIND_POINT_COMPUTE;
cmd.args.bindDescriptorSet.pipelineLayout = gfxPsD ? gfxPsD->layout : compPsD->layout; cmd.args.bindDescriptorSet.pipelineLayout = gfxPsD ? gfxPsD->layout : compPsD->layout;
cmd.args.bindDescriptorSet.descSet = srbD->descSets[descSetIdx]; cmd.args.bindDescriptorSet.descSet = srbD->descSets[currentFrameSlot];
cmd.args.bindDescriptorSet.dynamicOffsetCount = dynOfs.size(); cmd.args.bindDescriptorSet.dynamicOffsetCount = dynOfs.size();
cmd.args.bindDescriptorSet.dynamicOffsetIndex = cbD->pools.dynamicOffset.size(); cmd.args.bindDescriptorSet.dynamicOffsetIndex = cbD->pools.dynamicOffset.size();
cbD->pools.dynamicOffset.append(dynOfs.constData(), dynOfs.size()); cbD->pools.dynamicOffset.append(dynOfs.constData(), dynOfs.size());
@ -5778,7 +5772,7 @@ void QRhiVulkan::setShaderResources(QRhiCommandBuffer *cb, QRhiShaderResourceBin
cbD->currentComputeSrb = srb; cbD->currentComputeSrb = srb;
} }
cbD->currentSrbGeneration = srbD->generation; cbD->currentSrbGeneration = srbD->generation;
cbD->currentDescSetSlot = descSetIdx; cbD->currentDescSetSlot = currentFrameSlot;
} }
srbD->lastActiveFrameSlot = currentFrameSlot; srbD->lastActiveFrameSlot = currentFrameSlot;
@ -7992,13 +7986,10 @@ bool QVkShaderResourceBindings::create()
std::copy(m_bindings.cbegin(), m_bindings.cend(), std::back_inserter(sortedBindings)); std::copy(m_bindings.cbegin(), m_bindings.cend(), std::back_inserter(sortedBindings));
std::sort(sortedBindings.begin(), sortedBindings.end(), QRhiImplementation::sortedBindingLessThan); std::sort(sortedBindings.begin(), sortedBindings.end(), QRhiImplementation::sortedBindingLessThan);
hasSlottedResource = false;
hasDynamicOffset = false; hasDynamicOffset = false;
for (const QRhiShaderResourceBinding &binding : std::as_const(sortedBindings)) { for (const QRhiShaderResourceBinding &binding : std::as_const(sortedBindings)) {
const QRhiShaderResourceBinding::Data *b = QRhiImplementation::shaderResourceBindingData(binding); const QRhiShaderResourceBinding::Data *b = QRhiImplementation::shaderResourceBindingData(binding);
if (b->type == QRhiShaderResourceBinding::UniformBuffer && b->u.ubuf.buf) { if (b->type == QRhiShaderResourceBinding::UniformBuffer && b->u.ubuf.buf) {
if (QRHI_RES(QVkBuffer, b->u.ubuf.buf)->type() == QRhiBuffer::Dynamic)
hasSlottedResource = true;
if (b->u.ubuf.hasDynamicOffset) if (b->u.ubuf.hasDynamicOffset)
hasDynamicOffset = true; hasDynamicOffset = true;
} }

View File

@ -252,7 +252,6 @@ struct QVkShaderResourceBindings : public QRhiShaderResourceBindings
void updateResources(UpdateFlags flags) override; void updateResources(UpdateFlags flags) override;
QVarLengthArray<QRhiShaderResourceBinding, 8> sortedBindings; QVarLengthArray<QRhiShaderResourceBinding, 8> sortedBindings;
bool hasSlottedResource = false;
bool hasDynamicOffset = false; bool hasDynamicOffset = false;
int poolIndex = -1; int poolIndex = -1;
VkDescriptorSetLayout layout = VK_NULL_HANDLE; VkDescriptorSetLayout layout = VK_NULL_HANDLE;
@ -860,7 +859,7 @@ public:
VkPipelineStageFlags srcStage, VkPipelineStageFlags dstStage, VkPipelineStageFlags srcStage, VkPipelineStageFlags dstStage,
int startLayer, int layerCount, int startLayer, int layerCount,
int startLevel, int levelCount); int startLevel, int levelCount);
void updateShaderResourceBindings(QRhiShaderResourceBindings *srb, int descSetIdx = -1); void updateShaderResourceBindings(QRhiShaderResourceBindings *srb);
void ensureCommandPoolForNewFrame(); void ensureCommandPoolForNewFrame();
double elapsedSecondsFromTimestamp(quint64 timestamp[2], bool *ok); double elapsedSecondsFromTimestamp(quint64 timestamp[2], bool *ok);
void printExtraErrorInfo(VkResult err); void printExtraErrorInfo(VkResult err);