Attention is currently required from: laforge, pespin. fixeria has posted comments on this change by pespin. ( https://gerrit.osmocom.org/c/osmo-bsc/+/38052?usp=email )
Change subject: Initial support for >256 BTS ...................................................................... Patch Set 2: (1 comment) Commit Message: https://gerrit.osmocom.org/c/osmo-bsc/+/38052/comment/c9c11906_b3072abb?usp=email : PS1, Line 14: The ipaccess OML bts_nr is still kept as uint8_t so nothing changes over : the wire. That's because in ipaccess OML, in general each BTS is : bt > Which other BTSs are we speaking about which are not ip.access connecting to > osmo-bsc? Ericsson om2k? nokia foobar? Yes, these. Check `enum gsm_bts_type`. > I don't know tbh, One would needs to check if those have some sort of > limitation ... At least a quick git-grep shows me that `bts->nr` is sent over the wire in the om2k, for instance: ``` src/osmo-bsc/abis_om2000.c:3189: om2k_mo_init(&trx->rbs2000.trxc.om2k_mo, OM2K_MO_CLS_TRXC, bts->nr, 255, trx->nr); src/osmo-bsc/abis_om2000.c:3192: om2k_mo_init(&trx->rbs2000.tx.om2k_mo, OM2K_MO_CLS_TX, bts->nr, 255, trx->nr); src/osmo-bsc/abis_om2000.c:3195: om2k_mo_init(&trx->rbs2000.rx.om2k_mo, OM2K_MO_CLS_RX, bts->nr, 255, trx->nr); src/osmo-bsc/abis_om2000.c:3200: om2k_mo_init(&ts->rbs2000.om2k_mo, OM2K_MO_CLS_TS, bts->nr, trx->nr, i); src/osmo-bsc/abis_om2000.c:3217: om2k_mo_init(&bts->rbs2000.cf.om2k_mo, OM2K_MO_CLS_CF, bts->nr, 0xFF, 0); src/osmo-bsc/abis_om2000.c:3220: om2k_mo_init(&bts->rbs2000.is.om2k_mo, OM2K_MO_CLS_IS, bts->nr, 0xFF, 0); src/osmo-bsc/abis_om2000.c:3223: om2k_mo_init(&bts->rbs2000.con.om2k_mo, OM2K_MO_CLS_CON, bts->nr, 0xFF, 0); src/osmo-bsc/abis_om2000.c:3226: om2k_mo_init(&bts->rbs2000.dp.om2k_mo, OM2K_MO_CLS_DP, bts->nr, 0xFF, 0); src/osmo-bsc/abis_om2000.c:3228: om2k_mo_init(&bts->rbs2000.tf.om2k_mo, OM2K_MO_CLS_TF, bts->nr, 0xFF, 0); src/osmo-bsc/abis_om2000.c:3231: om2k_mo_init(&bts->rbs2000.mctr.om2k_mo, OM2K_MO_CLS_MCTR, bts->nr, 0xFF, 0); ``` This looks like a problem to me, given the definition of `struct abis_om2k_mo`: ``` 551 /* Ericsson OM2000 Managed Object */ 552 struct abis_om2k_mo { 553 uint8_t class; 554 uint8_t bts; 555 uint8_t assoc_so; 556 uint8_t inst; 557 } __attribute__ ((packed)); ``` Harald definitely knows more about om2k, I am just raising a concern, which is specifically about non-ip.access BTS models osmo-bsc also supports. -- To view, visit https://gerrit.osmocom.org/c/osmo-bsc/+/38052?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: I607a68efeb5f4a50cce107d11d3c5126b7d8f81a Gerrit-Change-Number: 38052 Gerrit-PatchSet: 2 Gerrit-Owner: pespin <[email protected]> Gerrit-Reviewer: Jenkins Builder Gerrit-CC: fixeria <[email protected]> Gerrit-CC: laforge <[email protected]> Gerrit-Attention: laforge <[email protected]> Gerrit-Attention: pespin <[email protected]> Gerrit-Comment-Date: Mon, 09 Sep 2024 08:28:20 +0000 Gerrit-HasComments: Yes Gerrit-Has-Labels: No Comment-In-Reply-To: laforge <[email protected]> Comment-In-Reply-To: pespin <[email protected]> Comment-In-Reply-To: fixeria <[email protected]>
