[asterisk-dev] MMX in Zaptel

[Tzafrir Cohen]

On Sat, Sep 22, 2007 at 12:33:17PM -0500, Matthew Fredrickson wrote:
> Tzafrir Cohen wrote:
> > Hi
> > 
> > Reminder: zconfig.h includes a defina called CONFIG_ZAPTEL_MMX,
> > undefined by default, to enable/disable MMX support in Zaptel. It also
> > warns of problems with various CPUs.
> > 
> > See also the previous thread:
> > http://lists.digium.com/pipermail/asterisk-dev/2007-August/029221.html
> > 
> > I have just commited a change to the kernel 2.6 Makefile to enable this
> > if the CPU supports MMX:
> > http://svn.digium.com/view/zaptel?view=rev&revision=3076
> > This is disabled by default. You need to explicitly edit that file or
> > something of the sort of:
> > 
> >   ZAPTEL_MMX_AUTO=yes make
> > 
> > to enable it. I do plan to enable it by default in the following release
> > (1.4.7 ).
> 
> Many have already replied to this thread giving similar sentiments. 
> That code is potentially broken and definitely unreliable.  Do not 
> enable it by default.  If you want to work on it to improve it, do so, 
> but until there is a LOT of positive feedback, I don't want to see it 
> enabled by default.

The previous thread on this subject left me more optimistic. Two have
replied to this message and reported it as "broken" but were not too
specific.

So it indeed calls for further testing.

-- 
               Tzafrir Cohen       
icq#16849755                    jabber:tzafrir at jabber.org
+972-50-7952406           mailto:tzafrir.cohen at xorcom.com       
http://www.xorcom.com  iax:guest at local.xorcom.com/tzafrir