<p style="white-space: pre-wrap; word-wrap: break-word;">apart from the NOTICE I think it's good to get merged.</p><p><a href="https://gerrit.osmocom.org/c/osmo-bsc/+/23574">View Change</a></p><p>2 comments:</p><ul style="list-style: none; padding: 0;"><li style="margin: 0; padding: 0;"><p><a href="https://gerrit.osmocom.org/c/osmo-bsc/+/23574/3/src/osmo-bsc/abis_rsl.c">File src/osmo-bsc/abis_rsl.c:</a></p><ul style="list-style: none; padding: 0;"><li style="margin: 0; padding: 0 0 0 16px;"><p style="margin-bottom: 4px;"><a href="https://gerrit.osmocom.org/c/osmo-bsc/+/23574/3/src/osmo-bsc/abis_rsl.c@1530">Patch Set #3, Line 1530:</a> <code style="font-family:monospace,monospace">LOGL_NOTICE</code></p><p style="white-space: pre-wrap; word-wrap: break-word;">not sure we want 'notice' on all of these, if this happens quite frequently on some BTSs.  In the end, it is you (Rhizomatica) who suffers most from all of these ending up in logs ;)</p><p style="white-space: pre-wrap; word-wrap: break-word;">We do have the counter, which is great.  I personally wouldn't go beyond DEBUG for the log, but it's of course always a bit of a matter of taste.</p></li></ul></li><li style="margin: 0; padding: 0;"><p><a href="https://gerrit.osmocom.org/c/osmo-bsc/+/23574/3/src/osmo-bsc/bsc_vty.c">File src/osmo-bsc/bsc_vty.c:</a></p><ul style="list-style: none; padding: 0;"><li style="margin: 0; padding: 0 0 0 16px;"><p style="margin-bottom: 4px;"><a href="https://gerrit.osmocom.org/c/osmo-bsc/+/23574/3/src/osmo-bsc/bsc_vty.c@2755">Patch Set #3, Line 2755:</a> <code style="font-family:monospace,monospace">            "rach max-delay <0-127>",</code></p><p><blockquote style="border-left: 1px solid #aaa; margin: 10px 0; padding: 0 10px;">Also, allowing 0 is strange.. This would essentially disable Access. […]</blockquote></p><p style="white-space: pre-wrap; word-wrap: break-word;">I think 1-127 or 1-255 is a reasonably permitted range. 0 is indeed odd.</p></li></ul></li></ul><p>To view, visit <a href="https://gerrit.osmocom.org/c/osmo-bsc/+/23574">change 23574</a>. To unsubscribe, or for help writing mail filters, visit <a href="https://gerrit.osmocom.org/settings">settings</a>.</p><div itemscope itemtype="http://schema.org/EmailMessage"><div itemscope itemprop="action" itemtype="http://schema.org/ViewAction"><link itemprop="url" href="https://gerrit.osmocom.org/c/osmo-bsc/+/23574"/><meta itemprop="name" content="View Change"/></div></div>

<div style="display:none"> Gerrit-Project: osmo-bsc </div>
<div style="display:none"> Gerrit-Branch: master </div>
<div style="display:none"> Gerrit-Change-Id: Ie8987bcc0e43921bc753162b77a0efc68799b3ce </div>
<div style="display:none"> Gerrit-Change-Number: 23574 </div>
<div style="display:none"> Gerrit-PatchSet: 3 </div>
<div style="display:none"> Gerrit-Owner: keith <keith@rhizomatica.org> </div>
<div style="display:none"> Gerrit-Reviewer: Jenkins Builder </div>
<div style="display:none"> Gerrit-Reviewer: fixeria <vyanitskiy@sysmocom.de> </div>
<div style="display:none"> Gerrit-CC: laforge <laforge@osmocom.org> </div>
<div style="display:none"> Gerrit-Comment-Date: Fri, 02 Apr 2021 11:57:21 +0000 </div>
<div style="display:none"> Gerrit-HasComments: Yes </div>
<div style="display:none"> Gerrit-Has-Labels: No </div>
<div style="display:none"> Comment-In-Reply-To: keith <keith@rhizomatica.org> </div>
<div style="display:none"> Gerrit-MessageType: comment </div>