[FFmpeg-devel] [PATCH 3/4] libavcodec/qsvdec.c: The ff_qsv_decode() now guarantees the consumption of whole packet.

Ivan Uskov ivan.uskov at nablet.com
Fri Jul 24 10:59:06 CEST 2015


Hello Michael,

Friday, July 24, 2015, 2:50:49 AM, you wrote:
>> The reason of replacing of av_usleep(1) to av_usleep(500) is
>> following: I believe it is bad idea to ask hardware 1000000 times per
>> second if it is busy, especially in main decoding loop.
>> Peak qsv decoding performance is about 2000 fps when destination is GPU memory.
>> For system memory the performance is less then 1000fps usually. So delay in 500microseconds (i.e. 0.5 ms)
>> is much more appropriate.

>> I can move it to separate patch but since main decoding loop totally
>> re-designed possible it can be applied as?

MN> i think it should be split, as it would allow very easy regression
MN> testing with bisect, of the change in the future
Ok, no problem, I will left av_usleep(1) for this patch.


-- 
Best regards,
 Ivan                            mailto:ivan.uskov at nablet.com



More information about the ffmpeg-devel mailing list