radv/video: Don't use op_close in encode

This should only be used on video session destroy, but at that point
we don't have command buffer. RENCODE_IB_OP_CLOSE_SESSION is optional,
so it's okay not using it at all.

Reviewed-by: Dave Airlie <airlied@redhat.com>
Part-of: <https://gitlab.freedesktop.org/mesa/mesa/-/merge_requests/31078>
This commit is contained in:
David Rosca 2024-09-08 07:08:56 +02:00 committed by Dave Airlie
parent 2a5a12cb71
commit 60c38b9886

View file

@ -1461,15 +1461,6 @@ radv_enc_op_init(struct radv_cmd_buffer *cmd_buffer)
ENC_END;
}
static void
radv_enc_op_close(struct radv_cmd_buffer *cmd_buffer)
{
struct radeon_cmdbuf *cs = cmd_buffer->cs;
ENC_BEGIN;
radeon_emit(cs, RENCODE_IB_OP_CLOSE_SESSION);
ENC_END;
}
static void
radv_enc_op_enc(struct radv_cmd_buffer *cmd_buffer)
{
@ -1652,17 +1643,6 @@ begin(struct radv_cmd_buffer *cmd_buffer, const VkVideoEncodeInfoKHR *enc_info)
radeon_emit_direct(cmd_buffer->cs, enc->task_size_offset, enc->total_task_size);
}
static void
destroy(struct radv_cmd_buffer *cmd_buffer)
{
struct radv_enc_state *enc = &cmd_buffer->video.enc;
radv_enc_session_info(cmd_buffer);
cmd_buffer->video.enc.total_task_size = 0;
radv_enc_task_info(cmd_buffer, false);
radv_enc_op_close(cmd_buffer);
radeon_emit_direct(cmd_buffer->cs, enc->task_size_offset, enc->total_task_size);
}
static void
radv_vcn_encode_video(struct radv_cmd_buffer *cmd_buffer, const VkVideoEncodeInfoKHR *enc_info)
{
@ -1752,8 +1732,6 @@ radv_vcn_encode_video(struct radv_cmd_buffer *cmd_buffer, const VkVideoEncodeInf
radv_enc_op_enc(cmd_buffer);
radeon_emit_direct(cmd_buffer->cs, enc->task_size_offset, enc->total_task_size);
destroy(cmd_buffer);
}
static void