[FFmpeg-devel] [PATCH] lavc/amfenc: Retain a reference to D3D frames used as input during the encoding process

[James Almer]

On 4/6/2018 7:25 AM, Alexander Kravchenko wrote:
>>
>> This breaks the testcase described in
>> https://trac.ffmpeg.org/ticket/6990 which is basically the same as the
>> one you described in this patch.
>>
>> I get the following spammed repeatedly:
>>
>> [AVHWFramesContext @ 000000000502d340] Static surface pool size exceeded.
>> [mpeg2video @ 0000000002f8d400] get_buffer() failed
>> [mpeg2video @ 0000000002f8d400] thread_get_buffer() failed
>> [mpeg2video @ 0000000002f8d400] get_buffer() failed (-12 0000000000000000)
>> Error while decoding stream #0:1: Operation not permitted
> 
> Hi, could you please review the following updated patch
> I added queue limit according initial pool size of incoming frame context.
> This solves the problem running this pipeline without -extra_hw_frames 16 option, but -extra_hw_frames option can be used to have bigger queue for encoder.

Yes, this solves it, and the output does indeed look good now. Thanks.

I'll leave reviewing this patch to someone more familiar with hw encoding.