<p>pespin has uploaded this change for <strong>review</strong>.</p><p><a href="https://gerrit.osmocom.org/c/osmo-pcu/+/23871">View Change</a></p><pre style="font-family: monospace,monospace; white-space: pre-wrap;">pdch: tbf_by_tfi(): Allow returning TBFs in state RELEASING<br><br>During RELEASING state the TFI, USFs, etc. are still reserved and<br>assigned to the TBF, and hence the TBF may still use it.<br>If callers of this function rely on not taking TBFs under RELEASING<br>state, they should check that explicitly.<br><br>It still makes sense being to operate on RELEASING TBFs, since under<br>some circumstances the TBF may go under a previous state. See for<br>instance 3GPP TS 44.060 sec 8.1.1.3a.2:<br><br>"""<br>If N3101 reaches the value N3101max, the network shall stop sending<br>PACKET UPLINK ACK/NACK messages to the mobile station for that TBF<br>and shall start timer T3169 for the TBF. If an RLC/MAC block is received<br>from the TBF when timer T3169 is running, the network shall stop timer<br>T3169 and resume sending PACKET UPLINK ACK/NACK messages to the TBF.<br>When T3169 expires, the network may consider the TBF as released and<br>reuse the TFI value.<br>"""<br><br>Change-Id: Ibb471e727388512d42794d3faa26597e2545b852<br>---<br>M src/pdch.cpp<br>1 file changed, 1 insertion(+), 8 deletions(-)<br><br></pre><pre style="font-family: monospace,monospace; white-space: pre-wrap;">git pull ssh://gerrit.osmocom.org:29418/osmo-pcu refs/changes/71/23871/1</pre><pre style="font-family: monospace,monospace; white-space: pre-wrap;"><span>diff --git a/src/pdch.cpp b/src/pdch.cpp</span><br><span>index baac0f7..76ca1d0 100644</span><br><span>--- a/src/pdch.cpp</span><br><span>+++ b/src/pdch.cpp</span><br><span>@@ -1044,14 +1044,7 @@</span><br><span> </span><br><span>       tbf = m_tbfs[dir][tfi];</span><br><span> </span><br><span style="color: hsl(0, 100%, 40%);">-     if (!tbf)</span><br><span style="color: hsl(0, 100%, 40%);">-               return NULL;</span><br><span style="color: hsl(0, 100%, 40%);">-</span><br><span style="color: hsl(0, 100%, 40%);">-    if (tbf->state_is_not(GPRS_RLCMAC_RELEASING)) {</span><br><span style="color: hsl(0, 100%, 40%);">-              return tbf;</span><br><span style="color: hsl(0, 100%, 40%);">-     }</span><br><span style="color: hsl(0, 100%, 40%);">-</span><br><span style="color: hsl(0, 100%, 40%);">-       return NULL;</span><br><span style="color: hsl(120, 100%, 40%);">+  return tbf;</span><br><span> }</span><br><span> </span><br><span> void gprs_rlcmac_pdch::attach_tbf(gprs_rlcmac_tbf *tbf)</span><br><span></span><br></pre><p>To view, visit <a href="https://gerrit.osmocom.org/c/osmo-pcu/+/23871">change 23871</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-pcu/+/23871"/><meta itemprop="name" content="View Change"/></div></div>

<div style="display:none"> Gerrit-Project: osmo-pcu </div>
<div style="display:none"> Gerrit-Branch: master </div>
<div style="display:none"> Gerrit-Change-Id: Ibb471e727388512d42794d3faa26597e2545b852 </div>
<div style="display:none"> Gerrit-Change-Number: 23871 </div>
<div style="display:none"> Gerrit-PatchSet: 1 </div>
<div style="display:none"> Gerrit-Owner: pespin <pespin@sysmocom.de> </div>
<div style="display:none"> Gerrit-MessageType: newchange </div>