<p>neels <strong>submitted</strong> this change.</p><p><a href="https://gerrit.osmocom.org/c/osmo-bsc/+/24002">View Change</a></p><div style="white-space:pre-wrap">Approvals:
  laforge: Looks good to me, but someone else must approve
  neels: Looks good to me, approved
  dexter: Looks good to me, but someone else must approve
  Jenkins Builder: Verified

</div><pre style="font-family: monospace,monospace; white-space: pre-wrap;">test_gsm48_multirate_config: rather keep 4x amr_mode<br><br>In osmo-bsc, gsm48_multirate_config() is used in a way where the struct<br>amr_mode *modes always points at the full set of configured modes, and<br>only the AMR bits m4_75 thru m12_2 are unset to filter the used modes.<br><br>In the current test, the bits are unset to filter, but also the struct<br>amr_mode *modes is reduced accordingly. Instead, keep the modes fully<br>populated and unset only bits, like osmo-bsc does in practice.<br><br>The test results should not (and do not) differ.<br><br>Change-Id: I545de6fb66a4b74a3f08899795b4b4d9c4538f58<br>---<br>M tests/gsm0408/gsm0408_test.c<br>1 file changed, 12 insertions(+), 9 deletions(-)<br><br></pre><pre style="font-family: monospace,monospace; white-space: pre-wrap;"><span>diff --git a/tests/gsm0408/gsm0408_test.c b/tests/gsm0408/gsm0408_test.c</span><br><span>index 9a1d3cf..8220c4f 100644</span><br><span>--- a/tests/gsm0408/gsm0408_test.c</span><br><span>+++ b/tests/gsm0408/gsm0408_test.c</span><br><span>@@ -824,12 +824,12 @@</span><br><span>    mr.ms_mode[0].mode = 2;</span><br><span>      mr.ms_mode[1].mode = 4;</span><br><span>      mr.ms_mode[2].mode = 5;</span><br><span style="color: hsl(0, 100%, 40%);">- mr.ms_mode[3].mode = 0;</span><br><span style="color: hsl(120, 100%, 40%);">+       mr.ms_mode[3].mode = 7;</span><br><span>      gsm48_ie->m12_2 = 0;</span><br><span>      mr.ms_mode[2].threshold = 0;</span><br><span>         mr.ms_mode[2].hysteresis = 0;</span><br><span> </span><br><span style="color: hsl(0, 100%, 40%);">-       rc = gsm48_multirate_config(lv, gsm48_ie, mr.ms_mode, 3);</span><br><span style="color: hsl(120, 100%, 40%);">+     rc = gsm48_multirate_config(lv, gsm48_ie, mr.ms_mode, 4);</span><br><span>    OSMO_ASSERT(rc == 0);</span><br><span>        printf("gsm48_multirate_config(): rc=%i, lv=%s\n", rc,</span><br><span>            osmo_hexdump_nospc(lv, 1 + lv[0]));</span><br><span>@@ -838,18 +838,19 @@</span><br><span>   mr.ms_mode[0].mode = 2;</span><br><span>      mr.ms_mode[2].mode = 4;</span><br><span>      mr.ms_mode[1].mode = 5;</span><br><span style="color: hsl(0, 100%, 40%);">- rc = gsm48_multirate_config(lv, gsm48_ie, mr.ms_mode, 3);</span><br><span style="color: hsl(120, 100%, 40%);">+     rc = gsm48_multirate_config(lv, gsm48_ie, mr.ms_mode, 4);</span><br><span>    OSMO_ASSERT(rc == -EINVAL);</span><br><span> </span><br><span>      /* Test #5: Normal configuration with 2 active set members */</span><br><span>        mr.ms_mode[0].mode = 2;</span><br><span>      mr.ms_mode[1].mode = 4;</span><br><span style="color: hsl(0, 100%, 40%);">- mr.ms_mode[2].mode = 0;</span><br><span style="color: hsl(120, 100%, 40%);">+       mr.ms_mode[2].mode = 5;</span><br><span style="color: hsl(120, 100%, 40%);">+       mr.ms_mode[3].mode = 7;</span><br><span>      gsm48_ie->m7_95 = 0;</span><br><span>      mr.ms_mode[1].threshold = 0;</span><br><span>         mr.ms_mode[1].hysteresis = 0;</span><br><span> </span><br><span style="color: hsl(0, 100%, 40%);">-       rc = gsm48_multirate_config(lv, gsm48_ie, mr.ms_mode, 2);</span><br><span style="color: hsl(120, 100%, 40%);">+     rc = gsm48_multirate_config(lv, gsm48_ie, mr.ms_mode, 4);</span><br><span>    OSMO_ASSERT(rc == 0);</span><br><span>        printf("gsm48_multirate_config(): rc=%i, lv=%s\n", rc,</span><br><span>            osmo_hexdump_nospc(lv, 1 + lv[0]));</span><br><span>@@ -857,24 +858,26 @@</span><br><span>   /* Test #6: 2 active set members, but wrong mode order: */</span><br><span>   mr.ms_mode[1].mode = 2;</span><br><span>      mr.ms_mode[0].mode = 4;</span><br><span style="color: hsl(0, 100%, 40%);">- rc = gsm48_multirate_config(lv, gsm48_ie, mr.ms_mode, 2);</span><br><span style="color: hsl(120, 100%, 40%);">+     rc = gsm48_multirate_config(lv, gsm48_ie, mr.ms_mode, 4);</span><br><span>    OSMO_ASSERT(rc == -EINVAL);</span><br><span> </span><br><span>      /* Test #7: Normal configuration with 1 active set member */</span><br><span>         mr.ms_mode[0].mode = 2;</span><br><span style="color: hsl(0, 100%, 40%);">- mr.ms_mode[1].mode = 0;</span><br><span style="color: hsl(120, 100%, 40%);">+       mr.ms_mode[1].mode = 4;</span><br><span style="color: hsl(120, 100%, 40%);">+       mr.ms_mode[2].mode = 5;</span><br><span style="color: hsl(120, 100%, 40%);">+       mr.ms_mode[3].mode = 7;</span><br><span>      gsm48_ie->m7_40 = 0;</span><br><span>      mr.ms_mode[0].threshold = 0;</span><br><span>         mr.ms_mode[0].hysteresis = 0;</span><br><span> </span><br><span style="color: hsl(0, 100%, 40%);">-       rc = gsm48_multirate_config(lv, gsm48_ie, mr.ms_mode, 1);</span><br><span style="color: hsl(120, 100%, 40%);">+     rc = gsm48_multirate_config(lv, gsm48_ie, mr.ms_mode, 4);</span><br><span>    OSMO_ASSERT(rc == 0);</span><br><span>        printf("gsm48_multirate_config(): rc=%i, lv=%s\n", rc,</span><br><span>            osmo_hexdump_nospc(lv, 1 + lv[0]));</span><br><span> </span><br><span>       /* Test #8: 0 active set members: */</span><br><span>         mr.ms_mode[0].mode = 0;</span><br><span style="color: hsl(0, 100%, 40%);">- rc = gsm48_multirate_config(lv, gsm48_ie, mr.ms_mode, 1);</span><br><span style="color: hsl(120, 100%, 40%);">+     rc = gsm48_multirate_config(lv, gsm48_ie, mr.ms_mode, 4);</span><br><span>    OSMO_ASSERT(rc == -EINVAL);</span><br><span> }</span><br><span> </span><br><span></span><br></pre><p>To view, visit <a href="https://gerrit.osmocom.org/c/osmo-bsc/+/24002">change 24002</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/+/24002"/><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: I545de6fb66a4b74a3f08899795b4b4d9c4538f58 </div>
<div style="display:none"> Gerrit-Change-Number: 24002 </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: dexter <pmaier@sysmocom.de> </div>
<div style="display:none"> Gerrit-Reviewer: laforge <laforge@osmocom.org> </div>
<div style="display:none"> Gerrit-Reviewer: neels <nhofmeyr@sysmocom.de> </div>
<div style="display:none"> Gerrit-CC: pespin <pespin@sysmocom.de> </div>
<div style="display:none"> Gerrit-MessageType: merged </div>