Patch Set 1: (2 comments)
https://gerrit.osmocom.org/#/c/7080/1/src/common/l1sap.c File src/common/l1sap.c: Line 1199 this busy_thresh check here is above the "ber10k" check... Line 1234: btsb->load.rach.busy++; ...but here the "busy_thresh" check comes after the rach_pass_filter() which now contains the "ber10k" check. Is that intended? If yes would be good to explain in the commit log. -- To view, visit https://gerrit.osmocom.org/7080 To unsubscribe, visit https://gerrit.osmocom.org/settings Gerrit-MessageType: comment Gerrit-Change-Id: I08277e215c4c16ac91ebd3533172a8f861d6eb46 Gerrit-PatchSet: 1 Gerrit-Project: osmo-bts Gerrit-Branch: master Gerrit-Owner: Vadim Yanitskiy <[email protected]> Gerrit-Reviewer: Jenkins Builder Gerrit-Reviewer: Neels Hofmeyr <[email protected]> Gerrit-HasComments: Yes
