Module: Mesa Branch: master Commit: 5010436e09f3a7acae679e192d2fe65f04c87c79 URL: http://cgit.freedesktop.org/mesa/mesa/commit/?id=5010436e09f3a7acae679e192d2fe65f04c87c79
Author: Samuel Pitoiset <[email protected]> Date: Mon Oct 30 19:37:39 2017 +0100 radv: bail out when binding the same vertex buffers Signed-off-by: Samuel Pitoiset <[email protected]> Reviewed-by: Bas Nieuwenhuizen <[email protected]> --- src/amd/vulkan/radv_cmd_buffer.c | 18 ++++++++++++++++-- 1 file changed, 16 insertions(+), 2 deletions(-) diff --git a/src/amd/vulkan/radv_cmd_buffer.c b/src/amd/vulkan/radv_cmd_buffer.c index 00ed7182a7..4b38ece10f 100644 --- a/src/amd/vulkan/radv_cmd_buffer.c +++ b/src/amd/vulkan/radv_cmd_buffer.c @@ -2259,14 +2259,28 @@ void radv_CmdBindVertexBuffers( { RADV_FROM_HANDLE(radv_cmd_buffer, cmd_buffer, commandBuffer); struct radv_vertex_binding *vb = cmd_buffer->state.vertex_bindings; + bool changed = false; /* We have to defer setting up vertex buffer since we need the buffer * stride from the pipeline. */ assert(firstBinding + bindingCount <= MAX_VBS); for (uint32_t i = 0; i < bindingCount; i++) { - vb[firstBinding + i].buffer = radv_buffer_from_handle(pBuffers[i]); - vb[firstBinding + i].offset = pOffsets[i]; + uint32_t idx = firstBinding + i; + + if (!changed && + (vb[idx].buffer != radv_buffer_from_handle(pBuffers[i]) || + vb[idx].offset != pOffsets[i])) { + changed = true; + } + + vb[idx].buffer = radv_buffer_from_handle(pBuffers[i]); + vb[idx].offset = pOffsets[i]; + } + + if (!changed) { + /* No state changes. */ + return; } cmd_buffer->state.vb_dirty = true; _______________________________________________ mesa-commit mailing list [email protected] https://lists.freedesktop.org/mailman/listinfo/mesa-commit
