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

[M] Change in onomondo-ipa[master]: V1.2: Add StateChangeCause to GetEimPackageRequest
by jolly 21 Aug '26

21 Aug '26

Attention is currently required from: jolly. Hello Jenkins Builder, dexter, I'd like you to reexamine a change. Please visit https://gerrit.osmocom.org/c/onomondo-ipa/+/43057?usp=email to look at the new patch set (#7). 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: V1.2: Add StateChangeCause to GetEimPackageRequest ...................................................................... V1.2: Add StateChangeCause to GetEimPackageRequest State change notification is optional and currently not supported by IPA. Reference: SGP.32 Section 6.3.2.6 Related: SYS#8101 Change-Id: I1ae561f6709a9eb490fa7a53420c332ff20bd058 --- M asn1/SGP32Definitions.asn M src/ipa/libasn/CMakeLists.txt M src/ipa/libasn/GetEimPackageRequest.c M src/ipa/libasn/GetEimPackageRequest.h A src/ipa/libasn/StateChangeCause.c A src/ipa/libasn/StateChangeCause.h 6 files changed, 124 insertions(+), 13 deletions(-) git pull ssh://gerrit.osmocom.org:29418/onomondo-ipa refs/changes/57/43057/7 -- To view, visit https://gerrit.osmocom.org/c/onomondo-ipa/+/43057?usp=email To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings?usp=email Gerrit-MessageType: newpatchset Gerrit-Project: onomondo-ipa Gerrit-Branch: master Gerrit-Change-Id: I1ae561f6709a9eb490fa7a53420c332ff20bd058 Gerrit-Change-Number: 43057 Gerrit-PatchSet: 7 Gerrit-Owner: jolly <andreas(a)eversberg.eu> Gerrit-Reviewer: Jenkins Builder Gerrit-Reviewer: dexter <pmaier(a)sysmocom.de> Gerrit-Attention: jolly <andreas(a)eversberg.eu>

1 0

[L] Change in onomondo-ipa[master]: V1.2: Add function ES10b.GetConnectivityParameters
by jolly 21 Aug '26

21 Aug '26

Attention is currently required from: dexter, jolly, laforge. Hello Jenkins Builder, dexter, laforge, I'd like you to reexamine a change. Please visit https://gerrit.osmocom.org/c/onomondo-ipa/+/43054?usp=email to look at the new patch set (#8). The following approvals got outdated and were removed: Code-Review+1 by dexter, Code-Review+1 by laforge, Verified+1 by Jenkins Builder Change subject: V1.2: Add function ES10b.GetConnectivityParameters ...................................................................... V1.2: Add function ES10b.GetConnectivityParameters Add new library function to get connectivity parameters from IoT eUICC. Also add an option to the command line interface to call the library function. The function cannot be emulated with non-IoT eUICC. Reference: SGP.32 Section 5.9.24 Related: SYS#8101 Change-Id: I075590e095dbbe05fa6cdb0a96d5b38c7a17ff78 --- M asn1/SGP32Definitions.asn M include/onomondo/ipa/ipad.h M src/ipa/libasn/CMakeLists.txt A src/ipa/libasn/ConnectivityParameters.c A src/ipa/libasn/ConnectivityParameters.h A src/ipa/libasn/ConnectivityParametersError.c A src/ipa/libasn/ConnectivityParametersError.h A src/ipa/libasn/GetConnectivityParametersRequest.c A src/ipa/libasn/GetConnectivityParametersRequest.h A src/ipa/libasn/GetConnectivityParametersResponse.c A src/ipa/libasn/GetConnectivityParametersResponse.h M src/ipa/libipa/CMakeLists.txt A src/ipa/libipa/es10b_get_conn_params.c A src/ipa/libipa/es10b_get_conn_params.h M src/ipa/libipa/ipad.c M src/ipa/main.c 16 files changed, 588 insertions(+), 0 deletions(-) git pull ssh://gerrit.osmocom.org:29418/onomondo-ipa refs/changes/54/43054/8 -- To view, visit https://gerrit.osmocom.org/c/onomondo-ipa/+/43054?usp=email To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings?usp=email Gerrit-MessageType: newpatchset Gerrit-Project: onomondo-ipa Gerrit-Branch: master Gerrit-Change-Id: I075590e095dbbe05fa6cdb0a96d5b38c7a17ff78 Gerrit-Change-Number: 43054 Gerrit-PatchSet: 8 Gerrit-Owner: jolly <andreas(a)eversberg.eu> Gerrit-Reviewer: Jenkins Builder Gerrit-Reviewer: dexter <pmaier(a)sysmocom.de> Gerrit-Reviewer: laforge <laforge(a)osmocom.org> Gerrit-Attention: jolly <andreas(a)eversberg.eu> Gerrit-Attention: laforge <laforge(a)osmocom.org> Gerrit-Attention: dexter <pmaier(a)sysmocom.de>

