[FFmpeg-devel] [PATCH v2] lavc/vaapi_encode_h264: fix poc incorrect issue after meeting idr frame.

[Andy Furniss]

Mark Thompson wrote:
> On 14/11/16 07:11, Jun Zhao wrote:
>> V2 : - Change the last_idr_frame filed location based on Mark code review.
>>       - Modify the commit message to actually explain the problem.
>>
>>  From a1bf2b021effd36f8297b331855a282d775f2a44 Mon Sep 17 00:00:00 2001
>> From: Jun Zhao <jun.zhao at intel.com>
>> Date: Fri, 11 Nov 2016 14:53:49 +0800
>> Subject: [PATCH v2] lavc/vaapi_encode_h264: fix poc incorrect issue after
>>   meeting idr frame.
>>
>> when meeting IDR frame, vaapi_encode_h264 poc number don't reset, now fix
>> this issue based on h264 spec. Some decoder don't care this case, but this
>> fix will enhance the encoder action. Before this fix, poc number is
>> negative in some case.
>>
>> Reviewed-by: Jun Zhao <jun.zhao at intel.com>
>> Signed-off-by: Wang, Yi A <yi.a.wang at intel.com>
>
> Applied as <http://git.videolan.org/?p=ffmpeg.git;a=commit;h=e72662e131e5099e34d5a7519c5690d2fff7b83f>.

Oh, nice, and thanks for getting all the other patches in ffmpeg.

On AMD this does fix the negative pocs issue, but there may still be
another, as I mentioned on mesa dev, -g 30 makes an IDR followed by 30
P frames. I assume the IDR should be included in the count?