<p><a href="https://gerrit.osmocom.org/12180">View Change</a></p><p>3 comments:</p><ul style="list-style: none; padding: 0;"><li style="margin: 0; padding: 0;"><p><a href="https://gerrit.osmocom.org/#/c/12180/1//COMMIT_MSG">Commit Message:</a></p><ul style="list-style: none; padding: 0;"><li style="margin: 0; padding: 0 0 0 16px;"><p style="margin-bottom: 4px;"><a href="https://gerrit.osmocom.org/#/c/12180/1//COMMIT_MSG@19">Patch Set #1, Line 19:</a> <code style="font-family:monospace,monospace">rsl.c:2103 (bts=0,trx=0,ts=3,ss=0) Tx PDCH DEACT ACK</code></p><p><blockquote style="border-left: 1px solid #aaa; margin: 10px 0; padding: 0 10px;">one thing I don't understand yet in the overall failure cause: is it an osmo-bsc bug? is it sending  […]</blockquote></p><p style="white-space: pre-wrap; word-wrap: break-word;">no, in here I'm just showing that scenario 2 still works fine with and without this patch (BTS sends a PDCH DEACT ACK, so no need to touch that code path).</p></li></ul></li><li style="margin: 0; padding: 0;"><p><a href="https://gerrit.osmocom.org/#/c/12180/1/src/common/rsl.c">File src/common/rsl.c:</a></p><ul style="list-style: none; padding: 0;"><li style="margin: 0; padding: 0 0 0 16px;"><p style="margin-bottom: 4px;"><a href="https://gerrit.osmocom.org/#/c/12180/1/src/common/rsl.c@2207">Patch Set #1, Line 2207:</a> <code style="font-family:monospace,monospace">            /* TCH takes preference over PDCH so allow forcing PDCH DEACT,</code></p><p><blockquote style="border-left: 1px solid #aaa; margin: 10px 0; padding: 0 10px;">plz explain: if a TS is in PDCH mode, sending a DEACT is not forcing anything. […]</blockquote></p><p style="white-space: pre-wrap; word-wrap: break-word;">From looking at BTS code, it looks like PDCH in general is expected to be !NONE too, but I didn't go into detail since wasn't needed to fix the issue (the scenario is quite narrow and the code was more or less already there before I touched it).</p><p style="white-space: pre-wrap; word-wrap: break-word;">Maybe PDCH is !NONE if there's some TBF active or similar, I don't know tbh.</p></li><li style="margin: 0; padding: 0 0 0 16px;"><p style="margin-bottom: 4px;"><a href="https://gerrit.osmocom.org/#/c/12180/1/src/common/rsl.c@2208">Patch Set #1, Line 2208:</a> <code style="font-family:monospace,monospace">           * but forbid forcing PDCH ACT if lchan still active */</code></p><p><blockquote style="border-left: 1px solid #aaa; margin: 10px 0; padding: 0 10px;">(say it without the word "forcing"?)</blockquote></p><p style="white-space: pre-wrap; word-wrap: break-word;">If we allowed it, we would force the BTS to drop an active channel which may be in use in order to switch it to PDCH.To me that's forcing. Otherwise it would go through another code path.</p></li></ul></li></ul><p>To view, visit <a href="https://gerrit.osmocom.org/12180">change 12180</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/12180"/><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-MessageType: comment </div>
<div style="display:none"> Gerrit-Change-Id: I6d6d12aec10c801fe55012ca6e58d0bc8755b15d </div>
<div style="display:none"> Gerrit-Change-Number: 12180 </div>
<div style="display:none"> Gerrit-PatchSet: 1 </div>
<div style="display:none"> Gerrit-Owner: Pau Espin Pedrol <pespin@sysmocom.de> </div>
<div style="display:none"> Gerrit-Reviewer: Jenkins Builder (1000002) </div>
<div style="display:none"> Gerrit-Reviewer: Neels Hofmeyr <nhofmeyr@sysmocom.de> </div>
<div style="display:none"> Gerrit-Reviewer: Pau Espin Pedrol <pespin@sysmocom.de> </div>
<div style="display:none"> Gerrit-Comment-Date: Fri, 07 Dec 2018 10:26:27 +0000 </div>
<div style="display:none"> Gerrit-HasComments: Yes </div>
<div style="display:none"> Gerrit-HasLabels: No </div>