pespin has submitted this change. ( https://gerrit.osmocom.org/c/libosmo-sigtran/+/43485?usp=email )
Change subject: xua_as_fsm: Do not send NOTIFY with all AS of ASP
......................................................................
xua_as_fsm: Do not send NOTIFY with all AS of ASP
The NOTIFY message is sharing Routing Context (AS) state change.
When the FSM of a given AS changes, we don't want to signal the state
change of that single AS to all its ASPs, not all other AS served by
those ASPs.
This partially reverts old commit
e4fc91b9fc3aa3194eea02ef10635323b7558c18.
Related: OS#7062
Change-Id: I252f9e16ab92a264559385d47690552d75837569
---
M src/xua_as_fsm.c
1 file changed, 10 insertions(+), 11 deletions(-)
Approvals:
fixeria: Looks good to me, approved
osmith: Looks good to me, but someone else must approve
Jenkins Builder: Verified
diff --git a/src/xua_as_fsm.c b/src/xua_as_fsm.c
index 0ba1b2a..9460cf8 100644
--- a/src/xua_as_fsm.c
+++ b/src/xua_as_fsm.c
@@ -43,14 +43,6 @@
return msg;
}
-static int fill_notify_route_ctx(const struct osmo_ss7_asp *asp, struct osmo_xlm_prim_notify *npar)
-{
- npar->route_ctx_count = ss7_asp_get_all_rctx(asp, npar->route_ctx, ARRAY_SIZE(npar->route_ctx), NULL);
- if (npar->route_ctx_count > 0)
- npar->presence |= NOTIFY_PAR_P_ROUTE_CTX;
- return 0;
-}
-
static void tx_notify(struct osmo_ss7_asp *asp, struct osmo_xlm_prim_notify *npar)
{
const char *type_name, *info_name, *info_str;
@@ -60,7 +52,6 @@
LOGPASP(asp, DLSS7, LOGL_INFO, "Tx NOTIFY Type %s:%s (%s)\n",
type_name, info_name, info_str);
- fill_notify_route_ctx(asp, npar);
struct msgb *msg = encode_notify(npar);
osmo_ss7_asp_send(asp, msg);
}
@@ -139,8 +130,6 @@
npar->presence &= ~NOTIFY_PAR_P_ASP_ID;
}
- fill_notify_route_ctx(asp, npar);
-
msg = encode_notify(npar);
osmo_ss7_asp_send(asp, msg);
sent++;
@@ -248,10 +237,20 @@
static void fill_notify_statchg_pars(const struct osmo_fsm_inst *fi, struct osmo_xlm_prim_notify *npar)
{
+ struct xua_as_fsm_priv *xafp = (struct xua_as_fsm_priv *) fi->priv;
+ struct osmo_ss7_as *as = xafp->as;
+
*npar = (struct osmo_xlm_prim_notify){
.status_type = M3UA_NOTIFY_T_STATCHG,
};
+ /* Add the routing context, if it is configured */
+ if (as->cfg.routing_key.context > 0) {
+ npar->presence |= NOTIFY_PAR_P_ROUTE_CTX;
+ npar->route_ctx[0] = as->cfg.routing_key.context;
+ npar->route_ctx_count = 1;
+ }
+
switch (fi->state) {
case XUA_AS_S_INACTIVE:
npar->status_info = M3UA_NOTIFY_I_AS_INACT;
--
To view, visit https://gerrit.osmocom.org/c/libosmo-sigtran/+/43485?usp=email
To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings?usp=email
Gerrit-MessageType: merged
Gerrit-Project: libosmo-sigtran
Gerrit-Branch: master
Gerrit-Change-Id: I252f9e16ab92a264559385d47690552d75837569
Gerrit-Change-Number: 43485
Gerrit-PatchSet: 1
Gerrit-Owner: pespin <pespin(a)sysmocom.de>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: fixeria <vyanitskiy(a)sysmocom.de>
Gerrit-Reviewer: laforge <laforge(a)osmocom.org>
Gerrit-Reviewer: osmith <osmith(a)sysmocom.de>
Gerrit-Reviewer: pespin <pespin(a)sysmocom.de>
Attention is currently required from: fixeria, laforge, osmith.
pespin has posted comments on this change by pespin. ( https://gerrit.osmocom.org/c/libosmo-sigtran/+/43500?usp=email )
Change subject: xUA: Validate multiple Routing Context in SNM messages
......................................................................
Patch Set 1:
(1 comment)
Commit Message:
https://gerrit.osmocom.org/c/libosmo-sigtran/+/43500/comment/9278a0ec_1b043… :
PS1, Line 14: work.
> Depending on how long this will take to implement, it might make sense to print a warning if there i […]
It's being tracked in a ticket, and it now becomes obvious in the code that only the first one is being picked, so I think it's fine.It's obvious anyway that people is not using that feature so far, since it's not really ready yet.
--
To view, visit https://gerrit.osmocom.org/c/libosmo-sigtran/+/43500?usp=email
To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings?usp=email
Gerrit-MessageType: comment
Gerrit-Project: libosmo-sigtran
Gerrit-Branch: master
Gerrit-Change-Id: Ic3a9dc13d128a0c5777981616923ee1856210b21
Gerrit-Change-Number: 43500
Gerrit-PatchSet: 1
Gerrit-Owner: pespin <pespin(a)sysmocom.de>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: fixeria <vyanitskiy(a)sysmocom.de>
Gerrit-Reviewer: laforge <laforge(a)osmocom.org>
Gerrit-Reviewer: osmith <osmith(a)sysmocom.de>
Gerrit-Attention: osmith <osmith(a)sysmocom.de>
Gerrit-Attention: laforge <laforge(a)osmocom.org>
Gerrit-Attention: fixeria <vyanitskiy(a)sysmocom.de>
Gerrit-Comment-Date: Wed, 02 Sep 2026 10:36:32 +0000
Gerrit-HasComments: Yes
Gerrit-Has-Labels: No
Comment-In-Reply-To: osmith <osmith(a)sysmocom.de>
Attention is currently required from: lynxis lazus.
fixeria has posted comments on this change by lynxis lazus. ( https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/43453?usp=email )
Change subject: SGSN: f_routing_area_update(): expect a Location Update by default
......................................................................
Patch Set 1:
(1 comment)
Commit Message:
https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/43453/comment/6994f43a_12e9… :
PS1, Line 10: location is still located
location located? Do you mean the MS?
--
To view, visit https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/43453?usp=email
To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings?usp=email
Gerrit-MessageType: comment
Gerrit-Project: osmo-ttcn3-hacks
Gerrit-Branch: master
Gerrit-Change-Id: Ib67af3197399f1bf94e8d691f4e356f9a926c0d5
Gerrit-Change-Number: 43453
Gerrit-PatchSet: 1
Gerrit-Owner: lynxis lazus <lynxis(a)fe80.eu>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: daniel <dwillmann(a)sysmocom.de>
Gerrit-Reviewer: lynxis lazus <lynxis(a)fe80.eu>
Gerrit-Reviewer: pespin <pespin(a)sysmocom.de>
Gerrit-CC: fixeria <vyanitskiy(a)sysmocom.de>
Gerrit-Attention: lynxis lazus <lynxis(a)fe80.eu>
Gerrit-Comment-Date: Wed, 02 Sep 2026 10:31:55 +0000
Gerrit-HasComments: Yes
Gerrit-Has-Labels: No
Attention is currently required from: lynxis lazus.
Hello Jenkins Builder, fixeria, laforge, pespin,
I'd like you to reexamine a change. Please visit
https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/39254?usp=email
to look at the new patch set (#12).
The following approvals got outdated and were removed:
Verified+1 by Jenkins Builder
The change is no longer submittable: Verified is unsatisfied now.
Change subject: SGSN: TC_attach_gmm_attach_req_while_gmm_attach: fix test case
......................................................................
SGSN: TC_attach_gmm_attach_req_while_gmm_attach: fix test case
The SGSN has changed the behavior:
- removed auth-policy - not supported anymore
- require authentication
- remove ID Req(IMEI), not used by new libvlr SGSN
Change-Id: I026499844940d4c3d82b0fbd61be636c6ca348e6
---
M sgsn/BSSGP_ConnHdlr.ttcn
M sgsn/SGSN_Tests.ttcn
2 files changed, 18 insertions(+), 24 deletions(-)
git pull ssh://gerrit.osmocom.org:29418/osmo-ttcn3-hacks refs/changes/54/39254/12
--
To view, visit https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/39254?usp=email
To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings?usp=email
Gerrit-MessageType: newpatchset
Gerrit-Project: osmo-ttcn3-hacks
Gerrit-Branch: master
Gerrit-Change-Id: I026499844940d4c3d82b0fbd61be636c6ca348e6
Gerrit-Change-Number: 39254
Gerrit-PatchSet: 12
Gerrit-Owner: lynxis lazus <lynxis(a)fe80.eu>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: fixeria <vyanitskiy(a)sysmocom.de>
Gerrit-Reviewer: laforge <laforge(a)osmocom.org>
Gerrit-Reviewer: lynxis lazus <lynxis(a)fe80.eu>
Gerrit-Reviewer: pespin <pespin(a)sysmocom.de>
Gerrit-Attention: lynxis lazus <lynxis(a)fe80.eu>