Attention is currently required from: neels, laforge, pespin. dexter has posted comments on this change. ( https://gerrit.osmocom.org/c/osmo-bsc/+/31618 ) Change subject: pcu_sock: handle multiple BTSs with multiple BSC co-located PCUs ...................................................................... Patch Set 6: (10 comments) Commit Message: https://gerrit.osmocom.org/c/osmo-bsc/+/31618/comment/6d9ced58_61673261 PS5, Line 15: RBS) are configured. It shouldn't disturb other (ip Access) BTSs. (see my comment above) Patchset: PS5: The commit message was not updated. It now should support multiple Ericsson BTS along with other BTSs. It doesn't break other BTSs since the we filter for Ericsson BTSs before we send any update to the PCU. We also filter when we activate/deactivate the PDCH on PCU reconnects. File src/osmo-bsc/pcu_sock.c: https://gerrit.osmocom.org/c/osmo-bsc/+/31618/comment/a4f2103a_4a9892a7 PS5, Line 367: pcu_tx_info_ind(bts); That is correct. We must not send any info indications to the PCU for BTSs that have a built in PCU. However in a follow up patch I will change this so that it only relies on pcu_connected() since BTSs with built in PCU must not be able to accept connections from a BSC co located PCU anyway. https://gerrit.osmocom.org/c/osmo-bsc/+/31618/comment/53363abf_26e7aa92 PS5, Line 810: llist_for_each_entry(bts, &state->net->bts_list, list) { (see comment below) https://gerrit.osmocom.org/c/osmo-bsc/+/31618/comment/99071479_17e1c76c PS5, Line 960: rc Done https://gerrit.osmocom.org/c/osmo-bsc/+/31618/comment/333db815_734e64e8 PS5, Line 960: Done https://gerrit.osmocom.org/c/osmo-bsc/+/31618/comment/fe33bcfd_64f5eb23 PS5, Line 966: if (conn_bfd->fd >= 0) { (see below) https://gerrit.osmocom.org/c/osmo-bsc/+/31618/comment/802b5202_43cb366c PS5, Line 969: osmo_fd_read_disable(&state->listen_bfd); This is borrowed from osmo-bts. To me this looks like a bug. conn_bfd->fd should be closed instead so that the new connection replaces the old one. I have also fixed this in osmo-bts now. https://gerrit.osmocom.org/c/osmo-bsc/+/31618/comment/2e7be381_9fd629a2 PS5, Line 986: llist_for_each_entry(bts, &state->net->bts_list, list) { Thanks. This makes sense. I also found that the VTY code already allows one socket path per BTS but the status struct had only a pointer to net. Yes, we can just use a backpointer to the BTS in the status struct and we will have one PCU connection per BTS (1:1) and that also means we can run as many Ericsson BTSs we want in parallel, each one with its individual PCU. https://gerrit.osmocom.org/c/osmo-bsc/+/31618/comment/fd69d95f_1fdba550 PS5, Line 1032: bts->pcu_state = state; Thats correct. (see comment above) -- To view, visit https://gerrit.osmocom.org/c/osmo-bsc/+/31618 To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings Gerrit-Project: osmo-bsc Gerrit-Branch: master Gerrit-Change-Id: I0b42c2c130106f6ffca2dd08d079e1a7bda41f0b Gerrit-Change-Number: 31618 Gerrit-PatchSet: 6 Gerrit-Owner: dexter <pmaier(a)sysmocom.de> Gerrit-Reviewer: Jenkins Builder Gerrit-Reviewer: laforge <laforge(a)osmocom.org> Gerrit-Reviewer: pespin <pespin(a)sysmocom.de> Gerrit-CC: neels <nhofmeyr(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-Comment-Date: Mon, 06 Mar 2023 15:47:48 +0000 Gerrit-HasComments: Yes Gerrit-Has-Labels: No Comment-In-Reply-To: neels <nhofmeyr(a)sysmocom.de> Comment-In-Reply-To: pespin <pespin(a)sysmocom.de> Gerrit-MessageType: comment