<p>dexter <strong>uploaded patch set #12</strong> to this change.</p><p><a href="https://gerrit.osmocom.org/12625">View Change</a></p><pre style="font-family: monospace,monospace; white-space: pre-wrap;">assignment_fsm: fix channel allocator preferences<br><br>When the MSC allocates a channel through the ASSIGNMENT REQUEST, it may<br>ask for a TCH/H and a TCH/F at the same time and tell which of the two<br>types it prefers.<br><br>The process of channel allocation currently selects, based on the BTS,<br>MSC and MS capabilites exactly one apropriate codec/rate (e.g. TCH/H)<br>and then tries to allocate it. If that allocation fails, there is no way<br>to try the second choice and the assignment fails.<br><br>For example: The MSC asks for TCH/F and TCH/H, prefering TCH/F, then the<br>channel allocator will try TCH/F and if it fails (all TCH/F are<br>currently in use), then TCH/H is never tried.<br><br>Since the BSC currently only trys the first best codec/rate that is<br>supported it also ignores the preference.<br><br>Lets fix those problems by including the preference information and both<br>possible codec/rate settings into the channel allocation decision.<br><br>Change-Id: I5239e05c1cfbcb8af28f43373a58fa6c2d216c51<br>Related: OS#3503<br>---<br>M include/osmocom/bsc/codec_pref.h<br>M include/osmocom/bsc/gsm_data.h<br>M src/osmo-bsc/assignment_fsm.c<br>M src/osmo-bsc/bsc_vty.c<br>M src/osmo-bsc/codec_pref.c<br>M src/osmo-bsc/handover_fsm.c<br>M src/osmo-bsc/osmo_bsc_bssap.c<br>M tests/codec_pref/codec_pref_test.c<br>8 files changed, 321 insertions(+), 113 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/25/12625/12</pre><p>To view, visit <a href="https://gerrit.osmocom.org/12625">change 12625</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/12625"/><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-MessageType: newpatchset </div>
<div style="display:none"> Gerrit-Change-Id: I5239e05c1cfbcb8af28f43373a58fa6c2d216c51 </div>
<div style="display:none"> Gerrit-Change-Number: 12625 </div>
<div style="display:none"> Gerrit-PatchSet: 12 </div>
<div style="display:none"> Gerrit-Owner: dexter <pmaier@sysmocom.de> </div>
<div style="display:none"> Gerrit-Assignee: Neels Hofmeyr <nhofmeyr@sysmocom.de> </div>
<div style="display:none"> Gerrit-Reviewer: Harald Welte <laforge@gnumonks.org> </div>
<div style="display:none"> Gerrit-Reviewer: Jenkins Builder (1000002) </div>
<div style="display:none"> Gerrit-Reviewer: Max <msuraev@sysmocom.de> </div>
<div style="display:none"> Gerrit-Reviewer: Neels Hofmeyr <nhofmeyr@sysmocom.de> </div>
<div style="display:none"> Gerrit-Reviewer: dexter <pmaier@sysmocom.de> </div>
<div style="display:none"> Gerrit-CC: Vadim Yanitskiy <axilirator@gmail.com> </div>
<div style="display:none"> Gerrit-CC: tnt <tnt@246tNt.com> </div>