<p>laforge <strong>uploaded patch set #3</strong> to the change originally created by pespin.</p><p><a href="https://gerrit.osmocom.org/c/osmo-pcu/+/26251">View Change</a></p><pre style="font-family: monospace,monospace; white-space: pre-wrap;">T_defs_pcu: Set default val for X2000 to 0 ms<br><br>That timer is really only useful to free the tbf asynchronously after<br>generating the Pkt Access Reject message, since we have nothing to do<br>with it after the message is sent, and the dummt TBF doesn't really hold<br>any reserved resource such as USF or TFI.<br>The timer is useful to still do the freeing asyncrhonously, since the<br>scheduler is interacting with the TBF during the code path, but there's<br>no real need to keep the object alive for 2 ms afterwards. Having a<br>default value of 0 ms is enough, since it fullfills the requirement<br>of freeing asnchronously.<br><br>The value of 2 ms was set initially when the reject support was added<br>here (e9a138e11111f509f988807bbdc5ca8cce2b3d3d), with no specific<br>explanation on the 2 ms value. It was just probably picked as a<br>convinience one, but 0 is actually more convinient.<br><br>Change-Id: I60e34e643f5c9d9afaf85530c54ab3232dc8f0be<br>---<br>M src/gprs_pcu.c<br>1 file changed, 1 insertion(+), 1 deletion(-)<br><br></pre><pre style="font-family: monospace,monospace; white-space: pre-wrap;">git pull ssh://gerrit.osmocom.org:29418/osmo-pcu refs/changes/51/26251/3</pre><p>To view, visit <a href="https://gerrit.osmocom.org/c/osmo-pcu/+/26251">change 26251</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/+/26251"/><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: I60e34e643f5c9d9afaf85530c54ab3232dc8f0be </div>
<div style="display:none"> Gerrit-Change-Number: 26251 </div>
<div style="display:none"> Gerrit-PatchSet: 3 </div>
<div style="display:none"> Gerrit-Owner: pespin <pespin@sysmocom.de> </div>
<div style="display:none"> Gerrit-Reviewer: Jenkins Builder </div>
<div style="display:none"> Gerrit-Reviewer: pespin <pespin@sysmocom.de> </div>
<div style="display:none"> Gerrit-MessageType: newpatchset </div>