<p>dexter <strong>uploaded patch set #8</strong> to this change.</p><p><a href="https://gerrit.osmocom.org/13093">View Change</a></p><pre style="font-family: monospace,monospace; white-space: pre-wrap;">codec_pref: fix special handling for AMR rate configuration  (S15-S0)<br><br>When match_codec_pref() is called and the codec selections from the<br>ASSIGNMENT COMMAND are matched against the interal capabilities, the<br>configurations are checked one by one. When a match is found that match<br>is returned.<br><br>However, the implementation currently does not check the AMR S15-S0 bits<br>when the actual matching happens. This is done afterwards in case AMR<br>gets picked. Unfortunately if the MSC implementation is not obeying the<br>settings the MSC has previously communicated in the L3 COMPL message we<br>may end up with an S15-S0 configuration that has all rate selection<br>(which eventually end up as active set in RSL) bits set to zero. This is<br>an invalid configuration and should be prevented. Also the handling of<br>the S15-S0 bits should happen as part of the matching so that there is a<br>chance to try the nect codec in the list if AMR is unuseable.<br><br>Also S15-S0 has one special setting "Config-NB-Code = 1" (S1 = 1). When<br>this bit is set, the four (in HR only three) most common rates are<br>selected into the active set. If there are also other S-bits set besides<br>S1 we should prefer S1 and discard the other bits.<br><br>- Perform handling of S15-S0 while matching<br>- If S1 is set, prefer this setting and discard all other settings<br><br>Change-Id: Ie52376b51fe07ed07056e8df2e9557293ff67a78<br>Related: SYS#4470<br>---<br>M src/osmo-bsc/codec_pref.c<br>M tests/codec_pref/codec_pref_test.ok<br>2 files changed, 62 insertions(+), 35 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/93/13093/8</pre><p>To view, visit <a href="https://gerrit.osmocom.org/13093">change 13093</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/13093"/><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: Ie52376b51fe07ed07056e8df2e9557293ff67a78 </div>
<div style="display:none"> Gerrit-Change-Number: 13093 </div>
<div style="display:none"> Gerrit-PatchSet: 8 </div>
<div style="display:none"> Gerrit-Owner: dexter <pmaier@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: dexter <pmaier@sysmocom.de> </div>