1 0

[L] Change in onomondo-ipa[master]: V1.2: Change fields in ProvideEimPackage(Result)
by jolly 21 Aug '26

21 Aug '26

Attention is currently required from: dexter, jolly, laforge. Hello Jenkins Builder, dexter, laforge, I'd like you to reexamine a change. Please visit https://gerrit.osmocom.org/c/onomondo-ipa/+/43055?usp=email to look at the new patch set (#7). The following approvals got outdated and were removed: Code-Review+1 by dexter, Code-Review+1 by laforge, Verified+1 by Jenkins Builder Change subject: V1.2: Change fields in ProvideEimPackage(Result) ...................................................................... V1.2: Change fields in ProvideEimPackage(Result) Change parameters in ProvideEimPackage and ProvideEimPackageResult. The structure is changed in IPA too. The new field eimTransactionId of EimPackageResultResponseError is filled with the eimTransactionId of IpaEuiccDataRequest, if exists. Reference: SGP.32 Section 5.14.6 Related: SYS#8101 Change-Id: I24baab4f9351aec27c58403b94974cba4b274c34 --- M asn1/SGP32Definitions.asn M src/ipa/libasn/CMakeLists.txt A src/ipa/libasn/EimPackageResult.c A src/ipa/libasn/EimPackageResult.h A src/ipa/libasn/EimPackageResultErrorCode.c A src/ipa/libasn/EimPackageResultErrorCode.h A src/ipa/libasn/EimPackageResultResponseError.c A src/ipa/libasn/EimPackageResultResponseError.h M src/ipa/libasn/ProvideEimPackageResult.c M src/ipa/libasn/ProvideEimPackageResult.h M src/ipa/libasn/ProvideEimPackageResultResponse.c M src/ipa/libasn/ProvideEimPackageResultResponse.h M src/ipa/libipa/esipa_prvde_eim_pkg_rslt.c M src/ipa/libipa/esipa_prvde_eim_pkg_rslt.h M src/ipa/libipa/proc_euicc_data_req.c M src/ipa/libipa/proc_euicc_pkg_dwnld_exec.c 16 files changed, 637 insertions(+), 220 deletions(-) git pull ssh://gerrit.osmocom.org:29418/onomondo-ipa refs/changes/55/43055/7 -- To view, visit https://gerrit.osmocom.org/c/onomondo-ipa/+/43055?usp=email To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings?usp=email Gerrit-MessageType: newpatchset Gerrit-Project: onomondo-ipa Gerrit-Branch: master Gerrit-Change-Id: I24baab4f9351aec27c58403b94974cba4b274c34 Gerrit-Change-Number: 43055 Gerrit-PatchSet: 7 Gerrit-Owner: jolly <andreas(a)eversberg.eu> Gerrit-Reviewer: Jenkins Builder Gerrit-Reviewer: dexter <pmaier(a)sysmocom.de> Gerrit-Reviewer: laforge <laforge(a)osmocom.org> Gerrit-Attention: jolly <andreas(a)eversberg.eu> Gerrit-Attention: laforge <laforge(a)osmocom.org> Gerrit-Attention: dexter <pmaier(a)sysmocom.de>

1 0

[S] Change in onomondo-ipa[master]: V1.2: Add eimTransactionId to InitiateAuthenticationRequestEsipa
by jolly 21 Aug '26

21 Aug '26

