Change in osmo-bts[master]: rsl: exclude disabled timeslots from interference reports

This is merely a historical archive of years 2008-2021, before the migration to mailman3.

A maintained and still updated list archive can be found at https://lists.osmocom.org/hyperkitty/list/gerrit-log@lists.osmocom.org/.

fixeria gerrit-no-reply at lists.osmocom.org
Tue Nov 9 12:28:19 UTC 2021


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


Change subject: rsl: exclude disabled timeslots from interference reports
......................................................................

rsl: exclude disabled timeslots from interference reports

It may happen after the A-bis connection recovery that the RF RESource
INDication message gets sent too early, while some timeslots are not
yet configured.  This confuses the BSC and provokes error messages.

Change-Id: I00bc6fe67ea1bbedcd5d8640e73bd8b16b9e667f
Related: SYS#4971
---
M src/common/l1sap.c
M src/common/pcu_sock.c
M src/common/rsl.c
3 files changed, 14 insertions(+), 0 deletions(-)



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

diff --git a/src/common/l1sap.c b/src/common/l1sap.c
index 8dee209..502bcef 100644
--- a/src/common/l1sap.c
+++ b/src/common/l1sap.c
@@ -597,6 +597,11 @@
 	for (tn = 0; tn < ARRAY_SIZE(trx->ts); tn++) {
 		struct gsm_bts_trx_ts *ts = &trx->ts[tn];
 
+		if (ts->mo.nm_state.operational != NM_OPSTATE_ENABLED)
+			continue;
+		if (ts->mo.nm_state.availability != NM_AVSTATE_OK)
+			continue;
+
 		for (ln = 0; ln < ARRAY_SIZE(ts->lchan); ln++) {
 			struct gsm_lchan *lchan = &ts->lchan[ln];
 
diff --git a/src/common/pcu_sock.c b/src/common/pcu_sock.c
index f8a0f55..0018acb 100644
--- a/src/common/pcu_sock.c
+++ b/src/common/pcu_sock.c
@@ -579,6 +579,10 @@
 		const struct gsm_bts_trx_ts *ts = &trx->ts[tn];
 		const struct gsm_lchan *lchan = &ts->lchan[0];
 
+		if (ts->mo.nm_state.operational != NM_OPSTATE_ENABLED)
+			continue;
+		if (ts->mo.nm_state.availability != NM_AVSTATE_OK)
+			continue;
 		if (ts_pchan(ts) != GSM_PCHAN_PDCH)
 			continue;
 
diff --git a/src/common/rsl.c b/src/common/rsl.c
index 4c84f21..9b73869 100644
--- a/src/common/rsl.c
+++ b/src/common/rsl.c
@@ -442,6 +442,11 @@
 	for (tn = 0; tn < ARRAY_SIZE(trx->ts); tn++) {
 		const struct gsm_bts_trx_ts *ts = &trx->ts[tn];
 
+		if (ts->mo.nm_state.operational != NM_OPSTATE_ENABLED)
+			continue;
+		if (ts->mo.nm_state.availability != NM_AVSTATE_OK)
+			continue;
+
 		for (ln = 0; ln < ARRAY_SIZE(ts->lchan); ln++) {
 			const struct gsm_lchan *lchan = &ts->lchan[ln];
 

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

Gerrit-Project: osmo-bts
Gerrit-Branch: master
Gerrit-Change-Id: I00bc6fe67ea1bbedcd5d8640e73bd8b16b9e667f
Gerrit-Change-Number: 26178
Gerrit-PatchSet: 1
Gerrit-Owner: fixeria <vyanitskiy at sysmocom.de>
Gerrit-MessageType: newchange
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.osmocom.org/pipermail/gerrit-log/attachments/20211109/2f529241/attachment.htm>


More information about the gerrit-log mailing list