- gerrit-log - lists.osmocom.org

[XS] Change in osmo-ttcn3-hacks[master]: deps: Update titan.ProtocolModules.M3UA to upstream master
by pespin 11 Mar '25

11 Mar '25

pespin has uploaded this change for review. ( https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/39764?usp=email ) Change subject: deps: Update titan.ProtocolModules.M3UA to upstream master ...................................................................... deps: Update titan.ProtocolModules.M3UA to upstream master MR contained in our fork was merged [1], so we can use upstream again. [1] https://gitlab.eclipse.org/eclipse/titan/titan.ProtocolModules.M3UA/-/merge… This patch doesn't contain an update hash because it was merged as fast-forward so the commit hash now in master HEAD did not change from the one in our repo fork branch. Change-Id: Icf411559d3ba5bc699d8a66d7c42e7034392a9e6 --- M deps/Makefile 1 file changed, 2 insertions(+), 2 deletions(-) git pull ssh://gerrit.osmocom.org:29418/osmo-ttcn3-hacks refs/changes/64/39764/1 diff --git a/deps/Makefile b/deps/Makefile index 61daa48..9e0229f 100644 --- a/deps/Makefile +++ b/deps/Makefile @@ -34,6 +34,7 @@ titan.ProtocolModules.JSON_v07_2006 \ titan.ProtocolModules.L2TP \ titan.ProtocolModules.LLC_v7.1.0 \ + titan.ProtocolModules.M3UA \ titan.ProtocolModules.MobileL3_v13.4.0 \ titan.ProtocolModules.SMPP \ titan.ProtocolModules.TCP \ @@ -65,8 +66,7 @@ OSMOGITHUB_REPOS= titan.TestPorts.SCTPasp \ titan.TestPorts.MTP3asp \ titan.ProtocolEmulations.SCCP \ - titan.ProtocolModules.BSSGP_v13.0.0 \ - titan.ProtocolModules.M3UA + titan.ProtocolModules.BSSGP_v13.0.0 OSMOGERRIT_REPOS= osmo-uecups -- To view, visit https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/39764?usp=email To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings?usp=email Gerrit-MessageType: newchange Gerrit-Project: osmo-ttcn3-hacks Gerrit-Branch: master Gerrit-Change-Id: Icf411559d3ba5bc699d8a66d7c42e7034392a9e6 Gerrit-Change-Number: 39764 Gerrit-PatchSet: 1 Gerrit-Owner: pespin <pespin(a)sysmocom.de>

1 0

[S] Change in aram-applet[master]: AccessRuleMaster: allow locking of STORE DATA commands
by osmith 11 Mar '25

11 Mar '25

