<p>neels <strong>uploaded patch set #2</strong> to this change.</p><p><a href="https://gerrit.osmocom.org/c/osmo-bsc/+/22267">View Change</a></p><pre style="font-family: monospace,monospace; white-space: pre-wrap;">lchan_avail(): omit logging for handover decision 2<br><br>Add bool log argument to lchan_avail_by_type() and omit logging when<br>passed as false. >From handover_decision_2.c, pass 'log' as false, from<br>all other callers pass true, i.e. for unchanged behavior.<br><br>Rationale:<br><br>Usually, we use lchan_avail_by_type() to select a new lchan to initiate<br>actual service. For that, it is interesting to see how osmo-bsc decides<br>which lchan will be used.<br><br>For handover decision 2, we since recently call lchan_avail_by_type()<br>for each and every handover candidate, to determine whether it will<br>occupy a dynamic timeslot or not (to know whether we would congest the<br>other TCH kind). So this happens for each permutation of source lchan<br>and target cell. That produces a lot of logging, out of proportion of<br>being useful to the maintainer.<br><br>Change-Id: Ia403f8fc853ca9ea9e81f7a7395df6b23845ebed<br>---<br>M include/osmocom/bsc/lchan_select.h<br>M src/osmo-bsc/abis_rsl.c<br>M src/osmo-bsc/handover_decision_2.c<br>M src/osmo-bsc/lchan_select.c<br>4 files changed, 25 insertions(+), 23 deletions(-)<br><br></pre><pre style="font-family: monospace,monospace; white-space: pre-wrap;">git pull ssh://gerrit.osmocom.org:29418/osmo-bsc refs/changes/67/22267/2</pre><p>To view, visit <a href="https://gerrit.osmocom.org/c/osmo-bsc/+/22267">change 22267</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/+/22267"/><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: Ia403f8fc853ca9ea9e81f7a7395df6b23845ebed </div>
<div style="display:none"> Gerrit-Change-Number: 22267 </div>
<div style="display:none"> Gerrit-PatchSet: 2 </div>
<div style="display:none"> Gerrit-Owner: neels <nhofmeyr@sysmocom.de> </div>
<div style="display:none"> Gerrit-Reviewer: Jenkins Builder </div>
<div style="display:none"> Gerrit-Reviewer: pespin <pespin@sysmocom.de> </div>
<div style="display:none"> Gerrit-MessageType: newpatchset </div>