<p>neels <strong>uploaded patch set #3</strong> to this change.</p><p><a href="https://gerrit.osmocom.org/c/osmo-bsc/+/24358">View Change</a></p><pre style="font-family: monospace,monospace; white-space: pre-wrap;">AMR config cleanup step 3: generate AMR LV on msg composition<br><br>Firstly, do not store the encoded AMR length-value bits in gsm_lchan->*<br>before an activation/modify has actually succeeded.<br><br>And secondly, do not store the AMR LV structure in struct gsm_lchan at<br>all, but only generate the TLV exactly when a message is being composed.<br><br>In gsm48_multirate_config(), generate the LV directly to a msgb instead<br>of a static buffer first. gsm0408_test.c expected output verifies that<br>the generated LV bytes remain unchanged.<br><br>In lchan_mr_config(), introduce a target mr_conf argument, so that Chan<br>Act and Mode Modify may generate the filtered AMR config to different<br>locations (lchan->{activate,modify}.mr_conf_filtered).<br><br>Only after receiving an ACK for Activate/Modify, set<br>lchan->current_mr_conf from lchan->{activate,modify}.mr_conf_filtered.<br><br>Use the properly scoped lchan->activate.mr_conf_filtered for Chan Act,<br>lchan->modify.mr_conf_filtered for Mode Modify and<br>new_lchan->current_mr_conf for Handover Command as appropriate.<br><br>Related: SYS#5315 OS#4940 OS#3787 OS#3833<br>Change-Id: Ie57f9d0e3912632903d9740291225bfd1634ed47<br>---<br>M include/osmocom/bsc/gsm_04_08_rr.h<br>M include/osmocom/bsc/gsm_data.h<br>M src/osmo-bsc/abis_rsl.c<br>M src/osmo-bsc/gsm_04_08_rr.c<br>M src/osmo-bsc/lchan_fsm.c<br>M tests/gsm0408/gsm0408_test.c<br>6 files changed, 140 insertions(+), 100 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/58/24358/3</pre><p>To view, visit <a href="https://gerrit.osmocom.org/c/osmo-bsc/+/24358">change 24358</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/+/24358"/><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: Ie57f9d0e3912632903d9740291225bfd1634ed47 </div>
<div style="display:none"> Gerrit-Change-Number: 24358 </div>
<div style="display:none"> Gerrit-PatchSet: 3 </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>