[FFmpeg-devel] [PATCH v4 2/2] lavc/vulkan_av1: port to the new stable API
Benjamin Cheng
ben at bcheng.me
Tue Feb 13 02:58:24 EET 2024
On Sun Feb 11, 2024 at 10:15 AM EST, Lynne wrote:
> From: Lynne <dev at lynne.ee>
> Date: Fri, 19 Jan 2024 10:49:02 +1000
> Subject: [PATCH v4 2/2] lavc/vulkan_av1: port to the new stable API
>
> Most of this patch was written by Dave Airlie <airlied at redhat.com>,
> with some additions by me.
> ---
> configure | 4 +-
> libavcodec/Makefile | 3 +-
> libavcodec/av1.h | 2 +
> libavcodec/vulkan_av1.c | 477 ++++++++++--------
> libavcodec/vulkan_decode.c | 24 +-
> libavcodec/vulkan_decode.h | 2 +-
> libavcodec/vulkan_video.c | 2 +-
> libavcodec/vulkan_video.h | 2 -
> libavcodec/vulkan_video_codec_av1std.h | 403 ---------------
> libavcodec/vulkan_video_codec_av1std_decode.h | 36 --
> libavutil/hwcontext_vulkan.c | 2 +-
> libavutil/vulkan_functions.h | 2 +-
> libavutil/vulkan_loader.h | 2 +-
> 13 files changed, 277 insertions(+), 684 deletions(-)
> delete mode 100644 libavcodec/vulkan_video_codec_av1std.h
> delete mode 100644 libavcodec/vulkan_video_codec_av1std_decode.h
>
> diff --git a/configure b/configure
> index f72533b7d2..9a3a451115 100755
> --- a/configure
> +++ b/configure
> @@ -7225,8 +7225,8 @@ enabled crystalhd && check_lib crystalhd "stdint.h libcrystalhd/libcrystalhd_if.
> "in maintaining it."
>
> if enabled vulkan; then
> - check_pkg_config_header_only vulkan "vulkan >= 1.3.255" "vulkan/vulkan.h" "defined VK_VERSION_1_3" ||
> - check_cpp_condition vulkan "vulkan/vulkan.h" "defined(VK_VERSION_1_4) || (defined(VK_VERSION_1_3) && VK_HEADER_VERSION >= 255)"
> + check_pkg_config_header_only vulkan "vulkan >= 1.3.277" "vulkan/vulkan.h" "defined VK_VERSION_1_3" ||
> + check_cpp_condition vulkan "vulkan/vulkan.h" "defined(VK_VERSION_1_4) || (defined(VK_VERSION_1_3) && VK_HEADER_VERSION >= 277)"
> fi
>
> if disabled vulkan; then
> diff --git a/libavcodec/Makefile b/libavcodec/Makefile
> index 470d7cb9b1..581415f3b7 100644
> --- a/libavcodec/Makefile
> +++ b/libavcodec/Makefile
> @@ -1262,7 +1262,6 @@ SKIPHEADERS += %_tablegen.h \
> aacenc_quantization.h \
> aacenc_quantization_misc.h \
> bitstream_template.h \
> - vulkan_video_codec_av1std.h \
> $(ARCH)/vpx_arith.h \
>
> SKIPHEADERS-$(CONFIG_AMF) += amfenc.h
> @@ -1285,7 +1284,7 @@ SKIPHEADERS-$(CONFIG_XVMC) += xvmc.h
> SKIPHEADERS-$(CONFIG_VAAPI) += vaapi_decode.h vaapi_hevc.h vaapi_encode.h
> SKIPHEADERS-$(CONFIG_VDPAU) += vdpau.h vdpau_internal.h
> SKIPHEADERS-$(CONFIG_VIDEOTOOLBOX) += videotoolbox.h vt_internal.h
> -SKIPHEADERS-$(CONFIG_VULKAN) += vulkan.h vulkan_video.h vulkan_decode.h vulkan_video_codec_av1std_decode.h
> +SKIPHEADERS-$(CONFIG_VULKAN) += vulkan.h vulkan_video.h vulkan_decode.h
> SKIPHEADERS-$(CONFIG_V4L2_M2M) += v4l2_buffers.h v4l2_context.h v4l2_m2m.h
> SKIPHEADERS-$(CONFIG_ZLIB) += zlib_wrapper.h
>
> diff --git a/libavcodec/av1.h b/libavcodec/av1.h
> index 8704bc41c1..18d5fa9e7f 100644
> --- a/libavcodec/av1.h
> +++ b/libavcodec/av1.h
> @@ -121,6 +121,8 @@ enum {
> AV1_DIV_LUT_NUM = 257,
>
> AV1_MAX_LOOP_FILTER = 63,
> +
> + AV1_RESTORATION_TILESIZE_MAX = 256,
> };
>
>
> diff --git a/libavcodec/vulkan_av1.c b/libavcodec/vulkan_av1.c
> index 9730e4b08d..641ee3e4bf 100644
> --- a/libavcodec/vulkan_av1.c
> +++ b/libavcodec/vulkan_av1.c
> @@ -31,22 +31,36 @@ const VkExtensionProperties ff_vk_dec_av1_ext = {
> typedef struct AV1VulkanDecodePicture {
> FFVulkanDecodePicture vp;
>
> - /* Workaround for a spec issue.
> - *Can be removed once no longer needed, and threading can be enabled. */
> + /* TODO: investigate if this can be removed to make decoding completely
> + * independent. */
> FFVulkanDecodeContext *dec;
>
> - StdVideoAV1MESATile tiles[MAX_TILES];
> - StdVideoAV1MESATileList tile_list;
> - const uint32_t *tile_offsets;
> + uint32_t tile_count;
> + uint32_t tile_sizes[MAX_TILES];
> + const uint32_t *tile_offsets;
>
> /* Current picture */
> - VkVideoDecodeAV1DpbSlotInfoMESA vkav1_ref;
> - StdVideoAV1MESAFrameHeader av1_frame_header;
> - VkVideoDecodeAV1PictureInfoMESA av1_pic_info;
> + StdVideoDecodeAV1ReferenceInfo std_ref;
> + VkVideoDecodeAV1DpbSlotInfoKHR vkav1_ref;
> + uint16_t width_in_sbs_minus1[64];
> + uint16_t height_in_sbs_minus1[64];
> + uint16_t mi_col_starts[64];
> + uint16_t mi_row_starts[64];
> + StdVideoAV1TileInfo tile_info;
> + StdVideoAV1Quantization quantization;
> + StdVideoAV1Segmentation segmentation;
> + StdVideoAV1LoopFilter loop_filter;
> + StdVideoAV1CDEF cdef;
> + StdVideoAV1LoopRestoration loop_restoration;
> + StdVideoAV1GlobalMotion global_motion;
> + StdVideoAV1FilmGrain film_grain;
> + StdVideoDecodeAV1PictureInfo std_pic_info;
> + VkVideoDecodeAV1PictureInfoKHR av1_pic_info;
>
> /* Picture refs */
> const AV1Frame *ref_src [AV1_NUM_REF_FRAMES];
> - VkVideoDecodeAV1DpbSlotInfoMESA vkav1_refs[AV1_NUM_REF_FRAMES];
> + StdVideoDecodeAV1ReferenceInfo std_ref_info[AV1_NUM_REF_FRAMES];
> + VkVideoDecodeAV1DpbSlotInfoKHR vkav1_refs[AV1_NUM_REF_FRAMES];
>
> uint8_t frame_id_set;
> uint8_t frame_id;
> @@ -55,44 +69,60 @@ typedef struct AV1VulkanDecodePicture {
> static int vk_av1_fill_pict(AVCodecContext *avctx, const AV1Frame **ref_src,
> VkVideoReferenceSlotInfoKHR *ref_slot, /* Main structure */
> VkVideoPictureResourceInfoKHR *ref, /* Goes in ^ */
> - VkVideoDecodeAV1DpbSlotInfoMESA *vkav1_ref, /* Goes in ^ */
> - const AV1Frame *pic, int is_current, int has_grain,
> - int dpb_slot_index)
> + StdVideoDecodeAV1ReferenceInfo *vkav1_std_ref,
> + VkVideoDecodeAV1DpbSlotInfoKHR *vkav1_ref, /* Goes in ^ */
> + const AV1Frame *pic, int is_current, int has_grain)
> {
> FFVulkanDecodeContext *dec = avctx->internal->hwaccel_priv_data;
> AV1VulkanDecodePicture *hp = pic->hwaccel_picture_private;
> FFVulkanDecodePicture *vkpic = &hp->vp;
> + AV1DecContext *s = avctx->priv_data;
> + CodedBitstreamAV1Context *cbs_ctx;
>
> int err = ff_vk_decode_prepare_frame(dec, pic->f, vkpic, is_current,
> has_grain || dec->dedicated_dpb);
> if (err < 0)
> return err;
>
> - *vkav1_ref = (VkVideoDecodeAV1DpbSlotInfoMESA) {
> - .sType = VK_STRUCTURE_TYPE_VIDEO_DECODE_AV1_DPB_SLOT_INFO_MESA,
> - .frameIdx = hp->frame_id,
> + cbs_ctx = (CodedBitstreamAV1Context *)(s->cbc->priv_data);
> +
> + *vkav1_std_ref = (StdVideoDecodeAV1ReferenceInfo) {
> + .flags = (StdVideoDecodeAV1ReferenceInfoFlags) {
> + .disable_frame_end_update_cdf = pic->raw_frame_header->disable_frame_end_update_cdf,
> + .segmentation_enabled = pic->raw_frame_header->segmentation_enabled,
> + },
> + .frame_type = pic->raw_frame_header->frame_type,
> + .OrderHint = pic->raw_frame_header->order_hint,
> + .RefFrameSignBias = 0x0,
> };
>
> - for (unsigned i = 0; i < 7; i++) {
> - const int idx = pic->raw_frame_header->ref_frame_idx[i];
> - vkav1_ref->ref_order_hint[i] = pic->raw_frame_header->ref_order_hint[idx];
> + for (int i = 0; i < AV1_TOTAL_REFS_PER_FRAME; i++) {
> + vkav1_std_ref->SavedOrderHints[i] = cbs_ctx->order_hints[i];
> + vkav1_std_ref->RefFrameSignBias |= cbs_ctx->ref_frame_sign_bias[i] << i;
> }
>
> - vkav1_ref->disable_frame_end_update_cdf = pic->raw_frame_header->disable_frame_end_update_cdf;
> + *vkav1_ref = (VkVideoDecodeAV1DpbSlotInfoKHR) {
> + .sType = VK_STRUCTURE_TYPE_VIDEO_DECODE_AV1_DPB_SLOT_INFO_KHR,
> + .pStdReferenceInfo = vkav1_std_ref,
> + };
> +
> + vkav1_std_ref->flags.disable_frame_end_update_cdf = pic->raw_frame_header->disable_frame_end_update_cdf;
> + vkav1_std_ref->flags.segmentation_enabled = pic->raw_frame_header->segmentation_enabled;
> + vkav1_std_ref->frame_type = pic->raw_frame_header->frame_type;
>
> *ref = (VkVideoPictureResourceInfoKHR) {
> .sType = VK_STRUCTURE_TYPE_VIDEO_PICTURE_RESOURCE_INFO_KHR,
> .codedOffset = (VkOffset2D){ 0, 0 },
> .codedExtent = (VkExtent2D){ pic->f->width, pic->f->height },
> .baseArrayLayer = ((has_grain || dec->dedicated_dpb) && dec->layered_dpb) ?
> - dpb_slot_index : 0,
> + hp->frame_id : 0,
> .imageViewBinding = vkpic->img_view_ref,
> };
>
> *ref_slot = (VkVideoReferenceSlotInfoKHR) {
> .sType = VK_STRUCTURE_TYPE_VIDEO_REFERENCE_SLOT_INFO_KHR,
> .pNext = vkav1_ref,
> - .slotIndex = dpb_slot_index,
> + .slotIndex = hp->frame_id,
> .pPictureResource = ref,
> };
>
> @@ -110,15 +140,37 @@ static int vk_av1_create_params(AVCodecContext *avctx, AVBufferRef **buf)
>
> const AV1RawSequenceHeader *seq = s->raw_seq;
>
> - StdVideoAV1MESASequenceHeader av1_sequence_header;
> - VkVideoDecodeAV1SessionParametersAddInfoMESA av1_params_info;
> - VkVideoDecodeAV1SessionParametersCreateInfoMESA av1_params;
> + StdVideoAV1SequenceHeader av1_sequence_header;
> + StdVideoAV1TimingInfo av1_timing_info;
> + StdVideoAV1ColorConfig av1_color_config;
> + VkVideoDecodeAV1SessionParametersCreateInfoKHR av1_params;
> VkVideoSessionParametersCreateInfoKHR session_params_create;
>
> int err;
>
> - av1_sequence_header = (StdVideoAV1MESASequenceHeader) {
> - .flags = (StdVideoAV1MESASequenceHeaderFlags) {
> + av1_timing_info = (StdVideoAV1TimingInfo) {
> + .flags = (StdVideoAV1TimingInfoFlags) {
> + .equal_picture_interval = seq->timing_info.equal_picture_interval,
> + },
> + .num_units_in_display_tick = seq->timing_info.num_units_in_display_tick,
> + .time_scale = seq->timing_info.time_scale,
> + .num_ticks_per_picture_minus_1 = seq->timing_info.num_ticks_per_picture_minus_1,
> + };
> +
> + av1_color_config = (StdVideoAV1ColorConfig) {
> + .flags = (StdVideoAV1ColorConfigFlags) {
> + .mono_chrome = seq->color_config.mono_chrome,
> + .color_range = seq->color_config.color_range,
> + .separate_uv_delta_q = seq->color_config.separate_uv_delta_q,
> + },
> + .BitDepth = seq->color_config.twelve_bit ? 12 :
> + seq->color_config.high_bitdepth ? 10 : 8,
> + .subsampling_x = seq->color_config.subsampling_x,
> + .subsampling_y = seq->color_config.subsampling_y,
> + };
> +
> + av1_sequence_header = (StdVideoAV1SequenceHeader) {
> + .flags = (StdVideoAV1SequenceHeaderFlags) {
> .still_picture = seq->still_picture,
> .reduced_still_picture_header = seq->reduced_still_picture_header,
> .use_128x128_superblock = seq->use_128x128_superblock,
> @@ -147,34 +199,13 @@ static int vk_av1_create_params(AVCodecContext *avctx, AVBufferRef **buf)
> .delta_frame_id_length_minus_2 = seq->delta_frame_id_length_minus_2,
> .additional_frame_id_length_minus_1 = seq->additional_frame_id_length_minus_1,
> .order_hint_bits_minus_1 = seq->order_hint_bits_minus_1,
> - .timing_info = (StdVideoAV1MESATimingInfo) {
> - .flags = (StdVideoAV1MESATimingInfoFlags) {
> - .equal_picture_interval = seq->timing_info.equal_picture_interval,
> - },
> - .num_units_in_display_tick = seq->timing_info.num_units_in_display_tick,
> - .time_scale = seq->timing_info.time_scale,
> - .num_ticks_per_picture_minus_1 = seq->timing_info.num_ticks_per_picture_minus_1,
> - },
> - .color_config = (StdVideoAV1MESAColorConfig) {
> - .flags = (StdVideoAV1MESAColorConfigFlags) {
> - .mono_chrome = seq->color_config.mono_chrome,
> - .color_range = seq->color_config.color_range,
> - .separate_uv_delta_q = seq->color_config.separate_uv_delta_q,
> - },
> - .bit_depth = seq->color_config.twelve_bit ? 12 :
> - seq->color_config.high_bitdepth ? 10 : 8,
> - .subsampling_x = seq->color_config.subsampling_x,
> - .subsampling_y = seq->color_config.subsampling_y,
> - },
> + .pTimingInfo = &av1_timing_info,
> + .pColorConfig = &av1_color_config,
> };
>
> - av1_params_info = (VkVideoDecodeAV1SessionParametersAddInfoMESA) {
> - .sType = VK_STRUCTURE_TYPE_VIDEO_DECODE_AV1_SESSION_PARAMETERS_ADD_INFO_MESA,
> - .sequence_header = &av1_sequence_header,
> - };
> - av1_params = (VkVideoDecodeAV1SessionParametersCreateInfoMESA) {
> - .sType = VK_STRUCTURE_TYPE_VIDEO_DECODE_AV1_SESSION_PARAMETERS_CREATE_INFO_MESA,
> - .pParametersAddInfo = &av1_params_info,
> + av1_params = (VkVideoDecodeAV1SessionParametersCreateInfoKHR) {
> + .sType = VK_STRUCTURE_TYPE_VIDEO_DECODE_AV1_SESSION_PARAMETERS_CREATE_INFO_KHR,
> + .pStdSequenceHeader = &av1_sequence_header,
> };
> session_params_create = (VkVideoSessionParametersCreateInfoKHR) {
> .sType = VK_STRUCTURE_TYPE_VIDEO_SESSION_PARAMETERS_CREATE_INFO_KHR,
> @@ -208,6 +239,10 @@ static int vk_av1_start_frame(AVCodecContext *avctx,
> const AV1RawFilmGrainParams *film_grain = &s->cur_frame.film_grain;
> const int apply_grain = !(avctx->export_side_data & AV_CODEC_EXPORT_DATA_FILM_GRAIN) &&
> film_grain->apply_grain;
> + StdVideoAV1FrameRestorationType remap_lr_type[4] = { STD_VIDEO_AV1_FRAME_RESTORATION_TYPE_NONE,
> + STD_VIDEO_AV1_FRAME_RESTORATION_TYPE_SWITCHABLE,
> + STD_VIDEO_AV1_FRAME_RESTORATION_TYPE_WIENER,
> + STD_VIDEO_AV1_FRAME_RESTORATION_TYPE_SGRPROJ };
>
> if (!dec->session_params) {
> err = vk_av1_create_params(avctx, &dec->session_params);
> @@ -235,8 +270,8 @@ static int vk_av1_start_frame(AVCodecContext *avctx,
> continue;
>
> err = vk_av1_fill_pict(avctx, &ap->ref_src[i], &vp->ref_slots[i],
> - &vp->refs[i], &ap->vkav1_refs[i],
> - ref_frame, 0, 0, i);
> + &vp->refs[i], &ap->std_ref_info[i], &ap->vkav1_refs[i],
> + ref_frame, 0, 0);
> if (err < 0)
> return err;
>
> @@ -244,20 +279,25 @@ static int vk_av1_start_frame(AVCodecContext *avctx,
> }
>
> err = vk_av1_fill_pict(avctx, NULL, &vp->ref_slot, &vp->ref,
> + &ap->std_ref,
> &ap->vkav1_ref,
> - pic, 1, apply_grain, 8);
> + pic, 1, apply_grain);
> if (err < 0)
> return err;
>
> - ap->tile_list.nb_tiles = 0;
> - ap->tile_list.tile_list = ap->tiles;
> + ap->tile_count = 0;
>
> - ap->av1_pic_info = (VkVideoDecodeAV1PictureInfoMESA) {
> - .sType = VK_STRUCTURE_TYPE_VIDEO_DECODE_AV1_PICTURE_INFO_MESA,
> - .frame_header = &ap->av1_frame_header,
> - .tile_list = &ap->tile_list,
> + ap->av1_pic_info = (VkVideoDecodeAV1PictureInfoKHR) {
> + .sType = VK_STRUCTURE_TYPE_VIDEO_DECODE_AV1_PICTURE_INFO_KHR,
> + .pStdPictureInfo = &ap->std_pic_info,
> + .tileCount = ap->tile_count,
> + .pTileOffsets = ap->tile_offsets,
Mentioned this on IRC earlier, but this means the driver always get
pTileOffsets = NULL, since ap->tile_offsets only gets filled out during
vk_av1_decode_slice().
> + .pTileSizes = ap->tile_sizes,
> };
>
> + for (int i = 0; i < STD_VIDEO_AV1_REFS_PER_FRAME; i++)
> + ap->av1_pic_info.referenceNameSlotIndices[i] = vp->ref_slots[frame_header->ref_frame_idx[i]].slotIndex;
> +
> vp->decode_info = (VkVideoDecodeInfoKHR) {
> .sType = VK_STRUCTURE_TYPE_VIDEO_DECODE_INFO_KHR,
> .pNext = &ap->av1_pic_info,
> @@ -274,9 +314,94 @@ static int vk_av1_start_frame(AVCodecContext *avctx,
> },
> };
>
> + ap->tile_info = (StdVideoAV1TileInfo) {
> + .flags = (StdVideoAV1TileInfoFlags) {
> + .uniform_tile_spacing_flag = frame_header->uniform_tile_spacing_flag,
> + },
> + .TileCols = frame_header->tile_cols,
> + .TileRows = frame_header->tile_rows,
> + .context_update_tile_id = frame_header->context_update_tile_id,
> + .tile_size_bytes_minus_1 = frame_header->tile_size_bytes_minus1,
> + .pWidthInSbsMinus1 = ap->width_in_sbs_minus1,
> + .pHeightInSbsMinus1 = ap->height_in_sbs_minus1,
> + .pMiColStarts = ap->mi_col_starts,
> + .pMiRowStarts = ap->mi_row_starts,
> + };
> +
> + ap->quantization = (StdVideoAV1Quantization) {
> + .flags.using_qmatrix = frame_header->using_qmatrix,
> + .flags.diff_uv_delta = frame_header->diff_uv_delta,
> + .base_q_idx = frame_header->base_q_idx,
> + .DeltaQYDc = frame_header->delta_q_y_dc,
> + .DeltaQUDc = frame_header->delta_q_u_dc,
> + .DeltaQUAc = frame_header->delta_q_u_ac,
> + .DeltaQVDc = frame_header->delta_q_v_dc,
> + .DeltaQVAc = frame_header->delta_q_v_ac,
> + .qm_y = frame_header->qm_y,
> + .qm_u = frame_header->qm_u,
> + .qm_v = frame_header->qm_v,
> + };
> +
> + for (int i = 0; i < STD_VIDEO_AV1_MAX_SEGMENTS; i++) {
> + for (int j = 0; j < STD_VIDEO_AV1_SEG_LVL_MAX; j++) {
> + ap->segmentation.FeatureEnabled[i] |= frame_header->feature_enabled[i][j] << j;
> + ap->segmentation.FeatureData[i][j] = frame_header->feature_value[i][j];
> + }
> + }
> +
> + ap->loop_filter = (StdVideoAV1LoopFilter) {
> + .flags = (StdVideoAV1LoopFilterFlags) {
> + .loop_filter_delta_enabled = frame_header->loop_filter_delta_enabled,
> + .loop_filter_delta_update = frame_header->loop_filter_delta_update,
> + },
> + .loop_filter_sharpness = frame_header->loop_filter_sharpness,
> + };
> +
> + for (int i = 0; i < STD_VIDEO_AV1_MAX_LOOP_FILTER_STRENGTHS; i++)
> + ap->loop_filter.loop_filter_level[i] = frame_header->loop_filter_level[i];
> + for (int i = 0; i < STD_VIDEO_AV1_LOOP_FILTER_ADJUSTMENTS; i++)
> + ap->loop_filter.loop_filter_mode_deltas[i] = frame_header->loop_filter_mode_deltas[i];
> +
> + ap->cdef = (StdVideoAV1CDEF) {
> + .cdef_damping_minus_3 = frame_header->cdef_damping_minus_3,
> + .cdef_bits = frame_header->cdef_bits,
> + };
> +
> + ap->loop_restoration = (StdVideoAV1LoopRestoration) {
> + .FrameRestorationType[0] = remap_lr_type[frame_header->lr_type[0]],
> + .FrameRestorationType[1] = remap_lr_type[frame_header->lr_type[1]],
> + .FrameRestorationType[2] = remap_lr_type[frame_header->lr_type[2]],
> + .LoopRestorationSize[0] = AV1_RESTORATION_TILESIZE_MAX >> (2 - frame_header->lr_unit_shift),
> + .LoopRestorationSize[1] = ap->loop_restoration.LoopRestorationSize[0] >> frame_header->lr_uv_shift,
> + .LoopRestorationSize[2] = ap->loop_restoration.LoopRestorationSize[0] >> frame_header->lr_uv_shift,
> + };
Not sure what the C spec says, but at least on GCC 13.2.1 it seems like
the assignment for LoopRestorationSize[1] and [2] happens before the
initalization of LoopRestorationSize[0].
> +
> + ap->film_grain = (StdVideoAV1FilmGrain) {
> + .flags = (StdVideoAV1FilmGrainFlags) {
> + .chroma_scaling_from_luma = film_grain->chroma_scaling_from_luma,
> + .overlap_flag = film_grain->overlap_flag,
> + .clip_to_restricted_range = film_grain->clip_to_restricted_range,
> + },
> + .grain_scaling_minus_8 = film_grain->grain_scaling_minus_8,
> + .ar_coeff_lag = film_grain->ar_coeff_lag,
> + .ar_coeff_shift_minus_6 = film_grain->ar_coeff_shift_minus_6,
> + .grain_scale_shift = film_grain->grain_scale_shift,
> + .grain_seed = film_grain->grain_seed,
> + .film_grain_params_ref_idx = film_grain->film_grain_params_ref_idx,
> + .num_y_points = film_grain->num_y_points,
> + .num_cb_points = film_grain->num_cb_points,
> + .num_cr_points = film_grain->num_cr_points,
> + .cb_mult = film_grain->cb_mult,
> + .cb_luma_mult = film_grain->cb_luma_mult,
> + .cb_offset = film_grain->cb_offset,
> + .cr_mult = film_grain->cr_mult,
> + .cr_luma_mult = film_grain->cr_luma_mult,
> + .cr_offset = film_grain->cr_offset,
> + };
> +
> /* Setup frame header */
> - ap->av1_frame_header = (StdVideoAV1MESAFrameHeader) {
> - .flags = (StdVideoAV1MESAFrameHeaderFlags) {
> + ap->std_pic_info = (StdVideoDecodeAV1PictureInfo) {
> + .flags = (StdVideoDecodeAV1PictureInfoFlags) {
> .error_resilient_mode = frame_header->error_resilient_mode,
> .disable_cdf_update = frame_header->disable_cdf_update,
> .use_superres = frame_header->use_superres,
> @@ -297,174 +422,88 @@ static int vk_av1_start_frame(AVCodecContext *avctx,
> .reference_select = frame_header->reference_select,
> .skip_mode_present = frame_header->skip_mode_present,
> .delta_q_present = frame_header->delta_q_present,
> + .delta_lf_present = frame_header->delta_lf_present,
> + .delta_lf_multi = frame_header->delta_lf_multi,
> + .segmentation_enabled = frame_header->segmentation_enabled,
> + .segmentation_update_map = frame_header->segmentation_update_map,
> + .segmentation_temporal_update = frame_header->segmentation_temporal_update,
> + .segmentation_update_data = frame_header->segmentation_update_data,
> + .UsesLr = frame_header->lr_type[0] || frame_header->lr_type[1] || frame_header->lr_type[2],
> + .apply_grain = apply_grain,
> },
> - .frame_to_show_map_idx = frame_header->frame_to_show_map_idx,
> - .frame_presentation_time = frame_header->frame_presentation_time,
> - .display_frame_id = frame_header->display_frame_id,
> .frame_type = frame_header->frame_type,
> .current_frame_id = frame_header->current_frame_id,
> - .order_hint = frame_header->order_hint,
> + .OrderHint = frame_header->order_hint,
> .primary_ref_frame = frame_header->primary_ref_frame,
> - .frame_width_minus_1 = frame_header->frame_width_minus_1,
> - .frame_height_minus_1 = frame_header->frame_height_minus_1,
> - .coded_denom = frame_header->coded_denom,
> - .render_width_minus_1 = frame_header->render_width_minus_1,
> - .render_height_minus_1 = frame_header->render_height_minus_1,
> .refresh_frame_flags = frame_header->refresh_frame_flags,
> .interpolation_filter = frame_header->interpolation_filter,
> - .tx_mode = frame_header->tx_mode,
> - .tiling = (StdVideoAV1MESATileInfo) {
> - .flags = (StdVideoAV1MESATileInfoFlags) {
> - .uniform_tile_spacing_flag = frame_header->uniform_tile_spacing_flag,
> - },
> - .tile_cols = frame_header->tile_cols,
> - .tile_rows = frame_header->tile_rows,
> - .context_update_tile_id = frame_header->context_update_tile_id,
> - .tile_size_bytes_minus1 = frame_header->tile_size_bytes_minus1,
> - },
> - .quantization = (StdVideoAV1MESAQuantization) {
> - .flags.using_qmatrix = frame_header->using_qmatrix,
> - .base_q_idx = frame_header->base_q_idx,
> - .delta_q_y_dc = frame_header->delta_q_y_dc,
> - .diff_uv_delta = frame_header->diff_uv_delta,
> - .delta_q_u_dc = frame_header->delta_q_u_dc,
> - .delta_q_u_ac = frame_header->delta_q_u_ac,
> - .delta_q_v_dc = frame_header->delta_q_v_dc,
> - .delta_q_v_ac = frame_header->delta_q_v_ac,
> - .qm_y = frame_header->qm_y,
> - .qm_u = frame_header->qm_u,
> - .qm_v = frame_header->qm_v,
> - },
> - .delta_q = (StdVideoAV1MESADeltaQ) {
> - .flags = (StdVideoAV1MESADeltaQFlags) {
> - .delta_lf_present = frame_header->delta_lf_present,
> - .delta_lf_multi = frame_header->delta_lf_multi,
> - },
> - .delta_q_res = frame_header->delta_q_res,
> - .delta_lf_res = frame_header->delta_lf_res,
> - },
> - .loop_filter = (StdVideoAV1MESALoopFilter) {
> - .flags = (StdVideoAV1MESALoopFilterFlags) {
> - .delta_enabled = frame_header->loop_filter_delta_enabled,
> - .delta_update = frame_header->loop_filter_delta_update,
> - },
> - .level = {
> - frame_header->loop_filter_level[0], frame_header->loop_filter_level[1],
> - frame_header->loop_filter_level[2], frame_header->loop_filter_level[3],
> - },
> - .sharpness = frame_header->loop_filter_sharpness,
> - .mode_deltas = {
> - frame_header->loop_filter_mode_deltas[0], frame_header->loop_filter_mode_deltas[1],
> - },
> - },
> - .cdef = (StdVideoAV1MESACDEF) {
> - .damping_minus_3 = frame_header->cdef_damping_minus_3,
> - .bits = frame_header->cdef_bits,
> - },
> - .lr = (StdVideoAV1MESALoopRestoration) {
> - .lr_unit_shift = frame_header->lr_unit_shift,
> - .lr_uv_shift = frame_header->lr_uv_shift,
> - .lr_type = { frame_header->lr_type[0], frame_header->lr_type[1], frame_header->lr_type[2] },
> - },
> - .segmentation = (StdVideoAV1MESASegmentation) {
> - .flags = (StdVideoAV1MESASegmentationFlags) {
> - .enabled = frame_header->segmentation_enabled,
> - .update_map = frame_header->segmentation_update_map,
> - .temporal_update = frame_header->segmentation_temporal_update,
> - .update_data = frame_header->segmentation_update_data,
> - },
> - },
> - .film_grain = (StdVideoAV1MESAFilmGrainParameters) {
> - .flags = (StdVideoAV1MESAFilmGrainFlags) {
> - .apply_grain = apply_grain,
> - .chroma_scaling_from_luma = film_grain->chroma_scaling_from_luma,
> - .overlap_flag = film_grain->overlap_flag,
> - .clip_to_restricted_range = film_grain->clip_to_restricted_range,
> - },
> - .grain_scaling_minus_8 = film_grain->grain_scaling_minus_8,
> - .ar_coeff_lag = film_grain->ar_coeff_lag,
> - .ar_coeff_shift_minus_6 = film_grain->ar_coeff_shift_minus_6,
> - .grain_scale_shift = film_grain->grain_scale_shift,
> - .grain_seed = film_grain->grain_seed,
> - .num_y_points = film_grain->num_y_points,
> - .num_cb_points = film_grain->num_cb_points,
> - .num_cr_points = film_grain->num_cr_points,
> - .cb_mult = film_grain->cb_mult,
> - .cb_luma_mult = film_grain->cb_luma_mult,
> - .cb_offset = film_grain->cb_offset,
> - .cr_mult = film_grain->cr_mult,
> - .cr_luma_mult = film_grain->cr_luma_mult,
> - .cr_offset = film_grain->cr_offset,
> - },
> + .TxMode = frame_header->tx_mode,
> + .delta_q_res = frame_header->delta_q_res,
> + .delta_lf_res = frame_header->delta_lf_res,
> + .SkipModeFrame[0] = s->cur_frame.skip_mode_frame_idx[0],
> + .SkipModeFrame[1] = s->cur_frame.skip_mode_frame_idx[1],
> + .coded_denom = frame_header->coded_denom,
> + .pTileInfo = &ap->tile_info,
> + .pQuantization = &ap->quantization,
> + .pSegmentation = &ap->segmentation,
> + .pLoopFilter = &ap->loop_filter,
> + .pCDEF = &ap->cdef,
> + .pLoopRestoration = &ap->loop_restoration,
> + .pGlobalMotion = &ap->global_motion,
> + .pFilmGrain = apply_grain ? &ap->film_grain : NULL,
> };
>
> for (int i = 0; i < 64; i++) {
> - ap->av1_frame_header.tiling.width_in_sbs_minus_1[i] = frame_header->width_in_sbs_minus_1[i];
> - ap->av1_frame_header.tiling.height_in_sbs_minus_1[i] = frame_header->height_in_sbs_minus_1[i];
> - ap->av1_frame_header.tiling.tile_start_col_sb[i] = frame_header->tile_start_col_sb[i];
> - ap->av1_frame_header.tiling.tile_start_row_sb[i] = frame_header->tile_start_row_sb[i];
> + ap->width_in_sbs_minus1[i] = frame_header->width_in_sbs_minus_1[i];
> + ap->height_in_sbs_minus1[i] = frame_header->height_in_sbs_minus_1[i];
> + ap->mi_col_starts[i] = frame_header->tile_start_col_sb[i];
> + ap->mi_row_starts[i] = frame_header->tile_start_row_sb[i];
> }
>
> for (int i = 0; i < 8; i++) {
> - ap->av1_frame_header.segmentation.feature_enabled_bits[i] = 0;
> - for (int j = 0; j < 8; j++) {
> - ap->av1_frame_header.segmentation.feature_enabled_bits[i] |= (frame_header->feature_enabled[i][j] << j);
> - ap->av1_frame_header.segmentation.feature_data[i][j] = frame_header->feature_value[i][j];
> + ap->segmentation.FeatureEnabled[i] = 0x0;
> + for (int j = 0; j < STD_VIDEO_AV1_SEG_LVL_MAX; j++) {
> + ap->segmentation.FeatureEnabled[i] |= (frame_header->feature_enabled[i][j] << j);
> + ap->segmentation.FeatureData[i][j] = frame_header->feature_value[i][j];
> }
>
> - ap->av1_frame_header.loop_filter.ref_deltas[i] = frame_header->loop_filter_ref_deltas[i];
> -
> - ap->av1_frame_header.cdef.y_pri_strength[i] = frame_header->cdef_y_pri_strength[i];
> - ap->av1_frame_header.cdef.y_sec_strength[i] = frame_header->cdef_y_sec_strength[i];
> - ap->av1_frame_header.cdef.uv_pri_strength[i] = frame_header->cdef_uv_pri_strength[i];
> - ap->av1_frame_header.cdef.uv_sec_strength[i] = frame_header->cdef_uv_sec_strength[i];
> -
> - ap->av1_frame_header.ref_order_hint[i] = frame_header->ref_order_hint[i];
> - ap->av1_frame_header.global_motion[i] = (StdVideoAV1MESAGlobalMotion) {
> - .flags = (StdVideoAV1MESAGlobalMotionFlags) {
> - .gm_invalid = s->cur_frame.gm_invalid[i],
> - },
> - .gm_type = s->cur_frame.gm_type[i],
> - .gm_params = {
> - s->cur_frame.gm_params[i][0], s->cur_frame.gm_params[i][1],
> - s->cur_frame.gm_params[i][2], s->cur_frame.gm_params[i][3],
> - s->cur_frame.gm_params[i][4], s->cur_frame.gm_params[i][5],
> - },
> - };
> - }
> + ap->loop_filter.loop_filter_ref_deltas[i] = frame_header->loop_filter_ref_deltas[i];
>
> - for (int i = 0; i < 7; i++) {
> - ap->av1_frame_header.ref_frame_idx[i] = frame_header->ref_frame_idx[i];
> - ap->av1_frame_header.delta_frame_id_minus1[i] = frame_header->delta_frame_id_minus1[i];
> - }
> + ap->cdef.cdef_y_pri_strength[i] = frame_header->cdef_y_pri_strength[i];
> + ap->cdef.cdef_y_sec_strength[i] = frame_header->cdef_y_sec_strength[i];
> + ap->cdef.cdef_uv_pri_strength[i] = frame_header->cdef_uv_pri_strength[i];
> + ap->cdef.cdef_uv_sec_strength[i] = frame_header->cdef_uv_sec_strength[i];
>
> - ap->av1_pic_info.skip_mode_frame_idx[0] = s->cur_frame.skip_mode_frame_idx[0];
> - ap->av1_pic_info.skip_mode_frame_idx[1] = s->cur_frame.skip_mode_frame_idx[1];
> + /* Reference frames */
> + ap->std_pic_info.OrderHints[i] = frame_header->ref_order_hint[i];
> + ap->global_motion.GmType[i] = s->cur_frame.gm_type[i];
> + for (int j = 0; j < STD_VIDEO_AV1_GLOBAL_MOTION_PARAMS; j++) {
> + ap->global_motion.gm_params[i][j] = s->cur_frame.gm_params[i][j];
> + }
> + }
>
> if (apply_grain) {
> - for (int i = 0; i < 14; i++) {
> - ap->av1_frame_header.film_grain.point_y_value[i] = film_grain->point_y_value[i];
> - ap->av1_frame_header.film_grain.point_y_scaling[i] = film_grain->point_y_scaling[i];
> + for (int i = 0; i < STD_VIDEO_AV1_MAX_NUM_Y_POINTS; i++) {
> + ap->film_grain.point_y_value[i] = film_grain->point_y_value[i];
> + ap->film_grain.point_y_scaling[i] = film_grain->point_y_scaling[i];
> }
>
> - for (int i = 0; i < 10; i++) {
> - ap->av1_frame_header.film_grain.point_cb_value[i] = film_grain->point_cb_value[i];
> - ap->av1_frame_header.film_grain.point_cb_scaling[i] = film_grain->point_cb_scaling[i];
> - ap->av1_frame_header.film_grain.point_cr_value[i] = film_grain->point_cr_value[i];
> - ap->av1_frame_header.film_grain.point_cr_scaling[i] = film_grain->point_cr_scaling[i];
> + for (int i = 0; i < STD_VIDEO_AV1_MAX_NUM_CB_POINTS; i++) {
> + ap->film_grain.point_cb_value[i] = film_grain->point_cb_value[i];
> + ap->film_grain.point_cb_scaling[i] = film_grain->point_cb_scaling[i];
> + ap->film_grain.point_cr_value[i] = film_grain->point_cr_value[i];
> + ap->film_grain.point_cr_scaling[i] = film_grain->point_cr_scaling[i];
> }
>
> - for (int i = 0; i < 24; i++) {
> - ap->av1_frame_header.film_grain.ar_coeffs_y_plus_128[i] = film_grain->ar_coeffs_y_plus_128[i];
> - ap->av1_frame_header.film_grain.ar_coeffs_cb_plus_128[i] = film_grain->ar_coeffs_cb_plus_128[i];
> - ap->av1_frame_header.film_grain.ar_coeffs_cr_plus_128[i] = film_grain->ar_coeffs_cr_plus_128[i];
> - }
> + for (int i = 0; i < STD_VIDEO_AV1_MAX_NUM_POS_LUMA; i++)
> + ap->film_grain.ar_coeffs_y_plus_128[i] = film_grain->ar_coeffs_y_plus_128[i];
>
> - ap->av1_frame_header.film_grain.ar_coeffs_cb_plus_128[24] = film_grain->ar_coeffs_cb_plus_128[24];
> - ap->av1_frame_header.film_grain.ar_coeffs_cr_plus_128[24] = film_grain->ar_coeffs_cr_plus_128[24];
> + for (int i = 0; i < STD_VIDEO_AV1_MAX_NUM_POS_CHROMA; i++) {
> + ap->film_grain.ar_coeffs_cb_plus_128[i] = film_grain->ar_coeffs_cb_plus_128[i];
> + ap->film_grain.ar_coeffs_cr_plus_128[i] = film_grain->ar_coeffs_cr_plus_128[i];
> + }
> }
>
> - /* Workaround for a spec issue. */
> ap->dec = dec;
>
> return 0;
> @@ -479,27 +518,23 @@ static int vk_av1_decode_slice(AVCodecContext *avctx,
> AV1VulkanDecodePicture *ap = s->cur_frame.hwaccel_picture_private;
> FFVulkanDecodePicture *vp = &ap->vp;
>
> + /* Too many tiles, exceeding all defined levels in the AV1 spec */
> + if (ap->tile_count > MAX_TILES)
> + return AVERROR(ENOSYS);
> +
> for (int i = s->tg_start; i <= s->tg_end; i++) {
> - ap->tiles[ap->tile_list.nb_tiles] = (StdVideoAV1MESATile) {
> - .size = s->tile_group_info[i].tile_size,
> - .offset = s->tile_group_info[i].tile_offset,
> - .row = s->tile_group_info[i].tile_row,
> - .column = s->tile_group_info[i].tile_column,
> - .tg_start = s->tg_start,
> - .tg_end = s->tg_end,
> - };
> + ap->tile_sizes[ap->tile_count] = s->tile_group_info[i].tile_size;
>
> err = ff_vk_decode_add_slice(avctx, vp,
> data + s->tile_group_info[i].tile_offset,
> s->tile_group_info[i].tile_size, 0,
> - &ap->tile_list.nb_tiles,
> + &ap->tile_count,
> &ap->tile_offsets);
> if (err < 0)
> return err;
> -
> - ap->tiles[ap->tile_list.nb_tiles - 1].offset = ap->tile_offsets[ap->tile_list.nb_tiles - 1];
> }
>
> + ap->av1_pic_info.tileCount = ap->tile_count;
> return 0;
> }
>
> @@ -513,7 +548,7 @@ static int vk_av1_end_frame(AVCodecContext *avctx)
> FFVulkanDecodePicture *rvp[AV1_NUM_REF_FRAMES] = { 0 };
> AVFrame *rav[AV1_NUM_REF_FRAMES] = { 0 };
>
> - if (!ap->tile_list.nb_tiles)
> + if (!ap->tile_count)
> return 0;
>
> if (!dec->session_params) {
> @@ -531,7 +566,7 @@ static int vk_av1_end_frame(AVCodecContext *avctx)
> }
>
> av_log(avctx, AV_LOG_VERBOSE, "Decoding frame, %"SIZE_SPECIFIER" bytes, %i tiles\n",
> - vp->slices_size, ap->tile_list.nb_tiles);
> + vp->slices_size, ap->tile_count);
>
> return ff_vk_decode_frame(avctx, pic->f, vp, rav, rvp);
> }
> @@ -575,8 +610,6 @@ const FFHWAccel ff_av1_vulkan_hwaccel = {
> * flexibility, this index cannot be present anywhere.
> * The current implementation tracks the index for the driver and submits it
> * as necessary information. Due to needing to modify the decoding context,
> - * which is not thread-safe, on frame free, threading is disabled.
> - * In the future, once this is fixed in the spec, the workarounds may be removed
> - * and threading enabled. */
> + * which is not thread-safe, on frame free, threading is disabled. */
> .caps_internal = HWACCEL_CAP_ASYNC_SAFE,
> };
> diff --git a/libavcodec/vulkan_decode.c b/libavcodec/vulkan_decode.c
> index fdbcbb450a..64ad60cecb 100644
> --- a/libavcodec/vulkan_decode.c
> +++ b/libavcodec/vulkan_decode.c
> @@ -50,7 +50,7 @@ static const VkVideoProfileInfoKHR *get_video_profile(FFVulkanDecodeShared *ctx,
> VkStructureType profile_struct_type =
> codec_id == AV_CODEC_ID_H264 ? VK_STRUCTURE_TYPE_VIDEO_DECODE_H264_PROFILE_INFO_KHR :
> codec_id == AV_CODEC_ID_HEVC ? VK_STRUCTURE_TYPE_VIDEO_DECODE_H265_PROFILE_INFO_KHR :
> - codec_id == AV_CODEC_ID_AV1 ? VK_STRUCTURE_TYPE_VIDEO_DECODE_AV1_PROFILE_INFO_MESA :
> + codec_id == AV_CODEC_ID_AV1 ? VK_STRUCTURE_TYPE_VIDEO_DECODE_AV1_PROFILE_INFO_KHR :
> 0;
>
> profile_list = ff_vk_find_struct(ctx->s.hwfc->create_pnext,
> @@ -663,7 +663,7 @@ static VkResult vulkan_setup_profile(AVCodecContext *avctx,
> const struct FFVkCodecMap *vk_codec,
> VkVideoDecodeH264CapabilitiesKHR *h264_caps,
> VkVideoDecodeH265CapabilitiesKHR *h265_caps,
> - VkVideoDecodeAV1CapabilitiesMESA *av1_caps,
> + VkVideoDecodeAV1CapabilitiesKHR *av1_caps,
> VkVideoCapabilitiesKHR *caps,
> VkVideoDecodeCapabilitiesKHR *dec_caps,
> int cur_profile)
> @@ -674,7 +674,7 @@ static VkResult vulkan_setup_profile(AVCodecContext *avctx,
>
> VkVideoDecodeH264ProfileInfoKHR *h264_profile = &prof->h264_profile;
> VkVideoDecodeH265ProfileInfoKHR *h265_profile = &prof->h265_profile;
> - VkVideoDecodeAV1ProfileInfoMESA *av1_profile = &prof->av1_profile;
> + VkVideoDecodeAV1ProfileInfoKHR *av1_profile = &prof->av1_profile;
>
> const AVPixFmtDescriptor *desc = av_pix_fmt_desc_get(avctx->sw_pix_fmt);
> if (!desc)
> @@ -702,8 +702,9 @@ static VkResult vulkan_setup_profile(AVCodecContext *avctx,
> } else if (avctx->codec_id == AV_CODEC_ID_AV1) {
> dec_caps->pNext = av1_caps;
> usage->pNext = av1_profile;
> - av1_profile->sType = VK_STRUCTURE_TYPE_VIDEO_DECODE_AV1_PROFILE_INFO_MESA;
> - av1_profile->stdProfileIdc = cur_profile;
> + av1_profile->sType = VK_STRUCTURE_TYPE_VIDEO_DECODE_AV1_PROFILE_INFO_KHR;
> + av1_profile->stdProfile = cur_profile;
> + av1_profile->filmGrainSupport = !(avctx->export_side_data & AV_CODEC_EXPORT_DATA_FILM_GRAIN);
> }
>
> usage->sType = VK_STRUCTURE_TYPE_VIDEO_DECODE_USAGE_INFO_KHR;
> @@ -758,8 +759,8 @@ static int vulkan_decode_get_profile(AVCodecContext *avctx, AVBufferRef *frames_
> VkVideoDecodeH265CapabilitiesKHR h265_caps = {
> .sType = VK_STRUCTURE_TYPE_VIDEO_DECODE_H265_CAPABILITIES_KHR,
> };
> - VkVideoDecodeAV1CapabilitiesMESA av1_caps = {
> - .sType = VK_STRUCTURE_TYPE_VIDEO_DECODE_AV1_CAPABILITIES_MESA,
> + VkVideoDecodeAV1CapabilitiesKHR av1_caps = {
> + .sType = VK_STRUCTURE_TYPE_VIDEO_DECODE_AV1_CAPABILITIES_KHR,
> };
>
> VkPhysicalDeviceVideoFormatInfoKHR fmt_info = {
> @@ -782,7 +783,7 @@ static int vulkan_decode_get_profile(AVCodecContext *avctx, AVBufferRef *frames_
> cur_profile = avctx->profile;
> base_profile = avctx->codec_id == AV_CODEC_ID_H264 ? AV_PROFILE_H264_CONSTRAINED_BASELINE :
> avctx->codec_id == AV_CODEC_ID_H265 ? AV_PROFILE_HEVC_MAIN :
> - avctx->codec_id == AV_CODEC_ID_AV1 ? STD_VIDEO_AV1_MESA_PROFILE_MAIN :
> + avctx->codec_id == AV_CODEC_ID_AV1 ? STD_VIDEO_AV1_PROFILE_MAIN :
> 0;
>
> ret = vulkan_setup_profile(avctx, prof, hwctx, vk, vk_codec,
> @@ -830,7 +831,7 @@ static int vulkan_decode_get_profile(AVCodecContext *avctx, AVBufferRef *frames_
>
> max_level = avctx->codec_id == AV_CODEC_ID_H264 ? ff_vk_h264_level_to_av(h264_caps.maxLevelIdc) :
> avctx->codec_id == AV_CODEC_ID_H265 ? ff_vk_h265_level_to_av(h265_caps.maxLevelIdc) :
> - avctx->codec_id == AV_CODEC_ID_AV1 ? av1_caps.maxLevelIdc :
> + avctx->codec_id == AV_CODEC_ID_AV1 ? av1_caps.maxLevel :
> 0;
>
> av_log(avctx, AV_LOG_VERBOSE, "Decoder capabilities for %s profile \"%s\":\n",
> @@ -911,7 +912,6 @@ static int vulkan_decode_get_profile(AVCodecContext *avctx, AVBufferRef *frames_
> /* TODO: make dedicated_dpb tunable */
> dec->dedicated_dpb = !(dec_caps->flags & VK_VIDEO_DECODE_CAPABILITY_DPB_AND_OUTPUT_COINCIDE_BIT_KHR);
> dec->layered_dpb = !(caps->flags & VK_VIDEO_CAPABILITY_SEPARATE_REFERENCE_IMAGES_BIT_KHR);
> - dec->external_fg = av1_caps.flags & VK_VIDEO_DECODE_AV1_CAPABILITY_EXTERNAL_FILM_GRAIN_MESA;
>
> if (dec->dedicated_dpb) {
> fmt_info.imageUsage = VK_IMAGE_USAGE_VIDEO_DECODE_DPB_BIT_KHR;
> @@ -1118,8 +1118,8 @@ int ff_vk_decode_init(AVCodecContext *avctx)
> VkVideoDecodeH265SessionParametersCreateInfoKHR h265_params = {
> .sType = VK_STRUCTURE_TYPE_VIDEO_DECODE_H265_SESSION_PARAMETERS_CREATE_INFO_KHR,
> };
> - VkVideoDecodeAV1SessionParametersCreateInfoMESA av1_params = {
> - .sType = VK_STRUCTURE_TYPE_VIDEO_DECODE_AV1_SESSION_PARAMETERS_CREATE_INFO_MESA,
> + VkVideoDecodeAV1SessionParametersCreateInfoKHR av1_params = {
> + .sType = VK_STRUCTURE_TYPE_VIDEO_DECODE_AV1_SESSION_PARAMETERS_CREATE_INFO_KHR,
> };
> VkVideoSessionParametersCreateInfoKHR session_params_create = {
> .sType = VK_STRUCTURE_TYPE_VIDEO_SESSION_PARAMETERS_CREATE_INFO_KHR,
> diff --git a/libavcodec/vulkan_decode.h b/libavcodec/vulkan_decode.h
> index b0b1ff5614..9991de6794 100644
> --- a/libavcodec/vulkan_decode.h
> +++ b/libavcodec/vulkan_decode.h
> @@ -28,7 +28,7 @@
> typedef struct FFVulkanDecodeProfileData {
> VkVideoDecodeH264ProfileInfoKHR h264_profile;
> VkVideoDecodeH265ProfileInfoKHR h265_profile;
> - VkVideoDecodeAV1ProfileInfoMESA av1_profile;
> + VkVideoDecodeAV1ProfileInfoKHR av1_profile;
> VkVideoDecodeUsageInfoKHR usage;
> VkVideoProfileInfoKHR profile;
> VkVideoProfileListInfoKHR profile_list;
> diff --git a/libavcodec/vulkan_video.c b/libavcodec/vulkan_video.c
> index fb20315db4..9622128b17 100644
> --- a/libavcodec/vulkan_video.c
> +++ b/libavcodec/vulkan_video.c
> @@ -37,7 +37,7 @@ const FFVkCodecMap ff_vk_codec_map[AV_CODEC_ID_FIRST_AUDIO] = {
> 0,
> 0,
> FF_VK_EXT_VIDEO_DECODE_AV1,
> - 0x01000000 /* TODO fix this */
> + VK_VIDEO_CODEC_OPERATION_DECODE_AV1_BIT_KHR
> },
> };
>
> diff --git a/libavcodec/vulkan_video.h b/libavcodec/vulkan_video.h
> index b28e3fe0bd..54cd75372d 100644
> --- a/libavcodec/vulkan_video.h
> +++ b/libavcodec/vulkan_video.h
> @@ -23,8 +23,6 @@
> #include "vulkan.h"
>
> #include <vk_video/vulkan_video_codecs_common.h>
> -#include "vulkan_video_codec_av1std.h"
> -#include "vulkan_video_codec_av1std_decode.h"
>
> #define CODEC_VER_MAJ(ver) (ver >> 22)
> #define CODEC_VER_MIN(ver) ((ver >> 12) & ((1 << 10) - 1))
> diff --git a/libavcodec/vulkan_video_codec_av1std.h b/libavcodec/vulkan_video_codec_av1std.h
> deleted file mode 100644
> index c46236c457..0000000000
> --- a/libavcodec/vulkan_video_codec_av1std.h
> +++ /dev/null
> @@ -1,403 +0,0 @@
> -/* Copyright 2023 Lynne
> - * Copyright 2023 Dave Airlie
> - *
> - * Licensed under the Apache License, Version 2.0 (the "License");
> - * you may not use this file except in compliance with the License.
> - * You may obtain a copy of the License at
> - *
> - * http://www.apache.org/licenses/LICENSE-2.0
> - *
> - * Unless required by applicable law or agreed to in writing, software
> - * distributed under the License is distributed on an "AS IS" BASIS,
> - * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
> - * See the License for the specific language governing permissions and
> - * limitations under the License.
> - */
> -
> -#ifndef VULKAN_VIDEO_CODEC_AV1STD_H_
> -#define VULKAN_VIDEO_CODEC_AV1STD_H_ 1
> -
> -/*
> -** This header is NOT YET generated from the Khronos Vulkan XML API Registry.
> -**
> -*/
> -
> -#ifdef __cplusplus
> -extern "C" {
> -#endif
> -#define vulkan_video_codec_av1std 1
> -
> -#define VK_MAKE_VIDEO_STD_VERSION(major, minor, patch) \
> - ((((uint32_t)(major)) << 22) | (((uint32_t)(minor)) << 12) | ((uint32_t)(patch)))
> -#define VK_STD_VULKAN_VIDEO_CODEC_AV1_DECODE_API_VERSION_0_1_0 VK_MAKE_VIDEO_STD_VERSION(0, 1, 0)
> -#define VK_STD_VULKAN_VIDEO_CODEC_AV1_DECODE_SPEC_VERSION VK_STD_VULKAN_VIDEO_CODEC_AV1_DECODE_API_VERSION_0_1_0
> -#define VK_STD_VULKAN_VIDEO_CODEC_AV1_DECODE_EXTENSION_NAME "VK_STD_vulkan_video_codec_av1_decode"
> -
> -typedef enum StdVideoAV1MESAProfile {
> - STD_VIDEO_AV1_MESA_PROFILE_MAIN = 0,
> - STD_VIDEO_AV1_MESA_PROFILE_HIGH = 1,
> - STD_VIDEO_AV1_MESA_PROFILE_PROFESSIONAL = 2,
> -} StdVideoAV1MESAProfile;
> -
> -typedef enum StdVideoAV1MESALevel {
> - STD_VIDEO_AV1_MESA_LEVEL_2_0 = 0,
> - STD_VIDEO_AV1_MESA_LEVEL_2_1 = 1,
> - STD_VIDEO_AV1_MESA_LEVEL_2_2 = 2,
> - STD_VIDEO_AV1_MESA_LEVEL_2_3 = 3,
> - STD_VIDEO_AV1_MESA_LEVEL_3_0 = 4,
> - STD_VIDEO_AV1_MESA_LEVEL_3_1 = 5,
> - STD_VIDEO_AV1_MESA_LEVEL_3_2 = 6,
> - STD_VIDEO_AV1_MESA_LEVEL_3_3 = 7,
> - STD_VIDEO_AV1_MESA_LEVEL_4_0 = 8,
> - STD_VIDEO_AV1_MESA_LEVEL_4_1 = 9,
> - STD_VIDEO_AV1_MESA_LEVEL_4_2 = 10,
> - STD_VIDEO_AV1_MESA_LEVEL_4_3 = 11,
> - STD_VIDEO_AV1_MESA_LEVEL_5_0 = 12,
> - STD_VIDEO_AV1_MESA_LEVEL_5_1 = 13,
> - STD_VIDEO_AV1_MESA_LEVEL_5_2 = 14,
> - STD_VIDEO_AV1_MESA_LEVEL_5_3 = 15,
> - STD_VIDEO_AV1_MESA_LEVEL_6_0 = 16,
> - STD_VIDEO_AV1_MESA_LEVEL_6_1 = 17,
> - STD_VIDEO_AV1_MESA_LEVEL_6_2 = 18,
> - STD_VIDEO_AV1_MESA_LEVEL_6_3 = 19,
> - STD_VIDEO_AV1_MESA_LEVEL_7_0 = 20,
> - STD_VIDEO_AV1_MESA_LEVEL_7_1 = 21,
> - STD_VIDEO_AV1_MESA_LEVEL_7_2 = 22,
> - STD_VIDEO_AV1_MESA_LEVEL_7_3 = 23,
> - STD_VIDEO_AV1_MESA_LEVEL_MAX = 31,
> -} StdVideoAV1MESALevel;
> -
> -typedef struct StdVideoAV1MESAFilmGrainFlags {
> - uint8_t apply_grain;
> - uint8_t chroma_scaling_from_luma;
> - uint8_t overlap_flag;
> - uint8_t clip_to_restricted_range;
> -} StdVideoAV1MESAFilmGrainFlags;
> -
> -typedef struct StdVideoAV1MESAFilmGrainParameters {
> - StdVideoAV1MESAFilmGrainFlags flags;
> - uint32_t grain_scaling_minus_8;
> - uint32_t ar_coeff_lag;
> - uint32_t ar_coeff_shift_minus_6;
> - uint32_t grain_scale_shift;
> -
> - uint16_t grain_seed;
> - uint8_t num_y_points;
> - uint8_t point_y_value[14];
> - uint8_t point_y_scaling[14];
> -
> - uint8_t num_cb_points;
> - uint8_t point_cb_value[10];
> - uint8_t point_cb_scaling[10];
> -
> - uint8_t num_cr_points;
> - uint8_t point_cr_value[10];
> - uint8_t point_cr_scaling[10];
> -
> - int8_t ar_coeffs_y_plus_128[24];
> - int8_t ar_coeffs_cb_plus_128[25];
> - int8_t ar_coeffs_cr_plus_128[25];
> - uint8_t cb_mult;
> - uint8_t cb_luma_mult;
> - uint16_t cb_offset;
> - uint8_t cr_mult;
> - uint8_t cr_luma_mult;
> - uint16_t cr_offset;
> -} StdVideoAV1MESAFilmGrainParameters;
> -
> -typedef struct StdVideoAV1MESAGlobalMotionFlags {
> - uint8_t gm_invalid;
> -} StdVideoAV1MESAGlobalMotionFlags;
> -
> -typedef struct StdVideoAV1MESAGlobalMotion {
> - StdVideoAV1MESAGlobalMotionFlags flags;
> - uint8_t gm_type;
> - uint32_t gm_params[6];
> -} StdVideoAV1MESAGlobalMotion;
> -
> -typedef struct StdVideoAV1MESALoopRestoration {
> - uint8_t lr_type[3];
> - uint8_t lr_unit_shift;
> - uint8_t lr_uv_shift;
> -} StdVideoAV1MESALoopRestoration;
> -
> -typedef struct StdVideoAV1MESATileInfoFlags {
> - uint8_t uniform_tile_spacing_flag;
> -} StdVideoAV1MESATileInfoFlags;
> -
> -typedef struct StdVideoAV1MESATileInfo {
> - StdVideoAV1MESATileInfoFlags flags;
> - uint8_t tile_cols;
> - uint8_t tile_rows;
> - uint8_t tile_start_col_sb[64];
> - uint8_t tile_start_row_sb[64];
> - uint8_t width_in_sbs_minus_1[64];
> - uint8_t height_in_sbs_minus_1[64];
> - uint16_t context_update_tile_id;
> - uint8_t tile_size_bytes_minus1;
> -} StdVideoAV1MESATileInfo;
> -
> -typedef struct StdVideoAV1MESAQuantizationFlags {
> - uint8_t using_qmatrix;
> -} StdVideoAV1MESAQuantizationFlags;
> -
> -typedef struct StdVideoAV1MESAQuantization {
> - StdVideoAV1MESAQuantizationFlags flags;
> - uint8_t base_q_idx;
> - int8_t delta_q_y_dc;
> - uint8_t diff_uv_delta;
> - int8_t delta_q_u_dc;
> - int8_t delta_q_u_ac;
> - int8_t delta_q_v_dc;
> - int8_t delta_q_v_ac;
> - uint8_t qm_y;
> - uint8_t qm_u;
> - uint8_t qm_v;
> -} StdVideoAV1MESAQuantization;
> -
> -typedef struct StdVideoAV1MESACDEF {
> - uint8_t damping_minus_3;
> - uint8_t bits;
> - uint8_t y_pri_strength[8];
> - uint8_t y_sec_strength[8];
> - uint8_t uv_pri_strength[8];
> - uint8_t uv_sec_strength[8];
> -} StdVideoAV1MESACDEF;
> -
> -typedef struct StdVideoAV1MESADeltaQFlags {
> - uint8_t delta_lf_present;
> - uint8_t delta_lf_multi;
> -} StdVideoAV1MESADeltaQFlags;
> -
> -typedef struct StdVideoAV1MESADeltaQ {
> - StdVideoAV1MESADeltaQFlags flags;
> - uint8_t delta_q_res;
> - uint8_t delta_lf_res;
> -} StdVideoAV1MESADeltaQ;
> -
> -typedef struct StdVideoAV1MESASegmentationFlags {
> - uint8_t enabled;
> - uint8_t update_map;
> - uint8_t temporal_update;
> - uint8_t update_data;
> -} StdVideoAV1MESASegmentationFlags;
> -
> -typedef struct StdVideoAV1MESASegmentation {
> - StdVideoAV1MESASegmentationFlags flags;
> - uint8_t feature_enabled_bits[8];
> - int16_t feature_data[8][8];
> -} StdVideoAV1MESASegmentation;
> -
> -typedef struct StdVideoAV1MESALoopFilterFlags {
> - uint8_t delta_enabled;
> - uint8_t delta_update;
> -} StdVideoAV1MESALoopFilterFlags;
> -
> -typedef struct StdVideoAV1MESALoopFilter {
> - StdVideoAV1MESALoopFilterFlags flags;
> - uint8_t level[4];
> - uint8_t sharpness;
> - int8_t ref_deltas[8];
> - int8_t mode_deltas[2];
> -} StdVideoAV1MESALoopFilter;
> -
> -typedef struct StdVideoAV1MESAFrameHeaderFlags {
> - uint8_t error_resilient_mode;
> - uint8_t disable_cdf_update;
> - uint8_t use_superres;
> - uint8_t render_and_frame_size_different;
> - uint8_t allow_screen_content_tools;
> - uint8_t is_filter_switchable;
> - uint8_t force_integer_mv;
> - uint8_t frame_size_override_flag;
> - uint8_t buffer_removal_time_present_flag;
> - uint8_t allow_intrabc;
> - uint8_t frame_refs_short_signaling;
> - uint8_t allow_high_precision_mv;
> - uint8_t is_motion_mode_switchable;
> - uint8_t use_ref_frame_mvs;
> - uint8_t disable_frame_end_update_cdf;
> - uint8_t allow_warped_motion;
> - uint8_t reduced_tx_set;
> - uint8_t reference_select;
> - uint8_t skip_mode_present;
> - uint8_t delta_q_present;
> - uint8_t UsesLr;
> -} StdVideoAV1MESAFrameHeaderFlags;
> -
> -typedef struct StdVideoAV1MESAFrameHeader {
> - StdVideoAV1MESAFrameHeaderFlags flags;
> -
> - uint32_t frame_presentation_time;
> - uint32_t display_frame_id;
> - uint32_t current_frame_id;
> - uint8_t frame_to_show_map_idx;
> - uint8_t frame_type;
> - uint8_t order_hint;
> - uint8_t primary_ref_frame;
> - uint16_t frame_width_minus_1;
> - uint16_t frame_height_minus_1;
> - uint16_t render_width_minus_1;
> - uint16_t render_height_minus_1;
> - uint8_t coded_denom;
> -
> - uint8_t refresh_frame_flags;
> - uint8_t ref_order_hint[8];
> - int8_t ref_frame_idx[7];
> - uint32_t delta_frame_id_minus1[7];
> -
> - uint8_t interpolation_filter;
> - uint8_t tx_mode;
> -
> - StdVideoAV1MESATileInfo tiling;
> - StdVideoAV1MESAQuantization quantization;
> - StdVideoAV1MESASegmentation segmentation;
> - StdVideoAV1MESADeltaQ delta_q;
> - StdVideoAV1MESALoopFilter loop_filter;
> - StdVideoAV1MESACDEF cdef;
> - StdVideoAV1MESALoopRestoration lr;
> - StdVideoAV1MESAGlobalMotion global_motion[8]; // One per ref frame
> - StdVideoAV1MESAFilmGrainParameters film_grain;
> -} StdVideoAV1MESAFrameHeader;
> -
> -typedef struct StdVideoAV1MESAScreenCoding {
> - uint8_t seq_force_screen_content_tools;
> -} StdVideoAV1MESAScreenCoding;
> -
> -typedef struct StdVideoAV1MESATimingInfoFlags {
> - uint8_t equal_picture_interval;
> -} StdVideoAV1MESATimingInfoFlags;
> -
> -typedef struct StdVideoAV1MESATimingInfo {
> - StdVideoAV1MESATimingInfoFlags flags;
> - uint32_t num_units_in_display_tick;
> - uint32_t time_scale;
> - uint32_t num_ticks_per_picture_minus_1;
> -} StdVideoAV1MESATimingInfo;
> -
> -typedef struct StdVideoAV1MESAColorConfigFlags {
> - uint8_t mono_chrome;
> - uint8_t color_range;
> - uint8_t separate_uv_delta_q;
> -} StdVideoAV1MESAColorConfigFlags;
> -
> -typedef struct StdVideoAV1MESAColorConfig {
> - StdVideoAV1MESAColorConfigFlags flags;
> - uint8_t bit_depth;
> - uint8_t subsampling_x;
> - uint8_t subsampling_y;
> -} StdVideoAV1MESAColorConfig;
> -
> -typedef struct StdVideoAV1MESASequenceHeaderFlags {
> - uint8_t still_picture;
> - uint8_t reduced_still_picture_header;
> - uint8_t use_128x128_superblock;
> - uint8_t enable_filter_intra;
> - uint8_t enable_intra_edge_filter;
> - uint8_t enable_interintra_compound;
> - uint8_t enable_masked_compound;
> - uint8_t enable_warped_motion;
> - uint8_t enable_dual_filter;
> - uint8_t enable_order_hint;
> - uint8_t enable_jnt_comp;
> - uint8_t enable_ref_frame_mvs;
> - uint8_t frame_id_numbers_present_flag;
> - uint8_t enable_superres;
> - uint8_t enable_cdef;
> - uint8_t enable_restoration;
> - uint8_t film_grain_params_present;
> - uint8_t timing_info_present_flag;
> - uint8_t initial_display_delay_present_flag;
> -} StdVideoAV1MESASequenceHeaderFlags;
> -
> -typedef struct StdVideoAV1MESASequenceHeader {
> - StdVideoAV1MESASequenceHeaderFlags flags;
> -
> - StdVideoAV1MESAProfile seq_profile;
> - uint8_t frame_width_bits_minus_1;
> - uint8_t frame_height_bits_minus_1;
> - uint16_t max_frame_width_minus_1;
> - uint16_t max_frame_height_minus_1;
> - uint8_t delta_frame_id_length_minus_2;
> - uint8_t additional_frame_id_length_minus_1;
> - uint8_t order_hint_bits_minus_1;
> - uint8_t seq_choose_integer_mv;
> - uint8_t seq_force_integer_mv;
> -
> - StdVideoAV1MESATimingInfo timing_info;
> - StdVideoAV1MESAColorConfig color_config;
> -} StdVideoAV1MESASequenceHeader;
> -
> -typedef struct StdVideoAV1MESATile {
> - uint16_t tg_start;
> - uint16_t tg_end;
> - uint16_t row;
> - uint16_t column;
> - uint32_t size;
> - uint32_t offset;
> -} StdVideoAV1MESATile;
> -
> -typedef struct StdVideoAV1MESATileList {
> - StdVideoAV1MESATile *tile_list;
> - uint32_t nb_tiles;
> -} StdVideoAV1MESATileList;
> -
> -typedef struct VkVideoDecodeAV1PictureInfoMESA {
> - VkStructureType sType;
> - const void *pNext;
> - StdVideoAV1MESAFrameHeader *frame_header;
> - StdVideoAV1MESATileList *tile_list;
> - uint8_t skip_mode_frame_idx[2];
> -} VkVideoDecodeAV1PictureInfoMESA;
> -
> -typedef struct VkVideoDecodeAV1DpbSlotInfoMESA {
> - VkStructureType sType;
> - const void *pNext;
> - uint8_t frameIdx;
> - uint8_t ref_order_hint[7];
> - uint8_t disable_frame_end_update_cdf;
> -} VkVideoDecodeAV1DpbSlotInfoMESA;
> -
> -typedef struct VkVideoDecodeAV1SessionParametersAddInfoMESA {
> - VkStructureType sType;
> - const void *pNext;
> - StdVideoAV1MESASequenceHeader *sequence_header;
> -} VkVideoDecodeAV1SessionParametersAddInfoMESA;
> -
> -typedef struct VkVideoDecodeAV1SessionParametersCreateInfoMESA {
> - VkStructureType sType;
> - const void *pNext;
> - const VkVideoDecodeAV1SessionParametersAddInfoMESA *pParametersAddInfo;
> -} VkVideoDecodeAV1SessionParametersCreateInfoMESA;
> -
> -typedef struct VkVideoDecodeAV1ProfileInfoMESA {
> - VkStructureType sType;
> - const void *pNext;
> - StdVideoAV1MESAProfile stdProfileIdc;
> -} VkVideoDecodeAV1ProfileInfoMESA;
> -
> -typedef enum VkVideoDecodeAV1CapabilityFlagBitsMESA {
> - VK_VIDEO_DECODE_AV1_CAPABILITY_EXTERNAL_FILM_GRAIN_MESA = 0x00000001,
> - VK_VIDEO_DECODE_AV1_CAPABILITY_FLAG_BITS_MAX_ENUM_MESA = 0x7FFFFFFF
> -} VkVideoDecodeAV1CapabilityFlagBitsMESA;
> -typedef VkFlags VkVideoDecodeAV1CapabilityFlagsMESA;
> -
> -typedef struct VkVideoDecodeAV1CapabilitiesMESA {
> - VkStructureType sType;
> - const void *pNext;
> - VkVideoDecodeAV1CapabilityFlagsMESA flags;
> - StdVideoAV1MESALevel maxLevelIdc;
> -} VkVideoDecodeAV1CapabilitiesMESA;
> -
> -#define VK_STRUCTURE_TYPE_VIDEO_DECODE_AV1_PICTURE_INFO_MESA 1000509000
> -#define VK_STRUCTURE_TYPE_VIDEO_DECODE_AV1_SESSION_PARAMETERS_CREATE_INFO_MESA 1000509001
> -#define VK_STRUCTURE_TYPE_VIDEO_DECODE_AV1_SESSION_PARAMETERS_ADD_INFO_MESA 1000509002
> -#define VK_STRUCTURE_TYPE_VIDEO_DECODE_AV1_DPB_SLOT_INFO_MESA 1000509003
> -#define VK_STRUCTURE_TYPE_VIDEO_DECODE_AV1_CAPABILITIES_MESA 1000509004
> -#define VK_STRUCTURE_TYPE_VIDEO_DECODE_AV1_PROFILE_INFO_MESA 1000509005
> -
> -#ifdef __cplusplus
> -}
> -#endif
> -
> -#endif
> diff --git a/libavcodec/vulkan_video_codec_av1std_decode.h b/libavcodec/vulkan_video_codec_av1std_decode.h
> deleted file mode 100644
> index a697c00593..0000000000
> --- a/libavcodec/vulkan_video_codec_av1std_decode.h
> +++ /dev/null
> @@ -1,36 +0,0 @@
> -/* Copyright 2023 Lynne
> - * Copyright 2023 Dave Airlie
> - *
> - * Licensed under the Apache License, Version 2.0 (the "License");
> - * you may not use this file except in compliance with the License.
> - * You may obtain a copy of the License at
> - *
> - * http://www.apache.org/licenses/LICENSE-2.0
> - *
> - * Unless required by applicable law or agreed to in writing, software
> - * distributed under the License is distributed on an "AS IS" BASIS,
> - * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
> - * See the License for the specific language governing permissions and
> - * limitations under the License.
> - */
> -
> -#ifndef VULKAN_VIDEO_CODEC_AV1STD_DECODE_H_
> -#define VULKAN_VIDEO_CODEC_AV1STD_DECODE_H_ 1
> -
> -/*
> -** This header is NOT YET generated from the Khronos Vulkan XML API Registry.
> -**
> -*/
> -
> -#ifdef __cplusplus
> -extern "C" {
> -#endif
> -#define vulkan_video_codec_av1std_decode 1
> -
> -
> -
> -#ifdef __cplusplus
> -}
> -#endif
> -
> -#endif
> diff --git a/libavutil/hwcontext_vulkan.c b/libavutil/hwcontext_vulkan.c
> index a84713e621..44b763550d 100644
> --- a/libavutil/hwcontext_vulkan.c
> +++ b/libavutil/hwcontext_vulkan.c
> @@ -436,7 +436,7 @@ static const VulkanOptExtension optional_device_exts[] = {
> { VK_KHR_VIDEO_DECODE_QUEUE_EXTENSION_NAME, FF_VK_EXT_VIDEO_DECODE_QUEUE },
> { VK_KHR_VIDEO_DECODE_H264_EXTENSION_NAME, FF_VK_EXT_VIDEO_DECODE_H264 },
> { VK_KHR_VIDEO_DECODE_H265_EXTENSION_NAME, FF_VK_EXT_VIDEO_DECODE_H265 },
> - { "VK_MESA_video_decode_av1", FF_VK_EXT_VIDEO_DECODE_AV1 },
> + { VK_KHR_VIDEO_DECODE_AV1_EXTENSION_NAME, FF_VK_EXT_VIDEO_DECODE_AV1 },
> };
>
> static VkBool32 VKAPI_CALL vk_dbg_callback(VkDebugUtilsMessageSeverityFlagBitsEXT severity,
> diff --git a/libavutil/vulkan_functions.h b/libavutil/vulkan_functions.h
> index 65021b04b1..6b379acf93 100644
> --- a/libavutil/vulkan_functions.h
> +++ b/libavutil/vulkan_functions.h
> @@ -43,7 +43,7 @@ typedef enum FFVulkanExtensions {
> FF_VK_EXT_VIDEO_DECODE_QUEUE = 1ULL << 11, /* VK_KHR_video_decode_queue */
> FF_VK_EXT_VIDEO_DECODE_H264 = 1ULL << 12, /* VK_EXT_video_decode_h264 */
> FF_VK_EXT_VIDEO_DECODE_H265 = 1ULL << 13, /* VK_EXT_video_decode_h265 */
> - FF_VK_EXT_VIDEO_DECODE_AV1 = 1ULL << 14, /* VK_MESA_video_decode_av1 */
> + FF_VK_EXT_VIDEO_DECODE_AV1 = 1ULL << 14, /* VK_KHR_video_decode_av1 */
> FF_VK_EXT_ATOMIC_FLOAT = 1ULL << 15, /* VK_EXT_shader_atomic_float */
> FF_VK_EXT_COOP_MATRIX = 1ULL << 16, /* VK_KHR_cooperative_matrix */
>
> diff --git a/libavutil/vulkan_loader.h b/libavutil/vulkan_loader.h
> index f88722f28f..fc854242dd 100644
> --- a/libavutil/vulkan_loader.h
> +++ b/libavutil/vulkan_loader.h
> @@ -56,7 +56,7 @@ static inline uint64_t ff_vk_extensions_to_mask(const char * const *extensions,
> { VK_KHR_VIDEO_DECODE_QUEUE_EXTENSION_NAME, FF_VK_EXT_VIDEO_DECODE_QUEUE },
> { VK_KHR_VIDEO_DECODE_H264_EXTENSION_NAME, FF_VK_EXT_VIDEO_DECODE_H264 },
> { VK_KHR_VIDEO_DECODE_H265_EXTENSION_NAME, FF_VK_EXT_VIDEO_DECODE_H265 },
> - { "VK_MESA_video_decode_av1", FF_VK_EXT_VIDEO_DECODE_AV1 },
> + { VK_KHR_VIDEO_DECODE_AV1_EXTENSION_NAME, FF_VK_EXT_VIDEO_DECODE_AV1 },
> };
>
> FFVulkanExtensions mask = 0x0;
> --
> 2.43.0.381.gb435a96ce8
More information about the ffmpeg-devel
mailing list