<p>laforge <strong>submitted</strong> this change.</p><p><a href="https://gerrit.osmocom.org/c/osmo-pcu/+/26251">View Change</a></p><div style="white-space:pre-wrap">Approvals:
  laforge: Looks good to me, approved
  pespin: Verified

</div><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;"><span>diff --git a/src/gprs_pcu.c b/src/gprs_pcu.c</span><br><span>index 8b8e46b..76dd5be 100644</span><br><span>--- a/src/gprs_pcu.c</span><br><span>+++ b/src/gprs_pcu.c</span><br><span>@@ -39,7 +39,7 @@</span><br><span>    { .T=PCU_TDEF_SI_CACHE_ALIVE,      .default_val=5,  .unit=OSMO_TDEF_S,   .desc="[RAC+CI]->[SI] resolution cache entry storage timeout (s)", .val=0 },</span><br><span>   { .T=-101,  .default_val=30,  .unit=OSMO_TDEF_S,  .desc="BSSGP (un)blocking procedures timer (s)",  .val=0 },</span><br><span>      { .T=-102,  .default_val=30,  .unit=OSMO_TDEF_S,  .desc="BSSGP reset procedure timer (s)",          .val=0 },</span><br><span style="color: hsl(0, 100%, 40%);">- { .T=-2000, .default_val=2,   .unit=OSMO_TDEF_MS, .desc="Delay release of UL TBF after tx Packet Access Reject (PACCH) (ms)", .val=0 },</span><br><span style="color: hsl(120, 100%, 40%);">+     { .T=-2000, .default_val=0,   .unit=OSMO_TDEF_MS, .desc="Delay release of UL TBF after tx Packet Access Reject (PACCH) (ms)", .val=0 },</span><br><span>    { .T=-2001, .default_val=2,   .unit=OSMO_TDEF_S,  .desc="PACCH assignment timeout (s)",               .val=0 },</span><br><span>    { .T=-2002, .default_val=200, .unit=OSMO_TDEF_MS, .desc="Waiting after IMM.ASS confirm timer (ms)", .val=0 },</span><br><span>      { .T=-2030, .default_val=60,  .unit=OSMO_TDEF_S,  .desc="Time to keep an idle MS object alive (s)", .val=0 }, /* slightly above T3314 (default 44s, 24.008, 11.2.2) */</span><br><span></span><br></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: 4 </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: laforge <laforge@osmocom.org> </div>
<div style="display:none"> Gerrit-Reviewer: pespin <pespin@sysmocom.de> </div>
<div style="display:none"> Gerrit-MessageType: merged </div>