Attention is currently required from: neels, 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 4:

(2 comments)

File src/osmo-bsc/nm_bb_transc_fsm.c:

https://gerrit.osmocom.org/c/osmo-bsc/+/29788/comment/8ba653ee_75e66ea9 PS1, Line 345: case NM_EV_RAMP_GO:

I've implemented it together with a libosmocore (ignore_invalid_event_mask). […]

I don't see why you'd need to add it in all FSM states. If the object is ENABLED that means the configuration already went one and hence how it is possible that the outside code send this event to allow start of configuration?

File src/osmo-bsc/nm_bts_fsm.c:

https://gerrit.osmocom.org/c/osmo-bsc/+/29788/comment/496f7252_60093b0c PS1, Line 92: struct gsm_bts *bts,

IMHO this is fine. last example of 6.2) looks the same. […]

You could check all the other configure_loop() functions and use the same format as in all of them ;)