Attention is currently required from: fixeria.

matanp has posted comments on this change. ( https://gerrit.osmocom.org/c/osmo-bsc/+/35475?usp=email )

Change subject: abis_rsl: Add rach expiry timeout ......................................................................

Patch Set 4:

(3 comments)

Commit Message:

https://gerrit.osmocom.org/c/osmo-bsc/+/35475/comment/2752ab53_466fba6e PS3, Line 7: abis_rsl: Separate rach expiry delay from radio link timeout

Please elaborate more why a separate parameter is needed (here in COMMIT_MSG).

Ack

File src/osmo-bsc/bts_vty.c:

https://gerrit.osmocom.org/c/osmo-bsc/+/35475/comment/6ad723c8_438622ae PS3, Line 777: expiry timeout

Let's combine it into a single command item `expiry-timeout`.

Ack

https://gerrit.osmocom.org/c/osmo-bsc/+/35475/comment/c8eba3d6_3ce9690e PS3, Line 4532: vty_out(vty, " rach max-delay %u%s", bts->rach_max_delay, VTY_NEWLINE);

You're missing to print `rach expiry-timeout` here.

Ack