Attention is currently required from: jolly. Hello Jenkins Builder, dexter, laforge, I'd like you to reexamine a change. Please visit https://gerrit.osmocom.org/c/onomondo-ipa/+/43056?usp=email to look at the new patch set (#7). 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: V1.2: Add eimTransactionId to InitiateAuthenticationRequestEsipa ...................................................................... V1.2: Add eimTransactionId to InitiateAuthenticationRequestEsipa Add new eimTransactionId field to InitiateAuthenticationRequestEsipa. The eimTransactionId walks through the IPA process like this: * The eimTransactionId is retrieved from ProfileDownloadTriggerRequest at proc_eim_pkg_retr.c. * Then it is handed to the Common Mutual Authentication Procedure at proc_indirect_prfle_dwnld.c. * There it is handed to the InitiateAuthentication function at proc_cmn_mtl_auth.c. * There it is stored in initiateAuthenticationRequestEsipa at esipa_init_auth.c. Reference: SGP.32 Section 5.14.1 Related: SYS#8101 Change-Id: I1c8ef6fc0a3f7ab5735e401597fc8c6b2021fc09 --- M asn1/SGP32Definitions.asn M src/ipa/libasn/InitiateAuthenticationRequestEsipa.c M src/ipa/libasn/InitiateAuthenticationRequestEsipa.h M src/ipa/libipa/esipa_init_auth.c M src/ipa/libipa/esipa_init_auth.h M src/ipa/libipa/proc_cmn_mtl_auth.c M src/ipa/libipa/proc_cmn_mtl_auth.h M src/ipa/libipa/proc_eim_pkg_retr.c M src/ipa/libipa/proc_indirect_prfle_dwnld.c M src/ipa/libipa/proc_indirect_prfle_dwnld.h 10 files changed, 31 insertions(+), 9 deletions(-) git pull ssh://gerrit.osmocom.org:29418/onomondo-ipa refs/changes/56/43056/7 -- To view, visit https://gerrit.osmocom.org/c/onomondo-ipa/+/43056?usp=email To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings?usp=email Gerrit-MessageType: newpatchset Gerrit-Project: onomondo-ipa Gerrit-Branch: master Gerrit-Change-Id: I1c8ef6fc0a3f7ab5735e401597fc8c6b2021fc09 Gerrit-Change-Number: 43056 Gerrit-PatchSet: 7 Gerrit-Owner: jolly <andreas(a)eversberg.eu> Gerrit-Reviewer: Jenkins Builder Gerrit-Reviewer: dexter <pmaier(a)sysmocom.de> Gerrit-Reviewer: laforge <laforge(a)osmocom.org> Gerrit-Attention: jolly <andreas(a)eversberg.eu>

1 0

[S] Change in osmo-ttcn3-hacks[master]: fixup Add close function to HTTP_Server_Emulation
by jolly 21 Aug '26

21 Aug '26

jolly has abandoned this change. ( https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/43282?usp=email ) Change subject: fixup Add close function to HTTP_Server_Emulation ...................................................................... Abandoned Needs to be squashed with other patch -- To view, visit https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/43282?usp=email To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings?usp=email Gerrit-MessageType: abandon Gerrit-Project: osmo-ttcn3-hacks Gerrit-Branch: master Gerrit-Change-Id: I8c82f0063770cd6d6a1a98c229d4122c5aa2b4bd Gerrit-Change-Number: 43282 Gerrit-PatchSet: 1 Gerrit-Owner: jolly <andreas(a)eversberg.eu> Gerrit-Reviewer: Jenkins Builder

1 0

[XS] Change in osmo-ttcn3-hacks[master]: hnbgw: Send correct recovery timestamp in PFCP Assoc Setup Resp
by pespin 21 Aug '26

21 Aug '26

