From: Samuel Pitoiset Date: Wed, 4 Oct 2017 20:27:38 +0000 (+0200) Subject: radv: check that pipeline is different before binding it X-Git-Url: https://git.libre-soc.org/?a=commitdiff_plain;h=b53c207659e65a29081c0f26b189f37fdd895406;p=mesa.git radv: check that pipeline is different before binding it We only need to dirty the descriptors when the pipeline is a new one, because user SGPRs can be potentially different. Signed-off-by: Samuel Pitoiset Reviewed-by: Bas Nieuwenhuizen --- diff --git a/src/amd/vulkan/radv_cmd_buffer.c b/src/amd/vulkan/radv_cmd_buffer.c index 61ea11c12a6..4b41b358e99 100644 --- a/src/amd/vulkan/radv_cmd_buffer.c +++ b/src/amd/vulkan/radv_cmd_buffer.c @@ -2454,14 +2454,20 @@ void radv_CmdBindPipeline( RADV_FROM_HANDLE(radv_cmd_buffer, cmd_buffer, commandBuffer); RADV_FROM_HANDLE(radv_pipeline, pipeline, _pipeline); - radv_mark_descriptor_sets_dirty(cmd_buffer); - switch (pipelineBindPoint) { case VK_PIPELINE_BIND_POINT_COMPUTE: + if (cmd_buffer->state.compute_pipeline == pipeline) + return; + radv_mark_descriptor_sets_dirty(cmd_buffer); + cmd_buffer->state.compute_pipeline = pipeline; cmd_buffer->push_constant_stages |= VK_SHADER_STAGE_COMPUTE_BIT; break; case VK_PIPELINE_BIND_POINT_GRAPHICS: + if (cmd_buffer->state.pipeline == pipeline) + return; + radv_mark_descriptor_sets_dirty(cmd_buffer); + cmd_buffer->state.pipeline = pipeline; if (!pipeline) break;