[FFmpeg-devel] [PATCH v2] avcodec/dnxhdenc: remove the unneed *thread[MAX_THREADS]

James Almer jamrial at gmail.com
Sun Sep 22 19:04:01 EEST 2019


On 9/22/2019 1:01 PM, Limin Wang wrote:
> On Sun, Sep 22, 2019 at 12:45:48PM -0300, James Almer wrote:
>> On 9/22/2019 12:43 PM, lance.lmwang at gmail.com wrote:
>>> From: Limin Wang <lance.lmwang at gmail.com>
>>>
>>> Signed-off-by: Limin Wang <lance.lmwang at gmail.com>
>>> ---
>>>  libavcodec/dnxhdenc.c | 47 ++++++++++-------------------------------------
>>>  libavcodec/dnxhdenc.h |  2 --
>>>  2 files changed, 10 insertions(+), 39 deletions(-)
>>
>> Err, what? I think you misunderstood what i called unnecessary in my
>> last email...
> No, I'm not familar with dnxhdenc too much, so I have pass "make fate" to testing
> the function(with THREAD=0 and THREAD=2)
> 
> For the performance, by my quick testing, the results are same as before.
> 
> I hope it's my misunderstanding, but the results are OK.

Try with THREAD_TYPE=slice, otherwise you'll be testing frame threading.

> 
> 
> Thanks,
> Limin
> 
> 
>> _______________________________________________
>> ffmpeg-devel mailing list
>> ffmpeg-devel at ffmpeg.org
>> https://ffmpeg.org/mailman/listinfo/ffmpeg-devel
>>
>> To unsubscribe, visit link above, or email
>> ffmpeg-devel-request at ffmpeg.org with subject "unsubscribe".
> _______________________________________________
> ffmpeg-devel mailing list
> ffmpeg-devel at ffmpeg.org
> https://ffmpeg.org/mailman/listinfo/ffmpeg-devel
> 
> To unsubscribe, visit link above, or email
> ffmpeg-devel-request at ffmpeg.org with subject "unsubscribe".
> 



More information about the ffmpeg-devel mailing list