Change in osmo-msc[master]: Convert paging response timer into an osmocom own X4 timer

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/.

lynxis lazus gerrit-no-reply at lists.osmocom.org
Fri Sep 25 03:29:15 UTC 2020


lynxis lazus has uploaded this change for review. ( https://gerrit.osmocom.org/c/osmo-msc/+/20280 )


Change subject: Convert paging response timer into an osmocom own X4 timer
......................................................................

Convert paging response timer into an osmocom own X4 timer

All timeout values should use tdef.

Change-Id: Ic050eb49ba0c5811b43d8d8b44145a1115fd8546
---
M include/osmocom/msc/gsm_data.h
M src/libmsc/msc_net_init.c
M src/libmsc/msc_vty.c
M src/libmsc/paging.c
M src/libmsc/ran_infra.c
M tests/test_nodes.vty
6 files changed, 21 insertions(+), 12 deletions(-)



  git pull ssh://gerrit.osmocom.org:29418/osmo-msc refs/changes/80/20280/1

diff --git a/include/osmocom/msc/gsm_data.h b/include/osmocom/msc/gsm_data.h
index 2122d4b..8e084b7 100644
--- a/include/osmocom/msc/gsm_data.h
+++ b/include/osmocom/msc/gsm_data.h
@@ -171,8 +171,6 @@
 	 */
 	struct llist_head trans_list;
 
-	unsigned int paging_response_timer;
-
 	/* Radio Resource Location Protocol (TS 04.31) */
 	struct {
 		enum rrlp_mode mode;
diff --git a/src/libmsc/msc_net_init.c b/src/libmsc/msc_net_init.c
index adb9ca7..d5cd1bd 100644
--- a/src/libmsc/msc_net_init.c
+++ b/src/libmsc/msc_net_init.c
@@ -70,8 +70,6 @@
 	net->mncc_guard_timeout = 180;
 	net->ncss_guard_timeout = 30;
 
-	net->paging_response_timer = MSC_PAGING_RESPONSE_TIMER_DEFAULT;
-
 	INIT_LLIST_HEAD(&net->trans_list);
 	INIT_LLIST_HEAD(&net->upqueue);
 	INIT_LLIST_HEAD(&net->neighbor_ident_list);
diff --git a/src/libmsc/msc_vty.c b/src/libmsc/msc_vty.c
index 29deb5e..95919b1 100644
--- a/src/libmsc/msc_vty.c
+++ b/src/libmsc/msc_vty.c
@@ -67,6 +67,7 @@
 #include <osmocom/msc/sgs_vty.h>
 #include <osmocom/msc/sccp_ran.h>
 #include <osmocom/msc/ran_peer.h>
+#include <osmocom/msc/ran_infra.h>
 
 static struct gsm_network *gsmnet = NULL;
 
@@ -553,7 +554,7 @@
 	return CMD_SUCCESS;
 }
 
-DEFUN(cfg_msc_paging_response_timer, cfg_msc_paging_response_timer_cmd,
+DEFUN_DEPRECATED(cfg_msc_paging_response_timer, cfg_msc_paging_response_timer_cmd,
       "paging response-timer (default|<1-65535>)",
       "Configure Paging\n"
       "Set Paging timeout, the minimum time to pass between (unsuccessful) Pagings sent towards"
@@ -561,10 +562,22 @@
       "Set to default timeout (" OSMO_STRINGIFY_VAL(MSC_PAGING_RESPONSE_TIMER_DEFAULT) " seconds)\n"
       "Set paging timeout in seconds\n")
 {
+	int rat;
+	int paging_response_timer;
 	if (!strcmp(argv[0], "default"))
-		gsmnet->paging_response_timer = MSC_PAGING_RESPONSE_TIMER_DEFAULT;
+		paging_response_timer = MSC_PAGING_RESPONSE_TIMER_DEFAULT;
 	else
-		gsmnet->paging_response_timer = atoi(argv[0]);
+		paging_response_timer = atoi(argv[0]);
+
+	for (rat=0; rat<OSMO_RAT_COUNT; rat++) {
+		osmo_tdef_set(msc_ran_infra[rat].tdefs, -4, paging_response_timer, OSMO_TDEF_S);
+	}
+
+	vty_out(vty, "%% paging response-timer is deprecated.%s"
+		"%% All ran timer has been modified.%s"
+		"%% use 'timer <geran|utran|sgs> X4 %s' instead%s",
+		VTY_NEWLINE, VTY_NEWLINE, argv[0], VTY_NEWLINE);
+
 	return CMD_SUCCESS;
 }
 
@@ -762,9 +775,6 @@
 			vty_out(vty, " check-imei-rqd 1%s", VTY_NEWLINE);
 	}
 
-	if (gsmnet->paging_response_timer != MSC_PAGING_RESPONSE_TIMER_DEFAULT)
-		vty_out(vty, " paging response-timer %u%s", gsmnet->paging_response_timer, VTY_NEWLINE);
-
 	if (gsmnet->emergency.route_to_msisdn) {
 		vty_out(vty, " emergency-call route-to-msisdn %s%s",
 			gsmnet->emergency.route_to_msisdn, VTY_NEWLINE);
diff --git a/src/libmsc/paging.c b/src/libmsc/paging.c
index b288414..5e1f154 100644
--- a/src/libmsc/paging.c
+++ b/src/libmsc/paging.c
@@ -84,6 +84,7 @@
 {
 	int rc;
 	struct paging_request *pr;
+	int paging_response_timer;
 	struct gsm_network *net = vsub->vlr->user_ctx;
 
 	pr = talloc(vsub, struct paging_request);
@@ -110,8 +111,9 @@
 		/* reduced on the first paging callback */
 		vlr_subscr_get(vsub, VSUB_USE_PAGING);
 		vsub->cs.is_paging = true;
+		paging_response_timer = osmo_tdef_get(msc_ran_infra[vsub->cs.attached_via_ran].tdefs, -4, OSMO_TDEF_S, 10);
 		osmo_timer_setup(&vsub->cs.paging_response_timer, paging_response_timer_cb, vsub);
-		osmo_timer_schedule(&vsub->cs.paging_response_timer, net->paging_response_timer, 0);
+		osmo_timer_schedule(&vsub->cs.paging_response_timer, paging_response_timer, 0);
 	}
 
 	llist_add_tail(&pr->entry, &vsub->cs.requests);
diff --git a/src/libmsc/ran_infra.c b/src/libmsc/ran_infra.c
index af40541..c0eaa1a 100644
--- a/src/libmsc/ran_infra.c
+++ b/src/libmsc/ran_infra.c
@@ -43,6 +43,7 @@
 	{ .T = -1, .default_val = 5, .desc = "RAN connection Complete Layer 3, Authentication and Ciphering timeout" }, \
 	{ .T = -2, .default_val = 30, .desc = "RAN connection release sanity timeout" }, \
 	{ .T = -3, .default_val = 10, .desc = "Timeout to find a target BSS after Handover Required" }, \
+	{ .T = -4, .default_val = 10, .desc = "Paging response timeout" }, \
 
 struct osmo_tdef msc_tdefs_geran[] = {
 	RAN_TDEFS
@@ -55,6 +56,7 @@
 };
 
 struct osmo_tdef msc_tdefs_sgs[] = {
+	{ .T = -4, .default_val = 10, .desc = "Paging response timeout" },
 	{}
 };
 
diff --git a/tests/test_nodes.vty b/tests/test_nodes.vty
index 520f07c..f86b96c 100644
--- a/tests/test_nodes.vty
+++ b/tests/test_nodes.vty
@@ -57,7 +57,6 @@
   check-imei-rqd (0|1|early)
   cs7-instance-a <0-15>
   cs7-instance-iu <0-15>
-  paging response-timer (default|<1-65535>)
   emergency-call route-to-msisdn MSISDN
   sms-over-gsup
   no sms-over-gsup

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

Gerrit-Project: osmo-msc
Gerrit-Branch: master
Gerrit-Change-Id: Ic050eb49ba0c5811b43d8d8b44145a1115fd8546
Gerrit-Change-Number: 20280
Gerrit-PatchSet: 1
Gerrit-Owner: lynxis lazus <lynxis at fe80.eu>
Gerrit-MessageType: newchange
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.osmocom.org/pipermail/gerrit-log/attachments/20200925/bf2f463c/attachment.htm>


More information about the gerrit-log mailing list