X-Git-Url: http://git.tdb.fi/?a=blobdiff_plain;f=source%2Fbackends%2Fvulkan%2Fpipelinestate_backend.cpp;h=5f9e7eb9185a4e812c1826edf0e071677e2b20d7;hb=2a70fecfbbe8708be2bdaa75d222dd5a889a8ed3;hp=7deae24c9b09b59ada978489feb1bdfd6b1637a9;hpb=d05d9963e185566f03ea348999655037617e4ba9;p=libs%2Fgl.git diff --git a/source/backends/vulkan/pipelinestate_backend.cpp b/source/backends/vulkan/pipelinestate_backend.cpp index 7deae24c..5f9e7eb9 100644 --- a/source/backends/vulkan/pipelinestate_backend.cpp +++ b/source/backends/vulkan/pipelinestate_backend.cpp @@ -10,6 +10,7 @@ #include "pipelinestate_backend.h" #include "program.h" #include "rect.h" +#include "renderpass.h" #include "sampler.h" #include "stenciltest.h" #include "structurebuilder.h" @@ -55,29 +56,25 @@ void VulkanPipelineState::update() const unapplied |= PipelineState::SHPROG; } - if(changes&(PipelineState::SHPROG|PipelineState::UNIFORMS|PipelineState::TEXTURES)) + if(changes&(PipelineState::SHPROG|PipelineState::RESOURCES)) { unsigned changed_sets = (changes&PipelineState::SHPROG ? ~0U : 0U); - for(const PipelineState::BoundUniformBlock &u: self.uniform_blocks) - if(u.changed || changed_sets==~0U) + for(const PipelineState::BoundResource &r: self.resources) + if(r.changed || changed_sets==~0U) { - if(u.block) - u.used = self.shprog->uses_uniform_block_binding(u.binding); - if(u.binding>=0) - changed_sets |= 1<<(u.binding>>20); - u.changed = false; - } - for(const PipelineState::BoundTexture &t: self.textures) - if(t.changed || changed_sets==~0U) - { - if(t.texture && t.sampler) - t.used = self.shprog->uses_texture_binding(t.binding); - changed_sets |= 1<<(t.binding>>20); - if(t.texture && t.level>=0) - t.texture->refresh_mip_views(); - if(t.sampler) - t.sampler->refresh(); - t.changed = false; + if(r.type==PipelineState::UNIFORM_BLOCK) + r.used = self.shprog->uses_uniform_block_binding(r.binding); + else if(r.type==PipelineState::SAMPLED_TEXTURE || r.type==PipelineState::STORAGE_TEXTURE) + { + r.used = self.shprog->uses_texture_binding(r.binding); + if(r.mip_level>=0) + r.texture->refresh_mip_views(); + if(r.type==PipelineState::SAMPLED_TEXTURE) + r.sampler->refresh(); + } + if(r.binding>=0) + changed_sets |= 1<<(r.binding>>20); + r.changed = false; } if(changed_sets) @@ -91,7 +88,7 @@ void VulkanPipelineState::update() const first_changed_desc_set = min(first_changed_desc_set, i); } - unapplied |= PipelineState::UNIFORMS; + unapplied |= PipelineState::RESOURCES; } } @@ -149,7 +146,9 @@ void VulkanPipelineState::fill_creation_info(vector &buffer) const const PipelineState &self = *static_cast(this); const FrameFormat &format = self.framebuffer->get_format(); - VkRenderPass render_pass = device.get_pipeline_cache().get_render_pass(format, false, false, false); + RenderPass render_pass; + render_pass.framebuffer = self.framebuffer; + render_pass.update(device); unsigned n_color_attachments = 0; for(FrameAttachment a: format) @@ -160,16 +159,16 @@ void VulkanPipelineState::fill_creation_info(vector &buffer) const } StructureBuilder sb(buffer, 10); - VkGraphicsPipelineCreateInfo *&pipeline_info = sb.add(); - VkPipelineInputAssemblyStateCreateInfo *&input_assembly_info = sb.add(); - VkPipelineViewportStateCreateInfo *&viewport_info = sb.add(); - VkPipelineRasterizationStateCreateInfo *&raster_info = sb.add(); - VkPipelineMultisampleStateCreateInfo *&multisample_info = sb.add(); - VkPipelineDepthStencilStateCreateInfo *&depth_stencil_info = sb.add(); - VkPipelineColorBlendStateCreateInfo *&blend_info = sb.add(); - VkPipelineColorBlendAttachmentState *&blend_attachments = sb.add(n_color_attachments); - VkPipelineDynamicStateCreateInfo *&dynamic_info = sb.add(); - VkDynamicState *&dynamic_states = sb.add(2); + VkGraphicsPipelineCreateInfo *const &pipeline_info = sb.add(); + VkPipelineInputAssemblyStateCreateInfo *const &input_assembly_info = sb.add(); + VkPipelineViewportStateCreateInfo *const &viewport_info = sb.add(); + VkPipelineRasterizationStateCreateInfo *const &raster_info = sb.add(); + VkPipelineMultisampleStateCreateInfo *const &multisample_info = sb.add(); + VkPipelineDepthStencilStateCreateInfo *const &depth_stencil_info = sb.add(); + VkPipelineColorBlendStateCreateInfo *const &blend_info = sb.add(); + VkPipelineColorBlendAttachmentState *const &blend_attachments = sb.add(n_color_attachments); + VkPipelineDynamicStateCreateInfo *const &dynamic_info = sb.add(); + VkDynamicState *const &dynamic_states = sb.add(2); input_assembly_info->sType = VK_STRUCTURE_TYPE_PIPELINE_INPUT_ASSEMBLY_STATE_CREATE_INFO; input_assembly_info->topology = static_cast(get_vulkan_primitive_type(self.primitive_type)); @@ -253,7 +252,7 @@ void VulkanPipelineState::fill_creation_info(vector &buffer) const pipeline_info->pDepthStencilState = depth_stencil_info; pipeline_info->pColorBlendState = blend_info; pipeline_info->pDynamicState = dynamic_info; - pipeline_info->renderPass = handle_cast<::VkRenderPass>(render_pass); + pipeline_info->renderPass = handle_cast<::VkRenderPass>(render_pass.handle); pipeline_info->subpass = 0; if(self.shprog) @@ -274,26 +273,29 @@ uint64_t VulkanPipelineState::compute_descriptor_set_hash(unsigned index) const uint64_t result = hash<64>(0, 0); bool empty = true; - auto i = lower_bound_member(self.uniform_blocks, static_cast(index)<<20, &PipelineState::BoundUniformBlock::binding); - for(; (i!=self.uniform_blocks.end() && static_cast(i->binding)>>20==index); ++i) - if(i->used) + auto i = lower_bound_member(self.resources, static_cast(index)<<20, &PipelineState::BoundResource::binding); + for(; (i!=self.resources.end() && static_cast(i->binding)>>20==index); ++i) + { + if(!i->used) + continue; + + result = hash_update<64>(result, i->binding); + result = hash_update<64>(result, i->type); + if(i->type==PipelineState::UNIFORM_BLOCK) { - result = hash_update<64>(result, i->binding); result = hash_update<64>(result, reinterpret_cast(i->block)); result = hash_update<64>(result, reinterpret_cast(i->buffer->handle)); - empty = false; } - - auto j = lower_bound_member(self.textures, index<<20, &PipelineState::BoundTexture::binding); - for(; (j!=self.textures.end() && j->binding>>20==index); ++j) - if(j->used) + else if(i->type==PipelineState::SAMPLED_TEXTURE) { - result = hash_update<64>(result, j->binding); - result = hash_update<64>(result, reinterpret_cast(j->texture->handle)); - result = hash_update<64>(result, reinterpret_cast(j->sampler->handle)); - result = hash_update<64>(result, j->level); - empty = false; + result = hash_update<64>(result, reinterpret_cast(i->texture->handle)); + result = hash_update<64>(result, reinterpret_cast(i->sampler->handle)); + result = hash_update<64>(result, i->mip_level); } + else if(i->type==PipelineState::STORAGE_TEXTURE) + result = hash_update<64>(result, reinterpret_cast(i->texture->handle)); + empty = false; + } if(!empty) result = hash_update<64>(result, self.shprog->stage_flags); @@ -305,9 +307,9 @@ bool VulkanPipelineState::is_descriptor_set_dynamic(unsigned index) const { const PipelineState &self = *static_cast(this); - auto i = lower_bound_member(self.uniform_blocks, static_cast(index)<<20, &PipelineState::BoundUniformBlock::binding); - for(; (i!=self.uniform_blocks.end() && static_cast(i->binding)>>20==index); ++i) - if(i->used && i->buffer && i->buffer->get_usage()==STREAMING) + auto i = lower_bound_member(self.resources, static_cast(index)<<20, &PipelineState::BoundResource::binding); + for(; (i!=self.resources.end() && static_cast(i->binding)>>20==index); ++i) + if(i->used && i->type==PipelineState::UNIFORM_BLOCK && i->buffer->get_usage()==STREAMING) return true; return false; @@ -322,30 +324,35 @@ unsigned VulkanPipelineState::fill_descriptor_writes(unsigned index, unsigned fr { const PipelineState &self = *static_cast(this); - auto u_begin = lower_bound_member(self.uniform_blocks, static_cast(index)<<20, &PipelineState::BoundUniformBlock::binding); - auto t_begin = lower_bound_member(self.textures, index<<20, &PipelineState::BoundTexture::binding); + auto begin = lower_bound_member(self.resources, static_cast(index)<<20, &PipelineState::BoundResource::binding); unsigned n_buffers = 0; - for(auto i=u_begin; (i!=self.uniform_blocks.end() && static_cast(i->binding)>>20==index); ++i) - if(i->used) - ++n_buffers; unsigned n_images = 0; - for(auto i=t_begin; (i!=self.textures.end() && i->binding>>20==index); ++i) + for(auto i=begin; (i!=self.resources.end() && static_cast(i->binding)>>20==index); ++i) if(i->used) - ++n_images; + { + if(i->type==PipelineState::UNIFORM_BLOCK) + ++n_buffers; + else if(i->type==PipelineState::SAMPLED_TEXTURE || i->type==PipelineState::STORAGE_TEXTURE) + ++n_images; + } unsigned n_writes = n_buffers+n_images; StructureBuilder sb(buffer, 3); - VkWriteDescriptorSet *&writes = sb.add(n_writes); - VkDescriptorBufferInfo *&buffers = sb.add(n_buffers); - VkDescriptorImageInfo *&images = sb.add(n_images); + VkWriteDescriptorSet *const &writes = sb.add(n_writes); + VkDescriptorBufferInfo *const &buffers = sb.add(n_buffers); + VkDescriptorImageInfo *const &images = sb.add(n_images); VkWriteDescriptorSet *write_ptr = writes; VkDescriptorBufferInfo *buffer_ptr = buffers; VkDescriptorImageInfo *image_ptr = images; - for(auto i=u_begin; (i!=self.uniform_blocks.end() && static_cast(i->binding)>>20==index); ++i) - if(i->used) + for(auto i=begin; (i!=self.resources.end() && static_cast(i->binding)>>20==index); ++i) + { + if(!i->used) + continue; + + if(i->type==PipelineState::UNIFORM_BLOCK) { buffer_ptr->buffer = handle_cast<::VkBuffer>(i->buffer->handle); buffer_ptr->offset = i->block->get_offset(); @@ -360,32 +367,49 @@ unsigned VulkanPipelineState::fill_descriptor_writes(unsigned index, unsigned fr write_ptr->pBufferInfo = buffer_ptr; ++buffer_ptr; - ++write_ptr; } - - for(auto i=t_begin; (i!=self.textures.end() && i->binding>>20==index); ++i) - if(i->used) + else if(i->type==PipelineState::SAMPLED_TEXTURE || i->type==PipelineState::STORAGE_TEXTURE) { - image_ptr->sampler = handle_cast<::VkSampler>(i->sampler->handle); - if(i->level<0) + if(i->mip_level<0) image_ptr->imageView = handle_cast<::VkImageView>(i->texture->view_handle); else - image_ptr->imageView = handle_cast<::VkImageView>(i->texture->mip_view_handles[i->level]); - image_ptr->imageLayout = VK_IMAGE_LAYOUT_SHADER_READ_ONLY_OPTIMAL; + image_ptr->imageView = handle_cast<::VkImageView>(i->texture->mip_view_handles[i->mip_level]); + + if(i->type==PipelineState::SAMPLED_TEXTURE) + { + image_ptr->sampler = handle_cast<::VkSampler>(i->sampler->handle); + image_ptr->imageLayout = VK_IMAGE_LAYOUT_SHADER_READ_ONLY_OPTIMAL; + write_ptr->descriptorType = VK_DESCRIPTOR_TYPE_COMBINED_IMAGE_SAMPLER; + } + else if(i->type==PipelineState::STORAGE_TEXTURE) + { + image_ptr->imageLayout = VK_IMAGE_LAYOUT_GENERAL; + write_ptr->descriptorType = VK_DESCRIPTOR_TYPE_STORAGE_IMAGE; + } write_ptr->sType = VK_STRUCTURE_TYPE_WRITE_DESCRIPTOR_SET; write_ptr->dstBinding = i->binding&0xFFFFF; write_ptr->descriptorCount = 1; - write_ptr->descriptorType = VK_DESCRIPTOR_TYPE_COMBINED_IMAGE_SAMPLER; write_ptr->pImageInfo = image_ptr; ++image_ptr; - ++write_ptr; } + ++write_ptr; + } + return n_writes; } +void VulkanPipelineState::synchronize_resources(bool discard_fb_contents) const +{ + const PipelineState &self = *static_cast(this); + + for(const PipelineState::BoundResource &r: self.resources) + if(r.type==PipelineState::STORAGE_TEXTURE) + r.texture->change_layout(-1, VK_IMAGE_LAYOUT_GENERAL, false); +} + void VulkanPipelineState::apply(const VulkanCommandRecorder &vkCmd, const VulkanPipelineState *last, unsigned frame, bool negative_viewport) const { const PipelineState &self = *static_cast(this); @@ -403,7 +427,7 @@ void VulkanPipelineState::apply(const VulkanCommandRecorder &vkCmd, const Vulkan unapplied |= PipelineState::SHPROG; if(self.push_const_compat!=last_ps.push_const_compat) { - unapplied |= PipelineState::UNIFORMS; + unapplied |= PipelineState::RESOURCES; first_changed_desc_set = 0; } } @@ -412,7 +436,7 @@ void VulkanPipelineState::apply(const VulkanCommandRecorder &vkCmd, const Vulkan for(unsigned i=0; i=last->descriptor_set_slots.size() || descriptor_set_slots[i]!=last->descriptor_set_slots[i]) { - unapplied |= PipelineState::UNIFORMS; + unapplied |= PipelineState::RESOURCES; first_changed_desc_set = min(first_changed_desc_set, i); break; } @@ -433,18 +457,18 @@ void VulkanPipelineState::apply(const VulkanCommandRecorder &vkCmd, const Vulkan vkCmd.BindIndexBuffer(vs->get_index_buffer()->handle, 0, index_type); } - if(!self.uniform_blocks.empty()) + if(!self.resources.empty()) { - const PipelineState::BoundUniformBlock &first_block = self.uniform_blocks.front(); - if(first_block.used && first_block.binding==ReflectData::PUSH_CONSTANT) + const PipelineState::BoundResource &first_res = self.resources.front(); + if(first_res.used && first_res.type==PipelineState::UNIFORM_BLOCK && first_res.binding==ReflectData::PUSH_CONSTANT) { - const UniformBlock &pc_block = *first_block.block; + const UniformBlock &pc_block = *first_res.block; vkCmd.PushConstants(self.shprog->layout_handle, self.shprog->stage_flags, pc_block.get_offset(), pc_block.get_data_size(), pc_block.get_data_pointer()); } } - if((unapplied&PipelineState::UNIFORMS) && !descriptor_set_slots.empty()) + if((unapplied&PipelineState::RESOURCES) && !descriptor_set_slots.empty()) { vector descriptor_set_handles; descriptor_set_handles.reserve(descriptor_set_slots.size()-first_changed_desc_set);