<p>pespin has uploaded this change for <strong>review</strong>.</p><p><a href="https://gerrit.osmocom.org/c/osmo-bts/+/26113">View Change</a></p><pre style="font-family: monospace,monospace; white-space: pre-wrap;">gsm_ts_release(): Make sure pchan{,is_want} is reset to NONE<br><br>Let's make sure all ts->pchan* related data is reset when we release the<br>TS. This is important for pchan_is, since in bts-trx upon shutdown<br>finish the PDCH lchan of a osmo dyn TS is set to<br>lchan->state=LCHAN_S_NONE, and as a result when the PCU tries to release<br>it by means of PCUIF act_req later, it may be ignored since the lchan is<br>already in erased state.<br><br>Related: SYS#4971<br>Change-Id: Ic7c34c0de23101ce094ffd03e00b4d2f6a551152<br>---<br>M src/common/gsm_data.c<br>1 file changed, 4 insertions(+), 0 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/13/26113/1</pre><pre style="font-family: monospace,monospace; white-space: pre-wrap;"><span>diff --git a/src/common/gsm_data.c b/src/common/gsm_data.c</span><br><span>index 4446958..e5dbf10 100644</span><br><span>--- a/src/common/gsm_data.c</span><br><span>+++ b/src/common/gsm_data.c</span><br><span>@@ -314,4 +314,8 @@</span><br><span>            struct gsm_lchan *lchan = &ts->lchan[ln];</span><br><span>             gsm_lchan_release(lchan, LCHAN_REL_ACT_OML);</span><br><span>         }</span><br><span style="color: hsl(120, 100%, 40%);">+     ts->pchan = GSM_PCHAN_NONE;</span><br><span style="color: hsl(120, 100%, 40%);">+        /* Make sure pchan_is is reset, since PCU act_req to release it will be</span><br><span style="color: hsl(120, 100%, 40%);">+        * ignored as the lchan will already be released. */</span><br><span style="color: hsl(120, 100%, 40%);">+  ts->dyn.pchan_is = ts->dyn.pchan_want = GSM_PCHAN_NONE;</span><br><span> }</span><br><span></span><br></pre><p>To view, visit <a href="https://gerrit.osmocom.org/c/osmo-bts/+/26113">change 26113</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/+/26113"/><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: Ic7c34c0de23101ce094ffd03e00b4d2f6a551152 </div>
<div style="display:none"> Gerrit-Change-Number: 26113 </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>