Change in osmo-bts[master]: bts-trx: Fix assert hit when rf_locked in .cfg and TS TCH/F_PDCH

[pespin]

pespin has uploaded this change for review. ( https://gerrit.osmocom.org/c/osmo-bts/+/19407 )


Change subject: bts-trx: Fix assert hit when rf_locked in .cfg and TS TCH/F_PDCH
......................................................................

bts-trx: Fix assert hit when rf_locked in .cfg and TS TCH/F_PDCH

If TRX is administratively locked during startup, for TS conigured as
TCH/F_PDCH the BSC will send a ACT PDCH message, but osmo-bts-trx won't
apply it by signalling it as available to the PCU, since the TRX is
locked.
That means the ts->flags contains the pending action to activate
it until it is unlocked. As a result, calling trx_set_ts() on it was
hitting the assert inside which expects not to apply the TS until it has
been confirmed by the PCU.
Let's still skip setting the TS and let pcu_tx_info_ind() trigger the
activation confirmation from PCU, since the TRX has just been unlocked.

Fixes following assert:
Assert failed (ts->flags & TS_F_PDCH_PENDING_MASK) == 0 /osmo-bts/src/osmo-bts-trx/l1_if.c:34

Change-Id: Ie3cad15d31870346d03a6e2f6dd32a9d2dd3067e
---
M src/osmo-bts-trx/l1_if.c
1 file changed, 7 insertions(+), 0 deletions(-)



  git pull ssh://gerrit.osmocom.org:29418/osmo-bts refs/changes/07/19407/1

diff --git a/src/osmo-bts-trx/l1_if.c b/src/osmo-bts-trx/l1_if.c
index 48305b5..a2b3415 100644
--- a/src/osmo-bts-trx/l1_if.c
+++ b/src/osmo-bts-trx/l1_if.c
@@ -702,6 +702,13 @@
 					 * internally restore the old status here.
 					 */
 					ts->dyn.pchan_is = ts->dyn.pchan_want;
+				} else if (ts->pchan == GSM_PCHAN_TCH_F_PDCH && ts->flags & TS_F_PDCH_PENDING_MASK) {
+					/* TS configuration already in progress,
+					 * waiting for PCU response, let it be
+					 * de/activated later by PCU ACT CNF as a
+					 * response to pcu_tx_info_ind()
+					 */
+					continue;
 				}
 				trx_set_ts(ts);
 			}

-- 
To view, visit https://gerrit.osmocom.org/c/osmo-bts/+/19407
To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings

Gerrit-Project: osmo-bts
Gerrit-Branch: master
Gerrit-Change-Id: Ie3cad15d31870346d03a6e2f6dd32a9d2dd3067e
Gerrit-Change-Number: 19407
Gerrit-PatchSet: 1
Gerrit-Owner: pespin <pespin at sysmocom.de>
Gerrit-MessageType: newchange
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.osmocom.org/pipermail/gerrit-log/attachments/20200727/482a02e0/attachment.htm>