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

[M] Change in osmo-bsc[pespin/rel-1.13.1]: Fix missing LAC/CI hashtable update upon CTRL SET

by osmith

Attention is currently required from: pespin. osmith has posted comments on this change by pespin. ( https://gerrit.osmocom.org/c/osmo-bsc/+/40996?usp=email ) Change subject: Fix missing LAC/CI hashtable update upon CTRL SET ...................................................................... Patch Set 1: Code-Review+2 -- To view, visit https://gerrit.osmocom.org/c/osmo-bsc/+/40996?usp=email To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings?usp=email Gerrit-MessageType: comment Gerrit-Project: osmo-bsc Gerrit-Branch: pespin/rel-1.13.1 Gerrit-Change-Id: If9509534bd4a75f8ed8fb3e0a6bec701c7596861 Gerrit-Change-Number: 40996 Gerrit-PatchSet: 1 Gerrit-Owner: pespin <pespin(a)sysmocom.de> Gerrit-Reviewer: osmith <osmith(a)sysmocom.de> Gerrit-CC: Jenkins Builder Gerrit-CC: keith <keith(a)rhizomatica.org> Gerrit-Attention: pespin <pespin(a)sysmocom.de> Gerrit-Comment-Date: Tue, 02 Sep 2025 13:57:28 +0000 Gerrit-HasComments: No Gerrit-Has-Labels: Yes

2 weeks, 5 days

1
0
0 0

[S] Change in osmo-bsc[pespin/rel-1.13.1]: Bump version: 1.13.1.2-9727f → 1.13.2

by pespin

pespin has uploaded this change for review. ( https://gerrit.osmocom.org/c/osmo-bsc/+/40998?usp=email ) Change subject: Bump version: 1.13.1.2-9727f → 1.13.2 ...................................................................... Bump version: 1.13.1.2-9727f → 1.13.2 Change-Id: Id1a598503349681fd27cbb374b3f31bab7a84996 --- M debian/changelog 1 file changed, 10 insertions(+), 0 deletions(-) git pull ssh://gerrit.osmocom.org:29418/osmo-bsc refs/changes/98/40998/1 diff --git a/debian/changelog b/debian/changelog index 9c5dc70..5d023e6 100644 --- a/debian/changelog +++ b/debian/changelog @@ -1,3 +1,13 @@ +osmo-bsc (1.13.2) unstable; urgency=medium + + [ Keith Whyte ] + * Fix missing LAC/CI hashtable update upon CTRL SET + + [ Pau Espin Pedrol ] + * Fix wrong list entry derreference during bts_depend_find_entry() + + -- Pau Espin Pedrol <pespin(a)sysmocom.de> Tue, 02 Sep 2025 15:54:44 +0200 + osmo-bsc (1.13.1) unstable; urgency=medium * Fix building for debian 13 armv7l -- To view, visit https://gerrit.osmocom.org/c/osmo-bsc/+/40998?usp=email To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings?usp=email Gerrit-MessageType: newchange Gerrit-Project: osmo-bsc Gerrit-Branch: pespin/rel-1.13.1 Gerrit-Change-Id: Id1a598503349681fd27cbb374b3f31bab7a84996 Gerrit-Change-Number: 40998 Gerrit-PatchSet: 1 Gerrit-Owner: pespin <pespin(a)sysmocom.de>

2 weeks, 5 days

1
0
0 0

[XS] Change in osmo-bsc[pespin/rel-1.13.1]: Fix wrong list entry derreference during bts_depend_find_entry()

by pespin

pespin has uploaded this change for review. ( https://gerrit.osmocom.org/c/osmo-bsc/+/40997?usp=email ) Change subject: Fix wrong list entry derreference during bts_depend_find_entry() ...................................................................... Fix wrong list entry derreference during bts_depend_find_entry() Commit 92dc38d1c2b49efd0d4626e7ad5fb2db5ae7e5bc converted bts->depends_on from bitmask to a list, but introduced an error while doing so, iterating the wrong list and hence derreferring wrong entries from it. This caused for instance sporadic malfunction where osmo-bsc refused a BTS from connecting with "Dependency not full-filled". Related: SYS#7062 Related: SYS#7624 Fixes: 92dc38d1c2b49efd0d4626e7ad5fb2db5ae7e5bc Change-Id: I50c8111d71ee4493076cd2cf3d4d850fee3c0414 (cherry picked from commit 5ee02ae7f6e4dc58882298c0e7055cd79f35c93b) --- M src/osmo-bsc/bts.c 1 file changed, 1 insertion(+), 1 deletion(-) git pull ssh://gerrit.osmocom.org:29418/osmo-bsc refs/changes/97/40997/1 diff --git a/src/osmo-bsc/bts.c b/src/osmo-bsc/bts.c index 730cf7a..2cc6153 100644 --- a/src/osmo-bsc/bts.c +++ b/src/osmo-bsc/bts.c @@ -932,7 +932,7 @@ static struct bts_depends_on_entry *bts_depend_find_entry(const struct gsm_bts *bts, gsm_bts_nr_t dep) { struct bts_depends_on_entry *entry; - llist_for_each_entry(entry, &bts->trx_list, list) { + llist_for_each_entry(entry, &bts->depends_on, list) { if (entry->bts_nr == dep) return entry; } -- To view, visit https://gerrit.osmocom.org/c/osmo-bsc/+/40997?usp=email To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings?usp=email Gerrit-MessageType: newchange Gerrit-Project: osmo-bsc Gerrit-Branch: pespin/rel-1.13.1 Gerrit-Change-Id: I50c8111d71ee4493076cd2cf3d4d850fee3c0414 Gerrit-Change-Number: 40997 Gerrit-PatchSet: 1 Gerrit-Owner: pespin <pespin(a)sysmocom.de>

2 weeks, 5 days

1
0
0 0

[M] Change in osmo-bsc[pespin/rel-1.13.1]: Fix missing LAC/CI hashtable update upon CTRL SET

by pespin

pespin has uploaded this change for review. ( https://gerrit.osmocom.org/c/osmo-bsc/+/40996?usp=email ) Change subject: Fix missing LAC/CI hashtable update upon CTRL SET ...................................................................... Fix missing LAC/CI hashtable update upon CTRL SET Add functions to set LAC/CI on a BTS and update hashtables Use these function from both the vty and the control interface. This commit fixes the issue where an update to LAC or CI via the control interface was not resulting in updated hashtables and therefore the BTS was not found during paging. Related: OS#6840 Fixes: 15284337ece94e3ff3ebf9575096290f67d69a88 Change-Id: If9509534bd4a75f8ed8fb3e0a6bec701c7596861 (cherry picked from commit e6b593f3a9dee85deb8682a7cfea0134db8d3982) --- M include/osmocom/bsc/bts.h M src/osmo-bsc/bts.c M src/osmo-bsc/bts_ctrl.c M src/osmo-bsc/bts_vty.c 4 files changed, 99 insertions(+), 17 deletions(-) git pull ssh://gerrit.osmocom.org:29418/osmo-bsc refs/changes/96/40996/1 diff --git a/include/osmocom/bsc/bts.h b/include/osmocom/bsc/bts.h index 0264dcf..a10d3c4 100644 --- a/include/osmocom/bsc/bts.h +++ b/include/osmocom/bsc/bts.h @@ -825,6 +825,9 @@ int gsm_set_bts_model(struct gsm_bts *bts, struct gsm_bts_model *model); int gsm_set_bts_type(struct gsm_bts *bts, enum gsm_bts_type type); +void gsm_bts_set_lac(struct gsm_bts *bts, uint16_t lac); +void gsm_bts_set_ci(struct gsm_bts *bts, uint16_t ci); + struct gsm_bts_trx *gsm_bts_trx_num(const struct gsm_bts *bts, int num); int bts_gprs_mode_is_compat(struct gsm_bts *bts, enum bts_gprs_mode mode); diff --git a/src/osmo-bsc/bts.c b/src/osmo-bsc/bts.c index dd60ef8..730cf7a 100644 --- a/src/osmo-bsc/bts.c +++ b/src/osmo-bsc/bts.c @@ -758,6 +758,30 @@ return 0; } +void gsm_bts_set_lac(struct gsm_bts *bts, uint16_t lac) +{ + bts->location_area_code = lac; + + hash_del(&bts->node_by_lac); + hash_del(&bts->node_by_lac_ci); + hash_add(bts->network->bts_by_lac, &bts->node_by_lac, bts->location_area_code); + hash_add(bts->network->bts_by_lac_ci, &bts->node_by_lac_ci, + LAC_CI_HASHTABLE_KEY(bts->location_area_code, bts->cell_identity)); +} + +void gsm_bts_set_ci(struct gsm_bts *bts, uint16_t ci) +{ + bts->cell_identity = ci; + + hash_del(&bts->node_by_ci); + hash_add(bts->network->bts_by_ci, &bts->node_by_ci, bts->cell_identity); + if (bts->location_area_code != GSM_LAC_RESERVED_DETACHED) { + hash_del(&bts->node_by_lac_ci); + hash_add(bts->network->bts_by_lac_ci, &bts->node_by_lac_ci, + LAC_CI_HASHTABLE_KEY(bts->location_area_code, bts->cell_identity)); + } +} + int bts_gprs_mode_is_compat(struct gsm_bts *bts, enum bts_gprs_mode mode) { if (mode != BTS_GPRS_NONE && diff --git a/src/osmo-bsc/bts_ctrl.c b/src/osmo-bsc/bts_ctrl.c index 2efa661..d63ad89 100644 --- a/src/osmo-bsc/bts_ctrl.c +++ b/src/osmo-bsc/bts_ctrl.c @@ -21,6 +21,7 @@ #include <errno.h> #include <time.h> +#include <inttypes.h> #include <osmocom/ctrl/control_cmd.h> @@ -244,9 +245,76 @@ return 1; } +static int get_bts_lac(struct ctrl_cmd *cmd, void *data) +{ + struct gsm_bts *bts = cmd->node; + cmd->reply = talloc_asprintf(cmd, "%" PRIu16, bts->location_area_code); + if (!cmd->reply) { + cmd->reply = "OOM"; + return CTRL_CMD_ERROR; + } + return CTRL_CMD_REPLY; +} + +static int set_bts_lac(struct ctrl_cmd *cmd, void *data) +{ + struct gsm_bts *bts = cmd->node; + int lac; + + osmo_str_to_int(&lac, cmd->value, 0, 0, 0xffff); + gsm_bts_set_lac(bts, lac); + return get_bts_lac(cmd, data); +} + +static int verify_bts_lac(struct ctrl_cmd *cmd, const char *value, void *data) +{ + int lac; + + if (osmo_str_to_int(&lac, cmd->value, 0, 0, 0xffff) < 0) { + cmd->reply = "Input not within the range"; + return -1; + } + if (lac == GSM_LAC_RESERVED_DETACHED || lac == GSM_LAC_RESERVED_ALL_BTS) { + cmd->reply = talloc_asprintf(cmd, "LAC %d is reserved by GSM 04.08", lac); + return -1; + } + return 0; +} + +static int get_bts_ci(struct ctrl_cmd *cmd, void *data) +{ + struct gsm_bts *bts = cmd->node; + cmd->reply = talloc_asprintf(cmd, "%" PRIu16, bts->cell_identity); + if (!cmd->reply) { + cmd->reply = "OOM"; + return CTRL_CMD_ERROR; + } + return CTRL_CMD_REPLY; +} + +static int set_bts_ci(struct ctrl_cmd *cmd, void *data) +{ + struct gsm_bts *bts = cmd->node; + int ci; + + osmo_str_to_int(&ci, cmd->value, 0, 0, 0xffff); + gsm_bts_set_ci(bts, ci); + return get_bts_ci(cmd, data); +} + +static int verify_bts_ci(struct ctrl_cmd *cmd, const char *value, void *data) +{ + int ci; + + if (osmo_str_to_int(&ci, cmd->value, 0, 0, 0xffff) < 0) { + cmd->reply = "Input not within the range"; + return -1; + } + return 0; +} /* BTS related commands below */ -CTRL_CMD_DEFINE_RANGE(bts_lac, "location-area-code", struct gsm_bts, location_area_code, 0, 65535); -CTRL_CMD_DEFINE_RANGE(bts_ci, "cell-identity", struct gsm_bts, cell_identity, 0, 65535); +CTRL_CMD_DEFINE(bts_lac, "location-area-code"); +CTRL_CMD_DEFINE(bts_ci, "cell-identity"); CTRL_CMD_DEFINE_RANGE(bts_bsic, "bsic", struct gsm_bts, bsic, 0, 63); CTRL_CMD_DEFINE_RANGE(bts_rach_max_delay, "rach-max-delay", struct gsm_bts, rach_max_delay, 1, 127); CTRL_CMD_DEFINE_RANGE(bts_rach_expiry_timeout, "rach-expiry-timeout", struct gsm_bts, rach_expiry_timeout, 4, 64); diff --git a/src/osmo-bsc/bts_vty.c b/src/osmo-bsc/bts_vty.c index 1babd4d..660439a 100644 --- a/src/osmo-bsc/bts_vty.c +++ b/src/osmo-bsc/bts_vty.c @@ -274,15 +274,8 @@ ci, VTY_NEWLINE); return CMD_WARNING; } - bts->cell_identity = ci; - hash_del(&bts->node_by_ci); - hash_add(bts->network->bts_by_ci, &bts->node_by_ci, bts->cell_identity); - if (bts->location_area_code != GSM_LAC_RESERVED_DETACHED) { - hash_del(&bts->node_by_lac_ci); - hash_add(bts->network->bts_by_lac_ci, &bts->node_by_lac_ci, - LAC_CI_HASHTABLE_KEY(bts->location_area_code, bts->cell_identity)); - } + gsm_bts_set_ci(bts, ci); return CMD_SUCCESS; } @@ -305,13 +298,7 @@ return CMD_WARNING; } - bts->location_area_code = lac; - hash_del(&bts->node_by_lac); - hash_del(&bts->node_by_lac_ci); - hash_add(bts->network->bts_by_lac, &bts->node_by_lac, bts->location_area_code); - hash_add(bts->network->bts_by_lac_ci, &bts->node_by_lac_ci, - LAC_CI_HASHTABLE_KEY(bts->location_area_code, bts->cell_identity)); - + gsm_bts_set_lac(bts, lac); return CMD_SUCCESS; } -- To view, visit https://gerrit.osmocom.org/c/osmo-bsc/+/40996?usp=email To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings?usp=email Gerrit-MessageType: newchange Gerrit-Project: osmo-bsc Gerrit-Branch: pespin/rel-1.13.1 Gerrit-Change-Id: If9509534bd4a75f8ed8fb3e0a6bec701c7596861 Gerrit-Change-Number: 40996 Gerrit-PatchSet: 1 Gerrit-Owner: pespin <pespin(a)sysmocom.de> Gerrit-CC: keith <keith(a)rhizomatica.org>

2 weeks, 5 days

1
0
0 0

[XS] Change in osmo-bsc[master]: Fix wrong list entry derreference during bts_depend_find_entry()

by pespin

pespin has submitted this change. ( https://gerrit.osmocom.org/c/osmo-bsc/+/40995?usp=email ) Change subject: Fix wrong list entry derreference during bts_depend_find_entry() ...................................................................... Fix wrong list entry derreference during bts_depend_find_entry() Commit 92dc38d1c2b49efd0d4626e7ad5fb2db5ae7e5bc converted bts->depends_on from bitmask to a list, but introduced an error while doing so, iterating the wrong list and hence derreferring wrong entries from it. This caused for instance sporadic malfunction where osmo-bsc refused a BTS from connecting with "Dependency not full-filled". Related: SYS#7062 Related: SYS#7624 Fixes: 92dc38d1c2b49efd0d4626e7ad5fb2db5ae7e5bc Change-Id: I50c8111d71ee4493076cd2cf3d4d850fee3c0414 --- M src/osmo-bsc/bts.c 1 file changed, 1 insertion(+), 1 deletion(-) Approvals: Jenkins Builder: Verified daniel: Looks good to me, but someone else must approve fixeria: Looks good to me, approved diff --git a/src/osmo-bsc/bts.c b/src/osmo-bsc/bts.c index 730cf7a..2cc6153 100644 --- a/src/osmo-bsc/bts.c +++ b/src/osmo-bsc/bts.c @@ -932,7 +932,7 @@ static struct bts_depends_on_entry *bts_depend_find_entry(const struct gsm_bts *bts, gsm_bts_nr_t dep) { struct bts_depends_on_entry *entry; - llist_for_each_entry(entry, &bts->trx_list, list) { + llist_for_each_entry(entry, &bts->depends_on, list) { if (entry->bts_nr == dep) return entry; } -- To view, visit https://gerrit.osmocom.org/c/osmo-bsc/+/40995?usp=email To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings?usp=email Gerrit-MessageType: merged Gerrit-Project: osmo-bsc Gerrit-Branch: master Gerrit-Change-Id: I50c8111d71ee4493076cd2cf3d4d850fee3c0414 Gerrit-Change-Number: 40995 Gerrit-PatchSet: 2 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: pespin <pespin(a)sysmocom.de>

2 weeks, 5 days

1
0
0 0

[L] Change in osmo-sgsn[master]: Introduce iu_rnc FSM

by osmith

Attention is currently required from: fixeria, lynxis lazus, pespin. osmith has posted comments on this change by pespin. ( https://gerrit.osmocom.org/c/osmo-sgsn/+/40988?usp=email ) Change subject: Introduce iu_rnc FSM ...................................................................... Patch Set 3: (1 comment) File src/sgsn/iu_rnc_fsm.c: https://gerrit.osmocom.org/c/osmo-sgsn/+/40988/comment/0802a735_4a876aa3?us… : PS3, Line 126: #define LEGACY_BEHAVIOR > This is basically the same ifdef we already have in osmo-msc ran_peer_fsm. […] Acknowledged -- To view, visit https://gerrit.osmocom.org/c/osmo-sgsn/+/40988?usp=email To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings?usp=email Gerrit-MessageType: comment Gerrit-Project: osmo-sgsn Gerrit-Branch: master Gerrit-Change-Id: I18b7803500163e78ff6a684095194174b0fb6ee1 Gerrit-Change-Number: 40988 Gerrit-PatchSet: 3 Gerrit-Owner: pespin <pespin(a)sysmocom.de> Gerrit-Reviewer: Jenkins Builder Gerrit-Reviewer: fixeria <vyanitskiy(a)sysmocom.de> Gerrit-Reviewer: lynxis lazus <lynxis(a)fe80.eu> Gerrit-Reviewer: osmith <osmith(a)sysmocom.de> Gerrit-Attention: pespin <pespin(a)sysmocom.de> Gerrit-Attention: fixeria <vyanitskiy(a)sysmocom.de> Gerrit-Attention: lynxis lazus <lynxis(a)fe80.eu> Gerrit-Comment-Date: Tue, 02 Sep 2025 12:47:07 +0000 Gerrit-HasComments: Yes Gerrit-Has-Labels: No Comment-In-Reply-To: osmith <osmith(a)sysmocom.de> Comment-In-Reply-To: pespin <pespin(a)sysmocom.de>

2 weeks, 5 days

1
0
0 0

[XS] Change in osmo-bsc[master]: Fix wrong list entry derreference during bts_depend_find_entry()

by fixeria

Attention is currently required from: pespin. fixeria has posted comments on this change by pespin. ( https://gerrit.osmocom.org/c/osmo-bsc/+/40995?usp=email ) Change subject: Fix wrong list entry derreference during bts_depend_find_entry() ...................................................................... Patch Set 2: Code-Review+2 -- To view, visit https://gerrit.osmocom.org/c/osmo-bsc/+/40995?usp=email To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings?usp=email Gerrit-MessageType: comment Gerrit-Project: osmo-bsc Gerrit-Branch: master Gerrit-Change-Id: I50c8111d71ee4493076cd2cf3d4d850fee3c0414 Gerrit-Change-Number: 40995 Gerrit-PatchSet: 2 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-Attention: pespin <pespin(a)sysmocom.de> Gerrit-Comment-Date: Tue, 02 Sep 2025 12:43:54 +0000 Gerrit-HasComments: No Gerrit-Has-Labels: Yes

2 weeks, 5 days

1
0
0 0

[XS] Change in osmo-bsc[master]: Fix wrong list entry derreference during bts_depend_find_entry()

by daniel

Attention is currently required from: pespin. daniel has posted comments on this change by pespin. ( https://gerrit.osmocom.org/c/osmo-bsc/+/40995?usp=email ) Change subject: Fix wrong list entry derreference during bts_depend_find_entry() ...................................................................... Patch Set 2: Code-Review+1 -- To view, visit https://gerrit.osmocom.org/c/osmo-bsc/+/40995?usp=email To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings?usp=email Gerrit-MessageType: comment Gerrit-Project: osmo-bsc Gerrit-Branch: master Gerrit-Change-Id: I50c8111d71ee4493076cd2cf3d4d850fee3c0414 Gerrit-Change-Number: 40995 Gerrit-PatchSet: 2 Gerrit-Owner: pespin <pespin(a)sysmocom.de> Gerrit-Reviewer: Jenkins Builder Gerrit-Reviewer: daniel <dwillmann(a)sysmocom.de> Gerrit-Attention: pespin <pespin(a)sysmocom.de> Gerrit-Comment-Date: Tue, 02 Sep 2025 12:39:59 +0000 Gerrit-HasComments: No Gerrit-Has-Labels: Yes

2 weeks, 5 days

1
0
0 0

[XS] Change in osmo-bsc[master]: Fix wrong list entry derreference during bts_depend_find_entry()

by pespin

Hello Jenkins Builder, I'd like you to reexamine a change. Please visit https://gerrit.osmocom.org/c/osmo-bsc/+/40995?usp=email to look at the new patch set (#2). Change subject: Fix wrong list entry derreference during bts_depend_find_entry() ...................................................................... Fix wrong list entry derreference during bts_depend_find_entry() Commit 92dc38d1c2b49efd0d4626e7ad5fb2db5ae7e5bc converted bts->depends_on from bitmask to a list, but introduced an error while doing so, iterating the wrong list and hence derreferring wrong entries from it. This caused for instance sporadic malfunction where osmo-bsc refused a BTS from connecting with "Dependency not full-filled". Related: SYS#7062 Related: SYS#7624 Fixes: 92dc38d1c2b49efd0d4626e7ad5fb2db5ae7e5bc Change-Id: I50c8111d71ee4493076cd2cf3d4d850fee3c0414 --- M src/osmo-bsc/bts.c 1 file changed, 1 insertion(+), 1 deletion(-) git pull ssh://gerrit.osmocom.org:29418/osmo-bsc refs/changes/95/40995/2 -- To view, visit https://gerrit.osmocom.org/c/osmo-bsc/+/40995?usp=email To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings?usp=email Gerrit-MessageType: newpatchset Gerrit-Project: osmo-bsc Gerrit-Branch: master Gerrit-Change-Id: I50c8111d71ee4493076cd2cf3d4d850fee3c0414 Gerrit-Change-Number: 40995 Gerrit-PatchSet: 2 Gerrit-Owner: pespin <pespin(a)sysmocom.de> Gerrit-Reviewer: Jenkins Builder

2 weeks, 5 days

1
0
0 0

[XS] Change in osmo-bsc[master]: Fix wrong list entry derreference during bts_depend_find_entry()

by pespin

pespin has uploaded this change for review. ( https://gerrit.osmocom.org/c/osmo-bsc/+/40995?usp=email ) Change subject: Fix wrong list entry derreference during bts_depend_find_entry() ...................................................................... Fix wrong list entry derreference during bts_depend_find_entry() Commit 92dc38d1c2b49efd0d4626e7ad5fb2db5ae7e5bc converted bts->depends_on from bitmask to a list, but introduced an error while doing so, iterating the wrong list and hence derreferring wrong entries from it. This caused for instnace sporadic malfunction where osmo-bsc refused a BTS from connecting with "Dependency not full-filled". Related: SYS#7062 Related: SYS#7624 Fixes: 92dc38d1c2b49efd0d4626e7ad5fb2db5ae7e5bc Change-Id: I50c8111d71ee4493076cd2cf3d4d850fee3c0414 --- M src/osmo-bsc/bts.c 1 file changed, 1 insertion(+), 1 deletion(-) git pull ssh://gerrit.osmocom.org:29418/osmo-bsc refs/changes/95/40995/1 diff --git a/src/osmo-bsc/bts.c b/src/osmo-bsc/bts.c index 730cf7a..2cc6153 100644 --- a/src/osmo-bsc/bts.c +++ b/src/osmo-bsc/bts.c @@ -932,7 +932,7 @@ static struct bts_depends_on_entry *bts_depend_find_entry(const struct gsm_bts *bts, gsm_bts_nr_t dep) { struct bts_depends_on_entry *entry; - llist_for_each_entry(entry, &bts->trx_list, list) { + llist_for_each_entry(entry, &bts->depends_on, list) { if (entry->bts_nr == dep) return entry; } -- To view, visit https://gerrit.osmocom.org/c/osmo-bsc/+/40995?usp=email To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings?usp=email Gerrit-MessageType: newchange Gerrit-Project: osmo-bsc Gerrit-Branch: master Gerrit-Change-Id: I50c8111d71ee4493076cd2cf3d4d850fee3c0414 Gerrit-Change-Number: 40995 Gerrit-PatchSet: 1 Gerrit-Owner: pespin <pespin(a)sysmocom.de>

2 weeks, 5 days

1
0
0 0

2025

2024

2023

2022

gerrit-log September 2025