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

[S] Change in osmo-ttcn3-hacks[master]: stp: Support expect RFC466 4.5.1 DUNA after ASP ACT

by pespin

pespin has uploaded this change for review. ( https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/40569?usp=email ) Change subject: stp: Support expect RFC466 4.5.1 DUNA after ASP ACT ...................................................................... stp: Support expect RFC466 4.5.1 DUNA after ASP ACT libosmo-sigtran recently introduced support in SG role to send DUNA for unreachablew destinations when a peer ASP becomes activated. This is currently sent upon first ASP going active, since it's the time where SG assumes the ASP peer may need that information. Depends: libosmo-sigtran.git Id9d92c3fda7423a9c79fffac6b650abba9651853 Change-Id: I6bee0f890a8baaa62be193824b1ae2fceb53fb45 --- M stp/STP_Tests_M3UA.ttcn 1 file changed, 31 insertions(+), 5 deletions(-) git pull ssh://gerrit.osmocom.org:29418/osmo-ttcn3-hacks refs/changes/69/40569/1 diff --git a/stp/STP_Tests_M3UA.ttcn b/stp/STP_Tests_M3UA.ttcn index a98ffdc..ebdb6e1 100644 --- a/stp/STP_Tests_M3UA.ttcn +++ b/stp/STP_Tests_M3UA.ttcn @@ -73,6 +73,7 @@ port M3UA_CODEC_PT M3UA[MAX_NR_M3UA]; var M3uaConfigs g_m3ua_configs; var integer g_m3ua_conn_id[MAX_NR_M3UA]; + var boolean g_iut_supports_rfc666_4_5_1_duna_after_act := true; /* RFC4666 4.5.1 */ } friend function f_m3ua_get_config_idx(charstring asp_name) runs on RAW_M3UA_CT return integer { @@ -336,6 +337,9 @@ f_init_common(); + /* Set always to true once libosmo-sigtran > 2.1.0 is released: */ + g_iut_supports_rfc666_4_5_1_duna_after_act := Misc_Helpers.f_osmo_repo_is("nightly"); + if (not istemplatekind(m3ua_configs, "omit")) { g_m3ua_configs := valueof(m3ua_configs); } else { @@ -459,7 +463,8 @@ friend function f_M3UA_asp_up_act(integer idx, template (omit) M3UA_Traffic_Mode_Type tmt := omit, template (omit) OCT4 rctx := omit, template (omit) OCT2 ntfy_after_up := c_M3UA_ST_I_AS_INACTIVE, - template (omit) OCT2 ntfy_after_act := c_M3UA_ST_I_AS_ACTIVE) + template (omit) OCT2 ntfy_after_act := c_M3UA_ST_I_AS_ACTIVE, + template (omit) boolean exp_duna_after_act := omit /* auto */) runs on RAW_M3UA_CT { f_M3UA_asp_up(idx, omit); if (not istemplatekind(ntfy_after_up, "omit")) { @@ -469,6 +474,20 @@ if (not istemplatekind(ntfy_after_act, "omit")) { f_M3UA_exp(idx, tr_M3UA_NOTIFY(c_M3UA_ST_T_STATE_CHG, ntfy_after_act, *)); } + if (g_iut_supports_rfc666_4_5_1_duna_after_act) { + var boolean exp_duna; + if (istemplatekind(exp_duna_after_act, "omit")) { + /* By default, we expect to receive RFC4666 4.5.1 DUNA + * if AS becomes active, since at that time SG expects us + * to not know current state of other AS destinations. */ + exp_duna := not istemplatekind(ntfy_after_act, "omit") and valueof(ntfy_after_act) == c_M3UA_ST_I_AS_ACTIVE; + } else { + exp_duna := valueof(exp_duna_after_act); + } + if (exp_duna) { + f_M3UA_exp(idx, tr_M3UA_DUNA); + } + } } @@ -1010,8 +1029,9 @@ testcase TC_rkm_unreg_active() runs on RAW_M3UA_CT { var Misc_Helpers.ro_charstring asps := { "asp-sender" }; var M3uaConfigs m3ua_configs := m3ua_build_configs(asps); - var OCT3 dpc := int2oct(123, 3); + var OCT3 dpcA := int2oct(123, 3); var OCT4 rctxA := int2oct(1234, 4); + var OCT3 dpcB := int2oct(m3ua_configs[0].point_code, 4); // must match config var OCT4 rctxB := int2oct(m3ua_configs[0].routing_ctx, 4); // must match config f_init_common(); @@ -1019,7 +1039,7 @@ f_init_m3ua(m3ua_configs := m3ua_configs); /* first register the routing key */ - f_M3UA_rkm_register(0, local_id:='30000099'O, dpc:=dpc, rctx:=rctxA, ntfy_after_reg := omit); + f_M3UA_rkm_register(0, local_id:='30000099'O, dpc:=dpcA, rctx:=rctxA, ntfy_after_reg := omit); /* then activate the ASP, this turns the 2 AS (one for each rctx, 1234 and default 1023) * INACTIVE, handle the 2 NOTIFY manually afterwards: */ @@ -1028,9 +1048,15 @@ f_M3UA_exp(0, tr_M3UA_NOTIFY(c_M3UA_ST_T_STATE_CHG, c_M3UA_ST_I_AS_INACTIVE, (rctxA & rctxB, rctxB & rctxA))); f_M3UA_asp_act(0, omit, rctxB); f_M3UA_exp(0, tr_M3UA_NOTIFY(c_M3UA_ST_T_STATE_CHG, c_M3UA_ST_I_AS_ACTIVE, (rctxA & rctxB, rctxB & rctxA))); - f_M3UA_exp(0, tr_M3UA_DAVA({*}, rctxA)); + f_M3UA_exp(0, tr_M3UA_DAVA({ts_M3UA_PC(oct2int(dpcB))}, rctxA)); + if (g_iut_supports_rfc666_4_5_1_duna_after_act) { + f_M3UA_exp(0, tr_M3UA_DUNA({*}, rctxB)); + } f_M3UA_exp(0, tr_M3UA_NOTIFY(c_M3UA_ST_T_STATE_CHG, c_M3UA_ST_I_AS_ACTIVE, (rctxA & rctxB, rctxB & rctxA))); - f_M3UA_exp(0, tr_M3UA_DAVA({*}, rctxB)); + f_M3UA_exp(0, tr_M3UA_DAVA({ts_M3UA_PC(oct2int(dpcA))}, rctxB)); + if (g_iut_supports_rfc666_4_5_1_duna_after_act) { + f_M3UA_exp(0, tr_M3UA_DUNA({*}, rctxA)); + } /* then try to de-register -> ERR_ASP_ACTIVE */ f_M3UA_send(0, ts_M3UA_DEREG_REQ(ts_M3UA_routing_ctx(rctxA))); -- To view, visit https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/40569?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: I6bee0f890a8baaa62be193824b1ae2fceb53fb45 Gerrit-Change-Number: 40569 Gerrit-PatchSet: 1 Gerrit-Owner: pespin <pespin(a)sysmocom.de>

