lynxis lazus has uploaded this change for review. ( https://gerrit.osmocom.org/c/osmo-msc/+/38107?usp=email )
Change subject: libvlr: replace direct call of paging_expired() into a callback
......................................................................
libvlr: replace direct call of paging_expired() into a callback
libmsc internal functions shouldn't be called from within the libvlr.
Re-use the paging callback with an osmocom specific indication to
inform it to cancel paging.
Change-Id: I7a3d15e8f0fb51c6b32add2de5024fc4d599ecf0
---
M src/libmsc/sgs_iface.c
M src/libvlr/vlr_sgs_fsm.c
2 files changed, 8 insertions(+), 3 deletions(-)
git pull ssh://gerrit.osmocom.org:29418/osmo-msc refs/changes/07/38107/1
diff --git a/src/libmsc/sgs_iface.c b/src/libmsc/sgs_iface.c
index a845ab8..c1b6b71 100644
--- a/src/libmsc/sgs_iface.c
+++ b/src/libmsc/sgs_iface.c
@@ -466,7 +466,7 @@
}
}
-/*! Page UE through SGs interface
+/*! Page UE through SGs interface or inform about an expired paging
* \param[in] vsub subscriber context
* \param[in] serv_ind service indicator (sms or voide)
* \returns 0 in case of success, -EINVAL in case of error. */
@@ -476,6 +476,11 @@
struct gsm29118_paging_req paging_params;
struct sgs_mme_ctx *mme;
+ if (serv_ind == SGSAP_SERV_IND_PAGING_TIMEOUT) {
+ paging_expired(vsub);
+ return 0;
+ }
+
/* See also: 3GPP TS 29.118, chapter 5.1.2.2 Paging Initiation */
if (vsub->sgs_fsm->state == SGS_UE_ST_NULL && vsub->conf_by_radio_contact_ind == true) {
LOGPFSMSL(vsub->sgs_fsm, DPAG, LOGL_ERROR, "Will not Page (conf_by_radio_contact_ind == true)\n");
diff --git a/src/libvlr/vlr_sgs_fsm.c b/src/libvlr/vlr_sgs_fsm.c
index 2771cf5..12cad1b 100644
--- a/src/libvlr/vlr_sgs_fsm.c
+++ b/src/libvlr/vlr_sgs_fsm.c
@@ -64,8 +64,8 @@
vlr_subscr_set_last_used_eutran_plmn_id(vsub, NULL);
/* Make sure any ongoing paging is aborted. */
- if (vsub->cs.is_paging)
- paging_expired(vsub);
+ if (vsub->cs.is_paging && vsub->sgs.paging_cb)
+ vsub->sgs.paging_cb(vsub, SGSAP_SERV_IND_PAGING_TIMEOUT);
/* Ensure that Ts5 (pending paging via SGs) is deleted */
if (vlr_sgs_pag_pend(vsub))
--
To view, visit https://gerrit.osmocom.org/c/osmo-msc/+/38107?usp=email
To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings?usp=email
Gerrit-MessageType: newchange
Gerrit-Project: osmo-msc
Gerrit-Branch: master
Gerrit-Change-Id: I7a3d15e8f0fb51c6b32add2de5024fc4d599ecf0
Gerrit-Change-Number: 38107
Gerrit-PatchSet: 1
Gerrit-Owner: lynxis lazus <lynxis(a)fe80.eu>
lynxis lazus has uploaded this change for review. ( https://gerrit.osmocom.org/c/osmo-msc/+/38109?usp=email )
Change subject: libvlr: fix tab/space issue
......................................................................
libvlr: fix tab/space issue
Change-Id: I7f831e7ee0014a2241a69f7c75f2c6404bbb7087
---
M src/libvlr/vlr.c
1 file changed, 1 insertion(+), 1 deletion(-)
git pull ssh://gerrit.osmocom.org:29418/osmo-msc refs/changes/09/38109/1
diff --git a/src/libvlr/vlr.c b/src/libvlr/vlr.c
index db4156f..e8ff1b1 100644
--- a/src/libvlr/vlr.c
+++ b/src/libvlr/vlr.c
@@ -1293,7 +1293,7 @@
}
/* Dispatch result to vsub->lu_fsm, which will either handle the result by itself (Check IMEI early) or dispatch
- * it further to lu_compl_vlr_fsm (Check IMEI after LU). */
+ * it further to lu_compl_vlr_fsm (Check IMEI after LU). */
if (gsup->message_type == OSMO_GSUP_MSGT_CHECK_IMEI_RESULT) {
if (gsup->imei_result == OSMO_GSUP_IMEI_RESULT_ACK)
osmo_fsm_inst_dispatch(vsub->lu_fsm, VLR_ULA_E_HLR_IMEI_ACK, NULL);
--
To view, visit https://gerrit.osmocom.org/c/osmo-msc/+/38109?usp=email
To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings?usp=email
Gerrit-MessageType: newchange
Gerrit-Project: osmo-msc
Gerrit-Branch: master
Gerrit-Change-Id: I7f831e7ee0014a2241a69f7c75f2c6404bbb7087
Gerrit-Change-Number: 38109
Gerrit-PatchSet: 1
Gerrit-Owner: lynxis lazus <lynxis(a)fe80.eu>
Attention is currently required from: osmith, pespin.
laforge has posted comments on this change by laforge. ( https://gerrit.osmocom.org/c/osmo-ci/+/38024?usp=email )
Change subject: gerrit-verifications,master-builds: Add pyosmocom
......................................................................
Patch Set 4:
(1 comment)
File jobs/master-builds.yml:
https://gerrit.osmocom.org/c/osmo-ci/+/38024/comment/38037688_f453eea1?usp=… :
PS4, Line 425: a1: !!python/tuple ["test", "pylint", "docs"]
> this is different from gerrit-verifications. should be the same? […]
the 'distcheck' is a NOP in jenkins.sh, so it doesn't make a practical diference. But I've removed it for gerrit-verifications.yml to be consistent.
--
To view, visit https://gerrit.osmocom.org/c/osmo-ci/+/38024?usp=email
To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings?usp=email
Gerrit-MessageType: comment
Gerrit-Project: osmo-ci
Gerrit-Branch: master
Gerrit-Change-Id: I9ec289f8d3116bbc8291298146c58d0209684439
Gerrit-Change-Number: 38024
Gerrit-PatchSet: 4
Gerrit-Owner: laforge <laforge(a)osmocom.org>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: osmith <osmith(a)sysmocom.de>
Gerrit-Reviewer: pespin <pespin(a)sysmocom.de>
Gerrit-Attention: osmith <osmith(a)sysmocom.de>
Gerrit-Attention: pespin <pespin(a)sysmocom.de>
Gerrit-Comment-Date: Wed, 11 Sep 2024 21:37:54 +0000
Gerrit-HasComments: Yes
Gerrit-Has-Labels: No
Comment-In-Reply-To: osmith <osmith(a)sysmocom.de>
Attention is currently required from: laforge, pespin.
Hello Jenkins Builder, osmith, pespin,
I'd like you to reexamine a change. Please visit
https://gerrit.osmocom.org/c/osmo-ci/+/38024?usp=email
to look at the new patch set (#5).
The following approvals got outdated and were removed:
Verified+1 by Jenkins Builder
Change subject: gerrit-verifications,master-builds: Add pyosmocom
......................................................................
gerrit-verifications,master-builds: Add pyosmocom
Change-Id: I9ec289f8d3116bbc8291298146c58d0209684439
---
M jobs/gerrit-verifications.yml
M jobs/master-builds.yml
2 files changed, 16 insertions(+), 0 deletions(-)
git pull ssh://gerrit.osmocom.org:29418/osmo-ci refs/changes/24/38024/5
--
To view, visit https://gerrit.osmocom.org/c/osmo-ci/+/38024?usp=email
To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings?usp=email
Gerrit-MessageType: newpatchset
Gerrit-Project: osmo-ci
Gerrit-Branch: master
Gerrit-Change-Id: I9ec289f8d3116bbc8291298146c58d0209684439
Gerrit-Change-Number: 38024
Gerrit-PatchSet: 5
Gerrit-Owner: laforge <laforge(a)osmocom.org>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: osmith <osmith(a)sysmocom.de>
Gerrit-Reviewer: pespin <pespin(a)sysmocom.de>
Gerrit-Attention: laforge <laforge(a)osmocom.org>
Gerrit-Attention: pespin <pespin(a)sysmocom.de>