[FFmpeg-devel] [PATCH] lazily create threads in pthread.c
Alexander Strange
astrange
Sun Aug 12 20:33:31 CEST 2007
On Aug 12, 2007, at 2:04 PM, Reimar D?ffinger wrote:
> Hello,
> On Sun, Aug 12, 2007 at 01:47:25PM -0400, Alexander Strange wrote:
>> This patch changes pthread.c so that threads aren't created until
>> execute()
>> requests them.
>> This way, you can unconditionally call avcodec_thread_init() and
>> not have
>> unused threads lying around if they aren't used.
>
> You forgot to say what the advantage is supposed to be...
>
>> It could also save one worker thread by reusing the main thread,
>> instead of
>> sleeping it; I'll probably look at that later.
>
> Again, advantage? I think any halfway recent OS is designed to easily
> handle hundreds of threads, I can't imagine ffmpeg using e.g. 4
> instead of 5
> threads will gain much if anything.
Pretty much cosmetic. A few more sleeping threads won't confuse any
decent OS,
they just clutter up gdb or profilers. And ps output, for those
pthread systems that fork for every thread.
I don't see any stability or performance changes introduced by this,
so it should be harmless otherwise.
More information about the ffmpeg-devel
mailing list