[FFmpeg-devel] [PATCH] libmpcodecs support

[Michael Niedermayer]

On Sat, Jan 15, 2011 at 03:45:09AM +0100, Michael Niedermayer wrote:
> On Sat, Jan 15, 2011 at 02:26:40AM +0100, Aurelien Jacobs wrote:
> > On Sat, Jan 15, 2011 at 12:44:26AM +0100, Michael Niedermayer wrote:
> > > On Fri, Jan 14, 2011 at 11:34:28PM +0100, Aurelien Jacobs wrote:
> > > > On Fri, Jan 14, 2011 at 05:37:45AM +0100, Michael Niedermayer wrote:
> > > > > Hi
> > > > > 
> > > > > Attached patchset makes libavfilter support most libmpcodecs filters
> > > > > The libmpcodecs filters are practically unmodified and for keeping it
> > > > > maintainable id like to keep them unmodified.
> > > > > 
> > > > > I will commit this soon but wont enable it yet. That way others can help
> > > > > cleanup the wraper (minus code that for maintainability should stay identical
> > > > > to the original), fix bugs, rename all global functions so they wont conflict
> > > > > with mplayers if they ever include this and generally help.
> > > > > 
> > > > > Please dont bikeshed this to death.
> > > > > 
> > > > > Ahh before i can commit this, libmpcodecs and subdirectories must not get
> > > > > blocked by the tab & trailing whitespace checkin scripts.
> > > > > 
> > > > 
> > > > 
> > > > > From 34072e13989ebed5e76a425b886b5b17756e6d5d Mon Sep 17 00:00:00 2001
> > > > > From: Michael Niedermayer <michaelni at gmx.at>
> > > > > Date: Fri, 14 Jan 2011 05:07:39 +0100
> > > > > Subject: [PATCH 7/7] Enable libmpcodecs support.
> > > > > 
> > > > > ---
> > > > >  libavfilter/Makefile     |   65 ++++++++++++++++++++++++++++++++++++++++++++++
> > > > >  libavfilter/allfilters.c |    1 +
> > > > >  2 files changed, 66 insertions(+), 0 deletions(-)
> > > > > 
> > > > > diff --git a/libavfilter/Makefile b/libavfilter/Makefile
> > > > > index fdb181e..0db4750 100644
> > > > > --- a/libavfilter/Makefile
> > > > > +++ b/libavfilter/Makefile
> > > > > @@ -30,6 +30,7 @@ OBJS-$(CONFIG_FREI0R_FILTER)                 += vf_frei0r.o
> > > > >  OBJS-$(CONFIG_GRADFUN_FILTER)                += vf_gradfun.o
> > > > >  OBJS-$(CONFIG_HFLIP_FILTER)                  += vf_hflip.o
> > > > >  OBJS-$(CONFIG_HQDN3D_FILTER)                 += vf_hqdn3d.o
> > > > > +OBJS-$(CONFIG_MP_FILTER)                     += vf_mp.o
> > > > >  OBJS-$(CONFIG_NOFORMAT_FILTER)               += vf_format.o
> > > > >  OBJS-$(CONFIG_NULL_FILTER)                   += vf_null.o
> > > > >  OBJS-$(CONFIG_OCV_FILTER)                    += vf_libopencv.o
> > > > > @@ -54,6 +55,70 @@ OBJS-$(CONFIG_NULLSRC_FILTER)                += vsrc_nullsrc.o
> > > > >  
> > > > >  OBJS-$(CONFIG_NULLSINK_FILTER)               += vsink_nullsink.o
> > > > >  
> > > > > +
> > > > > +OBJS += libmpcodecs/mp_image.o
> > > > > +OBJS += libmpcodecs/img_format.o
> > > > > +OBJS [...]
> > > > 
> > > > I guess all those libmpcodecs files should also be added to
> > > > OBJS-$(CONFIG_MP_FILTER). I don't think we want them compiled
> > > > unconditionally.
> > > 
> > > i had hoped other people would help but ok fixed locally
> > > 
> > > 
> > > [...]
> > > > IIUC you said that the ultimate goal is to have all those filters
> > > > converted to native libavfilter so that we can drop all the libmpcodecs
> > > > filters. If so, why does this libmpcodecs filter list includes some
> > > > filters which have already a native libavfilter version ?? Namely:
> 
> > > >   vf_cropdetect.o
> 
> quick test says same speed
> 
> 
> > > >   vf_dsize.o
> 
> > > >   vf_rectangle.o
> 
> not in svn
> 
> 
> > > >   vf_blackframe.o
> > > >   vf_mirror.o
> > > >   vf_rotate.o
> 
> wraped libmpcodec is faster than libavfilter
> 
> testing done by looking at the fps value from ffmpeg, differences where
> significant (on the order of 5%)

at least for rotate/transpose this is caused by free/malloc
This likely also is part of why our yadif is slower
i could fix that, well that is if i could finally commit this patchset and be
rid of it

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

It is not what we do, but why we do it that matters.
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 198 bytes
Desc: Digital signature
URL: <http://lists.mplayerhq.hu/pipermail/ffmpeg-devel/attachments/20110115/d179e813/attachment.pgp>