1 month

1
0
0 0

[M] Change in libosmo-sigtran[master]: Send DUNA during ASP ACT time

by pespin

Hello Jenkins Builder, I'd like you to reexamine a change. Please visit https://gerrit.osmocom.org/c/libosmo-sigtran/+/40564?usp=email to look at the new patch set (#3). The following approvals got outdated and were removed: Verified+1 by Jenkins Builder Change subject: Send DUNA during ASP ACT time ...................................................................... Send DUNA during ASP ACT time As explained in RFC 4666 4.5.1, this helps ASPs figuring quickly figuring out that some destinations may not be available. Change-Id: Id9d92c3fda7423a9c79fffac6b650abba9651853 --- M src/xua_as_fsm.c M src/xua_internal.h M src/xua_snm.c 3 files changed, 53 insertions(+), 3 deletions(-) git pull ssh://gerrit.osmocom.org:29418/libosmo-sigtran refs/changes/64/40564/3 -- To view, visit https://gerrit.osmocom.org/c/libosmo-sigtran/+/40564?usp=email To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings?usp=email Gerrit-MessageType: newpatchset Gerrit-Project: libosmo-sigtran Gerrit-Branch: master Gerrit-Change-Id: Id9d92c3fda7423a9c79fffac6b650abba9651853 Gerrit-Change-Number: 40564 Gerrit-PatchSet: 3 Gerrit-Owner: pespin <pespin(a)sysmocom.de> Gerrit-Reviewer: Jenkins Builder

1 month

1
0
0 0

[S] Change in osmo-ttcn3-hacks[master]: remsim: Bankd: check for a busy client

by lynxis lazus

lynxis lazus has uploaded this change for review. ( https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/40568?usp=email ) Change subject: remsim: Bankd: check for a busy client ...................................................................... remsim: Bankd: check for a busy client A busy client should be implicit replaced when ran against master. The latest release this check will fail. Change-Id: I57112758167c2a29fae9df0cf1e2691c5a8e7bed --- M remsim/RemsimBankd_Tests.ttcn 1 file changed, 16 insertions(+), 0 deletions(-) git pull ssh://gerrit.osmocom.org:29418/osmo-ttcn3-hacks refs/changes/68/40568/1 diff --git a/remsim/RemsimBankd_Tests.ttcn b/remsim/RemsimBankd_Tests.ttcn index a2686d3..637cb0f 100644 --- a/remsim/RemsimBankd_Tests.ttcn +++ b/remsim/RemsimBankd_Tests.ttcn @@ -118,6 +118,21 @@ Misc_Helpers.f_shutdown(__BFILE__, __LINE__, pass); } +/* attempt to create a mapping for a client that already has a mapping to a different slot */ +testcase TC_createMapping_busyClient() runs on bankd_test_CT { + f_init(); + as_connectBankReq(bid := mp_bank_id, nslots := mp_num_slots); + f_rspro_srv_reset_state(ok); + var BankSlot bs0 := { bankId := mp_bank_id, slotNr := 0 }; + var BankSlot bs1 := { bankId := mp_bank_id, slotNr := 1 }; + var ClientSlot cs := { clientId := 23, slotNr := 42 }; + /* create the mapping the first time */ + f_rspro_srv_create_slotmap(cs, bs0); + /* re-create the mapping a second time */ + f_rspro_srv_create_slotmap(cs, bs1); + Misc_Helpers.f_shutdown(__BFILE__, __LINE__, pass); +} + /* attempt to create a mapping for an out-of-range slot number */ testcase TC_createMapping_invalidSlot() runs on bankd_test_CT { f_init(); @@ -415,6 +430,7 @@ execute( TC_createMapping() ); execute( TC_createMapping_busySlot() ); + execute( TC_createMapping_busyClient() ); execute( TC_createMapping_invalidSlot() ); execute( TC_createMapping_invalidBank() ); -- To view, visit https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/40568?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: I57112758167c2a29fae9df0cf1e2691c5a8e7bed Gerrit-Change-Number: 40568 Gerrit-PatchSet: 1 Gerrit-Owner: lynxis lazus <lynxis(a)fe80.eu>

1 month

1
0
0 0

[XS] Change in osmo-ttcn3-hacks[master]: pcu: TC_sns_del: improve test stability

by lynxis lazus

lynxis lazus has submitted this change. ( https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/40565?usp=email ) Change subject: pcu: TC_sns_del: improve test stability ...................................................................... pcu: TC_sns_del: improve test stability Add a sleep 1 to ensure the BSSGP flow Ack flows before removing the SNS. Further add a data weight to the other NSVC to have at least one valid NSVC with data weights. Otherwise the NSEI would become unavailable. Related: OS#6751 Change-Id: Icb32e5f46ba7950848343efc46aee67e2397de5d --- M pcu/PCU_Tests_SNS.ttcn 1 file changed, 3 insertions(+), 1 deletion(-) Approvals: Jenkins Builder: Verified daniel: Looks good to me, but someone else must approve pespin: Looks good to me, approved diff --git a/pcu/PCU_Tests_SNS.ttcn b/pcu/PCU_Tests_SNS.ttcn index 3e7f81c..c2fa927 100644 --- a/pcu/PCU_Tests_SNS.ttcn +++ b/pcu/PCU_Tests_SNS.ttcn @@ -354,9 +354,11 @@ testcase TC_sns_del() runs on RAW_Test_CT { f_sns_bringup_1c1u(); + f_outgoing_sns_chg_weight(idx_chg := 0, w_sig := 1, w_user := 1, idx := 0); + + f_sleep(1.0); f_TC_sns_del(); - f_clean_ns_codec(); } -- To view, visit https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/40565?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: Icb32e5f46ba7950848343efc46aee67e2397de5d Gerrit-Change-Number: 40565 Gerrit-PatchSet: 1 Gerrit-Owner: lynxis lazus <lynxis(a)fe80.eu> Gerrit-Reviewer: Jenkins Builder Gerrit-Reviewer: daniel <dwillmann(a)sysmocom.de> Gerrit-Reviewer: lynxis lazus <lynxis(a)fe80.eu> Gerrit-Reviewer: pespin <pespin(a)sysmocom.de>

