[FFmpeg-devel] [PATCH] h264 parallelized

[Michael Niedermayer]

Hi

On Fri, Jun 29, 2007 at 02:41:33AM +0200, Michael Niedermayer wrote:
> Hi
> 
> On Mon, Jun 25, 2007 at 07:02:27PM +0200, Andreas ?man wrote:
> > Hi,
> > 
> > Michael Niedermayer wrote:
> > >Hi
> > >
> > >
> > >if this is a small change, that is MPV_common_*_threads() being called
> > >from 1-3 places iam ok with it, of it adds 10 calls to it iam not
> > >
> > >its not hard to add a codec != H264 check around the init code or even
> > >call MPV_common_*_threads() from MPV_common_*() if codec != H264
> > >
> > >(cleanup for mpegvideo stuff is very welcome, bloating it up is not)
> > >
> > 
> > Okay, here is an updated version.
> > I stuck to your codec_id == h264 idea, other than that I don't
> > have much more to say about it :)
> 
> what about the speed loss? it seems 2% slower here

ive rerun the benchmark and the values are below to make it more clear

svn  : 0m0.737s, 0m0.735s, 0m0.742s, 0m0.746s, 0m0.746s
patch: 0m0.756s, 0m0.763s, 0m0.757s, 0m0.761s, 0m0.752s

testing done with:
time ffmpeg/ffmpeg-threadslices -i test.h264 -f null /dev/null 2>/dev/null

[...]
-- 
Michael     GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB

No snowflake in an avalanche ever feels responsible. -- Voltaire
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 189 bytes
Desc: not available
URL: <http://lists.mplayerhq.hu/pipermail/ffmpeg-devel/attachments/20070629/7949fd84/attachment.pgp>