[FFmpeg-devel] [PATCH 3/5] avcodec/av1dec: Use ProgressFrames

James Almer jamrial at gmail.com
Sat Apr 20 23:22:35 EEST 2024


On 4/19/2024 1:07 PM, Andreas Rheinhardt wrote:
> AV1 can put a frame into multiple reference slots;
> up until now, this involved creating a new reference
> to the underlying AVFrame; therefore av1_frame_ref()
> could fail.
> This commit changes this by using the ProgressFrame API
> to share the underlying AVFrames.
> 
> (Hint: vaapi_av1_surface_id() checked whether the AV1Frames
> contained in the AV1DecContext were NULL or not (of course
> they were not); this has been changed to actually check for
> whether said AV1Frame is blank or not.)
> 
> Signed-off-by: Andreas Rheinhardt <andreas.rheinhardt at outlook.com>
> ---
> 1. The AV1 patches are basically untested (apart from FATE and
> compilation). Could someone please test them?

I tested this and patch 4/5 with d3d11/12, cuda and vulkan. No issues.
Can't test vaapi.


More information about the ffmpeg-devel mailing list