[MPlayer-dev-eng] [PATCH] altivec scaler performance enhancement

Diego Biurrun diego at biurrun.de
Sat Feb 11 15:13:53 CET 2006


On Fri, Feb 10, 2006 at 04:01:32PM +0100, Michael Niedermayer wrote:
> 
> On Fri, Feb 10, 2006 at 10:35:20AM +0100, Diego Biurrun wrote:
> > On Thu, Feb 09, 2006 at 10:12:14PM -0500, Alan Curry wrote:
> > > 
> > > This is the patch to move the v{Y,C}CoeffsBank vectors into the SwsContext,
> > > filling them in just once when the scaler is initialized, instead of building
> > > them and freeing them over and over. In my tests, it cuts CPU usage in half
> > > or better. Not just the scaler's CPU usage - the usage of the entire mplayer
> > > process! One of my tests went from being a 94% CPU hog to 33%.
> > 
> > Outstanding!  Works like a charm here, I'm seeing 30-50% performance
> > improvements.  Michael, OK to commit?
> 
> yes, after fixing the indention of the moved code

While the indentation looks peculiar, it does fit the code around it so
I've left it alone for now.

Patch committed.

Diego




More information about the MPlayer-dev-eng mailing list