<div dir="ltr">Hello Vorbis Developers / Monty,<div><br></div><div>Am I at the right place? I posted these patches about 2 weeks ago to &quot;<a href="mailto:vorbis-dev@xiph.org">vorbis-dev@xiph.org</a>&quot;</div><div><br></div><div>I haven&#39;t seen any response/reviews about this patch. Neither do I see any other emails from this mailing list.</div><div><br></div><div>Please advise.</div></div><div class="gmail_extra"><br clear="all"><div><div dir="ltr"><font face="comic sans ms, sans-serif">Regards,</font><div><font face="comic sans ms, sans-serif">Vish (Viswanath Puttagunta)</font></div><div><font face="comic sans ms, sans-serif">Cell: 972-342-0205</font></div><div><font face="comic sans ms, sans-serif">Technical Program Manager</font></div><div><font face="comic sans ms, sans-serif">Member Services, Linaro</font></div></div></div>
<br><div class="gmail_quote">On 10 September 2014 14:15, Viswanath Puttagunta <span dir="ltr">&lt;<a href="mailto:viswanath.puttagunta@linaro.org" target="_blank">viswanath.puttagunta@linaro.org</a>&gt;</span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">libvorbis does not currently have any simd/vectorization.<br>
<br>
Following patches add generic framework for simd/vectorization<br>
and on top, add ARM-NEON simd vectorization using intrinsics.<br>
<br>
I was able to get over 34% performance improvement on my<br>
Beaglebone Black which is single Cortex-A8 based CPU.<br>
<br>
You can find more information on metrics and procedure I used<br>
to measure at<br>
<a href="https://wiki.linaro.org/WorkingGroups/Middleware/Graphics/MediaLibs/libvorbis" target="_blank">https://wiki.linaro.org/WorkingGroups/Middleware/Graphics/MediaLibs/libvorbis</a><br>
<br>
As described in above link/doc, you can see my work-in-progress<br>
tree at<br>
<a href="https://git.linaro.org/people/viswanath.puttagunta/vorbis.git" target="_blank">https://git.linaro.org/people/viswanath.puttagunta/vorbis.git</a><br>
<br>
Also what standard should one follow when submitting patches<br>
to libvorbis? I&#39;m pretty sure linux kernel <a href="http://checkpatch.pl" target="_blank">checkpatch.pl</a> will<br>
fail all patches for libvorbis. Please advise.<br>
<br>
Viswanath Puttagunta (3):<br>
  lib/simd: Introduce vectorization framework for libvorbis<br>
  lib/block.c: Use optimized routine for wave_operation<br>
  mdct: implement arm simd implementation for mdct<br>
<br>
 <a href="http://configure.ac" target="_blank">configure.ac</a>         |    5 ++<br>
 lib/Makefile.am      |    6 +-<br>
 lib/block.c          |   14 ++---<br>
 lib/mdct.c           |  102 +-------------------------------<br>
 lib/simd/Makefile.am |    9 +++<br>
 lib/simd/neon_simd.c |  158 ++++++++++++++++++++++++++++++++++++++++++++++++++<br>
 lib/simd/no_simd.c   |  129 +++++++++++++++++++++++++++++++++++++++++<br>
 lib/simd/simd.h      |   40 +++++++++++++<br>
 8 files changed, 351 insertions(+), 112 deletions(-)<br>
 create mode 100644 lib/simd/Makefile.am<br>
 create mode 100644 lib/simd/neon_simd.c<br>
 create mode 100644 lib/simd/no_simd.c<br>
 create mode 100644 lib/simd/simd.h<br>
<span class="HOEnZb"><font color="#888888"><br>
--<br>
1.7.9.5<br>
<br>
</font></span></blockquote></div><br></div>