While this is a fix, I'm not sure if it is a fix we want without more
fixes. If we end up not finishing the flush_state, we could end up
emitting a draw packet without having emitted the shader which sounds
susceptible to hangs to me.
On Fri, Sep 8, 2017 at 3:15 PM, Samuel Pitoiset
<samuel.pitoi...@gmail.com> wrote:
> A return code error is stored in the command buffer and should
> be returned to the user via EndCommandBuffer().
>
> Signed-off-by: Samuel Pitoiset <samuel.pitoi...@gmail.com>
> ---
> src/amd/vulkan/radv_cmd_buffer.c | 12 ++++++++----
> 1 file changed, 8 insertions(+), 4 deletions(-)
>
> diff --git a/src/amd/vulkan/radv_cmd_buffer.c
> b/src/amd/vulkan/radv_cmd_buffer.c
> index a79b513ffc..07d9be9bf4 100644
> --- a/src/amd/vulkan/radv_cmd_buffer.c
> +++ b/src/amd/vulkan/radv_cmd_buffer.c
> @@ -1578,7 +1578,7 @@ static void radv_emit_primitive_reset_state(struct
> radv_cmd_buffer *cmd_buffer,
> }
> }
>
> -static void
> +static bool
> radv_cmd_buffer_update_vertex_descriptors(struct radv_cmd_buffer *cmd_buffer)
> {
> struct radv_device *device = cmd_buffer->device;
> @@ -1594,8 +1594,9 @@ radv_cmd_buffer_update_vertex_descriptors(struct
> radv_cmd_buffer *cmd_buffer)
> uint64_t va;
>
> /* allocate some descriptor state for vertex buffers */
> - radv_cmd_buffer_upload_alloc(cmd_buffer, count * 16, 256,
> - &vb_offset, &vb_ptr);
> + if (!radv_cmd_buffer_upload_alloc(cmd_buffer, count * 16, 256,
> + &vb_offset, &vb_ptr))
> + return false;
>
> for (i = 0; i < count; i++) {
> uint32_t *desc = &((uint32_t *)vb_ptr)[i * 4];
> @@ -1625,6 +1626,8 @@ radv_cmd_buffer_update_vertex_descriptors(struct
> radv_cmd_buffer *cmd_buffer)
> AC_UD_VS_VERTEX_BUFFERS, va);
> }
> cmd_buffer->state.vb_dirty = false;
> +
> + return true;
> }
>
> static void
> @@ -1639,7 +1642,8 @@ radv_cmd_buffer_flush_state(struct radv_cmd_buffer
> *cmd_buffer,
> MAYBE_UNUSED unsigned cdw_max =
> radeon_check_space(cmd_buffer->device->ws,
> cmd_buffer->cs,
> 4096);
>
> - radv_cmd_buffer_update_vertex_descriptors(cmd_buffer);
> + if (!radv_cmd_buffer_update_vertex_descriptors(cmd_buffer))
> + return;
>
> if (cmd_buffer->state.dirty & RADV_CMD_DIRTY_PIPELINE)
> radv_emit_graphics_pipeline(cmd_buffer, pipeline);
> --
> 2.14.1
>
> _______________________________________________
> mesa-dev mailing list
> mesa-dev@lists.freedesktop.org
> https://lists.freedesktop.org/mailman/listinfo/mesa-dev
_______________________________________________
mesa-dev mailing list
mesa-dev@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/mesa-dev