pespin has submitted this change. ( https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/43298?usp=email ) Change subject: hnbgw: Send correct recovery timestamp in PFCP Assoc Setup Resp ...................................................................... hnbgw: Send correct recovery timestamp in PFCP Assoc Setup Resp Change-Id: If57d58edbb4f4da529c6c189a84c25b153ac3326 --- M hnbgw/HNBGW_Tests.ttcn 1 file changed, 3 insertions(+), 2 deletions(-) Approvals: osmith: Looks good to me, approved Jenkins Builder: Verified fixeria: Looks good to me, but someone else must approve diff --git a/hnbgw/HNBGW_Tests.ttcn b/hnbgw/HNBGW_Tests.ttcn index fc33521..f43a738 100644 --- a/hnbgw/HNBGW_Tests.ttcn +++ b/hnbgw/HNBGW_Tests.ttcn @@ -263,6 +263,7 @@ private function f_pfcp_wait_assoc_setup_req(boolean fail_timeout := true, boolean fail_rx_others := true, float wait_timeout := 16.0) runs on PFCP_ConnHdlr { var PDU_PFCP m; timer T := wait_timeout; + var integer rts := f_PFCPEM_get_recovery_timestamp(); /* ask PFCPEM to route all PDUs to us */ f_PFCPEM_subscribe_bcast(); @@ -270,7 +271,7 @@ alt { [] PFCP.receive(tr_PFCP_Assoc_Setup_Req()) -> value m { PFCP.send(ts_PFCP_Assoc_Setup_Resp(m.sequence_number, ts_PFCP_Node_ID_fqdn(mp_pfcp_upf_node_id), - ts_PFCP_Cause(REQUEST_ACCEPTED), 1234)); + ts_PFCP_Cause(REQUEST_ACCEPTED), rts)); setverdict(pass); } [fail_rx_others] PFCP.receive(PDU_PFCP:?) -> value m { @@ -328,7 +329,7 @@ [not rx_assoc_setup_req] PFCP.receive(tr_PFCP_Assoc_Setup_Req()) -> value m { rx_assoc_setup_req := true; PFCP.send(ts_PFCP_Assoc_Setup_Resp(m.sequence_number, ts_PFCP_Node_ID_fqdn(mp_pfcp_upf_node_id), - ts_PFCP_Cause(REQUEST_ACCEPTED), 1234)); + ts_PFCP_Cause(REQUEST_ACCEPTED), rts)); if (not rx_hb_resp) { repeat; } -- To view, visit https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/43298?usp=email To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings?usp=email Gerrit-MessageType: merged Gerrit-Project: osmo-ttcn3-hacks Gerrit-Branch: master Gerrit-Change-Id: If57d58edbb4f4da529c6c189a84c25b153ac3326 Gerrit-Change-Number: 43298 Gerrit-PatchSet: 2 Gerrit-Owner: pespin <pespin(a)sysmocom.de> Gerrit-Reviewer: Jenkins Builder Gerrit-Reviewer: fixeria <vyanitskiy(a)sysmocom.de> Gerrit-Reviewer: osmith <osmith(a)sysmocom.de> Gerrit-Reviewer: pespin <pespin(a)sysmocom.de>

1 0

[M] Change in osmo-ttcn3-hacks[master]: hnbgw: Move generic PFCP ConnHdlr helpers to library/PFCP_Emulation.ttcn
by pespin 21 Aug '26

21 Aug '26

