<p>Vadim Yanitskiy <strong>uploaded patch set #3</strong> to this change.</p><p><a href="https://gerrit.osmocom.org/11138">View Change</a></p><pre style="font-family: monospace,monospace; white-space: pre-wrap;">trxcon/scheduler: fix Measurement Reporting on SACCH<br><br>According to 3GPP TS 04.08, section 3.4.1, SACCH logical channel<br>accompanies either a traffic or a signaling channel. It has the<br>particularity that continuous transmission must occur in both<br>directions, so on the Uplink direction measurement result messages<br>are sent at each possible occasion when nothing else has to be sent.<br>The LAPDm fill frames (0x01, 0x03, 0x01, 0x2b, ...) are not<br>applicable on SACCH channels!<br><br>Unfortunately, 3GPP TS 04.08 doesn't clearly state which "else<br>messages" besides Measurement Reports can be send by the MS on<br>SACCH channels. However, in sub-clause 3.4.1 it's stated that<br>the interval between two successive measurement result messages<br>shall not exceed one L2 frame.<br><br>This change introduces a separate handler for SACCH primitives,<br>which dequeues a SACCH primitive from transmit queue, if present.<br>Otherwise it dequeues a cached Measurement Report (the last<br>received one). Finally, if the cache is empty, a "dummy"<br>measurement report is used. When it's possible,<br>a non-MR primitive is prioritized.<br><br>Change-Id: If1b8dc74ced746d6270676fdde75fcda32f91a3d<br>Related: OS#2988<br>---<br>M src/host/trxcon/sched_prim.c<br>M src/host/trxcon/sched_trx.c<br>M src/host/trxcon/sched_trx.h<br>3 files changed, 204 insertions(+), 14 deletions(-)<br><br></pre><pre style="font-family: monospace,monospace; white-space: pre-wrap;">git pull ssh://gerrit.osmocom.org:29418/osmocom-bb refs/changes/38/11138/3</pre><p>To view, visit <a href="https://gerrit.osmocom.org/11138">change 11138</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/11138"/><meta itemprop="name" content="View Change"/></div></div>

<div style="display:none"> Gerrit-Project: osmocom-bb </div>
<div style="display:none"> Gerrit-Branch: master </div>
<div style="display:none"> Gerrit-MessageType: newpatchset </div>
<div style="display:none"> Gerrit-Change-Id: If1b8dc74ced746d6270676fdde75fcda32f91a3d </div>
<div style="display:none"> Gerrit-Change-Number: 11138 </div>
<div style="display:none"> Gerrit-PatchSet: 3 </div>
<div style="display:none"> Gerrit-Owner: Vadim Yanitskiy <axilirator@gmail.com> </div>
<div style="display:none"> Gerrit-Reviewer: Harald Welte <laforge@gnumonks.org> </div>
<div style="display:none"> Gerrit-Reviewer: Jenkins Builder (1000002) </div>
<div style="display:none"> Gerrit-Reviewer: Pau Espin Pedrol <pespin@sysmocom.de> </div>
<div style="display:none"> Gerrit-Reviewer: Vadim Yanitskiy <axilirator@gmail.com> </div>