Attention is currently required from: dexter. osmith has posted comments on this change by dexter. ( https://gerrit.osmocom.org/c/aram-applet/+/39615?usp=email ) Change subject: AccessRuleMaster: allow locking of STORE DATA commands ...................................................................... Patch Set 3: Code-Review+1 (1 comment) Patchset: PS3: I did not check the spec, but assuming processData can only be accessed by the secure domain (so with additional authentication) as it was explained, this should work. -- To view, visit https://gerrit.osmocom.org/c/aram-applet/+/39615?usp=email To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings?usp=email Gerrit-MessageType: comment Gerrit-Project: aram-applet Gerrit-Branch: master Gerrit-Change-Id: I86437844585c22fc4280cc48b99edbb56e3159db Gerrit-Change-Number: 39615 Gerrit-PatchSet: 3 Gerrit-Owner: dexter <pmaier(a)sysmocom.de> Gerrit-Reviewer: Jenkins Builder Gerrit-Reviewer: laforge <laforge(a)osmocom.org> Gerrit-Reviewer: osmith <osmith(a)sysmocom.de> Gerrit-Attention: dexter <pmaier(a)sysmocom.de> Gerrit-Comment-Date: Tue, 11 Mar 2025 07:54:47 +0000 Gerrit-HasComments: Yes Gerrit-Has-Labels: Yes

1 0

[L] Change in osmo-asf4-dfu[master]: clang-format
by osmith 11 Mar '25

11 Mar '25

Attention is currently required from: Hoernchen, laforge. osmith has posted comments on this change by Hoernchen. ( https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39470?usp=email ) Change subject: clang-format ...................................................................... Patch Set 3: Code-Review+1 (2 comments) Patchset: PS2: > I'd say this needs some more explanation and possibly integration? What is the purpose of the file? […] * pre-commit hook is there now * server side: needs clang-format added to the docker container first: https://gerrit.osmocom.org/c/docker-playground/+/39763, I can adjust contrib/jenkins.sh in a follow-up patch after both are merged so we don't need to block here File contrib/pre-commit-hook: https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39470/comment/562950a5_998dabd… : PS3, Line 1: #!/usr/bin/env bash maybe make this executable? -- To view, visit https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39470?usp=email To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings?usp=email Gerrit-MessageType: comment Gerrit-Project: osmo-asf4-dfu Gerrit-Branch: master Gerrit-Change-Id: I9768bc2590ca5306ee19d3276f0953c6b0b6f926 Gerrit-Change-Number: 39470 Gerrit-PatchSet: 3 Gerrit-Owner: Hoernchen <ewild(a)sysmocom.de> Gerrit-Reviewer: Jenkins Builder Gerrit-Reviewer: laforge <laforge(a)osmocom.org> Gerrit-Reviewer: osmith <osmith(a)sysmocom.de> Gerrit-Attention: Hoernchen <ewild(a)sysmocom.de> Gerrit-Attention: laforge <laforge(a)osmocom.org> Gerrit-Comment-Date: Tue, 11 Mar 2025 07:33:28 +0000 Gerrit-HasComments: Yes Gerrit-Has-Labels: Yes Comment-In-Reply-To: laforge <laforge(a)osmocom.org>

1 0

[XS] Change in docker-playground[master]: debian-bookworm-build: add linting tools
by osmith 11 Mar '25

11 Mar '25

osmith has uploaded this change for review. ( https://gerrit.osmocom.org/c/docker-playground/+/39763?usp=email ) Change subject: debian-bookworm-build: add linting tools ...................................................................... debian-bookworm-build: add linting tools Prepare for being able to run these linting tools in CI. * clang-format gets used for osmo-asf4-dfu * ruff gets used for testenv in osmo-ttcn3-hacks Related: osmo-asf4-dfu I9768bc2590ca5306ee19d3276f0953c6b0b6f926 Related: osmo-ttcn3-hacks 157897523e54762ec3739cd7842c7b5f5d724ac2 Change-Id: I0a287b991a3efa50090587739529f95de32225bc --- M debian-bookworm-build/Dockerfile 1 file changed, 2 insertions(+), 0 deletions(-) git pull ssh://gerrit.osmocom.org:29418/docker-playground refs/changes/63/39763/1 diff --git a/debian-bookworm-build/Dockerfile b/debian-bookworm-build/Dockerfile index 3efbc6e..8fbc9d3 100644 --- a/debian-bookworm-build/Dockerfile +++ b/debian-bookworm-build/Dockerfile @@ -61,6 +61,7 @@ bzip2 \ ca-certificates \ ccache \ + clang-format \ cmake \ coccinelle \ cppcheck \ @@ -180,6 +181,7 @@ 'git+https://github.com/podshumok/python-smpplib.git' \ 'pydbus' \ 'pysispm' \ + 'ruff' \ 'sphinx' \ 'sphinxcontrib-napoleon' \ -r /tmp/pysim_requirements.txt -- To view, visit https://gerrit.osmocom.org/c/docker-playground/+/39763?usp=email To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings?usp=email Gerrit-MessageType: newchange Gerrit-Project: docker-playground Gerrit-Branch: master Gerrit-Change-Id: I0a287b991a3efa50090587739529f95de32225bc Gerrit-Change-Number: 39763 Gerrit-PatchSet: 1 Gerrit-Owner: osmith <osmith(a)sysmocom.de>

1 0

[XS] Change in osmo-ggsn[master]: gtp: Send RAT type IE in update context request if present
by laforge 10 Mar '25

10 Mar '25

Attention is currently required from: daniel, lynxis lazus. laforge has posted comments on this change by daniel. ( https://gerrit.osmocom.org/c/osmo-ggsn/+/39700?usp=email ) Change subject: gtp: Send RAT type IE in update context request if present ...................................................................... Patch Set 1: Code-Review+2 -- To view, visit https://gerrit.osmocom.org/c/osmo-ggsn/+/39700?usp=email To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings?usp=email Gerrit-MessageType: comment Gerrit-Project: osmo-ggsn Gerrit-Branch: master Gerrit-Change-Id: Idbabd123838b88cb0a2407480ca5fab10312a547 Gerrit-Change-Number: 39700 Gerrit-PatchSet: 1 Gerrit-Owner: daniel <dwillmann(a)sysmocom.de> Gerrit-Reviewer: Jenkins Builder Gerrit-Reviewer: laforge <laforge(a)osmocom.org> Gerrit-Reviewer: lynxis lazus <lynxis(a)fe80.eu> Gerrit-Reviewer: pespin <pespin(a)sysmocom.de> Gerrit-Attention: daniel <dwillmann(a)sysmocom.de> Gerrit-Attention: lynxis lazus <lynxis(a)fe80.eu> Gerrit-Comment-Date: Mon, 10 Mar 2025 19:45:35 +0000 Gerrit-HasComments: No Gerrit-Has-Labels: Yes

1 0

[L] Change in ...logger_gsmtap[master]: Initial commit
by laforge 10 Mar '25

10 Mar '25

Attention is currently required from: fixeria. laforge has posted comments on this change by fixeria. ( https://gerrit.osmocom.org/c/erlang/logger_gsmtap/+/39552?usp=email ) Change subject: Initial commit ...................................................................... Patch Set 1: (1 comment) Patchset: PS1: can you simply set it manually to V+1? I guess the second or third commit should then add some build verfication? -- To view, visit https://gerrit.osmocom.org/c/erlang/logger_gsmtap/+/39552?usp=email To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings?usp=email Gerrit-MessageType: comment Gerrit-Project: erlang/logger_gsmtap Gerrit-Branch: master Gerrit-Change-Id: I30fad55b6e9d8d9f975de4deb2f97c1e98ae5c5e Gerrit-Change-Number: 39552 Gerrit-PatchSet: 1 Gerrit-Owner: fixeria <vyanitskiy(a)sysmocom.de> Gerrit-Reviewer: laforge <laforge(a)osmocom.org> Gerrit-Reviewer: pespin <pespin(a)sysmocom.de> Gerrit-Attention: fixeria <vyanitskiy(a)sysmocom.de> Gerrit-Comment-Date: Mon, 10 Mar 2025 19:45:10 +0000 Gerrit-HasComments: Yes Gerrit-Has-Labels: No

1 0

[M] Change in osmo-mgw[master]: E1: change dummy fill frame generation method
by falconia 10 Mar '25

10 Mar '25

falconia has submitted this change. ( https://gerrit.osmocom.org/c/osmo-mgw/+/39750?usp=email ) Change subject: E1: change dummy fill frame generation method ...................................................................... E1: change dummy fill frame generation method In the absence of a proper TFO transform (TS 28.062 section C3.2.1.1), OsmoMGW-E1 inserts constant fill frames when there is no RTP input, or when RTP input fails conversion to TRAU-DL and thus does not enqueue anything to the I.460 mux. (We accept TW-TS-001 and TW-TS-002 from RTP side, but bad frames cannot be converted to TRAU-DL without a TFO transform.) Prior to the present change, this idle fill of the DL was done using fully hard-coded frames, i.e., the operation of converting the desired fill to TRAU-DL bit format was done once externally and the raw bit frames were hard-coded. Change this approach: use fill frames in RTP format provided by libosmocodec, and execute osmo_rtp2trau() followed by osmo_trau_frame_encode() in the fill frame output path just like we do for regular RTP to DL. Depends: libosmocore.git I2c510ac62a0786c137115c45eee7a48b9736265f Change-Id: I123f77295c56d61ff9aac5f5d64b25eca01e0d63 --- M TODO-RELEASE M configure.ac M src/libosmo-mgcp/Makefile.am M src/libosmo-mgcp/mgcp_e1.c M src/osmo-mgw/Makefile.am M tests/mgcp/Makefile.am 6 files changed, 40 insertions(+), 157 deletions(-) Approvals: Jenkins Builder: Verified laforge: Looks good to me, approved pespin: Looks good to me, but someone else must approve diff --git a/TODO-RELEASE b/TODO-RELEASE index bdb3915..73a6cd6 100644 --- a/TODO-RELEASE +++ b/TODO-RELEASE @@ -9,3 +9,4 @@ #library what description / commit summary line libosmocore bump_dep; workaround Bump libosmocore version dependency after I68328adb952ca8833ba047cb3b49ccc6f8a1f1b5 has been merged to libosmocore.git; then remove my_msgb_copy_c wrapper function. +libosmocodec bump_dep We depend on the additions of I2c510ac62a0786c137115c45eee7a48b9736265f diff --git a/configure.ac b/configure.ac index 74933a7..8a0bd93 100644 --- a/configure.ac +++ b/configure.ac @@ -45,6 +45,7 @@ PKG_CHECK_MODULES(LIBOSMOCORE, libosmocore >= 1.11.0) +PKG_CHECK_MODULES(LIBOSMOCODEC, libosmocodec >= 1.11.0) PKG_CHECK_MODULES(LIBOSMOGSM, libosmogsm >= 1.11.0) PKG_CHECK_MODULES(LIBOSMOCTRL, libosmoctrl >= 1.11.0) PKG_CHECK_MODULES(LIBOSMOVTY, libosmovty >= 1.11.0) diff --git a/src/libosmo-mgcp/Makefile.am b/src/libosmo-mgcp/Makefile.am index c5e4bf4..e51da10 100644 --- a/src/libosmo-mgcp/Makefile.am +++ b/src/libosmo-mgcp/Makefile.am @@ -7,6 +7,7 @@ AM_CFLAGS = \ -Wall \ $(LIBOSMOCORE_CFLAGS) \ + $(LIBOSMOCODEC_CFLAGS) \ $(LIBOSMOGSM_CFLAGS) \ $(LIBOSMOVTY_CFLAGS) \ $(LIBOSMONETIF_CFLAGS) \ diff --git a/src/libosmo-mgcp/mgcp_e1.c b/src/libosmo-mgcp/mgcp_e1.c index f056d75..0e42e62 100644 --- a/src/libosmo-mgcp/mgcp_e1.c +++ b/src/libosmo-mgcp/mgcp_e1.c @@ -57,149 +57,6 @@ .sign_link = NULL, }; -/* The following EFR TRAU-DL frame is a dummy to be transmitted in the absence - * of RTP-derived TRAU-DL frames. The payload bit content here is the decoder - * homing frame (DHF) of TS 46.060 section 8.2 Table 7 - the best we can do - * in the absence of a proper TFO transform for EFR - while the full TRAU-DL - * frame was generated by passing said EFR DHF through osmo_rtp2trau(). - */ -static const ubit_t idle_tf_efr[] = { - 0, 0, 0, 0, 0, 0, 0, 0, - 0, 0, 0, 0, 0, 0, 0, 0, - 1, 1, 1, 0, 1, 0, 0, 0, - 0, 0, 0, 0, 1, 1, 1, 1, - 1, 1, 0, 0, 0, 0, 1, 0, - 0, 0, 0, 1, 0, 1, 1, 1, - 1, 1, 0, 1, 0, 1, 1, 0, - 1, 0, 0, 1, 0, 0, 1, 0, - 1, 0, 0, 0, 1, 1, 1, 1, - 1, 0, 0, 1, 0, 1, 0, 1, - 1, 0, 1, 0, 1, 1, 0, 1, - 0, 1, 1, 0, 0, 0, 0, 0, - 1, 0, 0, 1, 1, 1, 1, 1, - 0, 0, 0, 1, 1, 1, 0, 1, - 1, 0, 0, 0, 0, 1, 1, 0, - 0, 0, 0, 1, 1, 0, 0, 0, - 1, 0, 0, 0, 1, 1, 1, 1, - 0, 1, 1, 0, 1, 1, 0, 0, - 1, 0, 0, 1, 1, 0, 0, 0, - 1, 0, 0, 0, 0, 1, 0, 1, - 1, 1, 0, 0, 0, 0, 0, 0, - 1, 0, 0, 0, 0, 0, 0, 0, - 1, 0, 1, 0, 0, 1, 0, 0, - 0, 0, 0, 0, 0, 0, 1, 0, - 1, 0, 1, 0, 1, 0, 1, 0, - 1, 1, 0, 0, 0, 0, 0, 0, - 1, 0, 0, 0, 0, 0, 0, 0, - 0, 0, 0, 0, 0, 0, 0, 0, - 1, 0, 0, 0, 0, 0, 0, 0, - 0, 0, 0, 0, 0, 0, 0, 0, - 1, 0, 0, 0, 0, 0, 0, 0, - 0, 0, 0, 0, 1, 1, 1, 0, - 1, 1, 1, 0, 1, 0, 1, 1, - 0, 0, 0, 0, 0, 0, 0, 0, - 1, 0, 0, 0, 0, 0, 0, 0, - 0, 0, 0, 0, 0, 0, 0, 0, - 1, 0, 0, 0, 0, 0, 0, 0, - 0, 0, 0, 0, 0, 0, 0, 0, - 1, 0, 0, 0, 1, 1, 1, 1, - 1, 1, 1, 1, 1, 1, 1, 1, -}; - -/* The following FRv1 TRAU-DL frame is a dummy to be transmitted in the absence - * of RTP-derived TRAU-DL frames. The payload bit content here is the silence - * frame of TS 46.011 Table 1 - the best we can do without integrating the - * TFO transform for FRv1 from Themyscira libgsmfr2 - while the full TRAU-DL - * frame was generated by passing said FRv1 silence frame through - * osmo_rtp2trau(). - */ -static const ubit_t idle_tf_fr[] = { - 0, 0, 0, 0, 0, 0, 0, 0, - 0, 0, 0, 0, 0, 0, 0, 0, - 1, 1, 1, 1, 0, 0, 0, 0, - 0, 0, 0, 0, 1, 1, 1, 1, - 1, 0, 1, 0, 1, 0, 1, 1, - 1, 1, 0, 0, 1, 1, 0, 1, - 1, 0, 1, 0, 1, 0, 1, 0, - 1, 0, 0, 1, 0, 0, 1, 0, - 1, 1, 1, 0, 0, 1, 0, 0, - 0, 0, 1, 0, 1, 0, 0, 0, - 1, 1, 0, 0, 0, 0, 0, 0, - 0, 1, 1, 0, 0, 0, 1, 1, - 1, 1, 0, 0, 0, 1, 0, 0, - 1, 1, 1, 0, 1, 1, 0, 1, - 1, 1, 0, 1, 1, 0, 0, 0, - 1, 0, 0, 1, 1, 1, 0, 1, - 1, 1, 0, 0, 0, 0, 1, 0, - 1, 0, 0, 0, 1, 0, 0, 0, - 1, 0, 0, 0, 0, 1, 1, 0, - 0, 0, 1, 1, 1, 0, 0, 0, - 1, 1, 0, 0, 1, 1, 1, 0, - 1, 1, 0, 1, 1, 0, 1, 1, - 1, 0, 0, 0, 1, 0, 0, 1, - 1, 1, 0, 1, 1, 0, 0, 0, - 1, 0, 1, 0, 1, 0, 0, 0, - 1, 0, 0, 0, 0, 0, 0, 0, - 1, 1, 1, 0, 0, 0, 1, 1, - 1, 0, 0, 0, 1, 0, 0, 1, - 1, 1, 1, 0, 1, 1, 0, 1, - 1, 0, 1, 1, 0, 0, 0, 1, - 1, 0, 0, 1, 1, 1, 0, 1, - 1, 0, 0, 0, 0, 1, 0, 1, - 1, 0, 0, 0, 1, 0, 0, 0, - 0, 0, 0, 0, 1, 1, 0, 0, - 1, 0, 1, 1, 1, 0, 0, 0, - 1, 0, 0, 1, 1, 1, 0, 1, - 1, 1, 0, 1, 1, 0, 1, 1, - 0, 0, 0, 1, 0, 0, 1, 1, - 1, 1, 0, 1, 1, 0, 1, 1, - 1, 1, 1, 1, 1, 1, 1, 1, -}; - -/* Idle speech frame, see also GSM 08.60, chapter 3.4 */ -static const ubit_t idle_tf_spch[] = { - 0, 0, 0, 0, 0, 0, 0, 0, - 0, 0, 0, 0, 0, 0, 0, 0, - 1, 0, 1, 1, 1, 0, 0, 0, - 0, 0, 0, 0, 1, 0, 0, 0, - 1, 1, 1, 1, 1, 1, 1, 1, - 1, 1, 1, 1, 1, 1, 1, 1, - 1, 1, 1, 1, 1, 1, 1, 1, - 1, 1, 1, 1, 1, 1, 1, 1, - 1, 1, 1, 1, 1, 1, 1, 1, - 1, 1, 1, 1, 1, 1, 1, 1, - 1, 1, 1, 1, 1, 1, 1, 1, - 1, 1, 1, 1, 1, 1, 1, 1, - 1, 1, 1, 1, 1, 1, 1, 1, - 1, 1, 1, 1, 1, 1, 1, 1, - 1, 1, 1, 1, 1, 1, 1, 1, - 1, 1, 1, 1, 1, 1, 1, 1, - 1, 1, 1, 1, 1, 1, 1, 1, - 1, 1, 1, 1, 1, 1, 1, 1, - 1, 1, 1, 1, 1, 1, 1, 1, - 1, 1, 1, 1, 1, 1, 1, 1, - 1, 1, 1, 1, 1, 1, 1, 1, - 1, 1, 1, 1, 1, 1, 1, 1, - 1, 1, 1, 1, 1, 1, 1, 1, - 1, 1, 1, 1, 1, 1, 1, 1, - 1, 1, 1, 1, 1, 1, 1, 1, - 1, 1, 1, 1, 1, 1, 1, 1, - 1, 1, 1, 1, 1, 1, 1, 1, - 1, 1, 1, 1, 1, 1, 1, 1, - 1, 1, 1, 1, 1, 1, 1, 1, - 1, 1, 1, 1, 1, 1, 1, 1, - 1, 1, 1, 1, 1, 1, 1, 1, - 1, 1, 1, 1, 1, 1, 1, 1, - 1, 1, 1, 1, 1, 1, 1, 1, - 1, 1, 1, 1, 1, 1, 1, 1, - 1, 1, 1, 1, 1, 1, 1, 1, - 1, 1, 1, 1, 1, 1, 1, 1, - 1, 1, 1, 1, 1, 1, 1, 1, - 1, 1, 1, 1, 1, 1, 1, 1, - 1, 1, 1, 1, 1, 1, 1, 0, - 1, 1, 1, 1, 1, 1, 1, 1, -}; - /* If the RTP transmission has dropouts for some reason the I.460 TX-Queue may * run empty. In order to make sure that the TRAU frame transmission continues * we generate idle TRAU frames here. */ @@ -208,33 +65,54 @@ struct mgcp_endpoint *endp = user_data; struct rate_ctr_group *rate_ctrs = endp->trunk->ratectr.e1_stats; struct msgb *msg = msgb_alloc_c(endp->trunk, E1_TRAU_BITS_MSGB, "E1-I.460-IDLE-TX-TRAU-frame"); - uint8_t *ptr; - const uint8_t *ptr_ft; - enum osmo_trau_frame_type ft; + const uint8_t *dummy_fill_pl; + unsigned dummy_fill_pl_len; + struct osmo_trau_frame tf; + int rc; rate_ctr_inc(rate_ctr_group_get_ctr(rate_ctrs, E1_I460_TRAU_MUX_EMPTY_CTR)); - /* Choose an appropiate idle frame type */ - ft = endp->e1.trau_rtp_st->type; - switch (ft) { + /* choose dummy fill frame payload based on current codec */ + switch (endp->e1.trau_rtp_st->type) { case OSMO_TRAU16_FT_FR: - ptr_ft = idle_tf_fr; + dummy_fill_pl = osmo_gsm611_silence_frame; + dummy_fill_pl_len = GSM_FR_BYTES; break; case OSMO_TRAU16_FT_EFR: - ptr_ft = idle_tf_efr; + dummy_fill_pl = osmo_gsm660_homing_frame; + dummy_fill_pl_len = GSM_EFR_BYTES; break; default: - /* FIXME: What about 8k subslots and AMR frames? */ - ptr_ft = idle_tf_spch; + LOGPENDP(endp, DE1, LOGL_ERROR, "E1-I.460-IDLE-TX: unsupported frame type\n"); + goto skip; } - /* Put the replacement into a message buffer and enqueue it into the - * I.460 multiplexer */ - ptr = msgb_put(msg, E1_TRAU_BITS); - memcpy(ptr, ptr_ft, E1_TRAU_BITS); + /* turn it into a TRAU-DL frame */ + memset(&tf, 0, sizeof(tf)); + tf.dir = OSMO_TRAU_DIR_DL; + rc = osmo_rtp2trau(&tf, dummy_fill_pl, dummy_fill_pl_len, endp->e1.trau_rtp_st); + if (rc < 0) { + LOGPENDP(endp, DE1, LOGL_ERROR, + "E1-I.460-IDLE-TX: error converting dummy fill frame!\n"); + goto skip; + } + rc = osmo_trau_frame_encode(msgb_data(msg), msg->data_len, &tf); + if (rc < 0) { + LOGPENDP(endp, DE1, LOGL_ERROR, + "E1-I.460-IDLE-TX: error encoding dummy fill frame!\n"); + goto skip; + } + msgb_put(msg, rc); + + /* enqueue it into the I.460 multiplexer */ LOGPENDP(endp, DE1, LOGL_DEBUG, "E1-I.460-IDLE-TX: enquing %u trau frame bits: %s...\n", msgb_length(msg), osmo_ubit_dump(msgb_data(msg), msgb_length(msg) > DEBUG_BITS_MAX ? DEBUG_BITS_MAX : msgb_length(msg))); osmo_i460_mux_enqueue(endp->e1.schan, msg); + return; + +skip: + rate_ctr_inc(rate_ctr_group_get_ctr(rate_ctrs, E1_I460_TRAU_TX_FAIL_CTR)); + msgb_free(msg); } /* called by I.460 de-multiplexer; feed output of I.460 demux into TRAU frame sync */ diff --git a/src/osmo-mgw/Makefile.am b/src/osmo-mgw/Makefile.am index 928390b..0398f6d 100644 --- a/src/osmo-mgw/Makefile.am +++ b/src/osmo-mgw/Makefile.am @@ -27,6 +27,7 @@ osmo_mgw_LDADD = \ $(top_builddir)/src/libosmo-mgcp/libosmo-mgcp.a \ $(LIBOSMOCORE_LIBS) \ + $(LIBOSMOCODEC_LIBS) \ $(LIBOSMOVTY_LIBS) \ $(LIBOSMOGSM_LIBS) \ $(LIBOSMOCTRL_LIBS) \ diff --git a/tests/mgcp/Makefile.am b/tests/mgcp/Makefile.am index 78d9f6a..89c2cf1 100644 --- a/tests/mgcp/Makefile.am +++ b/tests/mgcp/Makefile.am @@ -36,6 +36,7 @@ mgcp_test_LDADD = \ $(top_builddir)/src/libosmo-mgcp/libosmo-mgcp.a \ $(LIBOSMOCORE_LIBS) \ + $(LIBOSMOCODEC_LIBS) \ $(LIBOSMOVTY_LIBS) \ $(LIBOSMOGSM_LIBS) \ $(LIBOSMOABIS_LIBS) \ -- To view, visit https://gerrit.osmocom.org/c/osmo-mgw/+/39750?usp=email To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings?usp=email Gerrit-MessageType: merged Gerrit-Project: osmo-mgw Gerrit-Branch: master Gerrit-Change-Id: I123f77295c56d61ff9aac5f5d64b25eca01e0d63 Gerrit-Change-Number: 39750 Gerrit-PatchSet: 5 Gerrit-Owner: falconia <falcon(a)freecalypso.org> Gerrit-Reviewer: Jenkins Builder Gerrit-Reviewer: falconia <falcon(a)freecalypso.org> Gerrit-Reviewer: laforge <laforge(a)osmocom.org> Gerrit-Reviewer: pespin <pespin(a)sysmocom.de>

1 0

[M] Change in osmo-mgw[master]: MGCP extension: add parameters for TW-TS-001 & TW-TS-002
by falconia 10 Mar '25

10 Mar '25

falconia has submitted this change. ( https://gerrit.osmocom.org/c/osmo-mgw/+/39730?usp=email ) Change subject: MGCP extension: add parameters for TW-TS-001 & TW-TS-002 ...................................................................... MGCP extension: add parameters for TW-TS-001 & TW-TS-002 TW-TS-001 and TW-TS-002 are enhanced RTP transport formats for GSM FR/HR/EFR codecs that restore the lost semantics of GSM 08.60 and 08.61 TRAU-UL frames. TW-TS-003 BSSMAP extension provides a way for a core network to ask the BSS to use these TRAU-like enhanced RTP formats instead of those specified in 3GPP TS 48.103; OsmoBSC already supports this mechanism when the BSS is comprised of IP-native OsmoBTS. However, in order to achieve the same effect when using E1-based legacy BTS hardware, the task of generating TW-TS-001/002 RTP packets for UL and accepting them for DL moves from OsmoBTS to the E1-Abis-interfacing MGW. osmo_trau2rtp() is already capable of generating these extended RTP formats on request, but until now there was no mechanism to signal from OsmoBSC to its associated E1 Abis MGW whether and when these extensions should be used. Considering that MGCP as it is used in Osmocom is essentially a private interface between OsmoBSC or OsmoMSC masters and OsmoMGW slaves, while the externally defined and generally interoperable interface is 3GPP TS 48.008, possibly extended with TW-TS-003, the sensible solution is to make a private extension to the way FR, EFR and HR codecs are described in SDP in the context of Osmocom-MGCP. The SDP extension birthed in the present patch consists of an fmtp parameter for GSM, GSM-EFR and GSM-HR-08 codecs that is structured just like the already implemented octet-align parameter for AMR. TW-TS-001 for FR and EFR shall be described as follows: m=audio 1234 RTP/AVP 3 110 a=rtpmap:3 GSM/8000/1 a=fmtp:3 tw-ts-001=1 a=rtpmap:110 GSM-EFR/8000/1 a=fmtp:110 tw-ts-001=1 TW-TS-002 for HR codec shall be described as follows: m=audio 1234 RTP/AVP 111 a=rtpmap:111 GSM-HR-08/8000/1 a=fmtp:111 tw-ts-002=1 The present patch affects two areas: * Experimental support for the newly defined extension is added to OsmoMGW-E1. This support is deemed experimental (not for production use) because even with this extension added, OsmoMGW-E1 is still unable to satisfy ThemWi requirements: neither ThemWi RTP endpoint library nor the TFO transform of TS 28.062 section C.3.2.1.1 are currently available in the repertoire of libraries whose use is allowed in mainline OsmoCNI components, yet both are required. * Support is added to libosmo-mgcp-client whereby OsmoBSC will be able to issue CRCX and MDCX commands to E1 Abis MGW endpoints with TW-TS-001 and/or TW-TS-002 enabled. Adding the necessary support to OsmoBSC will allow a complete working system to be deployed using OsmoBSC plus tw-e1abis-mgw, a replacement for OsmoMGW-E1 that works by using both Osmocom and ThemWi libraries. Related: OS#6614 Change-Id: I0d58e6d84418f50670c8ab7cf8490af3bc2f5c26 --- M doc/manuals/chapters/mgcp_extensions.adoc M include/osmocom/mgcp/mgcp_common.h M include/osmocom/mgcp/mgcp_endp.h M src/libosmo-mgcp-client/mgcp_client.c M src/libosmo-mgcp/mgcp_e1.c M src/libosmo-mgcp/mgcp_sdp.c 6 files changed, 160 insertions(+), 13 deletions(-) Approvals: laforge: Looks good to me, but someone else must approve Jenkins Builder: Verified pespin: Looks good to me, but someone else must approve falconia: Looks good to me, approved diff --git a/doc/manuals/chapters/mgcp_extensions.adoc b/doc/manuals/chapters/mgcp_extensions.adoc index df314bc..ebc0696 100644 --- a/doc/manuals/chapters/mgcp_extensions.adoc +++ b/doc/manuals/chapters/mgcp_extensions.adoc @@ -70,3 +70,85 @@ === `X-Osmux` See <<mgcp-extension-osmux>> + +=== Non-standard SDP codec parameters + +CRCX command message normally includes an SDP block that specifies the RTP +address of the connecting peer and the list of codecs accepted by that entity. +Codec specifications may include optional parameters given via `a=fmtp` lines; +one well-known parameter is `octet-align`, defined for AMR and AMR-WB codecs. + +In addition to standard (RFC-defined) codec parameters (of which the +just-mentioned `octet-align` parameter is the only one implemented so far), +OsmoMGW also understands some non-standard (Osmocom-private) codec parameters +as listed below. +These Osmocom-private codec parameters are meaningful only in the context +of OsmoBSC driving an E1 Abis MGW, which may be either OsmoMGW or a specialized +third-party replacement. + +The MGCP extension described in this section is meaningful only for E1 +endpoints - it has absolutely no effect on `rtpbridge` endpoints! + +==== `tw-ts-001` parameter for FR and EFR codecs + +GSM Full Rate codec is known in SDP as `GSM`; GSM Enhanced Full Rate codec +is known in SDP as `GSM-EFR`. +When either codec is to be transported in RTP, there is a choice between +two different payload formats: RFC 3551 or TW-TS-001. +RFC 3551 is the standard in non-GSM Internet applications, and it is the +format specified in later 3GPP releases for IP-based transport of +compressed speech - however, it exhibits functional regressions compared +to the original TRAU-UL frame format of GSM 08.60. +TW-TS-001 is an enhanced alternative payload format for the same codecs +that restores the original functionality and semantics of TRAU-UL frames. + +In the context of CRCX or MDCX command messages from OsmoBSC to an E1 Abis MGW, +`tw-ts-001` codec parameter (attached to either `GSM` or `GSM-EFR` codec) +instructs the MGW whether or not it should emit the extended payload format +of TW-TS-001 for GSM uplink traffic converted to RTP. +In the absence of this parameter, IETF-standard and 3GPP-standard RFC 3551 +format is emitted; if TW-TS-001 output is desired, please specify +`tw-ts-001=1` on the appropriate `a=fmtp` line. + +.Example: `CRCX` message requesting EFR codec with TW-TS-001 extensions +---- +CRCX 2 ds/e1-0/s-2/su16-2@mgw MGCP 1.0 +M: recvonly +C: 2 +L: p:20 + +v=0 +c=IN IP4 123.12.12.123 +m=audio 1234 RTP/AVP 110 +a=rtpmap:110 GSM-EFR/8000/1 +a=fmtp:110 tw-ts-001=1 +---- + +==== `tw-ts-002` parameter for HR codec + +GSM Half Rate codec is known in SDP as `GSM-HR-08`. +The same dichotomy that exists between RFC 3551 and TW-TS-001 RTP payload +formats for FR and EFR codecs also exists for HR codec, except that +the two opposing alternatives are now RFC 5993 and TW-TS-002. + +In the context of CRCX or MDCX command messages from OsmoBSC to an E1 Abis MGW, +`tw-ts-002` codec parameter (attached to `GSM-HR-08` codec) +instructs the MGW whether or not it should emit the extended payload format +of TW-TS-002 for GSM uplink traffic converted to RTP. +In the absence of this parameter, IETF-standard and 3GPP-standard RFC 5993 +format is emitted; if TW-TS-002 output is desired, please specify +`tw-ts-002=1` on the appropriate `a=fmtp` line. + +.Example: `CRCX` message requesting HR codec with TW-TS-002 extensions +---- +CRCX 2 ds/e1-0/s-2/su8-2@mgw MGCP 1.0 +M: recvonly +C: 2 +L: p:20 + +v=0 +c=IN IP4 123.12.12.123 +m=audio 1234 RTP/AVP 111 +a=rtpmap:111 GSM-HR-08/8000/1 +a=fmtp:111 tw-ts-002=1 +---- diff --git a/include/osmocom/mgcp/mgcp_common.h b/include/osmocom/mgcp/mgcp_common.h index 05ae95f..14b226f 100644 --- a/include/osmocom/mgcp/mgcp_common.h +++ b/include/osmocom/mgcp/mgcp_common.h @@ -63,6 +63,10 @@ struct mgcp_codec_param { bool amr_octet_aligned_present; bool amr_octet_aligned; + bool fr_efr_twts001_present; + bool fr_efr_twts001; + bool hr_twts002_present; + bool hr_twts002; }; /* Ensure that the msg->l2h is NUL terminated. */ diff --git a/include/osmocom/mgcp/mgcp_endp.h b/include/osmocom/mgcp/mgcp_endp.h index fbe0850..3366674 100644 --- a/include/osmocom/mgcp/mgcp_endp.h +++ b/include/osmocom/mgcp/mgcp_endp.h @@ -124,6 +124,7 @@ struct osmo_fsm_inst *trau_sync_fi; struct osmo_trau2rtp_state *trau_rtp_st; uint8_t last_amr_ft; + uint8_t rtp_extensions; struct mgcp_rtp_codec *last_codec; } e1; diff --git a/src/libosmo-mgcp-client/mgcp_client.c b/src/libosmo-mgcp-client/mgcp_client.c index ab8ba9f..63d03f9 100644 --- a/src/libosmo-mgcp-client/mgcp_client.c +++ b/src/libosmo-mgcp-client/mgcp_client.c @@ -1426,15 +1426,32 @@ /* Add optional codec parameters (fmtp) */ if (mgcp_msg->param_present) { for (i = 0; i < mgcp_msg->ptmap_len; i++) { - /* The following is only applicable for AMR */ - if (mgcp_msg->ptmap[i].codec != CODEC_AMR_8000_1 - && mgcp_msg->ptmap[i].codec != CODEC_AMRWB_16000_1) - continue; pt = mgcp_msg->ptmap[i].pt; - if (mgcp_msg->param.amr_octet_aligned_present && mgcp_msg->param.amr_octet_aligned) - MSGB_PRINTF_OR_RET("a=fmtp:%u octet-align=1\r\n", pt); - else if (mgcp_msg->param.amr_octet_aligned_present && !mgcp_msg->param.amr_octet_aligned) - MSGB_PRINTF_OR_RET("a=fmtp:%u octet-align=0\r\n", pt); + switch (mgcp_msg->ptmap[i].codec) { + case CODEC_AMR_8000_1: + case CODEC_AMRWB_16000_1: + if (!mgcp_msg->param.amr_octet_aligned_present) + break; + MSGB_PRINTF_OR_RET("a=fmtp:%u octet-align=%d\r\n", + pt, (int)mgcp_msg->param.amr_octet_aligned); + break; + case CODEC_GSM_8000_1: + case CODEC_GSMEFR_8000_1: + if (!mgcp_msg->param.fr_efr_twts001_present) + break; + MSGB_PRINTF_OR_RET("a=fmtp:%u tw-ts-001=%d\r\n", + pt, (int)mgcp_msg->param.fr_efr_twts001); + break; + case CODEC_GSMHR_8000_1: + if (!mgcp_msg->param.hr_twts002_present) + break; + MSGB_PRINTF_OR_RET("a=fmtp:%u tw-ts-002=%d\r\n", + pt, (int)mgcp_msg->param.hr_twts002); + break; + default: + /* no parameters for the remaining codecs */ + break; + } } } diff --git a/src/libosmo-mgcp/mgcp_e1.c b/src/libosmo-mgcp/mgcp_e1.c index 3a7a450..f056d75 100644 --- a/src/libosmo-mgcp/mgcp_e1.c +++ b/src/libosmo-mgcp/mgcp_e1.c @@ -40,6 +40,7 @@ #include <osmocom/mgcp/debug.h> #include <osmocom/mgcp/mgcp_e1.h> #include <osmocom/codec/codec.h> +#include <osmocom/gsm/rtp_extensions.h> #define DEBUG_BITS_MAX 80 #define DEBUG_BYTES_MAX 40 @@ -294,6 +295,7 @@ /* Convert decoded trau frame to RTP frame */ struct osmo_trau2rtp_state t2rs = { .type = fr.type, + .rtp_extensions = endp->e1.rtp_extensions, }; rc = osmo_trau2rtp(msgb_data(msg) + rtp_hdr_len, msg->data_len - rtp_hdr_len, &fr, &t2rs); if (rc <= 0) { @@ -682,6 +684,13 @@ determine_trau_fr_type(codec->subtype_name, endp->e1.scd.rate, endp->e1.last_amr_ft, endp); endp->e1.last_codec = codec; + /* possible RTP extensions, codec-associated */ + endp->e1.rtp_extensions = 0; + if (codec->param_present && codec->param.fr_efr_twts001) + endp->e1.rtp_extensions |= OSMO_RTP_EXT_TWTS001; + if (codec->param_present && codec->param.hr_twts002) + endp->e1.rtp_extensions |= OSMO_RTP_EXT_TWTS002; + /* Update sync pattern */ sync_pat_id = determine_trau_sync_pat(codec->subtype_name, endp->e1.scd.rate, endp->e1.last_amr_ft, endp); osmo_trau_sync_set_pat(endp->e1.trau_sync_fi, sync_pat_id); diff --git a/src/libosmo-mgcp/mgcp_sdp.c b/src/libosmo-mgcp/mgcp_sdp.c index 1d76ddb..2f21678 100644 --- a/src/libosmo-mgcp/mgcp_sdp.c +++ b/src/libosmo-mgcp/mgcp_sdp.c @@ -199,7 +199,7 @@ unsigned int pt; unsigned int count = 0; char delimiter; - unsigned int amr_octet_aligned; + unsigned int param_val; memset(fmtp_param, 0, sizeof(*fmtp_param)); @@ -239,12 +239,30 @@ if (delimiter == ';' || delimiter == ',') str_ptr[strlen(str_ptr) - 1] = '\0'; - /* AMR octet aligned parameter (see also RFC 3267, section 8.3) */ - if (sscanf(param_str, "octet-align=%d", &amr_octet_aligned) == 1) { + /* We support the following codec parameters: + * + * AMR: octet-align parameter of RFC 4867 section 8.3; + * FR & EFR: tw-ts-001 parameter, Osmocom private; + * HR: tw-ts-002 parameter, Osmocom private. + * + * tw-ts-001 and tw-ts-002 Osmocom-private parameters are used + * only in the context of OsmoBSC driving an E1 MGW endpoint. + */ + if (sscanf(param_str, "octet-align=%d", &param_val) == 1) { fmtp_param->param.amr_octet_aligned_present = true; fmtp_param->param.amr_octet_aligned = false; - if (amr_octet_aligned == 1) + if (param_val == 1) fmtp_param->param.amr_octet_aligned = true; + } else if (sscanf(param_str, "tw-ts-001=%d", &param_val) == 1) { + fmtp_param->param.fr_efr_twts001_present = true; + fmtp_param->param.fr_efr_twts001 = false; + if (param_val == 1) + fmtp_param->param.fr_efr_twts001 = true; + } else if (sscanf(param_str, "tw-ts-002=%d", &param_val) == 1) { + fmtp_param->param.hr_twts002_present = true; + fmtp_param->param.hr_twts002 = false; + if (param_val == 1) + fmtp_param->param.hr_twts002 = true; } param_str = strtok(NULL, " "); @@ -559,7 +577,23 @@ goto buffer_too_small; } - if (codec->param_present) { + /* Include a=fmtp line in MGCP response only for AMR + * octet-align parameter, not for tw-ts-* parameters. + * Rationale: + * + * - tw-ts-* parameters exist meaningfully only for E1 + * endpoints driven by OsmoBSC, not in any other + * context. libosmo-mgcp-client used by OsmoBSC + * completely ignores all fmtp lines, has no code + * to parse them at all. + * + * - The SDP we return is supposed to indicate what + * _we_ accept, conceptually independent from what + * the remote accepts. And we always accept TW-TS-001 + * and TW-TS-002 RTP formats going to E1 DL, whether + * or not we send them in UL per client request. + */ + if (codec->param_present && codec->param.amr_octet_aligned_present) { fmtp_param.payload_type = payload_type; fmtp_param.param = codec->param; fmtp_params[0] = fmtp_param; -- To view, visit https://gerrit.osmocom.org/c/osmo-mgw/+/39730?usp=email To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings?usp=email Gerrit-MessageType: merged Gerrit-Project: osmo-mgw Gerrit-Branch: master Gerrit-Change-Id: I0d58e6d84418f50670c8ab7cf8490af3bc2f5c26 Gerrit-Change-Number: 39730 Gerrit-PatchSet: 3 Gerrit-Owner: falconia <falcon(a)freecalypso.org> Gerrit-Reviewer: Jenkins Builder Gerrit-Reviewer: falconia <falcon(a)freecalypso.org> Gerrit-Reviewer: laforge <laforge(a)osmocom.org> Gerrit-Reviewer: pespin <pespin(a)sysmocom.de>

1 0

[XS] Change in osmo-mgw[master]: E1: implement dummy fill for HRv1 codec
by falconia 10 Mar '25

10 Mar '25

falconia has submitted this change. ( https://gerrit.osmocom.org/c/osmo-mgw/+/39751?usp=email ) Change subject: E1: implement dummy fill for HRv1 codec ...................................................................... E1: implement dummy fill for HRv1 codec In the absence of a proper TFO transform (TS 28.062 section C3.2.1.1), OsmoMGW-E1 inserts constant fill frames when there is no RTP input, or when RTP input fails conversion to TRAU-DL and thus does not enqueue anything to the I.460 mux. This dummy fill was previously implemented for FR and EFR codecs, but not for HRv1. Fix this omission. With this change and with a fix to OsmoBSC to select 8k subslot endpoints for TCH/H instead of 16k, OsmoMGW-E1 now supports HR codec to the same degree to which it supports FR and EFR: far from perfect, but mostly usable, especially if DTXu is disabled and there is no TrFO interworking with a far end that does DTXu. Change-Id: I188bc0a0468b19126281016f45e36f1de617e9ee --- M src/libosmo-mgcp/mgcp_e1.c 1 file changed, 5 insertions(+), 0 deletions(-) Approvals: laforge: Looks good to me, approved Jenkins Builder: Verified pespin: Looks good to me, but someone else must approve diff --git a/src/libosmo-mgcp/mgcp_e1.c b/src/libosmo-mgcp/mgcp_e1.c index 0e42e62..dc0acca 100644 --- a/src/libosmo-mgcp/mgcp_e1.c +++ b/src/libosmo-mgcp/mgcp_e1.c @@ -82,6 +82,11 @@ dummy_fill_pl = osmo_gsm660_homing_frame; dummy_fill_pl_len = GSM_EFR_BYTES; break; + case OSMO_TRAU16_FT_HR: + case OSMO_TRAU8_SPEECH: + dummy_fill_pl = osmo_gsm620_silence_frame; + dummy_fill_pl_len = GSM_HR_BYTES; + break; default: LOGPENDP(endp, DE1, LOGL_ERROR, "E1-I.460-IDLE-TX: unsupported frame type\n"); goto skip; -- To view, visit https://gerrit.osmocom.org/c/osmo-mgw/+/39751?usp=email To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings?usp=email Gerrit-MessageType: merged Gerrit-Project: osmo-mgw Gerrit-Branch: master Gerrit-Change-Id: I188bc0a0468b19126281016f45e36f1de617e9ee Gerrit-Change-Number: 39751 Gerrit-PatchSet: 6 Gerrit-Owner: falconia <falcon(a)freecalypso.org> Gerrit-Reviewer: Jenkins Builder Gerrit-Reviewer: falconia <falcon(a)freecalypso.org> Gerrit-Reviewer: laforge <laforge(a)osmocom.org> Gerrit-Reviewer: pespin <pespin(a)sysmocom.de>

1 0

[M] Change in osmo-mgw[master]: MGCP extension: add parameters for TW-TS-001 & TW-TS-002
by falconia 10 Mar '25

10 Mar '25

Attention is currently required from: pespin. falconia has posted comments on this change by falconia. ( https://gerrit.osmocom.org/c/osmo-mgw/+/39730?usp=email ) Change subject: MGCP extension: add parameters for TW-TS-001 & TW-TS-002 ...................................................................... Patch Set 2: Code-Review+2 (2 comments) Patchset: PS1: > Please see the new patch version. […] Done Patchset: PS2: Setting CR+2 based on CR+1 votes from pespin and laforge. -- To view, visit https://gerrit.osmocom.org/c/osmo-mgw/+/39730?usp=email To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings?usp=email Gerrit-MessageType: comment Gerrit-Project: osmo-mgw Gerrit-Branch: master Gerrit-Change-Id: I0d58e6d84418f50670c8ab7cf8490af3bc2f5c26 Gerrit-Change-Number: 39730 Gerrit-PatchSet: 2 Gerrit-Owner: falconia <falcon(a)freecalypso.org> Gerrit-Reviewer: Jenkins Builder Gerrit-Reviewer: falconia <falcon(a)freecalypso.org> Gerrit-Reviewer: laforge <laforge(a)osmocom.org> Gerrit-Reviewer: pespin <pespin(a)sysmocom.de> Gerrit-Attention: pespin <pespin(a)sysmocom.de> Gerrit-Comment-Date: Mon, 10 Mar 2025 19:26:28 +0000 Gerrit-HasComments: Yes Gerrit-Has-Labels: Yes Comment-In-Reply-To: falconia <falcon(a)freecalypso.org> Comment-In-Reply-To: pespin <pespin(a)sysmocom.de>

1 0

2026

2025

2024

2023

2022

gerrit-log