[MPlayer-dev-eng] Bugfix (sig11 if sdl, rgb and resize)
Felix Buenemann
atmosfear at users.sourceforge.net
Wed Sep 26 22:24:40 CEST 2001
On Wednesday, 26. September 2001 19:59, Michael Niedermayer wrote:
> On Wednesday 26 September 2001 13:06, Felix Buenemann wrote:
> > On Tuesday, 25. September 2001 22:16, Michael Niedermayer wrote:
> > > > ok, it'll be changed in cvs with the next commiting of local changes
> > > > (have other unfinished changes...).
> > >
> > > the priv->dblit=0 variant?
> > > it does an extra copy pass, and is slower (x2)!
> > > the second percent number (video_out cpu usage according to the faq)
> > > increased from 4-5% to 8% on my test movie.
> >
> > yes, but I think it's endian independent then.
>
> more bugs ...
> 1. video_out cpu usage doesnt count the time spent in flip_page
> 2. rgbsurface isnt aligned correctly
yea, but the problem is inside SDL where I can't change it, cause the malloc
is done by SDL_CreateRGBSurface, otherwise I could speed it up using
memalign() for malloc.
>
> i did a few more speed tests ...
> memcpy to priv->surface needs 4-5%
> SDL_CreateRGBSurfaceFrom & SDL_BlitSurface 8%
> memcpy to priv->rgbsurface needs 8%
> memcpy to priv->rgbsurface+24 needs 4% (neeeds to be aligned at 32byte?)
> SDL_BlitSurface from rgbsurface to surface needs 12%
yes I know that it's really slow, that's why I added the direct blitting
stuff which is enabled with dblit=1.
So I should take a look at your patch again.
Btw. if you remove SDL_ANYFORMAT flag it'll be veery slow :)
>
> the best solution IMHO for rgb would be to do a memcpy if possible
> (completly identical formats) and if thats not possible than
> SDL_CreateRGBSurfaceFrom & SDL_BlitSurface is next best, and could be used
> for rgb format conversation too.
>
yes.
> Michael
--
Best Regards,
Atmos
_____
RTFM
- Gabucino
More information about the MPlayer-dev-eng
mailing list