[FFmpeg-devel] [PATCH] H.264: faster skip decoding

Michael Niedermayer michaelni
Sun Jan 17 15:52:00 CET 2010


On Sun, Jan 17, 2010 at 02:45:32AM +0100, Michael Niedermayer wrote:
> On Sat, Jan 16, 2010 at 07:43:03PM -0500, Jason Garrett-Glaser wrote:
> > On Sat, Jan 16, 2010 at 7:17 PM, Michael Niedermayer <michaelni at gmx.at> wrote:
> > > On Sat, Jan 16, 2010 at 02:37:14PM -0500, Jason Garrett-Glaser wrote:
> > >> On Sat, Jan 16, 2010 at 2:34 PM, Jason Garrett-Glaser
> > >> <darkshikari at gmail.com> wrote:
> > >> > On Sat, Jan 16, 2010 at 2:25 PM, Jason Garrett-Glaser
> > >> > <darkshikari at gmail.com> wrote:
> > >> >> On a very skip-heavy source:
> > >> >>
> > >> >> After:
> > >> >> real ? ?12m52.048s
> > >> >> user ? ?10m6.360s
> > >> >> sys ? ? 1m53.584s
> > >> >>
> > >> >> Before:
> > >> >> real ? ?13m21.203s
> > >> >> user ? ?10m28.450s
> > >> >> sys ? ? 1m53.646s
> > >> >>
> > >> >> Dark Shikari
> > >> >>
> > >> >
> > >> > Ugh, it seems the hashes don't match, despite the fact that this patch
> > >> > really shouldn't break anything. ?Ideas are welcome as to what's wrong
> > >> > here, because the idea seems to be sound.
> > >> >
> > >> > Dark Shikari
> > >> >
> > >>
> > >> And the problem is deblocking. ?Ignore this patch, it doesn't work
> > >> with ffmpeg's deblocking model. ?Maybe if we did if(IS_SKIP(mbtype) &&
> > >> deblocking is off)...
> > >
> > > Iam currently trying to move the deblocing into a seperate pass
> > 
> > Are you sure this is a good idea? 
> 
> i dont know its an experiment
> if its slower it goes to /dev/null

btw current status is that it is a bit faster but there are hash differences
on several reference files,also various optimizations where disabled and i
hope i can reenable them once the code passes tests

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

The greatest way to live with honor in this world is to be what we pretend
to be. -- Socrates
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 189 bytes
Desc: Digital signature
URL: <http://lists.mplayerhq.hu/pipermail/ffmpeg-devel/attachments/20100117/d86a3c2c/attachment.pgp>



More information about the ffmpeg-devel mailing list