[MPlayer-dev-eng] Re: Bug#409431: mplayer: [ia64] unaligned access messages during playback

[Michael Niedermayer]

Hi

On Sat, Feb 03, 2007 at 02:11:16PM -0500, Rich Felker wrote:
> On Sat, Feb 03, 2007 at 02:59:19PM +0100, Diego Biurrun wrote:
> > On Sat, Feb 03, 2007 at 12:11:23PM +0100, Reimar D?ffinger wrote:
> > > On Sat, Feb 03, 2007 at 11:44:18AM +0100, Ivo wrote:
> > > > On Saturday 03 February 2007 10:58, Reimar D?ffinger wrote:
> > > > > Please try attached patch if it fixes it.
> > > > > http://www.mpg123.de/ is mentioned as upstream for this code, but the
> > > > > SVN there does not contain a sr1.c, so should this just be applied
> > > > > directly to MPlayer?
> > > > 
> > > > It seems it is in readers.c now, but they changed a lot of code in the 
> > > > meantime, so I don't think this can be easily applied there. I am not 
> > > > familiar with the mpg123 code, but by the looks of it, I think the 
> > > > unaligned access is fixed in svn. But I think it will be quite some work to 
> > > > merge the upstream mpg123 with MPlayer.
> > > 
> > > In that case it IMO is preferable to fix ffmp3 by implementing AVParser
> > > and slowly deprecating mp3lib.
> > 
> > (New) benchmarks are welcome, but ffmp3 is a fixed-point implementation
> > and considerably slower than mp3lib ...
> 
> No it's not. I wish it were fixed-point, but afaik only mad is
> fixed-point.

it is fixed point and that is why it is slower then mp3lib on modern cpus
i think we have disscussed this at least twice already

adding more optional float/SSE/3dnow support (currently only the antialias 
code exists in both float and int) should have a very significant
effect on speed for modern cpus

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

I have never wished to cater to the crowd; for what I know they do not
approve, and what they approve I do not know. -- Epicurus
-------------- 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/mplayer-dev-eng/attachments/20070204/cbbeddb5/attachment.pgp>