<blockquote style="border-left: 1px solid #aaa; margin: 10px 0; padding: 0 10px;"><p style="white-space: pre-wrap; word-wrap: break-word;">Patch Set 1:</p><blockquote style="border-left: 1px solid #aaa; margin: 10px 0; padding: 0 10px;"><p style="white-space: pre-wrap; word-wrap: break-word;">I was thinking about using struct l1sched_meas_set, but I was not sure if I should touch it. I also thought maybe using this then as a replacement for the remap functions too but this should be a follow up patch then. Ok, then I use struct l1sched_meas_set to keep track of the FN.</p></blockquote><p style="white-space: pre-wrap; word-wrap: break-word;">The problem is that this is a 'hot' code part that is executed quite often, so we should avoid adding more buffers and syscalls.  Your remap functions would still be useful, because in some cases the ring  buffer may not contain some samples (e.g. right after the channel activation).</p></blockquote><p style="white-space: pre-wrap; word-wrap: break-word;">I found a way to calculate the frame number from two bursts before. I have tested it and the results were the same as the ones I got before with the shift buffer. So we need no buffering at all now but I still use memset() and memcpy() to copy the measurement results around. I technically need only meas_avg.toa256 and meas_avg.rssi, so I could optimize this out, if necessary.</p><p><a href="https://gerrit.osmocom.org/c/osmo-bts/+/21329">View Change</a></p><ul style="list-style: none; padding: 0;"></ul><p>To view, visit <a href="https://gerrit.osmocom.org/c/osmo-bts/+/21329">change 21329</a>. To unsubscribe, or for help writing mail filters, visit <a href="https://gerrit.osmocom.org/settings">settings</a>.</p><div itemscope itemtype="http://schema.org/EmailMessage"><div itemscope itemprop="action" itemtype="http://schema.org/ViewAction"><link itemprop="url" href="https://gerrit.osmocom.org/c/osmo-bts/+/21329"/><meta itemprop="name" content="View Change"/></div></div>

<div style="display:none"> Gerrit-Project: osmo-bts </div>
<div style="display:none"> Gerrit-Branch: master </div>
<div style="display:none"> Gerrit-Change-Id: I1ad9fa3815feb2b4da608ab7df716a87ba1f2f91 </div>
<div style="display:none"> Gerrit-Change-Number: 21329 </div>
<div style="display:none"> Gerrit-PatchSet: 2 </div>
<div style="display:none"> Gerrit-Owner: dexter <pmaier@sysmocom.de> </div>
<div style="display:none"> Gerrit-Reviewer: Jenkins Builder </div>
<div style="display:none"> Gerrit-Reviewer: fixeria <vyanitskiy@sysmocom.de> </div>
<div style="display:none"> Gerrit-Comment-Date: Sun, 29 Nov 2020 20:38:32 +0000 </div>
<div style="display:none"> Gerrit-HasComments: No </div>
<div style="display:none"> Gerrit-Has-Labels: No </div>
<div style="display:none"> Gerrit-MessageType: comment </div>