1 month

1
0
0 0

[M] Change in libosmo-sigtran[master]: Introduce initial route status support

by pespin

Attention is currently required from: laforge. pespin has posted comments on this change by pespin. ( https://gerrit.osmocom.org/c/libosmo-sigtran/+/40556?usp=email ) Change subject: Introduce initial route status support ...................................................................... Patch Set 2: (1 comment) File src/xua_snm.c: https://gerrit.osmocom.org/c/libosmo-sigtran/+/40556/comment/1d27c27f_3b35f… : PS1, Line 173: > In here I think I still need: […] Done in follow-up patches. Please review. -- To view, visit https://gerrit.osmocom.org/c/libosmo-sigtran/+/40556?usp=email To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings?usp=email Gerrit-MessageType: comment Gerrit-Project: libosmo-sigtran Gerrit-Branch: master Gerrit-Change-Id: Icb4f78f8da9de5a611cb6311722bf3acf1a68fc5 Gerrit-Change-Number: 40556 Gerrit-PatchSet: 2 Gerrit-Owner: pespin <pespin(a)sysmocom.de> Gerrit-Reviewer: Jenkins Builder Gerrit-Reviewer: laforge <laforge(a)osmocom.org> Gerrit-CC: osmith <osmith(a)sysmocom.de> Gerrit-Attention: laforge <laforge(a)osmocom.org> Gerrit-Comment-Date: Wed, 02 Jul 2025 13:31:14 +0000 Gerrit-HasComments: Yes Gerrit-Has-Labels: No Comment-In-Reply-To: pespin <pespin(a)sysmocom.de>

1 month

1
0
0 0

[XS] Change in libosmo-sigtran[master]: Move ss7_as_alloc() to correct header

by pespin

pespin has submitted this change. ( https://gerrit.osmocom.org/c/libosmo-sigtran/+/40555?usp=email ) ( 1 is the latest approved patch-set. No files were changed between the latest approved patch-set and the submitted one. )Change subject: Move ss7_as_alloc() to correct header ...................................................................... Move ss7_as_alloc() to correct header Change-Id: I062ccf8d1ebd58b767652fb72cc056065749f2a5 --- M src/ss7_as.h M src/ss7_internal.h 2 files changed, 2 insertions(+), 4 deletions(-) Approvals: laforge: Looks good to me, approved Jenkins Builder: Verified diff --git a/src/ss7_as.h b/src/ss7_as.h index 5bc3574..fe1d86b 100644 --- a/src/ss7_as.h +++ b/src/ss7_as.h @@ -127,7 +127,8 @@ } loadshare; } cfg; }; - +struct osmo_ss7_as *ss7_as_alloc(struct osmo_ss7_instance *inst, const char *name, + enum osmo_ss7_asp_protocol proto); struct osmo_ss7_asp *ss7_as_select_asp(struct osmo_ss7_as *as, const struct osmo_mtp_transfer_param *mtp); unsigned int osmo_ss7_as_count_asp(const struct osmo_ss7_as *as); diff --git a/src/ss7_internal.h b/src/ss7_internal.h index 7dc2c57..d931351 100644 --- a/src/ss7_internal.h +++ b/src/ss7_internal.h @@ -15,9 +15,6 @@ bool ss7_ipv6_sctp_supported(const char *host, bool bind); -struct osmo_ss7_as *ss7_as_alloc(struct osmo_ss7_instance *inst, const char *name, - enum osmo_ss7_asp_protocol proto); - struct osmo_ss7_asp *ss7_asp_find_by_socket_addr(int fd, int trans_proto); bool ss7_asp_protocol_check_trans_proto(enum osmo_ss7_asp_protocol proto, int trans_proto); -- To view, visit https://gerrit.osmocom.org/c/libosmo-sigtran/+/40555?usp=email To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings?usp=email Gerrit-MessageType: merged Gerrit-Project: libosmo-sigtran Gerrit-Branch: master Gerrit-Change-Id: I062ccf8d1ebd58b767652fb72cc056065749f2a5 Gerrit-Change-Number: 40555 Gerrit-PatchSet: 2 Gerrit-Owner: pespin <pespin(a)sysmocom.de> Gerrit-Reviewer: Jenkins Builder Gerrit-Reviewer: laforge <laforge(a)osmocom.org> Gerrit-Reviewer: pespin <pespin(a)sysmocom.de>

1 month

1
0
0 0

[XS] Change in libosmo-sigtran[master]: xua_asp_fsm: Drop code branch never called in xUA ASP

by pespin

pespin has submitted this change. ( https://gerrit.osmocom.org/c/libosmo-sigtran/+/40549?usp=email ) Change subject: xua_asp_fsm: Drop code branch never called in xUA ASP ...................................................................... xua_asp_fsm: Drop code branch never called in xUA ASP ACTIVE state is only entered (xua_asp_fsm_active_onenter) from INACTIVE state; it cannot be entered from DOWN state. Hence, went went_up would always be false and the code never executed. This code is indeed not used since we already make sure we start BEAT procedure when entering INACTIVE state from DOWN state, and only disable it when we go back to DOWN state. Fixes: ce6b537fcc4c52a90d8072714ce41158524b3008 Change-Id: Ic0b8b4fbe31ab2e5206a4f4c7907868f20d2ef10 --- M src/xua_asp_fsm.c 1 file changed, 0 insertions(+), 6 deletions(-) Approvals: laforge: Looks good to me, but someone else must approve osmith: Looks good to me, but someone else must approve pespin: Looks good to me, approved Jenkins Builder: Verified diff --git a/src/xua_asp_fsm.c b/src/xua_asp_fsm.c index 26f3a78..86d86b7 100644 --- a/src/xua_asp_fsm.c +++ b/src/xua_asp_fsm.c @@ -695,12 +695,6 @@ { struct xua_asp_fsm_priv *xafp = fi->priv; struct osmo_ss7_asp *asp = xafp->asp; - bool went_up = (prev_state == XUA_ASP_S_DOWN); - - if (went_up) { - /* Now we are done with IPA handshake, Start Hearbeat Procedure, T(beat): */ - xua_t_beat_send(fi); - } dispatch_to_all_as(fi, XUA_ASPAS_ASP_ACTIVE_IND, asp); } -- To view, visit https://gerrit.osmocom.org/c/libosmo-sigtran/+/40549?usp=email To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings?usp=email Gerrit-MessageType: merged Gerrit-Project: libosmo-sigtran Gerrit-Branch: master Gerrit-Change-Id: Ic0b8b4fbe31ab2e5206a4f4c7907868f20d2ef10 Gerrit-Change-Number: 40549 Gerrit-PatchSet: 4 Gerrit-Owner: pespin <pespin(a)sysmocom.de> Gerrit-Reviewer: Jenkins Builder Gerrit-Reviewer: laforge <laforge(a)osmocom.org> Gerrit-Reviewer: osmith <osmith(a)sysmocom.de> Gerrit-Reviewer: pespin <pespin(a)sysmocom.de>

1 month

1
0
0 0

[XS] Change in libosmo-sigtran[master]: ss7_vty: Improve VTY output of vty_dump_rtable()

by pespin

pespin has submitted this change. ( https://gerrit.osmocom.org/c/libosmo-sigtran/+/40550?usp=email ) Change subject: ss7_vty: Improve VTY output of vty_dump_rtable() ...................................................................... ss7_vty: Improve VTY output of vty_dump_rtable() This same outputting hint is used by Cisco ITP. This commit doesn't change any .vty file because there's actually a bug when adding routes into existing combined linksets, hence the scenario in the .vty file doesn't end up containing a combined linkset with multiple routes, but actually multiple combined linksets each with 1 route. This issue is fixed in the follow-up commit. Change-Id: I87511084278976889ab031fcfd10c09cc60c7595 --- M src/ss7_vty.c 1 file changed, 6 insertions(+), 2 deletions(-) Approvals: Jenkins Builder: Verified osmith: Looks good to me, approved diff --git a/src/ss7_vty.c b/src/ss7_vty.c index 1f443d2..ab9784c 100644 --- a/src/ss7_vty.c +++ b/src/ss7_vty.c @@ -517,9 +517,13 @@ llist_for_each_entry(rt, &clset->routes, list) { bool rt_avail = ss7_route_is_available(rt); - + bool first_rt_in_clset = (rt == llist_first_entry(&clset->routes, struct osmo_ss7_route, list)); + /* Print route str only in first rt in combined linkset. + * This allows users to easily determine visually combined + * linksets: */ + const char *rt_str = first_rt_in_clset ? osmo_ss7_route_print(rt) : ""; vty_out(vty, "%-16s %-5s %c %c %u %-19s %-7s %-7s %-7s %-3s%s", - osmo_ss7_route_print(rt), + rt_str, rt_avail ? "acces" : "INACC", ' ', '0' + rt->cfg.qos_class, -- To view, visit https://gerrit.osmocom.org/c/libosmo-sigtran/+/40550?usp=email To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings?usp=email Gerrit-MessageType: merged Gerrit-Project: libosmo-sigtran Gerrit-Branch: master Gerrit-Change-Id: I87511084278976889ab031fcfd10c09cc60c7595 Gerrit-Change-Number: 40550 Gerrit-PatchSet: 4 Gerrit-Owner: pespin <pespin(a)sysmocom.de> Gerrit-Reviewer: Jenkins Builder Gerrit-Reviewer: laforge <laforge(a)osmocom.org> Gerrit-Reviewer: osmith <osmith(a)sysmocom.de> Gerrit-Reviewer: pespin <pespin(a)sysmocom.de>

1 month

1
0
0 0

[M] Change in libosmo-sigtran[master]: Differentiate between dynamic and static routes

by pespin

pespin has submitted this change. ( https://gerrit.osmocom.org/c/libosmo-sigtran/+/40543?usp=email ) Change subject: Differentiate between dynamic and static routes ...................................................................... Differentiate between dynamic and static routes Differentiation is important because for instance * It shall not be possible to delete dynamic routes over VTY "remove route" command. * Dynamic routes should not be stored in config, ie. not appear during "show running-config" or "write running-config". * During VTY "show cs7 instance 0 route", a "dyn" string keyword sohuld be displayed on each dynamic route row. Related: OS#6755 Change-Id: Ic6c6b46084a1e4063ebf1f5d13e0e03386bb4c45 --- M src/sccp_user.c M src/ss7_as_vty.c M src/ss7_route.c M src/ss7_route.h M src/ss7_route_table.c M src/ss7_route_table.h M src/ss7_vty.c M src/xua_as_fsm.c M src/xua_rkm.c M tests/ss7/ss7_test.c M tests/vty/osmo_stp_route_prio.vty 11 files changed, 85 insertions(+), 50 deletions(-) Approvals: Jenkins Builder: Verified osmith: Looks good to me, but someone else must approve lynxis lazus: Looks good to me, but someone else must approve pespin: Looks good to me, approved diff --git a/src/sccp_user.c b/src/sccp_user.c index 18561c4..bf92c1a 100644 --- a/src/sccp_user.c +++ b/src/sccp_user.c @@ -618,12 +618,12 @@ LOGP(DLSCCP, LOGL_NOTICE, "%s: Using AS instance %s\n", name, as->cfg.name); - /* Create a default route if necessary */ - rt = ss7_route_table_find_route_by_dpc_mask(ss7->rtable_system, 0, 0); + /* Create a default dynamic route if necessary */ + rt = ss7_route_table_find_route_by_dpc_mask(ss7->rtable_system, 0, 0, true); if (!rt) { LOGP(DLSCCP, LOGL_NOTICE, "%s: Creating default route\n", name); rt = ss7_route_create(ss7->rtable_system, 0, 0, - as->cfg.name); + true, as->cfg.name); if (!rt) goto out_as; rt_created = true; @@ -867,7 +867,7 @@ goto out_strings; /* route only selected PC to the client */ - rt = ss7_route_create(ss7->rtable_system, pc, 0xffff, as_name); + rt = ss7_route_create(ss7->rtable_system, pc, 0xffff, true, as_name); if (!rt) goto out_as; diff --git a/src/ss7_as_vty.c b/src/ss7_as_vty.c index c7658d1..c34bbed 100644 --- a/src/ss7_as_vty.c +++ b/src/ss7_as_vty.c @@ -299,12 +299,14 @@ /* When libosmo-sigtran is used in ASP role, the VTY routing table node * (config-cs7-rt) is not available. However, when we add a routing key - * to an AS we still have to put a matching route into the routing - * table. This is done automatically by first removing the old route + * to an AS we still have to put a matching dynamic route into the routing + * table. This is done automatically by first removing the old dynamic route * (users may change the routing key via VTY during runtime) and then - * putting a new route (see below). */ + * putting a new dynamic route (see below). */ if (cs7_role == CS7_ROLE_ASP) { - rt = ss7_route_table_find_route_by_dpc_mask(as->inst->rtable_system, rkey->pc, 0xffffff); + rt = ss7_route_table_find_route_by_dpc_mask(as->inst->rtable_system, + rkey->pc, 0xffffff, + true); if (rt) ss7_route_destroy(rt); } @@ -315,9 +317,9 @@ rkey->si = si ? get_string_value(mtp_si_vals, si) : 0; /* FIXME: input validation */ rkey->ssn = ssn ? atoi(ssn) : 0; /* FIXME: input validation */ - /* automatically add new route (see also comment above) */ + /* automatically add new dynamic route (see also comment above) */ if (cs7_role == CS7_ROLE_ASP) { - if (!ss7_route_create(as->inst->rtable_system, rkey->pc, 0xffffff, as->cfg.name)) { + if (!ss7_route_create(as->inst->rtable_system, rkey->pc, 0xffffff, true, as->cfg.name)) { vty_out(vty, "Cannot create route (pc=%s, linkset=%s) to AS %s", dpc, as->cfg.name, VTY_NEWLINE); return CMD_WARNING; } diff --git a/src/ss7_route.c b/src/ss7_route.c index 6350f71..e71b543 100644 --- a/src/ss7_route.c +++ b/src/ss7_route.c @@ -42,6 +42,7 @@ * \param[in] rtbl Routing Table where the route belongs * \param[in] pc Point Code of the destination of the route * \param[in] mask Mask of the destination Point Code \ref pc + * \param[in] dynamic Whether the route is dynamic * \returns Allocated route (not yet inserted into its rtbl), NULL on error * * The returned route has no linkset associated yet, user *must* associate it @@ -58,9 +59,14 @@ * The route entry allocated with this API can be destroyed/freed at any point using API * ss7_route_destroy(), regardless of it being already inserted or not in * its routing table. + * + * Dynamic routes are not configured by the user (VTY), and hence cannot be + * removed by the user. Dynamic routes are not stored in the config and hence + * they don't show up in eg "show running-config"; they can be listed using + * specific VTY commands like "show cs7 instance 0 route". */ struct osmo_ss7_route * -ss7_route_alloc(struct osmo_ss7_route_table *rtbl, uint32_t pc, uint32_t mask) +ss7_route_alloc(struct osmo_ss7_route_table *rtbl, uint32_t pc, uint32_t mask, bool dynamic) { struct osmo_ss7_route *rt; @@ -78,6 +84,7 @@ rt->cfg.mask = osmo_ss7_pc_normalize(&rtbl->inst->cfg.pc_fmt, mask); rt->cfg.pc = osmo_ss7_pc_normalize(&rtbl->inst->cfg.pc_fmt, pc); rt->cfg.priority = OSMO_SS7_ROUTE_PRIO_DEFAULT; + rt->cfg.dyn_allocated = dynamic; return rt; } @@ -160,7 +167,8 @@ if (clset) { /* check for duplicates */ struct osmo_ss7_route *prev_rt; llist_for_each_entry(prev_rt, &clset->routes, list) { - if (!strcmp(prev_rt->cfg.linkset_name, rt->cfg.linkset_name)) { + if (strcmp(prev_rt->cfg.linkset_name, rt->cfg.linkset_name) == 0 && + prev_rt->cfg.dyn_allocated == rt->cfg.dyn_allocated) { LOGSS7(rtbl->inst, LOGL_ERROR, "Refusing to create route with existing linkset name: pc=%u=%s mask=0x%x via linkset/AS '%s'\n", rt->cfg.pc, osmo_ss7_pointcode_print(rtbl->inst, rt->cfg.pc), @@ -181,6 +189,7 @@ * \param[in] rtbl Routing Table in which the route is to be created * \param[in] pc Point Code of the destination of the route * \param[in] mask Mask of the destination Point Code \ref pc + * \param[in] dynamic Whether the route is dynamic * \param[in] linkset_name string name of the linkset to be used * \returns callee-allocated + initialized route, NULL on error * @@ -192,12 +201,12 @@ */ struct osmo_ss7_route * ss7_route_create(struct osmo_ss7_route_table *rtbl, uint32_t pc, - uint32_t mask, const char *linkset_name) + uint32_t mask, bool dynamic, const char *linkset_name) { struct osmo_ss7_route *rt; int rc; - rt = ss7_route_alloc(rtbl, pc, mask); + rt = ss7_route_alloc(rtbl, pc, mask, dynamic); if (!rt) return NULL; @@ -210,7 +219,7 @@ /* Keep old behavior, return already existing route: */ if (rc == -EADDRINUSE) { talloc_free(rt); - return ss7_route_table_find_route_by_dpc_mask(rtbl, pc, mask); + return ss7_route_table_find_route_by_dpc_mask(rtbl, pc, mask, dynamic); } return rt; @@ -309,8 +318,11 @@ } while (0) APPEND("pc=%u=%s mask=0x%x=%s", - rt->cfg.pc, osmo_ss7_pointcode_print_buf(pc_str, sizeof(pc_str), inst, rt->cfg.pc), - rt->cfg.mask, osmo_ss7_pointcode_print_buf(mask_str, sizeof(mask_str), inst, rt->cfg.mask)); + rt->cfg.pc, osmo_ss7_pointcode_print_buf(pc_str, sizeof(pc_str), inst, rt->cfg.pc), + rt->cfg.mask, osmo_ss7_pointcode_print_buf(mask_str, sizeof(mask_str), inst, rt->cfg.mask)); + + if (rt->cfg.dyn_allocated) + APPEND(" dyn"); if (rt->dest.as) { struct osmo_ss7_as *as = rt->dest.as; diff --git a/src/ss7_route.h b/src/ss7_route.h index 0dcc34e..4bb7a2b 100644 --- a/src/ss7_route.h +++ b/src/ss7_route.h @@ -1,6 +1,7 @@ #pragma once #include <stdint.h> +#include <stdbool.h> #include <osmocom/core/linuxlist.h> /*********************************************************************** @@ -38,14 +39,15 @@ /*! lower priority is higher */ uint32_t priority; uint8_t qos_class; + bool dyn_allocated; } cfg; }; struct osmo_ss7_route * -ss7_route_alloc(struct osmo_ss7_route_table *rtbl, uint32_t pc, uint32_t mask); +ss7_route_alloc(struct osmo_ss7_route_table *rtbl, uint32_t pc, uint32_t mask, bool dynamic); struct osmo_ss7_route * ss7_route_create(struct osmo_ss7_route_table *rtbl, uint32_t dpc, - uint32_t mask, const char *linkset_name); + uint32_t mask, bool dynamic, const char *linkset_name); void ss7_route_destroy(struct osmo_ss7_route *rt); struct osmo_ss7_route * diff --git a/src/ss7_route_table.c b/src/ss7_route_table.c index d2f34a6..fb10df4 100644 --- a/src/ss7_route_table.c +++ b/src/ss7_route_table.c @@ -110,7 +110,7 @@ */ struct osmo_ss7_route * ss7_route_table_find_route_by_dpc_mask(struct osmo_ss7_route_table *rtbl, uint32_t dpc, - uint32_t mask) + uint32_t mask, bool dynamic) { struct osmo_ss7_combined_linkset *clset; struct osmo_ss7_route *rt; @@ -120,11 +120,21 @@ dpc = osmo_ss7_pc_normalize(&rtbl->inst->cfg.pc_fmt, dpc); mask = osmo_ss7_pc_normalize(&rtbl->inst->cfg.pc_fmt, mask); - clset = ss7_route_table_find_combined_linkset_by_dpc_mask(rtbl, dpc, mask); - if (!clset) - return NULL; - rt = llist_first_entry_or_null(&clset->routes, struct osmo_ss7_route, list); - return rt; + /* we assume the combined_links are sorted by mask length, i.e. more + * specific combined links first, and less specific combined links with shorter + * mask later */ + llist_for_each_entry(clset, &rtbl->combined_linksets, list) { + if ((dpc & clset->cfg.mask) != clset->cfg.pc) + continue; + if (mask != clset->cfg.mask) + continue; + llist_for_each_entry(rt, &clset->routes, list) { + if (rt->cfg.dyn_allocated != dynamic) + continue; + return rt; + } + } + return NULL; } struct osmo_ss7_combined_linkset * diff --git a/src/ss7_route_table.h b/src/ss7_route_table.h index 1de3be5..9f27412 100644 --- a/src/ss7_route_table.h +++ b/src/ss7_route_table.h @@ -39,7 +39,7 @@ struct osmo_ss7_route * ss7_route_table_find_route_by_dpc_mask(struct osmo_ss7_route_table *rtbl, uint32_t dpc, - uint32_t mask); + uint32_t mask, bool dynamic); struct osmo_ss7_route * ss7_route_table_lookup_route(struct osmo_ss7_route_table *rtbl, const struct osmo_ss7_route_label *rtlabel); diff --git a/src/ss7_vty.c b/src/ss7_vty.c index 66efc03..1f443d2 100644 --- a/src/ss7_vty.c +++ b/src/ss7_vty.c @@ -398,7 +398,7 @@ return CMD_WARNING; } - rt = ss7_route_alloc(rtable, dpc, mask); + rt = ss7_route_alloc(rtable, dpc, mask, false); if (!rt) { vty_out(vty, "%% Cannot allocate new route%s", VTY_NEWLINE); return CMD_WARNING; @@ -464,7 +464,7 @@ return CMD_WARNING; } - rt = ss7_route_table_find_route_by_dpc_mask(rtable, dpc, mask); + rt = ss7_route_table_find_route_by_dpc_mask(rtable, dpc, mask, false); if (!rt) { vty_out(vty, "cannot find route to be deleted%s", VTY_NEWLINE); return CMD_WARNING; @@ -484,6 +484,8 @@ vty_out(vty, " description %s%s", rtable->cfg.description, VTY_NEWLINE); llist_for_each_entry(clset, &rtable->combined_linksets, list) { llist_for_each_entry(rt, &clset->routes, list) { + if (rt->cfg.dyn_allocated) + continue; vty_out(vty, " update route %s %s linkset %s", osmo_ss7_pointcode_print(rtable->inst, rt->cfg.pc), osmo_ss7_pointcode_print2(rtable->inst, rt->cfg.mask), @@ -516,7 +518,7 @@ llist_for_each_entry(rt, &clset->routes, list) { bool rt_avail = ss7_route_is_available(rt); - vty_out(vty, "%-16s %-5s %c %c %u %-19s %-7s %-7s %-7s%s", + vty_out(vty, "%-16s %-5s %c %c %u %-19s %-7s %-7s %-7s %-3s%s", osmo_ss7_route_print(rt), rt_avail ? "acces" : "INACC", ' ', @@ -526,6 +528,7 @@ rt_avail ? "avail" : "UNAVAIL", "?", rt_avail ? "avail" : "UNAVAIL", + rt->cfg.dyn_allocated ? "dyn" : "", VTY_NEWLINE); } } diff --git a/src/xua_as_fsm.c b/src/xua_as_fsm.c index 3e117ee..6a8941b 100644 --- a/src/xua_as_fsm.c +++ b/src/xua_as_fsm.c @@ -277,12 +277,14 @@ struct osmo_ss7_as *as = xafp->as; struct osmo_ss7_instance *inst = as->inst; - if (ss7_route_table_find_route_by_dpc_mask(inst->rtable_system, as->cfg.routing_key.pc, 0xffffff)) + if (ss7_route_table_find_route_by_dpc_mask(inst->rtable_system, + as->cfg.routing_key.pc, 0xffffff, + true)) return; /* As opposed to M3UA, there is no RKM and we have to implicitly - * automatically add a route once an IPA connection has come up */ - if (ss7_route_create(inst->rtable_system, as->cfg.routing_key.pc, 0xffffff, as->cfg.name)) + * automatically add a dynamic route once an IPA connection has come up */ + if (ss7_route_create(inst->rtable_system, as->cfg.routing_key.pc, 0xffffff, true, as->cfg.name)) xafp->ipa_route_created = true; } @@ -298,7 +300,9 @@ return; /* find the route which we have created if we ever reached ipa_asp_fsm_wait_id_ack2 */ - rt = ss7_route_table_find_route_by_dpc_mask(inst->rtable_system, as->cfg.routing_key.pc, 0xffffff); + rt = ss7_route_table_find_route_by_dpc_mask(inst->rtable_system, + as->cfg.routing_key.pc, 0xffffff, + true); /* no route found, bail out */ if (!rt) { LOGPFSML(fi, LOGL_NOTICE, "Attempting to delete route for this IPA AS, but cannot " diff --git a/src/xua_rkm.c b/src/xua_rkm.c index 1ae5ea6..f03397c 100644 --- a/src/xua_rkm.c +++ b/src/xua_rkm.c @@ -279,8 +279,8 @@ as->cfg.routing_key.pc = dpc; as->cfg.routing_key.context = rctx; - /* add route for that routing key */ - rt = ss7_route_create(as->inst->rtable_system, dpc, 0xFFFFFF, namebuf); + /* add dynamic route for that routing key */ + rt = ss7_route_create(as->inst->rtable_system, dpc, 0xFFFFFF, true, namebuf); if (!rt) { LOGPASP(asp, DLSS7, LOGL_ERROR, "RKM: Cannot insert route for DPC %s / as %s\n", osmo_ss7_pointcode_print(asp->inst, dpc), namebuf); @@ -393,7 +393,9 @@ return -1; } - rt = ss7_route_table_find_route_by_dpc_mask(inst->rtable_system, as->cfg.routing_key.pc, 0xffffff); + rt = ss7_route_table_find_route_by_dpc_mask(inst->rtable_system, + as->cfg.routing_key.pc, 0xffffff, + true); if (!rt) { msgb_append_dereg_res(resp, M3UA_RKM_DEREG_ERR_UNKNOWN, 0); return -1; diff --git a/tests/ss7/ss7_test.c b/tests/ss7/ss7_test.c index bb96e71..c8e1c67 100644 --- a/tests/ss7/ss7_test.c +++ b/tests/ss7/ss7_test.c @@ -188,7 +188,7 @@ /* route with full mask */ route_label = RT_LABEL(0, 12, 0); OSMO_ASSERT(ss7_route_table_lookup_route(rtbl, &route_label) == NULL); - rt = ss7_route_create(rtbl, 12, 0xffff, "a"); + rt = ss7_route_create(rtbl, 12, 0xffff, false, "a"); printf("route with full mask: %s\n", osmo_ss7_route_print(rt)); OSMO_ASSERT(rt); route_label = RT_LABEL(0, 12, 0); @@ -196,7 +196,7 @@ ss7_route_destroy(rt); /* route with partial mask */ - rt = ss7_route_create(rtbl, 8, 0xfff8, "a"); + rt = ss7_route_create(rtbl, 8, 0xfff8, false, "a"); printf("route with partial mask: %s\n", osmo_ss7_route_print(rt)); route_label = RT_LABEL(0, 8, 0); OSMO_ASSERT(ss7_route_table_lookup_route(rtbl, &route_label) == rt); @@ -210,7 +210,7 @@ OSMO_ASSERT(ss7_route_table_lookup_route(rtbl, &route_label) == NULL); /* insert more specific route for 12, must have higher priority * than existing one */ - rt12 = ss7_route_create(rtbl, 12, 0xffff, "b"); + rt12 = ss7_route_create(rtbl, 12, 0xffff, false, "b"); route_label = RT_LABEL(0, 12, 0); OSMO_ASSERT(ss7_route_table_lookup_route(rtbl, &route_label) == rt12); route_label = RT_LABEL(0, 15, 0); @@ -218,7 +218,7 @@ route_label = RT_LABEL(0, 16, 0); OSMO_ASSERT(ss7_route_table_lookup_route(rtbl, &route_label) == NULL); /* add a default route, which should have lowest precedence */ - rtdef = ss7_route_create(rtbl, 0, 0, "a"); + rtdef = ss7_route_create(rtbl, 0, 0, false, "a"); route_label = RT_LABEL(0, 12, 0); OSMO_ASSERT(ss7_route_table_lookup_route(rtbl, &route_label) == rt12); route_label = RT_LABEL(0, 15, 0); @@ -230,7 +230,7 @@ ss7_route_destroy(rt12); ss7_route_destroy(rt); - rt = ss7_route_create(rtbl, 8, 0xfff9, "a"); + rt = ss7_route_create(rtbl, 8, 0xfff9, false, "a"); printf("route with non-consecutive mask: %s\n", osmo_ss7_route_print(rt)); ss7_route_destroy(rt); diff --git a/tests/vty/osmo_stp_route_prio.vty b/tests/vty/osmo_stp_route_prio.vty index 1a3b516..fe89e44 100644 --- a/tests/vty/osmo_stp_route_prio.vty +++ b/tests/vty/osmo_stp_route_prio.vty @@ -92,9 +92,9 @@ Destination C Q P Linkset Name Linkset Non-adj Route ---------------------- - - - ------------------- ------- ------- ------- -3.2.1/14 INACC 0 2 as3 UNAVAIL ? UNAVAIL -3.2.1/14 INACC 0 5 as2 UNAVAIL ? UNAVAIL -3.2.1/14 INACC 7 6 as1 UNAVAIL ? UNAVAIL +3.2.1/14 INACC 0 2 as3 UNAVAIL ? UNAVAIL +3.2.1/14 INACC 0 5 as2 UNAVAIL ? UNAVAIL +3.2.1/14 INACC 7 6 as1 UNAVAIL ? UNAVAIL OsmoSTP(config-cs7-rt)# ! NOW ADD MORE GENERIC ROUTES (SMALLER BITMASK LENGTH) OsmoSTP(config-cs7-rt)# update route 3.2.0 7.255.0 linkset as1 priority 1 @@ -118,12 +118,12 @@ Destination C Q P Linkset Name Linkset Non-adj Route ---------------------- - - - ------------------- ------- ------- ------- -3.2.1/14 INACC 0 2 as3 UNAVAIL ? UNAVAIL -3.2.1/14 INACC 0 5 as2 UNAVAIL ? UNAVAIL -3.2.1/14 INACC 7 6 as1 UNAVAIL ? UNAVAIL -3.2.0/11 INACC 0 1 as1 UNAVAIL ? UNAVAIL -3.2.0/11 INACC 0 1 as2 UNAVAIL ? UNAVAIL -3.2.0/11 INACC 0 1 as3 UNAVAIL ? UNAVAIL +3.2.1/14 INACC 0 2 as3 UNAVAIL ? UNAVAIL +3.2.1/14 INACC 0 5 as2 UNAVAIL ? UNAVAIL +3.2.1/14 INACC 7 6 as1 UNAVAIL ? UNAVAIL +3.2.0/11 INACC 0 1 as1 UNAVAIL ? UNAVAIL +3.2.0/11 INACC 0 1 as2 UNAVAIL ? UNAVAIL +3.2.0/11 INACC 0 1 as3 UNAVAIL ? UNAVAIL OsmoSTP(config-cs7-rt)# do show cs7 instance 0 route binding-table 3.2.1 -- To view, visit https://gerrit.osmocom.org/c/libosmo-sigtran/+/40543?usp=email To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings?usp=email Gerrit-MessageType: merged Gerrit-Project: libosmo-sigtran Gerrit-Branch: master Gerrit-Change-Id: Ic6c6b46084a1e4063ebf1f5d13e0e03386bb4c45 Gerrit-Change-Number: 40543 Gerrit-PatchSet: 5 Gerrit-Owner: pespin <pespin(a)sysmocom.de> Gerrit-Reviewer: Jenkins Builder Gerrit-Reviewer: laforge <laforge(a)osmocom.org> Gerrit-Reviewer: lynxis lazus <lynxis(a)fe80.eu> Gerrit-Reviewer: osmith <osmith(a)sysmocom.de> Gerrit-Reviewer: pespin <pespin(a)sysmocom.de>

1 month

1
0
0 0

[S] Change in libosmo-sigtran[master]: Fix combined linkset lookup when inserting route

by pespin

pespin has submitted this change. ( https://gerrit.osmocom.org/c/libosmo-sigtran/+/40551?usp=email ) Change subject: Fix combined linkset lookup when inserting route ...................................................................... Fix combined linkset lookup when inserting route The early loop termiantion logic was wrong, which meant ss7_route_table_find_combined_linkset() may return earlier than expected and hence a new combined linkset may have been created when adding a route instead of adding it to a previous existing matching one. The fix can be spotted by the fact that now "show cs7 instance 0 route" shows the 3 3.2.0/11 prio=1 routes properly in the same combined linkset in osmo_stp_route_prio.vty. Change-Id: I44d7545a3c6a3ea4839e983104112f8afa6b981a --- M src/ss7_route_table.c M tests/vty/osmo_stp_route_prio.vty 2 files changed, 12 insertions(+), 5 deletions(-) Approvals: pespin: Looks good to me, approved Jenkins Builder: Verified osmith: Looks good to me, but someone else must approve diff --git a/src/ss7_route_table.c b/src/ss7_route_table.c index fb10df4..c816e36 100644 --- a/src/ss7_route_table.c +++ b/src/ss7_route_table.c @@ -184,12 +184,13 @@ /* we assume the combined_links are sorted by mask length, i.e. more * specific routes first, and less specific routes with shorter - * mask later */ + * mask later. Finally, ordered based on priority (lower prio value means higher prio). + */ llist_for_each_entry(clset, &rtbl->combined_linksets, list) { - if (mask < clset->cfg.mask) + if (mask > clset->cfg.mask) break; if (dpc == clset->cfg.pc && mask == clset->cfg.mask) { - if (prio > clset->cfg.priority) + if (prio < clset->cfg.priority) break; if (prio == clset->cfg.priority) return clset; diff --git a/tests/vty/osmo_stp_route_prio.vty b/tests/vty/osmo_stp_route_prio.vty index fe89e44..f2bfd1e 100644 --- a/tests/vty/osmo_stp_route_prio.vty +++ b/tests/vty/osmo_stp_route_prio.vty @@ -97,9 +97,11 @@ 3.2.1/14 INACC 7 6 as1 UNAVAIL ? UNAVAIL OsmoSTP(config-cs7-rt)# ! NOW ADD MORE GENERIC ROUTES (SMALLER BITMASK LENGTH) +OsmoSTP(config-cs7-rt)# update route 3.2.0 7.255.0 linkset as3 priority 3 OsmoSTP(config-cs7-rt)# update route 3.2.0 7.255.0 linkset as1 priority 1 OsmoSTP(config-cs7-rt)# update route 3.2.0 7.255.0 linkset as2 priority 1 OsmoSTP(config-cs7-rt)# update route 3.2.0 7.255.0 linkset as3 priority 1 +OsmoSTP(config-cs7-rt)# update route 3.2.0 7.255.0 linkset as3 priority 2 OsmoSTP(config-cs7-rt)# show running-config ... cs7 instance 0 @@ -111,6 +113,8 @@ update route 3.2.0 7.255.0 linkset as1 priority 1 update route 3.2.0 7.255.0 linkset as2 priority 1 update route 3.2.0 7.255.0 linkset as3 priority 1 + update route 3.2.0 7.255.0 linkset as3 priority 2 + update route 3.2.0 7.255.0 linkset as3 priority 3 ... OsmoSTP(config-cs7-rt)# do show cs7 instance 0 route Routing table = system @@ -122,8 +126,10 @@ 3.2.1/14 INACC 0 5 as2 UNAVAIL ? UNAVAIL 3.2.1/14 INACC 7 6 as1 UNAVAIL ? UNAVAIL 3.2.0/11 INACC 0 1 as1 UNAVAIL ? UNAVAIL -3.2.0/11 INACC 0 1 as2 UNAVAIL ? UNAVAIL -3.2.0/11 INACC 0 1 as3 UNAVAIL ? UNAVAIL + INACC 0 1 as2 UNAVAIL ? UNAVAIL + INACC 0 1 as3 UNAVAIL ? UNAVAIL +3.2.0/11 INACC 0 2 as3 UNAVAIL ? UNAVAIL +3.2.0/11 INACC 0 3 as3 UNAVAIL ? UNAVAIL OsmoSTP(config-cs7-rt)# do show cs7 instance 0 route binding-table 3.2.1 -- To view, visit https://gerrit.osmocom.org/c/libosmo-sigtran/+/40551?usp=email To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings?usp=email Gerrit-MessageType: merged Gerrit-Project: libosmo-sigtran Gerrit-Branch: master Gerrit-Change-Id: I44d7545a3c6a3ea4839e983104112f8afa6b981a Gerrit-Change-Number: 40551 Gerrit-PatchSet: 4 Gerrit-Owner: pespin <pespin(a)sysmocom.de> Gerrit-Reviewer: Jenkins Builder Gerrit-Reviewer: laforge <laforge(a)osmocom.org> Gerrit-Reviewer: osmith <osmith(a)sysmocom.de> Gerrit-Reviewer: pespin <pespin(a)sysmocom.de>

1 month

1
0
0 0

2025

2024

2023

2022

gerrit-log July 2025