pespin has submitted this change. ( https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/43299?usp=email ) Change subject: hnbgw: Move generic PFCP ConnHdlr helpers to library/PFCP_Emulation.ttcn ...................................................................... hnbgw: Move generic PFCP ConnHdlr helpers to library/PFCP_Emulation.ttcn Change-Id: Ic40bea51934ad465a68e0d5690de066cbe1dc57d --- M hnbgw/HNBGW_Tests.ttcn M library/PFCP_Emulation.ttcn 2 files changed, 112 insertions(+), 98 deletions(-) Approvals: fixeria: Looks good to me, but someone else must approve osmith: Looks good to me, approved Jenkins Builder: Verified diff --git a/hnbgw/HNBGW_Tests.ttcn b/hnbgw/HNBGW_Tests.ttcn index f43a738..ac43d9f 100644 --- a/hnbgw/HNBGW_Tests.ttcn +++ b/hnbgw/HNBGW_Tests.ttcn @@ -260,100 +260,6 @@ return pfcp_associated; } -private function f_pfcp_wait_assoc_setup_req(boolean fail_timeout := true, boolean fail_rx_others := true, float wait_timeout := 16.0) runs on PFCP_ConnHdlr { - var PDU_PFCP m; - timer T := wait_timeout; - var integer rts := f_PFCPEM_get_recovery_timestamp(); - /* ask PFCPEM to route all PDUs to us */ - f_PFCPEM_subscribe_bcast(); - - T.start; - alt { - [] PFCP.receive(tr_PFCP_Assoc_Setup_Req()) -> value m { - PFCP.send(ts_PFCP_Assoc_Setup_Resp(m.sequence_number, ts_PFCP_Node_ID_fqdn(mp_pfcp_upf_node_id), - ts_PFCP_Cause(REQUEST_ACCEPTED), rts)); - setverdict(pass); - } - [fail_rx_others] PFCP.receive(PDU_PFCP:?) -> value m { - Misc_Helpers.f_shutdown(__BFILE__, __LINE__, fail, - log2str("Got an unexpected PFCP message ", m, " vs exp ", tr_PFCP_Assoc_Setup_Req())); - } - [fail_timeout] T.timeout { - Misc_Helpers.f_shutdown(__BFILE__, __LINE__, fail, - log2str("Timeout waiting for PFCP ", tr_PFCP_Assoc_Setup_Req())); - } - [not fail_timeout] T.timeout { - log("Timeout waiting for PFCP ", tr_PFCP_Assoc_Setup_Req()); - } - } - T.stop; - /* ask PFCPEM to *not* route all PDUs to us anymore */ - f_PFCPEM_unsubscribe_bcast(); -} - -altstep as_pfcp_handle_heartbeat_req(integer tx_recovery_timestamp) -runs on PFCP_ConnHdlr { - var PDU_PFCP m; - - [] PFCP.receive(tr_PFCP_Heartbeat_Req) -> value m { - var PDU_PFCP tx_pfcp; - tx_pfcp := valueof(ts_PFCP_Heartbeat_Resp(tx_recovery_timestamp)); - tx_pfcp.sequence_number := m.sequence_number; - PFCP.send(tx_pfcp); - } -} - -/* Notify peer that we restarted, it will then try to associate again. */ -private function f_pfcp_force_assoc_setup_req(boolean fail_timeout := true, - boolean fail_rx_others := true, - float wait_timeout := 16.0) -runs on PFCP_ConnHdlr { - var boolean rx_hb_resp := false; - var boolean rx_assoc_setup_req := false; - var integer rts := f_PFCPEM_get_recovery_timestamp(); - var PDU_PFCP m; - timer T := wait_timeout; - - /* ask PFCPEM to route all PDUs to us */ - f_PFCPEM_subscribe_bcast(); - - T.start; - PFCP.send(ts_PFCP_Heartbeat_Req(rts)); - alt { - [not rx_hb_resp] PFCP.receive(tr_PFCP_Heartbeat_Resp) { - rx_hb_resp := true; - if (not rx_assoc_setup_req) { - repeat; - } - } - [not rx_assoc_setup_req] PFCP.receive(tr_PFCP_Assoc_Setup_Req()) -> value m { - rx_assoc_setup_req := true; - PFCP.send(ts_PFCP_Assoc_Setup_Resp(m.sequence_number, ts_PFCP_Node_ID_fqdn(mp_pfcp_upf_node_id), - ts_PFCP_Cause(REQUEST_ACCEPTED), rts)); - if (not rx_hb_resp) { - repeat; - } - } - [] as_pfcp_handle_heartbeat_req(rts) { repeat; } - [fail_rx_others] PFCP.receive(PDU_PFCP:?) -> value m { - Misc_Helpers.f_shutdown(__BFILE__, __LINE__, fail, - log2str("Got an unexpected PFCP message ", m, " vs exp ", tr_PFCP_Assoc_Setup_Req())); - } - [fail_timeout] T.timeout { - Misc_Helpers.f_shutdown(__BFILE__, __LINE__, fail, - log2str("Timeout waiting for PFCP ", tr_PFCP_Assoc_Setup_Req())); - } - [not fail_timeout] T.timeout { - log("Timeout waiting for PFCP ", tr_PFCP_Assoc_Setup_Req()); - } - } - T.stop; - setverdict(pass); - - /* ask PFCPEM to *not* route all PDUs to us anymore */ - f_PFCPEM_unsubscribe_bcast(); -} - function f_init_pfcp(charstring id) runs on test_CT { id := id & "-PFCP"; @@ -380,10 +286,12 @@ connect(self:PFCP_PROC, vc_PFCP:CLIENT_PROC); /* the non-nightly path can be removed once osmo-hnbgw >1.8.1 is released. */ if (Misc_Helpers.f_osmo_repo_is("nightly")) { - f_pfcp_force_assoc_setup_req(fail_timeout := true); + f_pfcp_force_assoc_setup_req(ts_PFCP_Node_ID_fqdn(mp_pfcp_upf_node_id), + fail_timeout := true); } else { if (not f_statsd_pfcp_associated()) { - f_pfcp_wait_assoc_setup_req(fail_timeout := true); + f_pfcp_wait_assoc_setup_req(ts_PFCP_Node_ID_fqdn(mp_pfcp_upf_node_id), + fail_timeout := true); } } disconnect(self:PFCP, vc_PFCP:CLIENT); @@ -2809,7 +2717,8 @@ RAN_CONN.receive(tr_MSC_CONN_PRIM_DISC_IND); as_pfcp_handle_del_req(); /* Wait for a 2nd ASSOC SETUP REQ after Heartbeat timeout: */ - f_pfcp_wait_assoc_setup_req(fail_timeout := true, fail_rx_others := false); + f_pfcp_wait_assoc_setup_req(ts_PFCP_Node_ID_fqdn(mp_pfcp_upf_node_id), + fail_timeout := true, fail_rx_others := false); /* After we answer the ASSOC SETUP RESP, IUT detects peer up: */ f_statsd_expect(expect_connected, wait_converge := true); @@ -2871,7 +2780,8 @@ f_PFCPEM_unsubscribe_bcast(); /* Wait for a 2nd ASSOC SETUP REQ after Recovery triggered: */ - f_pfcp_wait_assoc_setup_req(fail_timeout := true, fail_rx_others := false); + f_pfcp_wait_assoc_setup_req(ts_PFCP_Node_ID_fqdn(mp_pfcp_upf_node_id), + fail_timeout := true, fail_rx_others := false); /* After we answer the ASSOC SETUP RESP, IUT detects peer up: */ f_statsd_expect(expect_connected, wait_converge := true); diff --git a/library/PFCP_Emulation.ttcn b/library/PFCP_Emulation.ttcn index a5ec818..2471faf 100644 --- a/library/PFCP_Emulation.ttcn +++ b/library/PFCP_Emulation.ttcn @@ -401,4 +401,108 @@ } } +/* Wait for peer to send a PFCP HEARTBEAT REQ to us, answer it */ +altstep as_pfcp_handle_heartbeat_req(integer tx_recovery_timestamp) +runs on PFCP_ConnHdlr { + var PDU_PFCP m; + + [] PFCP.receive(tr_PFCP_Heartbeat_Req) -> value m { + var PDU_PFCP tx_pfcp; + tx_pfcp := valueof(ts_PFCP_Heartbeat_Resp(tx_recovery_timestamp)); + tx_pfcp.sequence_number := m.sequence_number; + PFCP.send(tx_pfcp); + } +} + +/* Wait for peer to send a PFCP ASSOC SETUP REQ to us, answer it */ +function f_pfcp_wait_assoc_setup_req(template (value) Node_ID local_node_id, + boolean fail_timeout := true, + boolean fail_rx_others := true, + float wait_timeout := 16.0) runs on PFCP_ConnHdlr { + var PDU_PFCP m; + timer T := wait_timeout; + var integer rts := f_PFCPEM_get_recovery_timestamp(); + /* ask PFCPEM to route all PDUs to us */ + f_PFCPEM_subscribe_bcast(); + + T.start; + alt { + [] PFCP.receive(tr_PFCP_Assoc_Setup_Req()) -> value m { + PFCP.send(ts_PFCP_Assoc_Setup_Resp(m.sequence_number, + local_node_id, + ts_PFCP_Cause(REQUEST_ACCEPTED), + rts)); + setverdict(pass); + } + [fail_rx_others] PFCP.receive(PDU_PFCP:?) -> value m { + Misc_Helpers.f_shutdown(__BFILE__, __LINE__, fail, + log2str("Got an unexpected PFCP message ", m, " vs exp ", tr_PFCP_Assoc_Setup_Req())); + } + [fail_timeout] T.timeout { + Misc_Helpers.f_shutdown(__BFILE__, __LINE__, fail, + log2str("Timeout waiting for PFCP ", tr_PFCP_Assoc_Setup_Req())); + } + [not fail_timeout] T.timeout { + log("Timeout waiting for PFCP ", tr_PFCP_Assoc_Setup_Req()); + } + } + T.stop; + /* ask PFCPEM to *not* route all PDUs to us anymore */ + f_PFCPEM_unsubscribe_bcast(); +} + +/* Notify peer that we restarted, it will then try to associate again. */ +function f_pfcp_force_assoc_setup_req(template (value) Node_ID local_node_id, + boolean fail_timeout := true, + boolean fail_rx_others := true, + float wait_timeout := 16.0) +runs on PFCP_ConnHdlr { + var boolean rx_hb_resp := false; + var boolean rx_assoc_setup_req := false; + var integer rts := f_PFCPEM_get_recovery_timestamp(); + var PDU_PFCP m; + timer T := wait_timeout; + + /* ask PFCPEM to route all PDUs to us */ + f_PFCPEM_subscribe_bcast(); + + T.start; + PFCP.send(ts_PFCP_Heartbeat_Req(rts)); + alt { + [not rx_hb_resp] PFCP.receive(tr_PFCP_Heartbeat_Resp) { + rx_hb_resp := true; + if (not rx_assoc_setup_req) { + repeat; + } + } + [not rx_assoc_setup_req] PFCP.receive(tr_PFCP_Assoc_Setup_Req()) -> value m { + rx_assoc_setup_req := true; + PFCP.send(ts_PFCP_Assoc_Setup_Resp(m.sequence_number, + local_node_id, + ts_PFCP_Cause(REQUEST_ACCEPTED), + rts)); + if (not rx_hb_resp) { + repeat; + } + } + [] as_pfcp_handle_heartbeat_req(rts) { repeat; } + [fail_rx_others] PFCP.receive(PDU_PFCP:?) -> value m { + Misc_Helpers.f_shutdown(__BFILE__, __LINE__, fail, + log2str("Got an unexpected PFCP message ", m, " vs exp ", tr_PFCP_Assoc_Setup_Req())); + } + [fail_timeout] T.timeout { + Misc_Helpers.f_shutdown(__BFILE__, __LINE__, fail, + log2str("Timeout waiting for PFCP ", tr_PFCP_Assoc_Setup_Req())); + } + [not fail_timeout] T.timeout { + log("Timeout waiting for PFCP ", tr_PFCP_Assoc_Setup_Req()); + } + } + T.stop; + setverdict(pass); + + /* ask PFCPEM to *not* route all PDUs to us anymore */ + f_PFCPEM_unsubscribe_bcast(); +} + } -- To view, visit https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/43299?usp=email To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings?usp=email Gerrit-MessageType: merged Gerrit-Project: osmo-ttcn3-hacks Gerrit-Branch: master Gerrit-Change-Id: Ic40bea51934ad465a68e0d5690de066cbe1dc57d Gerrit-Change-Number: 43299 Gerrit-PatchSet: 4 Gerrit-Owner: pespin <pespin(a)sysmocom.de> Gerrit-Reviewer: Jenkins Builder Gerrit-Reviewer: fixeria <vyanitskiy(a)sysmocom.de> Gerrit-Reviewer: osmith <osmith(a)sysmocom.de> Gerrit-Reviewer: pespin <pespin(a)sysmocom.de>

