<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><p style="white-space: pre-wrap; word-wrap: break-word;">My assumption was wrong, we already do send to BFIs in the current implementation:</p><p style="white-space: pre-wrap; word-wrap: break-word;">https://gerrit.osmocom.org/c/osmo-bts/+/20981 osmo-bts-trx/scheduler: fix comments related to FACCH/H and BFI</p><p style="white-space: pre-wrap; word-wrap: break-word;">so why would we end up with one measurement sample less then? I guess the problem is that one such BFI is ignored by the measurement logic as expected, while another one is not, because it holds a valid RSSI value? During the last call, I proposed to ignore the measurement samples carried by BFIs triggered by FACCH, but what if we take the opposite approach and ignore FACCH samples instead? In other words, you count speech frames as usual, including all BFIs, but do not take FACCH samples into account. This way there would be need to mess up with expected number of measurements, it would always be constant. And of course, this is only in speech mode. In signalling, we count FACCH frames as usual. What do you think?</p></blockquote><p style="white-space: pre-wrap; word-wrap: break-word;">This might be the solution to the problem. I will give it a try.</p><p><a href="https://gerrit.osmocom.org/c/osmo-bts/+/20980">View Change</a></p><p>3 comments:</p><ul style="list-style: none; padding: 0;"><li style="margin: 0; padding: 0;"><p><a href="https://gerrit.osmocom.org/c/osmo-bts/+/20980/1//COMMIT_MSG">Commit Message:</a></p><ul style="list-style: none; padding: 0;"><li style="margin: 0; padding: 0 0 0 16px;"><p style="margin-bottom: 4px;"><a href="https://gerrit.osmocom.org/c/osmo-bts/+/20980/1//COMMIT_MSG@9">Patch Set #1, Line 9:</a> <code style="font-family:monospace,monospace">FACCH channel ist</code></p><p><blockquote style="border-left: 1px solid #aaa; margin: 10px 0; padding: 0 10px;">It's not the 'channel' that is spread over 6 bursts, but a frame, right? […]</blockquote></p><p style="white-space: pre-wrap; word-wrap: break-word;">I think "Block" is more right here. To my knowledge a frame is the 8 timeslots...</p></li><li style="margin: 0; padding: 0 0 0 16px;"><p style="margin-bottom: 4px;"><a href="https://gerrit.osmocom.org/c/osmo-bts/+/20980/1//COMMIT_MSG@11">Patch Set #1, Line 11:</a> <code style="font-family:monospace,monospace">up with one uplink measurement sample less</code></p><p><blockquote style="border-left: 1px solid #aaa; margin: 10px 0; padding: 0 10px;">Yes, and this is not a valid behavior. […]</blockquote></p><p style="white-space: pre-wrap; word-wrap: break-word;">I think you mean a different bug. The BFI frames (blocks) that are handed up are ignored by measurement.c anyway since they are sent with RSSI=0 anyway. If you do the channel mapping drawings, you will see that there is indeed exactly one measurement value less when a FACCH is transmitted. (However, I think there are still bugs, but those affect the timing only.)</p></li></ul></li><li style="margin: 0; padding: 0;"><p><a href="https://gerrit.osmocom.org/c/osmo-bts/+/20980/1/src/common/measurement.c">File src/common/measurement.c:</a></p><ul style="list-style: none; padding: 0;"><li style="margin: 0; padding: 0 0 0 16px;"><p style="margin-bottom: 4px;"><a href="https://gerrit.osmocom.org/c/osmo-bts/+/20980/1/src/common/measurement.c@670">Patch Set #1, Line 670:</a> <code style="font-family:monospace,monospace">Lost %u UL measurements</code></p><p><blockquote style="border-left: 1px solid #aaa; margin: 10px 0; padding: 0 10px;">This looks and sounds confusing: we're not loosing anything...</blockquote></p><p style="white-space: pre-wrap; word-wrap: break-word;">Done</p></li></ul></li></ul><p>To view, visit <a href="https://gerrit.osmocom.org/c/osmo-bts/+/20980">change 20980</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/+/20980"/><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: Ibf693aede8fffa6432cdcdcf5d52910493a1104b </div>
<div style="display:none"> Gerrit-Change-Number: 20980 </div>
<div style="display:none"> Gerrit-PatchSet: 1 </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: Mon, 02 Nov 2020 15:06:23 +0000 </div>
<div style="display:none"> Gerrit-HasComments: Yes </div>
<div style="display:none"> Gerrit-Has-Labels: No </div>
<div style="display:none"> Comment-In-Reply-To: fixeria <vyanitskiy@sysmocom.de> </div>
<div style="display:none"> Gerrit-MessageType: comment </div>