October 2026 - gerrit-log - lists.osmocom.org

[XL] Change in pysim[master]: Remote Card Procedure Framework
by dexter 02 Oct '26

02 Oct '26

Attention is currently required from: laforge, lynxis lazus, pespin. dexter has posted comments on this change by dexter. ( https://gerrit.osmocom.org/c/pysim/+/42996?usp=email ) Change subject: Remote Card Procedure Framework ...................................................................... Patch Set 4: (1 comment) Patchset: PS3: > I agree with that. […] I am also a bit uncertain. Maybe it is enough to restructure the pySim directory a bit. osmo-smdpp.py also falls in that category, its a relatively big program with its own sub directory (smdpp-data). I think some structure within the pysim git would already make a huge difference. -- To view, visit https://gerrit.osmocom.org/c/pysim/+/42996?usp=email To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings?usp=email Gerrit-MessageType: comment Gerrit-Project: pysim Gerrit-Branch: master Gerrit-Change-Id: I8500a98492328ed5b3eb4bb177af2a4a8e481f80 Gerrit-Change-Number: 42996 Gerrit-PatchSet: 4 Gerrit-Owner: dexter <pmaier(a)sysmocom.de> Gerrit-Reviewer: Jenkins Builder Gerrit-Reviewer: laforge <laforge(a)osmocom.org> Gerrit-Reviewer: pespin <pespin(a)sysmocom.de> Gerrit-CC: lynxis lazus <lynxis(a)fe80.eu> Gerrit-Attention: laforge <laforge(a)osmocom.org> Gerrit-Attention: pespin <pespin(a)sysmocom.de> Gerrit-Attention: lynxis lazus <lynxis(a)fe80.eu> Gerrit-Comment-Date: Fri, 02 Oct 2026 09:15:30 +0000 Gerrit-HasComments: Yes Gerrit-Has-Labels: No Comment-In-Reply-To: laforge <laforge(a)osmocom.org> Comment-In-Reply-To: lynxis lazus <lynxis(a)fe80.eu>

1 0

[L] Change in libosmo-sigtran[master]: Add Cisco like DSCP configuration support
by jolly 02 Oct '26

02 Oct '26

Attention is currently required from: fixeria, pespin. Hello Jenkins Builder, fixeria, pespin, I'd like you to reexamine a change. Please visit https://gerrit.osmocom.org/c/libosmo-sigtran/+/43667?usp=email to look at the new patch set (#11). The following approvals got outdated and were removed: Verified+1 by Jenkins Builder Change subject: Add Cisco like DSCP configuration support ...................................................................... Add Cisco like DSCP configuration support Add Cisco like VTY configuration support for DSCP. QoS classes can be defined and configured per SS7 instance and the classes can be assigned to listener and connections like this: cs7 instance 0 qos class 3 qos-ip-dscp 36 ... asp myasp 2905 2905 m3ua qos-class 3 ... asp asp-srv-m3ua 2906 2905 m3ua qos-class 3 ... asp asp-clnt-m3ua 2905 2906 m3ua qos-class 3 ... Ensure that the ongoing listener or connection socket is updated whenever the QoS class is changed or deleted or its DSCP value is changed. Deprecated "ip-dscp" and "init-ip-dscp" configuration is still supported. When QoS class is assigned, the deprecated configuration is removed and vice versa. Update vty tests to the new VTY command layout and test the new VTY commands. Related: OS#6753 Depends-on: I06927855a173c5b6dbf5c93aa08ea75e1d9d5f37 (libosmocore) Change-Id: Ic346698fc63771d95e474aec53ffb6c776636b3b --- M TODO-RELEASE M src/Makefile.am M src/ss7_asp.c M src/ss7_asp.h M src/ss7_asp_vty.c M src/ss7_instance.c M src/ss7_instance.h A src/ss7_qos_class.c A src/ss7_qos_class.h A src/ss7_qos_class_vty.c M src/ss7_vty.c M src/ss7_vty.h M src/ss7_xua_srv.c M src/ss7_xua_srv.h M src/ss7_xua_srv_vty.c M tests/vty/osmo_stp_test.vty M tests/vty/ss7_asp_test.vty M tests/vty/vty_test_runner.py 18 files changed, 469 insertions(+), 52 deletions(-) git pull ssh://gerrit.osmocom.org:29418/libosmo-sigtran refs/changes/67/43667/11 -- To view, visit https://gerrit.osmocom.org/c/libosmo-sigtran/+/43667?usp=email To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings?usp=email Gerrit-MessageType: newpatchset Gerrit-Project: libosmo-sigtran Gerrit-Branch: master Gerrit-Change-Id: Ic346698fc63771d95e474aec53ffb6c776636b3b Gerrit-Change-Number: 43667 Gerrit-PatchSet: 11 Gerrit-Owner: jolly <andreas(a)eversberg.eu> Gerrit-Reviewer: Jenkins Builder Gerrit-Reviewer: fixeria <vyanitskiy(a)sysmocom.de> Gerrit-Reviewer: pespin <pespin(a)sysmocom.de> Gerrit-Attention: fixeria <vyanitskiy(a)sysmocom.de> Gerrit-Attention: pespin <pespin(a)sysmocom.de>

1 0

[L] Change in libosmo-sigtran[master]: Add Cisco like DSCP configuration support
by jolly 02 Oct '26

02 Oct '26

Attention is currently required from: fixeria, pespin. jolly has posted comments on this change by jolly. ( https://gerrit.osmocom.org/c/libosmo-sigtran/+/43667?usp=email ) Change subject: Add Cisco like DSCP configuration support ...................................................................... Patch Set 9: (2 comments) File src/ss7_qos_class.c: https://gerrit.osmocom.org/c/libosmo-sigtran/+/43667/comment/3fbb4cb8_e59d5… : PS7, Line 32: pointer to Application Server > Not addressed. Done https://gerrit.osmocom.org/c/libosmo-sigtran/+/43667/comment/72edf493_a5f15… : PS7, Line 50: pointer to Application Server > Not addressed. Done -- To view, visit https://gerrit.osmocom.org/c/libosmo-sigtran/+/43667?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: Ic346698fc63771d95e474aec53ffb6c776636b3b Gerrit-Change-Number: 43667 Gerrit-PatchSet: 9 Gerrit-Owner: jolly <andreas(a)eversberg.eu> Gerrit-Reviewer: Jenkins Builder Gerrit-Reviewer: fixeria <vyanitskiy(a)sysmocom.de> Gerrit-Reviewer: pespin <pespin(a)sysmocom.de> Gerrit-Attention: fixeria <vyanitskiy(a)sysmocom.de> Gerrit-Attention: pespin <pespin(a)sysmocom.de> Gerrit-Comment-Date: Fri, 02 Oct 2026 08:30:36 +0000 Gerrit-HasComments: Yes Gerrit-Has-Labels: No Comment-In-Reply-To: jolly <andreas(a)eversberg.eu> Comment-In-Reply-To: fixeria <vyanitskiy(a)sysmocom.de>

1 0

[L] Change in libosmo-sigtran[master]: Add Cisco like DSCP configuration support
by jolly 02 Oct '26

02 Oct '26

Attention is currently required from: jolly, pespin. Hello Jenkins Builder, fixeria, pespin, I'd like you to reexamine a change. Please visit https://gerrit.osmocom.org/c/libosmo-sigtran/+/43667?usp=email to look at the new patch set (#10). The following approvals got outdated and were removed: Verified+1 by Jenkins Builder Change subject: Add Cisco like DSCP configuration support ...................................................................... Add Cisco like DSCP configuration support Add Cisco like VTY configuration support for DSCP. QoS classes can be defined and configured per SS7 instance and the classes can be assigned to listener and connections like this: cs7 instance 0 qos class 3 qos-ip-dscp 36 ... asp myasp 2905 2905 m3ua qos-class 3 ... asp asp-srv-m3ua 2906 2905 m3ua qos-class 3 ... asp asp-clnt-m3ua 2905 2906 m3ua qos-class 3 ... Ensure that the ongoing listener or connection socket is updated whenever the QoS class is changed or deleted or its DSCP value is changed. Deprecated "ip-dscp" and "init-ip-dscp" configuration is still supported. When QoS class is assigned, the deprecated configuration is removed and vice versa. Update vty tests to the new VTY command layout and test the new VTY commands. Related: OS#6753 Depends-on: I06927855a173c5b6dbf5c93aa08ea75e1d9d5f37 (libosmocore) Change-Id: Ic346698fc63771d95e474aec53ffb6c776636b3b --- M TODO-RELEASE M src/Makefile.am M src/ss7_asp.c M src/ss7_asp.h M src/ss7_asp_vty.c M src/ss7_instance.c M src/ss7_instance.h A src/ss7_qos_class.c A src/ss7_qos_class.h A src/ss7_qos_class_vty.c M src/ss7_vty.c M src/ss7_vty.h M src/ss7_xua_srv.c M src/ss7_xua_srv.h M src/ss7_xua_srv_vty.c M tests/vty/osmo_stp_test.vty M tests/vty/ss7_asp_test.vty M tests/vty/vty_test_runner.py 18 files changed, 469 insertions(+), 52 deletions(-) git pull ssh://gerrit.osmocom.org:29418/libosmo-sigtran refs/changes/67/43667/10 -- To view, visit https://gerrit.osmocom.org/c/libosmo-sigtran/+/43667?usp=email To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings?usp=email Gerrit-MessageType: newpatchset Gerrit-Project: libosmo-sigtran Gerrit-Branch: master Gerrit-Change-Id: Ic346698fc63771d95e474aec53ffb6c776636b3b Gerrit-Change-Number: 43667 Gerrit-PatchSet: 10 Gerrit-Owner: jolly <andreas(a)eversberg.eu> Gerrit-Reviewer: Jenkins Builder Gerrit-Reviewer: fixeria <vyanitskiy(a)sysmocom.de> Gerrit-Reviewer: pespin <pespin(a)sysmocom.de> Gerrit-Attention: jolly <andreas(a)eversberg.eu> Gerrit-Attention: pespin <pespin(a)sysmocom.de>

1 0

[L] Change in libosmo-sigtran[master]: Add Cisco like DSCP configuration support
by fixeria 01 Oct '26

01 Oct '26

Attention is currently required from: jolly, pespin. fixeria has posted comments on this change by jolly. ( https://gerrit.osmocom.org/c/libosmo-sigtran/+/43667?usp=email ) Change subject: Add Cisco like DSCP configuration support ...................................................................... Patch Set 9: (2 comments) File src/ss7_qos_class.c: https://gerrit.osmocom.org/c/libosmo-sigtran/+/43667/comment/13d10fd7_9d1dc… : PS7, Line 32: pointer to Application Server > Done Not addressed. https://gerrit.osmocom.org/c/libosmo-sigtran/+/43667/comment/d30595a0_197e1… : PS7, Line 50: pointer to Application Server > Done Not addressed. -- To view, visit https://gerrit.osmocom.org/c/libosmo-sigtran/+/43667?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: Ic346698fc63771d95e474aec53ffb6c776636b3b Gerrit-Change-Number: 43667 Gerrit-PatchSet: 9 Gerrit-Owner: jolly <andreas(a)eversberg.eu> Gerrit-Reviewer: Jenkins Builder Gerrit-Reviewer: fixeria <vyanitskiy(a)sysmocom.de> Gerrit-Reviewer: pespin <pespin(a)sysmocom.de> Gerrit-Attention: jolly <andreas(a)eversberg.eu> Gerrit-Attention: pespin <pespin(a)sysmocom.de> Gerrit-Comment-Date: Thu, 01 Oct 2026 16:12:56 +0000 Gerrit-HasComments: Yes Gerrit-Has-Labels: No Comment-In-Reply-To: jolly <andreas(a)eversberg.eu> Comment-In-Reply-To: fixeria <vyanitskiy(a)sysmocom.de>

1 0

[XS] Change in osmo-trx[master]: proxy/vty: g_proxy_ctx->bind_addr cannot be NULL
by fixeria 01 Oct '26

01 Oct '26

fixeria has submitted this change. ( https://gerrit.osmocom.org/c/osmo-trx/+/43735?usp=email ) Change subject: proxy/vty: g_proxy_ctx->bind_addr cannot be NULL ...................................................................... proxy/vty: g_proxy_ctx->bind_addr cannot be NULL Change-Id: Ifbe1112ff73940154a861f6991211652d913d0b4 Fixes: CID#564330 --- M proxy/src/vty.c 1 file changed, 1 insertion(+), 2 deletions(-) Approvals: Jenkins Builder: Verified pespin: Looks good to me, approved diff --git a/proxy/src/vty.c b/proxy/src/vty.c index 6b32ec9..473ac7f 100644 --- a/proxy/src/vty.c +++ b/proxy/src/vty.c @@ -361,8 +361,7 @@ struct proxy_trx *trx; vty_out(vty, "proxy%s", VTY_NEWLINE); - if (g_proxy_ctx->bind_addr) - vty_out(vty, " bind-addr %s%s", g_proxy_ctx->bind_addr, VTY_NEWLINE); + vty_out(vty, " bind-addr %s%s", g_proxy_ctx->bind_addr, VTY_NEWLINE); if (clck_gen_get_start_fn() == CLCK_GEN_START_FN_RANDOM) vty_out(vty, " clck-gen start-fn random%s", VTY_NEWLINE); else -- To view, visit https://gerrit.osmocom.org/c/osmo-trx/+/43735?usp=email To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings?usp=email Gerrit-MessageType: merged Gerrit-Project: osmo-trx Gerrit-Branch: master Gerrit-Change-Id: Ifbe1112ff73940154a861f6991211652d913d0b4 Gerrit-Change-Number: 43735 Gerrit-PatchSet: 2 Gerrit-Owner: fixeria <vyanitskiy(a)sysmocom.de> Gerrit-Reviewer: Jenkins Builder Gerrit-Reviewer: fixeria <vyanitskiy(a)sysmocom.de> Gerrit-Reviewer: pespin <pespin(a)sysmocom.de>

1 0

[S] Change in osmo-trx[master]: proxy: clck_gen_timer_cb(): sanitize expire_count
by fixeria 01 Oct '26

01 Oct '26

fixeria has submitted this change. ( https://gerrit.osmocom.org/c/osmo-trx/+/43736?usp=email ) Change subject: proxy: clck_gen_timer_cb(): sanitize expire_count ...................................................................... proxy: clck_gen_timer_cb(): sanitize expire_count The number of timer expirations read from the timerfd is used as the loop boundary. It is already bounded indirectly by the PC clock skew check, but let's also check it explicitly against the same CLCK_GEN_MAX_FN_SKEW limit. Change-Id: I55f736c9bff58034c197f876206efc817da30204 Fixes: CID#564331 --- M proxy/src/clck_gen.c 1 file changed, 10 insertions(+), 0 deletions(-) Approvals: Jenkins Builder: Verified pespin: Looks good to me, but someone else must approve fixeria: Looks good to me, approved diff --git a/proxy/src/clck_gen.c b/proxy/src/clck_gen.c index 39abc6c..2798b88 100644 --- a/proxy/src/clck_gen.c +++ b/proxy/src/clck_gen.c @@ -99,6 +99,16 @@ return 0; } + /* the number of expirations is also bounded by the skew limit */ + if (expire_count > CLCK_GEN_MAX_FN_SKEW) { + LOGP(DTRXC, LOGL_FATAL, + "Too many timer expirations (%" PRIu64 "): " + "stopping the TDMA clock generator\n", + expire_count); + clck_gen_stop(gen); + return 0; + } + while (expire_count-- > 0) { const struct proxy_trx *trx; -- To view, visit https://gerrit.osmocom.org/c/osmo-trx/+/43736?usp=email To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings?usp=email Gerrit-MessageType: merged Gerrit-Project: osmo-trx Gerrit-Branch: master Gerrit-Change-Id: I55f736c9bff58034c197f876206efc817da30204 Gerrit-Change-Number: 43736 Gerrit-PatchSet: 2 Gerrit-Owner: fixeria <vyanitskiy(a)sysmocom.de> Gerrit-Reviewer: Jenkins Builder Gerrit-Reviewer: fixeria <vyanitskiy(a)sysmocom.de> Gerrit-Reviewer: pespin <pespin(a)sysmocom.de>

1 0

[S] Change in osmo-trx[master]: proxy: properly set .trx_num in {NOPE,BURST}.ind
by fixeria 01 Oct '26

01 Oct '26

fixeria has submitted this change. ( https://gerrit.osmocom.org/c/osmo-trx/+/43737?usp=email ) Change subject: proxy: properly set .trx_num in {NOPE,BURST}.ind ...................................................................... proxy: properly set .trx_num in {NOPE,BURST}.ind The .trx_num field shall indicate the destination channel number. Copying the value from the incoming {NOPE,BURST}.req is wrong. Change-Id: Ib748ed7b681b582fdfad829a1940d71148e63334 Fixes: 1f6f0559 ("proxy: implement burst forwarding (proxying) logic") Related: OS#6672 --- M proxy/src/burst_fwd.c 1 file changed, 9 insertions(+), 5 deletions(-) Approvals: Jenkins Builder: Verified fixeria: Looks good to me, approved pespin: Looks good to me, but someone else must approve diff --git a/proxy/src/burst_fwd.c b/proxy/src/burst_fwd.c index f56f316..2c89107 100644 --- a/proxy/src/burst_fwd.c +++ b/proxy/src/burst_fwd.c @@ -44,8 +44,7 @@ #define OSMO_TRXD_F_COMMON_MASK ( \ OSMO_TRXD_F_NOPE_REQ | \ OSMO_TRXD_F_MOD_TYPE | \ - OSMO_TRXD_F_TS_INFO | \ - OSMO_TRXD_F_TRX_NUM \ + OSMO_TRXD_F_TS_INFO \ ) /*! The BURST.req copy wrapped in a queued struct msgb @@ -60,14 +59,14 @@ const struct osmo_trxd_burst_req *br) { struct osmo_trxd_burst_ind bi = { - .flags = br->flags & OSMO_TRXD_F_COMMON_MASK, + .flags = (br->flags & OSMO_TRXD_F_COMMON_MASK) | OSMO_TRXD_F_TRX_NUM, .fn = br->fn, .tn = br->tn, .burst_len = br->burst_len, .mod = br->mod, .tsc_set = br->tsc_set, .tsc = br->tsc, - .trx_num = br->trx_num, + .trx_num = dst_chan, }; /* TRXDv0/v1 BURST.req PDUs carry no MTS field at all, so the true @@ -175,7 +174,12 @@ struct proxy_trx_chan *c = &trx->chans[chan]; for (unsigned int tn = 0; tn < PROXY_TRX_NUM_TS; tn++) { - c->ts[tn].bi = (struct osmo_trxd_burst_ind){ .fn = fn, .tn = tn }; + c->ts[tn].bi = (struct osmo_trxd_burst_ind){ + .flags = OSMO_TRXD_F_TRX_NUM, + .fn = fn, + .tn = tn, + .trx_num = chan, + }; path_sim_fill_nope(&c->ts[tn].bi, g_proxy_ctx->path_sim); } } -- To view, visit https://gerrit.osmocom.org/c/osmo-trx/+/43737?usp=email To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings?usp=email Gerrit-MessageType: merged Gerrit-Project: osmo-trx Gerrit-Branch: master Gerrit-Change-Id: Ib748ed7b681b582fdfad829a1940d71148e63334 Gerrit-Change-Number: 43737 Gerrit-PatchSet: 1 Gerrit-Owner: fixeria <vyanitskiy(a)sysmocom.de> Gerrit-Reviewer: Jenkins Builder Gerrit-Reviewer: fixeria <vyanitskiy(a)sysmocom.de> Gerrit-Reviewer: pespin <pespin(a)sysmocom.de>

1 0

[S] Change in osmo-trx[master]: proxy: properly set .trx_num in {NOPE,BURST}.ind
by fixeria 01 Oct '26

01 Oct '26

fixeria has posted comments on this change by fixeria. ( https://gerrit.osmocom.org/c/osmo-trx/+/43737?usp=email ) Change subject: proxy: properly set .trx_num in {NOPE,BURST}.ind ...................................................................... Patch Set 1: Code-Review+2 -- To view, visit https://gerrit.osmocom.org/c/osmo-trx/+/43737?usp=email To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings?usp=email Gerrit-MessageType: comment Gerrit-Project: osmo-trx Gerrit-Branch: master Gerrit-Change-Id: Ib748ed7b681b582fdfad829a1940d71148e63334 Gerrit-Change-Number: 43737 Gerrit-PatchSet: 1 Gerrit-Owner: fixeria <vyanitskiy(a)sysmocom.de> Gerrit-Reviewer: Jenkins Builder Gerrit-Reviewer: fixeria <vyanitskiy(a)sysmocom.de> Gerrit-Reviewer: pespin <pespin(a)sysmocom.de> Gerrit-Comment-Date: Thu, 01 Oct 2026 16:10:07 +0000 Gerrit-HasComments: No Gerrit-Has-Labels: Yes

1 0

[S] Change in osmo-trx[master]: proxy: clck_gen_timer_cb(): sanitize expire_count
by fixeria 01 Oct '26

01 Oct '26

fixeria has posted comments on this change by fixeria. ( https://gerrit.osmocom.org/c/osmo-trx/+/43736?usp=email ) Change subject: proxy: clck_gen_timer_cb(): sanitize expire_count ...................................................................... Patch Set 2: Code-Review+2 -- To view, visit https://gerrit.osmocom.org/c/osmo-trx/+/43736?usp=email To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings?usp=email Gerrit-MessageType: comment Gerrit-Project: osmo-trx Gerrit-Branch: master Gerrit-Change-Id: I55f736c9bff58034c197f876206efc817da30204 Gerrit-Change-Number: 43736 Gerrit-PatchSet: 2 Gerrit-Owner: fixeria <vyanitskiy(a)sysmocom.de> Gerrit-Reviewer: Jenkins Builder Gerrit-Reviewer: fixeria <vyanitskiy(a)sysmocom.de> Gerrit-Reviewer: pespin <pespin(a)sysmocom.de> Gerrit-Comment-Date: Thu, 01 Oct 2026 16:10:03 +0000 Gerrit-HasComments: No Gerrit-Has-Labels: Yes

1 0