[pulseaudio-discuss] [PATCH] core: Fix a litte-endian bug in ARM svolume code

[Arun Raghavan]

On Thu, 2012-10-18 at 13:33 +0200, Peter Meerwald wrote:
> 
> > The old code seems to be written for big-endian code.
>  
> > NB: This one's really strange since this effectively means that our soft
> > volumes on ARMv6 have been mostly broken (since almost everybody uses
> > litte-endian). Review would be appreciated!
> 
> benchmark results on Cortex-A8 were horrible 
> (http://lists.freedesktop.org/archives/pulseaudio-discuss/2012-February/012864.html):
> 
> runtime on beagle-xm, 800 MHz:
> 
> checking NEON volume_float32ne
> NEON: 10223 usec.
> ref: 46480 usec.
> checking NEON volume_s16ne
> NEON: 8484 usec.
> ARM: 339272 usec.
> ref: 20203 usec.

That's odd indeed. I have this on a Freescale i.mx53 (also Cortex A8)

Checking ARM svolume
func: 905150 usec (min = 9006, max = 9562, stddev = 76.1938).
orig: 2278824 usec (min = 22760, max = 23252, stddev = 65.5575).

Any chance you could rerun this?

Also, if anyone here has a Cortex A8 handy, please let me know. It would
be good to get some more results.

Cheers,
Arun