From 06a12f250f9f8d7790b7777240440c1ea525d908 Mon Sep 17 00:00:00 2001 From: Samuel Pitoiset Date: Wed, 13 Sep 2017 13:55:58 +0200 Subject: [PATCH] radv: only copy the dynamic states that changed When binding a new pipeline, we applied all dynamic states without checking if they really need to be re-emitted. This doesn't seem to be useful for the meta operations because only the viewports/scissors are updated. This should reduce the number of commands added to the IB when a new graphics pipeline is bound. Also, rename radv_dynamic_state_copy() to radv_bind_dynamic_state() and set the dirty flags directly there. Signed-off-by: Samuel Pitoiset Reviewed-by: Dave Airlie --- src/amd/vulkan/radv_cmd_buffer.c | 92 ++++++++++++++++++++++++-------- 1 file changed, 69 insertions(+), 23 deletions(-) diff --git a/src/amd/vulkan/radv_cmd_buffer.c b/src/amd/vulkan/radv_cmd_buffer.c index 7e5da5485b3..08a05277fa5 100644 --- a/src/amd/vulkan/radv_cmd_buffer.c +++ b/src/amd/vulkan/radv_cmd_buffer.c @@ -79,10 +79,12 @@ const struct radv_dynamic_state default_dynamic_state = { }; static void -radv_dynamic_state_copy(struct radv_dynamic_state *dest, +radv_bind_dynamic_state(struct radv_cmd_buffer *cmd_buffer, const struct radv_dynamic_state *src) { + struct radv_dynamic_state *dest = &cmd_buffer->state.dynamic; uint32_t copy_mask = src->mask; + uint32_t dest_mask = 0; /* Make sure to copy the number of viewports/scissors because they can * only be specified at pipeline creation time. @@ -91,35 +93,82 @@ radv_dynamic_state_copy(struct radv_dynamic_state *dest, dest->scissor.count = src->scissor.count; if (copy_mask & (1 << VK_DYNAMIC_STATE_VIEWPORT)) { - typed_memcpy(dest->viewport.viewports, src->viewport.viewports, - src->viewport.count); + if (memcmp(&dest->viewport.viewports, &src->viewport.viewports, + src->viewport.count * sizeof(VkViewport))) { + typed_memcpy(dest->viewport.viewports, + src->viewport.viewports, + src->viewport.count); + dest_mask |= 1 << VK_DYNAMIC_STATE_VIEWPORT; + } } if (copy_mask & (1 << VK_DYNAMIC_STATE_SCISSOR)) { - typed_memcpy(dest->scissor.scissors, src->scissor.scissors, - src->scissor.count); + if (memcmp(&dest->scissor.scissors, &src->scissor.scissors, + src->scissor.count * sizeof(VkRect2D))) { + typed_memcpy(dest->scissor.scissors, + src->scissor.scissors, src->scissor.count); + dest_mask |= 1 << VK_DYNAMIC_STATE_SCISSOR; + } } - if (copy_mask & (1 << VK_DYNAMIC_STATE_LINE_WIDTH)) - dest->line_width = src->line_width; + if (copy_mask & (1 << VK_DYNAMIC_STATE_LINE_WIDTH)) { + if (dest->line_width != src->line_width) { + dest->line_width = src->line_width; + dest_mask |= 1 << VK_DYNAMIC_STATE_LINE_WIDTH; + } + } - if (copy_mask & (1 << VK_DYNAMIC_STATE_DEPTH_BIAS)) - dest->depth_bias = src->depth_bias; + if (copy_mask & (1 << VK_DYNAMIC_STATE_DEPTH_BIAS)) { + if (memcmp(&dest->depth_bias, &src->depth_bias, + sizeof(src->depth_bias))) { + dest->depth_bias = src->depth_bias; + dest_mask |= 1 << VK_DYNAMIC_STATE_DEPTH_BIAS; + } + } - if (copy_mask & (1 << VK_DYNAMIC_STATE_BLEND_CONSTANTS)) - typed_memcpy(dest->blend_constants, src->blend_constants, 4); + if (copy_mask & (1 << VK_DYNAMIC_STATE_BLEND_CONSTANTS)) { + if (memcmp(&dest->blend_constants, &src->blend_constants, + sizeof(src->blend_constants))) { + typed_memcpy(dest->blend_constants, + src->blend_constants, 4); + dest_mask |= 1 << VK_DYNAMIC_STATE_BLEND_CONSTANTS; + } + } - if (copy_mask & (1 << VK_DYNAMIC_STATE_DEPTH_BOUNDS)) - dest->depth_bounds = src->depth_bounds; + if (copy_mask & (1 << VK_DYNAMIC_STATE_DEPTH_BOUNDS)) { + if (memcmp(&dest->depth_bounds, &src->depth_bounds, + sizeof(src->depth_bounds))) { + dest->depth_bounds = src->depth_bounds; + dest_mask |= 1 << VK_DYNAMIC_STATE_DEPTH_BOUNDS; + } + } + + if (copy_mask & (1 << VK_DYNAMIC_STATE_STENCIL_COMPARE_MASK)) { + if (memcmp(&dest->stencil_compare_mask, + &src->stencil_compare_mask, + sizeof(src->stencil_compare_mask))) { + dest->stencil_compare_mask = src->stencil_compare_mask; + dest_mask |= 1 << VK_DYNAMIC_STATE_STENCIL_COMPARE_MASK; + } + } - if (copy_mask & (1 << VK_DYNAMIC_STATE_STENCIL_COMPARE_MASK)) - dest->stencil_compare_mask = src->stencil_compare_mask; + if (copy_mask & (1 << VK_DYNAMIC_STATE_STENCIL_WRITE_MASK)) { + if (memcmp(&dest->stencil_write_mask, &src->stencil_write_mask, + sizeof(src->stencil_write_mask))) { + dest->stencil_write_mask = src->stencil_write_mask; + dest_mask |= 1 << VK_DYNAMIC_STATE_STENCIL_WRITE_MASK; + } + } - if (copy_mask & (1 << VK_DYNAMIC_STATE_STENCIL_WRITE_MASK)) - dest->stencil_write_mask = src->stencil_write_mask; + if (copy_mask & (1 << VK_DYNAMIC_STATE_STENCIL_REFERENCE)) { + if (memcmp(&dest->stencil_reference, &src->stencil_reference, + sizeof(src->stencil_reference))) { + dest->stencil_reference = src->stencil_reference; + dest_mask |= 1 << VK_DYNAMIC_STATE_STENCIL_REFERENCE; + } + } - if (copy_mask & (1 << VK_DYNAMIC_STATE_STENCIL_REFERENCE)) - dest->stencil_reference = src->stencil_reference; + cmd_buffer->state.dirty |= dest_mask; } bool radv_cmd_buffer_uses_mec(struct radv_cmd_buffer *cmd_buffer) @@ -2531,10 +2580,7 @@ void radv_CmdBindPipeline( cmd_buffer->state.dirty |= RADV_CMD_DIRTY_PIPELINE; cmd_buffer->push_constant_stages |= pipeline->active_stages; - /* Apply the dynamic state from the pipeline */ - cmd_buffer->state.dirty |= pipeline->dynamic_state.mask; - radv_dynamic_state_copy(&cmd_buffer->state.dynamic, - &pipeline->dynamic_state); + radv_bind_dynamic_state(cmd_buffer, &pipeline->dynamic_state); if (pipeline->graphics.esgs_ring_size > cmd_buffer->esgs_ring_size_needed) cmd_buffer->esgs_ring_size_needed = pipeline->graphics.esgs_ring_size; -- 2.30.2