<p>Neels Hofmeyr <strong>uploaded patch set #5</strong> to this change.</p><p><a href="https://gerrit.osmocom.org/9668">View Change</a></p><pre style="font-family: monospace,monospace; white-space: pre-wrap;">fix / clarify rsl dtap cache<br><br>In certain situations like handover or assignment, DTAP must not go out via RSL<br>directly but is cached to be submitted later. Make sure that all RSL DTAP<br>sending adheres to this:<br><br>gscon_submit_rsl_dtap() is the new "public" API to request an RSL DTAP to be<br>sent. Depending on the gscon's state, this ends up in the cache or is sent<br>directly. When caching, there is no way to tell whether sending will succeed or<br>not, so semantically it does not make sense to even return a result code. Just<br>return void. Change all "public" callers to gscon_submit_rsl_dtap().<br><br>Merge gsm0808_submit_dtap() and submit_dtap() guts to gsm0808_send_rsl_dtap(),<br>static in bsc_subscr_conn_fsm.c: directly send DTAP, assume a conn->lchan to be<br>present, or otherwise trigger a BSSMAP Clear Request.<br><br>The static submit_dtap() becomes a thin convenience wrapper.<br><br>Move ho_dtap_cache* functions to bsc_subscr_conn_fsm.c and rename to<br>gscon_dtap_cache_* -- they are not only for handover, also for assignment.<br><br>Function gsm0808_submit_dtap() m<br>Introduce function gscon_submit_rsl_dtap()<br><br>Change-Id: I6ffd7aa641c8905292c769400048c96aa0949585<br>---<br>M include/osmocom/bsc/bsc_subscr_conn_fsm.h<br>M include/osmocom/bsc/gsm_data.h<br>M src/osmo-bsc/bsc_api.c<br>M src/osmo-bsc/bsc_init.c<br>M src/osmo-bsc/bsc_subscr_conn_fsm.c<br>M src/osmo-bsc/gsm_04_08_utils.c<br>M src/osmo-bsc/gsm_04_80_utils.c<br>M src/osmo-bsc/osmo_bsc_api.c<br>M tests/bsc/bsc_test.c<br>9 files changed, 185 insertions(+), 160 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/68/9668/5</pre><p>To view, visit <a href="https://gerrit.osmocom.org/9668">change 9668</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/9668"/><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: I6ffd7aa641c8905292c769400048c96aa0949585 </div>
<div style="display:none"> Gerrit-Change-Number: 9668 </div>
<div style="display:none"> Gerrit-PatchSet: 5 </div>
<div style="display:none"> Gerrit-Owner: 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 </div>