Change in ...osmo-bts[master]: revert changes to T200

Hoernchen gerrit-no-reply at lists.osmocom.org
Fri Jun 21 16:38:40 UTC 2019


Hoernchen has uploaded this change for review. ( https://gerrit.osmocom.org/c/osmo-bts/+/14567


Change subject: revert changes to T200
......................................................................

revert changes to T200

The timers are unfortunately completely broken, so let's go back to the
long default timeout values.

Change-Id: Ia44310245a348675dbbf3ffc3dc5b6d207fd62d3
---
M src/common/oml.c
1 file changed, 17 insertions(+), 7 deletions(-)



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

diff --git a/src/common/oml.c b/src/common/oml.c
index 8ecafbb..3defa49 100644
--- a/src/common/oml.c
+++ b/src/common/oml.c
@@ -479,13 +479,13 @@
 /* The defaults below correspond to various sources/recommendations that could be found online.
  * The BSC should override this via OML anyway. */
 const unsigned int oml_default_t200_ms[7] = {
-	[T200_SDCCH]		=  220,
-	[T200_SDCCH_SAPI3]	=  450,
-	[T200_SACCH_SDCCH]	=  900,
-	[T200_FACCH_F]		=  250,
-	[T200_FACCH_H]		=  250,
-	[T200_SACCH_TCH_SAPI0]	= 1500,
-	[T200_SACCH_TCH_SAPI3]	= 1500,
+        [T200_SDCCH]		= 1000,
+        [T200_FACCH_F]		= 1000,
+        [T200_FACCH_H]		= 1000,
+        [T200_SACCH_TCH_SAPI0]	= 2000,
+        [T200_SACCH_SDCCH]	= 2000,
+        [T200_SDCCH_SAPI3]	= 1000,
+        [T200_SACCH_TCH_SAPI3]	= 2000,
 };
 
 /* 3GPP TS 52.021 §8.11.1 Get Attributes has been received */
@@ -626,10 +626,20 @@
 		payload = TLVP_VAL(&tp, NM_ATT_T200);
 		for (i = 0; i < ARRAY_SIZE(bts->t200_ms); i++) {
 			uint32_t t200_ms = payload[i] * abis_nm_t200_ms[i];
+#if 0
 			bts->t200_ms[i] = t200_ms;
 			DEBUGPFOH(DOML, foh, "T200[%u]: OML=%u, mult=%u => %u ms\n",
 				  i, payload[i], abis_nm_t200_ms[i],
 				  bts->t200_ms[i]);
+#else
+                        /* we'd rather use the 1s/2s (long) defaults by
+                         * libosmocore, as we appear to have some bug(s)
+                         * related to handling T200 expiration in
+                         * libosmogsm lapd(m) code? */
+                        LOGPFOH(DOML, LOGL_NOTICE, foh, "Ignoring T200[%u] (%u ms) "
+                                "as sent by BSC due to suspected LAPDm bug!\n",
+                                i, t200_ms);
+#endif
 		}
 	}
 

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

Gerrit-Project: osmo-bts
Gerrit-Branch: master
Gerrit-Change-Id: Ia44310245a348675dbbf3ffc3dc5b6d207fd62d3
Gerrit-Change-Number: 14567
Gerrit-PatchSet: 1
Gerrit-Owner: Hoernchen <ewild at sysmocom.de>
Gerrit-MessageType: newchange
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.osmocom.org/pipermail/gerrit-log/attachments/20190621/c45f3e26/attachment.html>


More information about the gerrit-log mailing list