[prev in list] [next in list] [prev in thread] [next in thread] 

List:       gnuradio-commit
Subject:    [Commit-gnuradio] [gnuradio] 20/22: volk: fix memory overrun/corruption in neon binary_slicer_8i
From:       git () gnuradio ! org
Date:       2014-10-31 19:22:31
Message-ID: 20141031192231.EB5DA1A82A2 () hertz ! gnuradio ! org
[Download RAW message or body]

This is an automated email from the git hooks/post-receive script.

jcorgan pushed a commit to branch master
in repository gnuradio.

commit 5d1c29c4491a2fcefc69857a7f2f557f6b8eff57
Author: Nathan West <nathan.west@okstate.edu>
Date:   Tue Oct 28 05:36:57 2014 -0500

    volk: fix memory overrun/corruption in neon binary_slicer_8i
---
 volk/kernels/volk/volk_32f_binary_slicer_8i.h | 10 +++++-----
 1 file changed, 5 insertions(+), 5 deletions(-)

diff --git a/volk/kernels/volk/volk_32f_binary_slicer_8i.h \
b/volk/kernels/volk/volk_32f_binary_slicer_8i.h index 17b48d7..ae4420b 100644
--- a/volk/kernels/volk/volk_32f_binary_slicer_8i.h
+++ b/volk/kernels/volk/volk_32f_binary_slicer_8i.h
@@ -221,7 +221,7 @@ volk_32f_binary_slicer_8i_neon(int8_t* cVector, const float* \
aVector,  int8_t* cPtr = cVector;
   const float* aPtr = aVector;
   unsigned int number = 0;
-  unsigned int n8points = num_points / 8;
+  unsigned int n16points = num_points / 16;
 
   float32x4x2_t input_val0, input_val1;
   float32x4_t zero_val;
@@ -237,7 +237,7 @@ volk_32f_binary_slicer_8i_neon(int8_t* cVector, const float* \
aVector,  // TODO: this is a good candidate for asm because the vcombines
   // can be eliminated simply by picking dst registers that are
   // adjacent.
-  for(number = 0; number < n8points; number++) {
+  for(number = 0; number < n16points; number++) {
     input_val0 = vld2q_f32(aPtr);
     input_val1 = vld2q_f32(aPtr+8);
 
@@ -269,12 +269,12 @@ volk_32f_binary_slicer_8i_neon(int8_t* cVector, const float* \
aVector,  res_u8.val[1] = vand_u8(one, res_u8.val[1]);
 
     vst2_u8((unsigned char*)cPtr, res_u8);
-    cPtr += 8;
-    aPtr += 8;
+    cPtr += 16;
+    aPtr += 16;
 
   }
 
-  for(number = n8points * 8; number < num_points; number++) {
+  for(number = n16points * 16; number < num_points; number++) {
     if(*aPtr++ >= 0) {
       *cPtr++ = 1;
     }

_______________________________________________
Commit-gnuradio mailing list
Commit-gnuradio@gnu.org
https://lists.gnu.org/mailman/listinfo/commit-gnuradio


[prev in list] [next in list] [prev in thread] [next in thread]