1 0

[S] Change in osmo-ttcn3-hacks[master]: pgw: Move generic helper find_teid to library/
by pespin 21 Aug '26

21 Aug '26

pespin has submitted this change. ( https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/43297?usp=email ) Change subject: pgw: Move generic helper find_teid to library/ ...................................................................... pgw: Move generic helper find_teid to library/ Change-Id: I7dc56e2462436edc058da72dff3dded6569a5e95 --- M library/GTPv2_Templates.ttcn M pgw/PGW_Session_CT.ttcn 2 files changed, 18 insertions(+), 18 deletions(-) Approvals: Jenkins Builder: Verified fixeria: Looks good to me, but someone else must approve osmith: Looks good to me, approved diff --git a/library/GTPv2_Templates.ttcn b/library/GTPv2_Templates.ttcn index d318ace..31ec999 100644 --- a/library/GTPv2_Templates.ttcn +++ b/library/GTPv2_Templates.ttcn @@ -607,6 +607,22 @@ additionalOctets := omit } +/* find TEID of given interface type (and optionally instance) */ +function f_GTP2C_find_teid(FullyQualifiedTEID_List list, + template (present) integer if_type, + template (present) BIT4 instance := ?) +return template (omit) FullyQualifiedTEID +{ + var integer i; + for (i := 0; i < lengthof(list); i := i+1) { + if (match(list[i].interfaceType, if_type) and + match(list[i].instance, instance)) { + return list[i]; + } + } + return omit; +} + /* 8.28 */ template (value) BearerContextIEs ts_GTP2C_BcContextIE(template (value) uint4_t ebi, diff --git a/pgw/PGW_Session_CT.ttcn b/pgw/PGW_Session_CT.ttcn index d45cad2..3de8d54 100644 --- a/pgw/PGW_Session_CT.ttcn +++ b/pgw/PGW_Session_CT.ttcn @@ -234,22 +234,6 @@ return (g_pars.rat_type == GTP2C_RAT_WLAN or g_pars.rat_type == GTP2C_RAT_Virtual); } -/* find TEID of given interface type (and optionally instance) */ -private function f_find_teid(FullyQualifiedTEID_List list, - template (present) integer if_type, - template (present) BIT4 instance := ?) -return template (omit) FullyQualifiedTEID -{ - var integer i; - for (i := 0; i < lengthof(list); i := i+1) { - if (match(list[i].interfaceType, if_type) and - match(list[i].instance, instance)) { - return list[i]; - } - } - return omit; -} - private function f_create_sess_tun() runs on PGW_Session_CT { var template (value) UECUPS_AddrType user_addr_type; @@ -320,8 +304,8 @@ /* find F-TEID of the P-GW U side */ var FullyQualifiedTEID rx_fteid; - rx_fteid := valueof(f_find_teid(bctx.bearerContextIEs.fullyQualifiedTEID, - exp_fteid_if_type, exp_fteid_instance)); + rx_fteid := valueof(f_GTP2C_find_teid(bctx.bearerContextIEs.fullyQualifiedTEID, + exp_fteid_if_type, exp_fteid_instance)); g_pars.bearer.teid_remote := rx_fteid.tEID_GRE_Key; if (rx_fteid.v4_Flag == '1'B) { g_gtpu4_remote := rx_fteid.iPv4_Address; -- To view, visit https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/43297?usp=email To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings?usp=email Gerrit-MessageType: merged Gerrit-Project: osmo-ttcn3-hacks Gerrit-Branch: master Gerrit-Change-Id: I7dc56e2462436edc058da72dff3dded6569a5e95 Gerrit-Change-Number: 43297 Gerrit-PatchSet: 2 Gerrit-Owner: pespin <pespin(a)sysmocom.de> Gerrit-Reviewer: Jenkins Builder Gerrit-Reviewer: fixeria <vyanitskiy(a)sysmocom.de> Gerrit-Reviewer: osmith <osmith(a)sysmocom.de> Gerrit-Reviewer: pespin <pespin(a)sysmocom.de>

1 0

[M] Change in osmo-ttcn3-hacks[master]: smf: Introduce initial PFCP support
by osmith 21 Aug '26

21 Aug '26

Attention is currently required from: pespin. osmith has posted comments on this change by pespin. ( https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/43308?usp=email ) Change subject: smf: Introduce initial PFCP support ...................................................................... Patch Set 1: (1 comment) File smf/SMF_Tests.ttcn: https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/43308/comment/a8336e76_9784… : PS1, Line 216: //if (Misc_Helpers.f_osmo_repo_is("nightly")) { lots of commented out code seems unintentional? (also connect and disconnect lines) -- To view, visit https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/43308?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: I3fb988b5ad351b40e18f02f9bfca7bbd267bb6a4 Gerrit-Change-Number: 43308 Gerrit-PatchSet: 1 Gerrit-Owner: pespin <pespin(a)sysmocom.de> Gerrit-Reviewer: Jenkins Builder Gerrit-CC: osmith <osmith(a)sysmocom.de> Gerrit-Attention: pespin <pespin(a)sysmocom.de> Gerrit-Comment-Date: Fri, 21 Aug 2026 11:32:14 +0000 Gerrit-HasComments: Yes Gerrit-Has-Labels: No

1 0

[XL] Change in osmo-ttcn3-hacks[master]: smf: Introduce testsuite
by osmith 21 Aug '26

21 Aug '26

Attention is currently required from: pespin. osmith has posted comments on this change by pespin. ( https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/43307?usp=email ) Change subject: smf: Introduce testsuite ...................................................................... Patch Set 1: Code-Review+1 (1 comment) File smf/README.md: https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/43307/comment/e61b014a_ca82… : PS1, Line 14: ATS [label="ATS\nC5G_Tests.ttcn"]; ```suggestion ATS [label="ATS\nSMF_Tests.ttcn"]; ``` -- To view, visit https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/43307?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: Ide6275323085f3d1f5d1ab01f3425fd6baa0f8b3 Gerrit-Change-Number: 43307 Gerrit-PatchSet: 1 Gerrit-Owner: pespin <pespin(a)sysmocom.de> Gerrit-Reviewer: Jenkins Builder Gerrit-Reviewer: osmith <osmith(a)sysmocom.de> Gerrit-Attention: pespin <pespin(a)sysmocom.de> Gerrit-Comment-Date: Fri, 21 Aug 2026 11:30:56 +0000 Gerrit-HasComments: Yes Gerrit-Has-Labels: Yes

1 0

2026

2025

2024

2023

2022

gerrit-log August 2026