diff options
author | Pau Espin Pedrol <pespin@sysmocom.de> | 2018-12-03 17:46:04 +0100 |
---|---|---|
committer | Harald Welte <laforge@gnumonks.org> | 2018-12-05 19:41:34 +0000 |
commit | f7331764ac22d3c25838be057835b79ef7ceaadd (patch) | |
tree | fff479bb9fe2eee69484e455b8a9eaa3b11b32d2 /Transceiver52M/sigProcLib.cpp | |
parent | 800c029c709fda37ec1dd37b833ae5decd03ec2c (diff) |
SigProcLib: Improve Vector buffer allocation mess
Original issue: In order to use SSE instructions, 16-byte aligned memory
chunks are needed, and C++ version < C++11 doesn't provide for a native
new/delete store. For that reason, memalign() must be used in the
implementation of convolve_h_alloc() for some buffers.
On the other side, The C++ code relies on C++ "new T[]" operator to
allocate a chunk of memory containing an array of class instances. As
classes are complex types, they cannot be allocated through C structures
(calling malloc). Experimentally can be seen too that it's unreliable
and the process will crash during startup if malloc() is used and then a
Complex<> deferred from it.
Previous implementation allowed for use of convolve_h_alloc or new[]
based on how the (signal)Vector is called, because then the buffer is
not going to be managed internally. But that's unreliable since resize()
calling resize() on it could use "delete" operator on a malloc'ed
buffer, and end up having a new new[] allocated buffer. It was also
found that some of the callers were actually leaking memory through ASan (because the
buffer is not managed by the Vector instance).
IMHO best option would be to rewrite all this code using C structures
and malloc/free exclusively, since it would make all this cod eeasier to
maintain.
But for now, let's extend the Vector class to allow specifying an
external alloc/free function and let the Vector instance take care of
the ownership of the buffer in all scenarios.
Change-Id: Ie484a4762a7f77fe1b105188ea03a6f025730b82
Diffstat (limited to 'Transceiver52M/sigProcLib.cpp')
-rw-r--r-- | Transceiver52M/sigProcLib.cpp | 34 |
1 files changed, 10 insertions, 24 deletions
diff --git a/Transceiver52M/sigProcLib.cpp b/Transceiver52M/sigProcLib.cpp index 28c4ded..f720828 100644 --- a/Transceiver52M/sigProcLib.cpp +++ b/Transceiver52M/sigProcLib.cpp @@ -84,14 +84,13 @@ static Resampler *dnsampler = NULL; * perform 16-byte memory alignment required by many SSE instructions. */ struct CorrelationSequence { - CorrelationSequence() : sequence(NULL), buffer(NULL) + CorrelationSequence() : sequence(NULL) { } ~CorrelationSequence() { delete sequence; - free(buffer); } signalVector *sequence; @@ -106,8 +105,7 @@ struct CorrelationSequence { * for SSE instructions. */ struct PulseSequence { - PulseSequence() : c0(NULL), c1(NULL), c0_inv(NULL), empty(NULL), - c0_buffer(NULL), c1_buffer(NULL), c0_inv_buffer(NULL) + PulseSequence() : c0(NULL), c1(NULL), c0_inv(NULL), empty(NULL) { } @@ -117,17 +115,12 @@ struct PulseSequence { delete c1; delete c0_inv; delete empty; - free(c0_buffer); - free(c1_buffer); } signalVector *c0; signalVector *c1; signalVector *c0_inv; signalVector *empty; - void *c0_buffer; - void *c1_buffer; - void *c0_inv_buffer; }; static CorrelationSequence *gMidambles[] = {NULL,NULL,NULL,NULL,NULL,NULL,NULL,NULL}; @@ -340,7 +333,7 @@ static signalVector *convolve(const signalVector *x, const signalVector *h, if (y && (len > y->size())) return NULL; if (!y) { - y = new signalVector(len); + y = new signalVector(len, convolve_h_alloc, free); alloc = true; } @@ -403,8 +396,7 @@ static bool generateInvertC0Pulse(PulseSequence *pulse) if (!pulse) return false; - pulse->c0_inv_buffer = convolve_h_alloc(5); - pulse->c0_inv = new signalVector((complex *) pulse->c0_inv_buffer, 0, 5); + pulse->c0_inv = new signalVector((complex *) convolve_h_alloc(5), 0, 5, convolve_h_alloc, free); pulse->c0_inv->isReal(true); pulse->c0_inv->setAligned(false); @@ -433,9 +425,7 @@ static bool generateC1Pulse(int sps, PulseSequence *pulse) return false; } - pulse->c1_buffer = convolve_h_alloc(len); - pulse->c1 = new signalVector((complex *) - pulse->c1_buffer, 0, len); + pulse->c1 = new signalVector((complex *) convolve_h_alloc(len), 0, len, convolve_h_alloc, free); pulse->c1->isReal(true); /* Enable alignment for SSE usage */ @@ -489,8 +479,7 @@ static PulseSequence *generateGSMPulse(int sps) len = 4; } - pulse->c0_buffer = convolve_h_alloc(len); - pulse->c0 = new signalVector((complex *) pulse->c0_buffer, 0, len); + pulse->c0 = new signalVector((complex *) convolve_h_alloc(len), 0, len, convolve_h_alloc, free); pulse->c0->isReal(true); /* Enable alingnment for SSE usage */ @@ -1019,7 +1008,7 @@ static void generateDelayFilters() for (int i = 0; i < DELAYFILTS; i++) { data = (complex *) convolve_h_alloc(h_len); - h = new signalVector(data, 0, h_len); + h = new signalVector(data, 0, h_len, convolve_h_alloc, free); h->setAligned(true); h->isReal(true); @@ -1263,7 +1252,7 @@ static bool generateMidamble(int sps, int tsc) /* For SSE alignment, reallocate the midamble sequence on 16-byte boundary */ data = (complex *) convolve_h_alloc(midMidamble->size()); - _midMidamble = new signalVector(data, 0, midMidamble->size()); + _midMidamble = new signalVector(data, 0, midMidamble->size(), convolve_h_alloc, free); _midMidamble->setAligned(true); midMidamble->copyTo(*_midMidamble); @@ -1274,7 +1263,6 @@ static bool generateMidamble(int sps, int tsc) } gMidambles[tsc] = new CorrelationSequence; - gMidambles[tsc]->buffer = data; gMidambles[tsc]->sequence = _midMidamble; gMidambles[tsc]->gain = peakDetect(*autocorr, &toa, NULL); @@ -1319,13 +1307,12 @@ static CorrelationSequence *generateEdgeMidamble(int tsc) conjugateVector(*midamble); data = (complex *) convolve_h_alloc(midamble->size()); - _midamble = new signalVector(data, 0, midamble->size()); + _midamble = new signalVector(data, 0, midamble->size(), convolve_h_alloc, free); _midamble->setAligned(true); midamble->copyTo(*_midamble); /* Channel gain is an empirically measured value */ seq = new CorrelationSequence; - seq->buffer = data; seq->sequence = _midamble; seq->gain = Complex<float>(-19.6432, 19.5006) / 1.18; seq->toa = 0; @@ -1360,7 +1347,7 @@ static bool generateRACHSequence(CorrelationSequence **seq, const BitVector &bv, /* For SSE alignment, reallocate the midamble sequence on 16-byte boundary */ data = (complex *) convolve_h_alloc(seq1->size()); - _seq1 = new signalVector(data, 0, seq1->size()); + _seq1 = new signalVector(data, 0, seq1->size(), convolve_h_alloc, free); _seq1->setAligned(true); seq1->copyTo(*_seq1); @@ -1372,7 +1359,6 @@ static bool generateRACHSequence(CorrelationSequence **seq, const BitVector &bv, *seq = new CorrelationSequence; (*seq)->sequence = _seq1; - (*seq)->buffer = data; (*seq)->gain = peakDetect(*autocorr, &toa, NULL); /* For 1 sps only |