<p>neels <strong>uploaded patch set #2</strong> to this change.</p><p><a href="https://gerrit.osmocom.org/c/osmo-bsc/+/24352">View Change</a></p><pre style="font-family: monospace,monospace; white-space: pre-wrap;">make sure channel mode and s15_s0 are updated only after an ACK<br><br>I noticed during testing that an lchan used as TCH/F in fact still had<br>its channel mode set to Signalling -- because on Assignment, the Speech<br>mode used to be placed in the *previous* lchan and the new lchan was<br>never updated after the Activ ACK. This is unbearable confusion which I<br>complained about numerous times, so far mostly for cosmetic reasons. But<br>implementing re-assignment properly actually requires this to be cleaned<br>up.<br><br>Keep all volatile chan mode settings in lchan->activate.* or<br>lchan->modify.*, and only update lchan->* members when an ACK has been<br>received for those settings. So a failed request keeps a sane state.<br><br>Make sure that those settings are in fact updated in the proper lchan,<br>upon an ACK, so that subsequent re-assignment or mode-modify know the<br>accurate lchan state.<br><br>Related are upcoming patches that sort out the AMR multirate<br>configuration in a similar fashion, see<br>Iebac2dc26412d877e5364f90d6f2ed7a7952351e<br>Ia7519d2fa9e7f0b61b222d27d077bde4660c40b9<br>Ie57f9d0e3912632903d9740291225bfd1634ed47.<br><br>Related: SYS#5315 OS#4940 OS#3787 OS#3833<br>Change-Id: Ie0da36124d73efc28a8809b63d7c96e2167fc412<br>---<br>M include/osmocom/bsc/gsm_data.h<br>M src/osmo-bsc/abis_rsl.c<br>M src/osmo-bsc/assignment_fsm.c<br>M src/osmo-bsc/bsc_vty.c<br>M src/osmo-bsc/gsm_04_08_rr.c<br>M src/osmo-bsc/gsm_data.c<br>M src/osmo-bsc/handover_decision_2.c<br>M src/osmo-bsc/handover_fsm.c<br>M src/osmo-bsc/lchan_fsm.c<br>M src/osmo-bsc/lchan_rtp_fsm.c<br>M src/osmo-bsc/osmo_bsc_bssap.c<br>M src/osmo-bsc/osmo_bsc_lcls.c<br>M tests/handover/handover_test.c<br>13 files changed, 187 insertions(+), 164 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/52/24352/2</pre><p>To view, visit <a href="https://gerrit.osmocom.org/c/osmo-bsc/+/24352">change 24352</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/+/24352"/><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: Ie0da36124d73efc28a8809b63d7c96e2167fc412 </div>
<div style="display:none"> Gerrit-Change-Number: 24352 </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-MessageType: newpatchset </div>