May 2025 - gerrit-log - lists.osmocom.org

[M] Change in ...osmo-epdg[master]: Enable the use of an UE requested IP type and address to be used for ...

by pespin

Attention is currently required from: clufn. pespin has posted comments on this change by clufn. ( https://gerrit.osmocom.org/c/erlang/osmo-epdg/+/40277?usp=email ) Change subject: Enable the use of an UE requested IP type and address to be used for gtpv2 PDN Address Allocation ...................................................................... Patch Set 1: (1 comment) Patchset: PS1: FYI, we have ttcn3 testsuite, you may want to add tests for IPv6 to it: https://jenkins.osmocom.org/jenkins/job/ttcn3-epdg-test/ https://gitea.osmocom.org/osmocom/docker-playground/src/branch/master/ttcn3… -- To view, visit https://gerrit.osmocom.org/c/erlang/osmo-epdg/+/40277?usp=email To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings?usp=email Gerrit-MessageType: comment Gerrit-Project: erlang/osmo-epdg Gerrit-Branch: master Gerrit-Change-Id: If0c77b1e366df36439d3a1b72030d831dd1b3a05 Gerrit-Change-Number: 40277 Gerrit-PatchSet: 1 Gerrit-Owner: clufn <callumfarrbennett(a)yahoo.com> Gerrit-Reviewer: Jenkins Builder Gerrit-Reviewer: fixeria <vyanitskiy(a)sysmocom.de> Gerrit-Reviewer: pespin <pespin(a)sysmocom.de> Gerrit-Attention: clufn <callumfarrbennett(a)yahoo.com> Gerrit-Comment-Date: Mon, 12 May 2025 09:38:20 +0000 Gerrit-HasComments: Yes Gerrit-Has-Labels: No

1 month, 2 weeks

1
0
0 0

[M] Change in ...osmo-epdg[master]: Enable the use of an UE requested IP type and address to be used for ...

by pespin

Attention is currently required from: clufn. pespin has posted comments on this change by clufn. ( https://gerrit.osmocom.org/c/erlang/osmo-epdg/+/40277?usp=email ) Change subject: Enable the use of an UE requested IP type and address to be used for gtpv2 PDN Address Allocation ...................................................................... Patch Set 1: Code-Review-1 (3 comments) File src/conv.erl: https://gerrit.osmocom.org/c/erlang/osmo-epdg/+/40277/comment/72f91d69_e8f9… : PS1, Line 140: #v2_pdn_address_allocation{type = ipv4v6, address = get_v4v6(maps:get(ipv4,Address),maps:get(ipv6,Address))}. missing space between parameters. Several times in lines 135-140. File src/epdg_ue_fsm.erl: https://gerrit.osmocom.org/c/erlang/osmo-epdg/+/40277/comment/31980744_b166… : PS1, Line 264: Data1 = Data#ue_fsm_data{paa = conv:pdp_address_to_gtp2_paa(PdpTypeNr, PdpAddress), apn = Apn}, I'd really prefer keeping the pdp address in our own format and only convert o gtp2 struct whenever passing it to gtp2 code. https://gerrit.osmocom.org/c/erlang/osmo-epdg/+/40277/comment/bb3beb7e_3612… : PS1, Line 353: Data#ue_fsm_data.paa, "conv:pdp_address_to_gtp2_paa(PdpTypeNr, PdpAddress)" here, or even better, do it inside epdg_gtpc_s2b:create_session_req() -- To view, visit https://gerrit.osmocom.org/c/erlang/osmo-epdg/+/40277?usp=email To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings?usp=email Gerrit-MessageType: comment Gerrit-Project: erlang/osmo-epdg Gerrit-Branch: master Gerrit-Change-Id: If0c77b1e366df36439d3a1b72030d831dd1b3a05 Gerrit-Change-Number: 40277 Gerrit-PatchSet: 1 Gerrit-Owner: clufn <callumfarrbennett(a)yahoo.com> Gerrit-Reviewer: Jenkins Builder Gerrit-Reviewer: fixeria <vyanitskiy(a)sysmocom.de> Gerrit-Reviewer: pespin <pespin(a)sysmocom.de> Gerrit-Attention: clufn <callumfarrbennett(a)yahoo.com> Gerrit-Comment-Date: Mon, 12 May 2025 09:37:13 +0000 Gerrit-HasComments: Yes Gerrit-Has-Labels: Yes

1 month, 2 weeks

1
0
0 0

[S] Change in osmo-ttcn3-hacks[master]: hnbgw: Introduce altstep as_pfcp_handle_del_req()

by pespin

Attention is currently required from: fixeria. pespin has posted comments on this change by pespin. ( https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/40271?usp=email ) Change subject: hnbgw: Introduce altstep as_pfcp_handle_del_req() ...................................................................... Patch Set 1: (2 comments) File hnbgw/ConnHdlr.ttcn: https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/40271/comment/38464fcd_b1f4… : PS1, Line 998: PFCP.receive(PDU_PFCP:?) > This catch-all `. […] I don't see a problem with this tbh, this is how it will be used so far. If later on we want to use it in some other way we can change it. File hnbgw/HNBGW_Tests.ttcn: https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/40271/comment/b5380fc4_243a… : PS1, Line 1545: as_pfcp_handle_del_req > Potential side-effect: the new altstep is doing `f_PFCPEM_unsubscribe_seid()`, which was not here be […] That's fine, the other tests was simply no calling this but it's actually more correct to call it like the altstep does. -- To view, visit https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/40271?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: I94101b5f0ad8634db68f0a9d5230c00926b02061 Gerrit-Change-Number: 40271 Gerrit-PatchSet: 1 Gerrit-Owner: pespin <pespin(a)sysmocom.de> Gerrit-Reviewer: Jenkins Builder Gerrit-Reviewer: osmith <osmith(a)sysmocom.de> Gerrit-CC: fixeria <vyanitskiy(a)sysmocom.de> Gerrit-Attention: fixeria <vyanitskiy(a)sysmocom.de> Gerrit-Comment-Date: Mon, 12 May 2025 09:25:46 +0000 Gerrit-HasComments: Yes Gerrit-Has-Labels: No Comment-In-Reply-To: fixeria <vyanitskiy(a)sysmocom.de>

1 month, 2 weeks

1
0
0 0

[M] Change in osmo-hnbgw[master]: Support tearing down PFCP upon PS-RAB-ASS-REQ (REL)

by daniel

Attention is currently required from: fixeria, laforge, neels, pespin. daniel has posted comments on this change by pespin. ( https://gerrit.osmocom.org/c/osmo-hnbgw/+/40275?usp=email ) Change subject: Support tearing down PFCP upon PS-RAB-ASS-REQ (REL) ...................................................................... Patch Set 5: Code-Review+2 -- To view, visit https://gerrit.osmocom.org/c/osmo-hnbgw/+/40275?usp=email To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings?usp=email Gerrit-MessageType: comment Gerrit-Project: osmo-hnbgw Gerrit-Branch: master Gerrit-Change-Id: Iea7f2cea4aa1fbab12cb8bf0275cd43d9e0713c2 Gerrit-Change-Number: 40275 Gerrit-PatchSet: 5 Gerrit-Owner: pespin <pespin(a)sysmocom.de> Gerrit-Reviewer: Jenkins Builder Gerrit-Reviewer: daniel <dwillmann(a)sysmocom.de> Gerrit-Reviewer: fixeria <vyanitskiy(a)sysmocom.de> Gerrit-Reviewer: laforge <laforge(a)osmocom.org> Gerrit-Reviewer: neels <nhofmeyr(a)sysmocom.de> Gerrit-Reviewer: osmith <osmith(a)sysmocom.de> Gerrit-Attention: neels <nhofmeyr(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: Mon, 12 May 2025 09:23:26 +0000 Gerrit-HasComments: No Gerrit-Has-Labels: Yes

1 month, 2 weeks

1
0
0 0

[M] Change in osmo-hnbgw[master]: Convert struct hnbgw_cnpool to be a talloc context

by pespin

pespin has submitted this change. ( https://gerrit.osmocom.org/c/osmo-hnbgw/+/40264?usp=email ) Change subject: Convert struct hnbgw_cnpool to be a talloc context ...................................................................... Convert struct hnbgw_cnpool to be a talloc context We currently have some memory corruption somewhere in osmo-hnbgw in production which we need to figure out. The previous talloc context hierarchy was really messy, and it was probably a left over of previous states of the code where we didn't have support for several sccp clients, etc. This patch is a step towards clarifying that hierarchy. Change-Id: I62a511e3ea0254d0327f2664d504b9195416bc76 --- M include/osmocom/hnbgw/hnbgw.h M include/osmocom/hnbgw/hnbgw_cn.h M src/osmo-hnbgw/hnbgw.c M src/osmo-hnbgw/hnbgw_cn.c M src/osmo-hnbgw/hnbgw_sccp.c M src/osmo-hnbgw/hnbgw_vty.c M src/osmo-hnbgw/osmo_hnbgw_main.c 7 files changed, 81 insertions(+), 68 deletions(-) Approvals: Jenkins Builder: Verified osmith: Looks good to me, approved diff --git a/include/osmocom/hnbgw/hnbgw.h b/include/osmocom/hnbgw/hnbgw.h index eb9c61c..685bc12 100644 --- a/include/osmocom/hnbgw/hnbgw.h +++ b/include/osmocom/hnbgw/hnbgw.h @@ -133,9 +133,9 @@ struct llist_head users; /* Pool of core network peers: MSCs for IuCS */ - struct hnbgw_cnpool cnpool_iucs; + struct hnbgw_cnpool *cnpool_iucs; /* Pool of core network peers: SGSNs for IuPS */ - struct hnbgw_cnpool cnpool_iups; + struct hnbgw_cnpool *cnpool_iups; } sccp; /* MGW pool, also includes the single MGCP client as fallback if no * pool is configured. */ diff --git a/include/osmocom/hnbgw/hnbgw_cn.h b/include/osmocom/hnbgw/hnbgw_cn.h index 5ca5675..fe4f7bb 100644 --- a/include/osmocom/hnbgw/hnbgw_cn.h +++ b/include/osmocom/hnbgw/hnbgw_cn.h @@ -56,11 +56,7 @@ struct rate_ctr_group *ctrs; }; -extern const struct rate_ctr_group_desc iucs_ctrg_desc; -extern const struct rate_ctr_group_desc iups_ctrg_desc; - -extern const struct rate_ctr_group_desc msc_ctrg_desc; -extern const struct rate_ctr_group_desc sgsn_ctrg_desc; +struct hnbgw_cnpool *hnbgw_cnpool_alloc(RANAP_CN_DomainIndicator_t domain); struct hnbgw_cnlink *hnbgw_cnlink_select(struct hnbgw_context_map *map); diff --git a/src/osmo-hnbgw/hnbgw.c b/src/osmo-hnbgw/hnbgw.c index c46ddad..d237928 100644 --- a/src/osmo-hnbgw/hnbgw.c +++ b/src/osmo-hnbgw/hnbgw.c @@ -235,35 +235,8 @@ g_hnbgw->config.pfcp.remote_port = OSMO_PFCP_PORT; #endif - g_hnbgw->sccp.cnpool_iucs = (struct hnbgw_cnpool){ - .domain = DOMAIN_CS, - .pool_name = "iucs", - .peer_name = "msc", - .default_remote_pc = DEFAULT_PC_MSC, - .vty = { - .nri_bitlen = OSMO_NRI_BITLEN_DEFAULT, - .null_nri_ranges = osmo_nri_ranges_alloc(g_hnbgw), - }, - .cnlink_ctrg_desc = &msc_ctrg_desc, - - .ctrs = rate_ctr_group_alloc(g_hnbgw, &iucs_ctrg_desc, 0), - }; - INIT_LLIST_HEAD(&g_hnbgw->sccp.cnpool_iucs.cnlinks); - - g_hnbgw->sccp.cnpool_iups = (struct hnbgw_cnpool){ - .domain = DOMAIN_PS, - .pool_name = "iups", - .peer_name = "sgsn", - .default_remote_pc = DEFAULT_PC_SGSN, - .vty = { - .nri_bitlen = OSMO_NRI_BITLEN_DEFAULT, - .null_nri_ranges = osmo_nri_ranges_alloc(g_hnbgw), - }, - .cnlink_ctrg_desc = &sgsn_ctrg_desc, - - .ctrs = rate_ctr_group_alloc(g_hnbgw, &iups_ctrg_desc, 0), - }; - INIT_LLIST_HEAD(&g_hnbgw->sccp.cnpool_iups.cnlinks); + g_hnbgw->sccp.cnpool_iucs = hnbgw_cnpool_alloc(DOMAIN_CS); + g_hnbgw->sccp.cnpool_iups = hnbgw_cnpool_alloc(DOMAIN_PS); osmo_timer_setup(&g_hnbgw->store_uptime_timer, hnbgw_store_hnb_uptime, g_hnbgw); osmo_timer_schedule(&g_hnbgw->store_uptime_timer, STORE_UPTIME_INTERVAL, 0); diff --git a/src/osmo-hnbgw/hnbgw_cn.c b/src/osmo-hnbgw/hnbgw_cn.c index e03f3c7..4420157 100644 --- a/src/osmo-hnbgw/hnbgw_cn.c +++ b/src/osmo-hnbgw/hnbgw_cn.c @@ -42,10 +42,6 @@ #include <osmocom/hnbgw/hnbgw_cn.h> #include <osmocom/hnbgw/context_map.h> -/*********************************************************************** - * Incoming primitives from SCCP User SAP - ***********************************************************************/ - static bool addr_has_pc_and_ssn(const struct osmo_sccp_addr *addr) { if (!(addr->presence & OSMO_SCCP_ADDR_T_SSN)) @@ -94,7 +90,7 @@ cnpool->use.nri_bitlen = cnpool->vty.nri_bitlen; osmo_nri_ranges_free(cnpool->use.null_nri_ranges); - cnpool->use.null_nri_ranges = osmo_nri_ranges_alloc(g_hnbgw); + cnpool->use.null_nri_ranges = osmo_nri_ranges_alloc(cnpool); llist_for_each_entry(r, &cnpool->vty.null_nri_ranges->entries, entry) osmo_nri_ranges_add(cnpool->use.null_nri_ranges, r); } @@ -269,7 +265,7 @@ */ struct hnbgw_cnlink *hnbgw_cnlink_select(struct hnbgw_context_map *map) { - struct hnbgw_cnpool *cnpool = map->is_ps ? &g_hnbgw->sccp.cnpool_iups : &g_hnbgw->sccp.cnpool_iucs; + struct hnbgw_cnpool *cnpool = map->is_ps ? g_hnbgw->sccp.cnpool_iups : g_hnbgw->sccp.cnpool_iucs; struct hnbgw_cnlink *cnlink; struct hnbgw_cnlink *round_robin_next = NULL; struct hnbgw_cnlink *round_robin_first = NULL; @@ -578,3 +574,51 @@ ARRAY_SIZE(cnpool_ctr_description), cnpool_ctr_description, }; + +static int hnbgw_cnpool_talloc_destructor(struct hnbgw_cnpool *cnpool) +{ + struct hnbgw_cnlink *cnlink; + osmo_nri_ranges_free(cnpool->vty.null_nri_ranges); + cnpool->vty.null_nri_ranges = NULL; + + while ((cnlink = llist_first_entry_or_null(&cnpool->cnlinks, struct hnbgw_cnlink, entry))) + hnbgw_cnlink_term_and_free(cnlink); + return 0; +} + +struct hnbgw_cnpool *hnbgw_cnpool_alloc(RANAP_CN_DomainIndicator_t domain) +{ + struct hnbgw_cnpool *cnpool = talloc_zero(g_hnbgw, struct hnbgw_cnpool); + OSMO_ASSERT(cnpool); + + cnpool->domain = domain; + cnpool->vty = (struct hnbgw_cnpool_cfg){ + .nri_bitlen = OSMO_NRI_BITLEN_DEFAULT, + .null_nri_ranges = osmo_nri_ranges_alloc(cnpool), + }; + OSMO_ASSERT(cnpool->vty.null_nri_ranges); + INIT_LLIST_HEAD(&cnpool->cnlinks); + + talloc_set_destructor(cnpool, hnbgw_cnpool_talloc_destructor); + + switch (domain) { + case DOMAIN_CS: + cnpool->pool_name = "iucs"; + cnpool->peer_name = "msc"; + cnpool->default_remote_pc = DEFAULT_PC_MSC; + cnpool->cnlink_ctrg_desc = &msc_ctrg_desc; + cnpool->ctrs = rate_ctr_group_alloc(cnpool, &iucs_ctrg_desc, 0); + break; + case DOMAIN_PS: + cnpool->pool_name = "iups"; + cnpool->peer_name = "sgsn"; + cnpool->default_remote_pc = DEFAULT_PC_SGSN; + cnpool->cnlink_ctrg_desc = &sgsn_ctrg_desc; + cnpool->ctrs = rate_ctr_group_alloc(cnpool, &iups_ctrg_desc, 0); + break; + default: + OSMO_ASSERT(0); + } + + return cnpool; +} diff --git a/src/osmo-hnbgw/hnbgw_sccp.c b/src/osmo-hnbgw/hnbgw_sccp.c index 434c58e..ed58b78 100644 --- a/src/osmo-hnbgw/hnbgw_sccp.c +++ b/src/osmo-hnbgw/hnbgw_sccp.c @@ -54,11 +54,11 @@ const struct osmo_sccp_addr *remote_addr) { struct hnbgw_cnlink *cnlink; - llist_for_each_entry(cnlink, &g_hnbgw->sccp.cnpool_iucs.cnlinks, entry) { + llist_for_each_entry(cnlink, &g_hnbgw->sccp.cnpool_iucs->cnlinks, entry) { if (cnlink_matches(cnlink, hsu, remote_addr)) return cnlink; } - llist_for_each_entry(cnlink, &g_hnbgw->sccp.cnpool_iups.cnlinks, entry) { + llist_for_each_entry(cnlink, &g_hnbgw->sccp.cnpool_iups->cnlinks, entry) { if (cnlink_matches(cnlink, hsu, remote_addr)) return cnlink; } @@ -188,9 +188,9 @@ static struct hnbgw_cnlink *cnlink_find_by_remote_pc(struct osmo_ss7_instance *cs7, uint32_t pc) { struct hnbgw_cnlink *cnlink; - cnlink = _cnlink_find_by_remote_pc(&g_hnbgw->sccp.cnpool_iucs, cs7, pc); + cnlink = _cnlink_find_by_remote_pc(g_hnbgw->sccp.cnpool_iucs, cs7, pc); if (!cnlink) - cnlink = _cnlink_find_by_remote_pc(&g_hnbgw->sccp.cnpool_iups, cs7, pc); + cnlink = _cnlink_find_by_remote_pc(g_hnbgw->sccp.cnpool_iups, cs7, pc); return cnlink; } diff --git a/src/osmo-hnbgw/hnbgw_vty.c b/src/osmo-hnbgw/hnbgw_vty.c index bfe0c73..52c5ba6 100644 --- a/src/osmo-hnbgw/hnbgw_vty.c +++ b/src/osmo-hnbgw/hnbgw_vty.c @@ -86,7 +86,7 @@ "iucs", "Configure IuCS options") { vty->node = IUCS_NODE; - vty->index = &g_hnbgw->sccp.cnpool_iucs; + vty->index = g_hnbgw->sccp.cnpool_iucs; return CMD_SUCCESS; } @@ -100,7 +100,7 @@ "iups", "Configure IuPS options") { vty->node = IUPS_NODE; - vty->index = &g_hnbgw->sccp.cnpool_iups; + vty->index = g_hnbgw->sccp.cnpool_iups; return CMD_SUCCESS; } @@ -167,10 +167,10 @@ { struct hnbgw_cnlink *cnlink; vty_out(vty, "IuCS: "); - llist_for_each_entry(cnlink, &g_hnbgw->sccp.cnpool_iucs.cnlinks, entry) + llist_for_each_entry(cnlink, &g_hnbgw->sccp.cnpool_iucs->cnlinks, entry) _show_cnlink(vty, cnlink); vty_out(vty, "IuPS: "); - llist_for_each_entry(cnlink, &g_hnbgw->sccp.cnpool_iups.cnlinks, entry) + llist_for_each_entry(cnlink, &g_hnbgw->sccp.cnpool_iups->cnlinks, entry) _show_cnlink(vty, cnlink); return CMD_SUCCESS; } @@ -511,7 +511,7 @@ "Configure an IuCS link to an MSC\n" "MSC nr\n") { - return cnlink_nr(vty, &g_hnbgw->sccp.cnpool_iucs, argc, argv); + return cnlink_nr(vty, g_hnbgw->sccp.cnpool_iucs, argc, argv); } /* 'sgsn 0' */ @@ -520,7 +520,7 @@ "Configure an IuPS link to an SGSN\n" "SGSN nr\n") { - return cnlink_nr(vty, &g_hnbgw->sccp.cnpool_iups, argc, argv); + return cnlink_nr(vty, g_hnbgw->sccp.cnpool_iups, argc, argv); } /* 'msc 0' / 'remote-addr my-msc' and @@ -716,14 +716,14 @@ * nri null add ... */ vty_out(vty, "hnbgw%s", VTY_NEWLINE); - cnpool_write_nri(vty, &g_hnbgw->sccp.cnpool_iucs, true); - cnpool_write_nri(vty, &g_hnbgw->sccp.cnpool_iups, true); + cnpool_write_nri(vty, g_hnbgw->sccp.cnpool_iucs, true); + cnpool_write_nri(vty, g_hnbgw->sccp.cnpool_iups, true); /* msc 0 * nri add ... */ - cnlinks_write_nri(vty, &g_hnbgw->sccp.cnpool_iucs, true); - cnlinks_write_nri(vty, &g_hnbgw->sccp.cnpool_iups, true); + cnlinks_write_nri(vty, g_hnbgw->sccp.cnpool_iucs, true); + cnlinks_write_nri(vty, g_hnbgw->sccp.cnpool_iups, true); return CMD_SUCCESS; } @@ -739,9 +739,9 @@ { struct hnbgw_cnpool *cnpool; if (!strcmp("msc", argv[0])) - cnpool = &g_hnbgw->sccp.cnpool_iucs; + cnpool = g_hnbgw->sccp.cnpool_iucs; else - cnpool = &g_hnbgw->sccp.cnpool_iups; + cnpool = g_hnbgw->sccp.cnpool_iups; cnpool->round_robin_next_nr = atoi(argv[1]); return CMD_SUCCESS; } @@ -760,9 +760,9 @@ int nr = atoi(argv[1]); if (!strcmp("msc", msc_sgsn)) - cnpool = &g_hnbgw->sccp.cnpool_iucs; + cnpool = g_hnbgw->sccp.cnpool_iucs; else - cnpool = &g_hnbgw->sccp.cnpool_iups; + cnpool = g_hnbgw->sccp.cnpool_iups; cnlink = cnlink_get_nr(cnpool, nr, false); if (!cnlink) { @@ -801,11 +801,11 @@ { struct hnbgw_cnpool *cnpool; - cnpool = &g_hnbgw->sccp.cnpool_iucs; + cnpool = g_hnbgw->sccp.cnpool_iucs; hnbgw_cnpool_apply_cfg(cnpool); hnbgw_cnpool_cnlinks_start_or_restart(cnpool); - cnpool = &g_hnbgw->sccp.cnpool_iups; + cnpool = g_hnbgw->sccp.cnpool_iups; hnbgw_cnpool_apply_cfg(cnpool); hnbgw_cnpool_cnlinks_start_or_restart(cnpool); @@ -1014,8 +1014,8 @@ llist_for_each_entry(hnbp, &g_hnbgw->hnb_persistent_list, list) write_one_hnbp(vty, hnbp); - _config_write_cnpool(vty, &g_hnbgw->sccp.cnpool_iucs); - _config_write_cnpool(vty, &g_hnbgw->sccp.cnpool_iups); + _config_write_cnpool(vty, g_hnbgw->sccp.cnpool_iucs); + _config_write_cnpool(vty, g_hnbgw->sccp.cnpool_iups); if (g_hnbgw->config.nft_kpi.enable) vty_out(vty, " nft-kpi%s%s%s", @@ -1065,13 +1065,13 @@ static int config_write_msc(struct vty *vty) { - _config_write_cnlink(vty, &g_hnbgw->sccp.cnpool_iucs); + _config_write_cnlink(vty, g_hnbgw->sccp.cnpool_iucs); return CMD_SUCCESS; } static int config_write_sgsn(struct vty *vty) { - _config_write_cnlink(vty, &g_hnbgw->sccp.cnpool_iups); + _config_write_cnlink(vty, g_hnbgw->sccp.cnpool_iups); return CMD_SUCCESS; } diff --git a/src/osmo-hnbgw/osmo_hnbgw_main.c b/src/osmo-hnbgw/osmo_hnbgw_main.c index 845002f..0f932d9 100644 --- a/src/osmo-hnbgw/osmo_hnbgw_main.c +++ b/src/osmo-hnbgw/osmo_hnbgw_main.c @@ -345,8 +345,8 @@ #endif } - hnbgw_cnpool_start(&g_hnbgw->sccp.cnpool_iucs); - hnbgw_cnpool_start(&g_hnbgw->sccp.cnpool_iups); + hnbgw_cnpool_start(g_hnbgw->sccp.cnpool_iucs); + hnbgw_cnpool_start(g_hnbgw->sccp.cnpool_iups); if (hnbgw_cmdline_config.daemonize) { rc = osmo_daemonize(); -- To view, visit https://gerrit.osmocom.org/c/osmo-hnbgw/+/40264?usp=email To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings?usp=email Gerrit-MessageType: merged Gerrit-Project: osmo-hnbgw Gerrit-Branch: master Gerrit-Change-Id: I62a511e3ea0254d0327f2664d504b9195416bc76 Gerrit-Change-Number: 40264 Gerrit-PatchSet: 3 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 month, 2 weeks

1
0
0 0

[S] Change in osmo-ttcn3-hacks[master]: hnbgw: Introduce altstep as_pfcp_handle_del_req()

by fixeria

Attention is currently required from: pespin. fixeria has posted comments on this change by pespin. ( https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/40271?usp=email ) Change subject: hnbgw: Introduce altstep as_pfcp_handle_del_req() ...................................................................... Patch Set 1: (2 comments) File hnbgw/ConnHdlr.ttcn: https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/40271/comment/7fc00236_ecc0… : PS1, Line 998: PFCP.receive(PDU_PFCP:?) This catch-all `.receive` makes the altstep less useful, because this way it cannot be used among with other `PFCP.receive` statements / altsteps. If you want it to be self-contained, then maybe turn it into a generic `f_pfcp_receive()` function with the timeout. File hnbgw/HNBGW_Tests.ttcn: https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/40271/comment/787fe69b_30a2… : PS1, Line 1545: as_pfcp_handle_del_req Potential side-effect: the new altstep is doing `f_PFCPEM_unsubscribe_seid()`, which was not here before. -- To view, visit https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/40271?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: I94101b5f0ad8634db68f0a9d5230c00926b02061 Gerrit-Change-Number: 40271 Gerrit-PatchSet: 1 Gerrit-Owner: pespin <pespin(a)sysmocom.de> Gerrit-Reviewer: Jenkins Builder Gerrit-Reviewer: osmith <osmith(a)sysmocom.de> Gerrit-CC: fixeria <vyanitskiy(a)sysmocom.de> Gerrit-Attention: pespin <pespin(a)sysmocom.de> Gerrit-Comment-Date: Mon, 12 May 2025 09:19:47 +0000 Gerrit-HasComments: Yes Gerrit-Has-Labels: No

1 month, 2 weeks

1
0
0 0

[M] Change in osmo-hnbgw[master]: Support tearing down PFCP upon PS-RAB-ASS-REQ (REL)

by osmith

Attention is currently required from: fixeria, laforge, neels, pespin. osmith has posted comments on this change by pespin. ( https://gerrit.osmocom.org/c/osmo-hnbgw/+/40275?usp=email ) Change subject: Support tearing down PFCP upon PS-RAB-ASS-REQ (REL) ...................................................................... Patch Set 5: Code-Review+1 -- To view, visit https://gerrit.osmocom.org/c/osmo-hnbgw/+/40275?usp=email To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings?usp=email Gerrit-MessageType: comment Gerrit-Project: osmo-hnbgw Gerrit-Branch: master Gerrit-Change-Id: Iea7f2cea4aa1fbab12cb8bf0275cd43d9e0713c2 Gerrit-Change-Number: 40275 Gerrit-PatchSet: 5 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: neels <nhofmeyr(a)sysmocom.de> Gerrit-Reviewer: osmith <osmith(a)sysmocom.de> Gerrit-Attention: neels <nhofmeyr(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: Mon, 12 May 2025 08:58:41 +0000 Gerrit-HasComments: No Gerrit-Has-Labels: Yes

1 month, 2 weeks

1
0
0 0

[M] Change in osmo-hnbgw[master]: Convert struct hnbgw_cnpool to be a talloc context

by osmith

Attention is currently required from: fixeria, pespin. osmith has posted comments on this change by pespin. ( https://gerrit.osmocom.org/c/osmo-hnbgw/+/40264?usp=email ) Change subject: Convert struct hnbgw_cnpool to be a talloc context ...................................................................... Patch Set 3: Code-Review+2 -- To view, visit https://gerrit.osmocom.org/c/osmo-hnbgw/+/40264?usp=email To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings?usp=email Gerrit-MessageType: comment Gerrit-Project: osmo-hnbgw Gerrit-Branch: master Gerrit-Change-Id: I62a511e3ea0254d0327f2664d504b9195416bc76 Gerrit-Change-Number: 40264 Gerrit-PatchSet: 3 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-Attention: pespin <pespin(a)sysmocom.de> Gerrit-Attention: fixeria <vyanitskiy(a)sysmocom.de> Gerrit-Comment-Date: Mon, 12 May 2025 08:58:11 +0000 Gerrit-HasComments: No Gerrit-Has-Labels: Yes

1 month, 2 weeks

1
0
0 0

[M] Change in osmo-hnbgw[master]: Support tearing down PFCP upon PS-RAB-ASS-REQ (REL)

by pespin

Attention is currently required from: fixeria, laforge, neels, osmith. Hello Jenkins Builder, fixeria, laforge, neels, osmith, I'd like you to reexamine a change. Please visit https://gerrit.osmocom.org/c/osmo-hnbgw/+/40275?usp=email to look at the new patch set (#5). The following approvals got outdated and were removed: Code-Review+1 by osmith, Verified+1 by Jenkins Builder Change subject: Support tearing down PFCP upon PS-RAB-ASS-REQ (REL) ...................................................................... Support tearing down PFCP upon PS-RAB-ASS-REQ (REL) Before this patch, RAB Assign Request containing only a ReleaseList where being discarded with an error. This commit adds logic to allow passing those messages, and upon RAB Assign Response, tear down the related PFCP sessions before forwarding it back to the CN. This is scenario is tested in TTCN-3 testsuite HNBGW_Tests.TC_ps_rab_release. Related: SYS#7451 Change-Id: Iea7f2cea4aa1fbab12cb8bf0275cd43d9e0713c2 --- M include/osmocom/hnbgw/ps_rab_ass_fsm.h M include/osmocom/hnbgw/ps_rab_fsm.h M src/osmo-hnbgw/ps_rab_ass_fsm.c M src/osmo-hnbgw/ps_rab_fsm.c 4 files changed, 143 insertions(+), 66 deletions(-) git pull ssh://gerrit.osmocom.org:29418/osmo-hnbgw refs/changes/75/40275/5 -- To view, visit https://gerrit.osmocom.org/c/osmo-hnbgw/+/40275?usp=email To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings?usp=email Gerrit-MessageType: newpatchset Gerrit-Project: osmo-hnbgw Gerrit-Branch: master Gerrit-Change-Id: Iea7f2cea4aa1fbab12cb8bf0275cd43d9e0713c2 Gerrit-Change-Number: 40275 Gerrit-PatchSet: 5 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: neels <nhofmeyr(a)sysmocom.de> Gerrit-Reviewer: osmith <osmith(a)sysmocom.de> Gerrit-Attention: osmith <osmith(a)sysmocom.de> Gerrit-Attention: neels <nhofmeyr(a)sysmocom.de> Gerrit-Attention: laforge <laforge(a)osmocom.org> Gerrit-Attention: fixeria <vyanitskiy(a)sysmocom.de>

1 month, 2 weeks

1
0
0 0

[M] Change in osmo-hnbgw[master]: Convert struct hnbgw_cnpool to be a talloc context

by pespin

Attention is currently required from: fixeria, osmith. Hello Jenkins Builder, fixeria, osmith, I'd like you to reexamine a change. Please visit https://gerrit.osmocom.org/c/osmo-hnbgw/+/40264?usp=email to look at the new patch set (#3). The following approvals got outdated and were removed: Code-Review+1 by fixeria, Code-Review+1 by osmith Change subject: Convert struct hnbgw_cnpool to be a talloc context ...................................................................... Convert struct hnbgw_cnpool to be a talloc context We currently have some memory corruption somewhere in osmo-hnbgw in production which we need to figure out. The previous talloc context hierarchy was really messy, and it was probably a left over of previous states of the code where we didn't have support for several sccp clients, etc. This patch is a step towards clarifying that hierarchy. Change-Id: I62a511e3ea0254d0327f2664d504b9195416bc76 --- M include/osmocom/hnbgw/hnbgw.h M include/osmocom/hnbgw/hnbgw_cn.h M src/osmo-hnbgw/hnbgw.c M src/osmo-hnbgw/hnbgw_cn.c M src/osmo-hnbgw/hnbgw_sccp.c M src/osmo-hnbgw/hnbgw_vty.c M src/osmo-hnbgw/osmo_hnbgw_main.c 7 files changed, 81 insertions(+), 68 deletions(-) git pull ssh://gerrit.osmocom.org:29418/osmo-hnbgw refs/changes/64/40264/3 -- To view, visit https://gerrit.osmocom.org/c/osmo-hnbgw/+/40264?usp=email To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings?usp=email Gerrit-MessageType: newpatchset Gerrit-Project: osmo-hnbgw Gerrit-Branch: master Gerrit-Change-Id: I62a511e3ea0254d0327f2664d504b9195416bc76 Gerrit-Change-Number: 40264 Gerrit-PatchSet: 3 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-Attention: osmith <osmith(a)sysmocom.de> Gerrit-Attention: fixeria <vyanitskiy(a)sysmocom.de>

1 month, 2 weeks

1
0
0 0

2025

2024

2023

2022

gerrit-log May 2025