<p>fixeria <strong>uploaded patch set #4</strong> to this change.</p><p><a href="https://gerrit.osmocom.org/c/osmo-bts/+/26046">View Change</a></p><pre style="font-family: monospace,monospace; white-space: pre-wrap;">l1sap: rework handling of DATA.ind on SACCH<br><br>Currently an Uplink SACCH block is being passed to LAPDm first, and<br>then gets forwareded to the BSC in handle_ms_meas_report(), together<br>with the Uplink measurements collected so far.<br><br>This approach has a serious flaw: handle_ms_meas_report() won't be<br>called if an Uplink block contains SAPI=3 data (SMS) or was not<br>decoded at all (len=0) fow whatever reason.  Therefore, no RSL<br>MEASurement RESult message will be sent to the BSC.<br><br>Rename handle_ms_meas_report() to lchan_meas_handle_sacch(), and call<br>it from l1sap_ph_data_ind().  This way perioduc RSL MEASurement RESult<br>messages will be sent regardless of what happens on Uplink SACCH.<br><br>Change-Id: Ifed91f87fd653debc87a09da3fd31ad64a13f330<br>Fixes: TC_meas_res_speech_{tchf,tchh}_sapi3<br>Related: SYS#5319<br>---<br>M include/osmo-bts/measurement.h<br>M src/common/l1sap.c<br>M src/common/measurement.c<br>M src/common/rsl.c<br>4 files changed, 84 insertions(+), 74 deletions(-)<br><br></pre><pre style="font-family: monospace,monospace; white-space: pre-wrap;">git pull ssh://gerrit.osmocom.org:29418/osmo-bts refs/changes/46/26046/4</pre><p>To view, visit <a href="https://gerrit.osmocom.org/c/osmo-bts/+/26046">change 26046</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/+/26046"/><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: Ifed91f87fd653debc87a09da3fd31ad64a13f330 </div>
<div style="display:none"> Gerrit-Change-Number: 26046 </div>
<div style="display:none"> Gerrit-PatchSet: 4 </div>
<div style="display:none"> Gerrit-Owner: fixeria <vyanitskiy@sysmocom.de> </div>
<div style="display:none"> Gerrit-Reviewer: Jenkins Builder </div>
<div style="display:none"> Gerrit-Reviewer: laforge <laforge@osmocom.org> </div>
<div style="display:none"> Gerrit-Reviewer: pespin <pespin@sysmocom.de> </div>
<div style="display:none"> Gerrit-MessageType: newpatchset </div>