Attention is currently required from: fixeria, laforge, osmith, pespin.
Hello Jenkins Builder, fixeria, laforge, osmith,
I'd like you to reexamine a change. Please visit
https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/43383?usp=email
to look at the new patch set (#2).
The following approvals got outdated and were removed:
Code-Review+1 by osmith, Verified+1 by Jenkins Builder
Change subject: stp: Use SI in valid range when transmitting data
......................................................................
stp: Use SI in valid range when transmitting data
ITU Q.704 14.2.1 defines Service Indicator as being 4 bits, hence valid
range of the field is 0..0x0F. M3UA encodes it inside a full octet
"justified to the least significant bit. Unused bits are coded `0'."
Make sure we send a value inside range, since newer libosmo-sigtran will
validate the received m3ua field.
Use SI=0010 "spare" since we are not really sending any meaningful data.
Change-Id: I1d0553b0ed57b9283dd988e0f686dea8a4f17ebb
---
M stp/STP_Tests_M3UA.ttcn
1 file changed, 10 insertions(+), 10 deletions(-)
git pull ssh://gerrit.osmocom.org:29418/osmo-ttcn3-hacks refs/changes/83/43383/2
--
To view, visit https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/43383?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: I1d0553b0ed57b9283dd988e0f686dea8a4f17ebb
Gerrit-Change-Number: 43383
Gerrit-PatchSet: 2
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: pespin <pespin(a)sysmocom.de>
Gerrit-Attention: fixeria <vyanitskiy(a)sysmocom.de>
Attention is currently required from: biocape, laforge.
fixeria has posted comments on this change by biocape. ( https://gerrit.osmocom.org/c/pysim/+/41913?usp=email )
Change subject: Decode GSM 7-bit packed EF.PNN data
......................................................................
Patch Set 3: Code-Review+2
(1 comment)
File pySim/utils.py:
https://gerrit.osmocom.org/c/pysim/+/41913/comment/22f6be3a_8d75c061?usp=em… :
PS1, Line 81: de
> I would think those are useful/relevant beyond pySim and should go into pyosmocom instead?
Done
--
To view, visit https://gerrit.osmocom.org/c/pysim/+/41913?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: I4558fd011592aeeae2389fe8a1f62f3d7e21d219
Gerrit-Change-Number: 41913
Gerrit-PatchSet: 3
Gerrit-Owner: biocape <biofel(a)cape.co>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: fixeria <vyanitskiy(a)sysmocom.de>
Gerrit-Reviewer: laforge <laforge(a)osmocom.org>
Gerrit-Attention: laforge <laforge(a)osmocom.org>
Gerrit-Attention: biocape <biofel(a)cape.co>
Gerrit-Comment-Date: Tue, 25 Aug 2026 10:06:31 +0000
Gerrit-HasComments: Yes
Gerrit-Has-Labels: Yes
Comment-In-Reply-To: laforge <laforge(a)osmocom.org>
Attention is currently required from: fixeria, laforge, pespin.
osmith has posted comments on this change by pespin. ( https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/43383?usp=email )
Change subject: stp: Use SI in valid range when transmitting data
......................................................................
Patch Set 1: Code-Review+1
--
To view, visit https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/43383?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: I1d0553b0ed57b9283dd988e0f686dea8a4f17ebb
Gerrit-Change-Number: 43383
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: laforge <laforge(a)osmocom.org>
Gerrit-Attention: pespin <pespin(a)sysmocom.de>
Gerrit-Attention: fixeria <vyanitskiy(a)sysmocom.de>
Gerrit-Comment-Date: Tue, 25 Aug 2026 10:05:26 +0000
Gerrit-HasComments: No
Gerrit-Has-Labels: Yes
pespin has uploaded this change for review. ( https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/43383?usp=email )
Change subject: stp: Use SI in valid range when transmitting data
......................................................................
stp: Use SI in valid range when transmitting data
ITU Q.704 14.2.1 defines Service Indicator as being 4 bits, hence valid
range of the field is 0..0x0F. M3UA encodes it inside a full octet
"justified to the least significant bit. Unused bits are coded `0'."
Make sure we send a value inside range, since newer libosmo-sigtran will
validate the received m3ua field.
Use SI=0010 "spare" since we are not really sending any meaningful data.
Change-Id: I1d0553b0ed57b9283dd988e0f686dea8a4f17ebb
---
M stp/STP_Tests_M3UA.ttcn
1 file changed, 1 insertion(+), 1 deletion(-)
git pull ssh://gerrit.osmocom.org:29418/osmo-ttcn3-hacks refs/changes/83/43383/1
diff --git a/stp/STP_Tests_M3UA.ttcn b/stp/STP_Tests_M3UA.ttcn
index f1ce309..f740f15 100644
--- a/stp/STP_Tests_M3UA.ttcn
+++ b/stp/STP_Tests_M3UA.ttcn
@@ -776,7 +776,7 @@
/* Test if traffic is routed from idx_tx/pc_tx to idx_rx/pc_rx */
friend function f_test_traffic(integer idx_tx, template (omit) OCT4 rctx_sender, OCT4 pc_tx,
integer idx_rx, template (omit) OCT4 rctx_receiver, OCT4 pc_rx,
- OCT1 si := '23'O, OCT1 ni := mp_ni, OCT1 mp := '00'O, OCT1 sls := '00'O)
+ OCT1 si := '02'O, OCT1 ni := mp_ni, OCT1 mp := '00'O, OCT1 sls := '00'O)
runs on RAW_M3UA_CT {
var octetstring data := f_rnd_octstring_rnd_len(100);
f_M3UA_send(idx_tx, ts_M3UA_DATA(rctx_sender,
--
To view, visit https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/43383?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: I1d0553b0ed57b9283dd988e0f686dea8a4f17ebb
Gerrit-Change-Number: 43383
Gerrit-PatchSet: 1
Gerrit-Owner: pespin <pespin(a)sysmocom.de>
Attention is currently required from: biocape, dexter.
fixeria has posted comments on this change by biocape. ( https://gerrit.osmocom.org/c/python/pyosmocom/+/43371?usp=email )
Change subject: Add GSM 7-bit packing and Network Name IE support
......................................................................
Patch Set 1: Code-Review+1
(1 comment)
File src/osmocom/utils.py:
https://gerrit.osmocom.org/c/python/pyosmocom/+/43371/comment/cbd43c9e_754f… :
PS1, Line 121: int.from_bytes(packed, byteorder='little')
I like the approach, but there is a problem here: `int.from_bytes()` with `byteorder='little'` will drop any trailing 0x00 bytes in the stream. It's probably safe for the Network Name IE, but this function may also be used for general GSM 7-bit unpacking where data can end in zeros (like SMS text). You can append a dummy byte before converting to prevent bit-shifting into empty padding:
```suggestion
packed_int = int.from_bytes(packed + b'\x01', byteorder='little')
```
--
To view, visit https://gerrit.osmocom.org/c/python/pyosmocom/+/43371?usp=email
To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings?usp=email
Gerrit-MessageType: comment
Gerrit-Project: python/pyosmocom
Gerrit-Branch: master
Gerrit-Change-Id: I4661352e39031dc0041fc7851f675d6083a1035f
Gerrit-Change-Number: 43371
Gerrit-PatchSet: 1
Gerrit-Owner: biocape <biofel(a)cape.co>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: dexter <pmaier(a)sysmocom.de>
Gerrit-Reviewer: fixeria <vyanitskiy(a)sysmocom.de>
Gerrit-Reviewer: laforge <laforge(a)osmocom.org>
Gerrit-Attention: biocape <biofel(a)cape.co>
Gerrit-Attention: dexter <pmaier(a)sysmocom.de>
Gerrit-Comment-Date: Tue, 25 Aug 2026 10:02:08 +0000
Gerrit-HasComments: Yes
Gerrit-Has-Labels: Yes
Attention is currently required from: pespin.
laforge has posted comments on this change by pespin. ( https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/43379?usp=email )
Change subject: library/pfcp: Support passing CP/UP Function Features to Assoc Setup helpers
......................................................................
Patch Set 2: Code-Review+1
--
To view, visit https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/43379?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: I8d74d56e3a602890f2559abbc671bfa3ee3c6e0d
Gerrit-Change-Number: 43379
Gerrit-PatchSet: 2
Gerrit-Owner: pespin <pespin(a)sysmocom.de>
Gerrit-Reviewer: fixeria <vyanitskiy(a)sysmocom.de>
Gerrit-Reviewer: laforge <laforge(a)osmocom.org>
Gerrit-Reviewer: osmith <osmith(a)sysmocom.de>
Gerrit-Attention: pespin <pespin(a)sysmocom.de>
Gerrit-Comment-Date: Tue, 25 Aug 2026 09:23:37 +0000
Gerrit-HasComments: No
Gerrit-Has-Labels: Yes