Attention is currently required from: neels, laforge, fixeria, lynxis lazus.
pespin has posted comments on this change. (
https://gerrit.osmocom.org/c/osmo-bsc/+/29788
)
Change subject: Add BTS setup ramping to prevent BSC overloading
......................................................................
Patch Set 17:
(1 comment)
File src/osmo-bsc/bts_setup_ramp.c:
https://gerrit.osmocom.org/c/osmo-bsc/+/29788/comment/e06c8e2b_74e15919
PS16, Line 40: nm_fsm_dispatch_all(bts, NM_EV_RAMP_GO, NULL);
As long we don't check the state here of the NM
objects, we might end up sending events to the NM st […]
This shouldn't happen
in general since at the moment the RAMP_GO is sent, the BTS is not configured, so it's
in the expected state.
The only case would be where the "administrative" VTY command to unblock all BTS
are executed. That code path should only send the event to BTS requiring it.
--
To view, visit
https://gerrit.osmocom.org/c/osmo-bsc/+/29788
To unsubscribe, or for help writing mail filters, visit
https://gerrit.osmocom.org/settings
Gerrit-Project: osmo-bsc
Gerrit-Branch: master
Gerrit-Change-Id: Id56dde6d58f3d0d20352f6c306598d2cccc6345d
Gerrit-Change-Number: 29788
Gerrit-PatchSet: 17
Gerrit-Owner: lynxis lazus <lynxis(a)fe80.eu>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: fixeria <vyanitskiy(a)sysmocom.de>
Gerrit-Reviewer: pespin <pespin(a)sysmocom.de>
Gerrit-CC: laforge <laforge(a)osmocom.org>
Gerrit-CC: neels <nhofmeyr(a)sysmocom.de>
Gerrit-Attention: neels <nhofmeyr(a)sysmocom.de>
Gerrit-Attention: laforge <laforge(a)osmocom.org>
Gerrit-Attention: fixeria <vyanitskiy(a)sysmocom.de>
Gerrit-Attention: lynxis lazus <lynxis(a)fe80.eu>
Gerrit-Comment-Date: Mon, 14 Nov 2022 11:34:05 +0000
Gerrit-HasComments: Yes
Gerrit-Has-Labels: No
Comment-In-Reply-To: lynxis lazus <lynxis(a)fe80.eu>
Gerrit-MessageType: comment