<blockquote style="border-left: 1px solid #aaa; margin: 10px 0; padding: 0 10px;"><p style="white-space: pre-wrap; word-wrap: break-word;">build fails with:<br>====<br>handover_fsm.c: In function 'send_handover_performed':<br>handover_fsm.c:678:36: error: storage size of 'ho_perf_params'<br>isn't known<br>struct gsm0808_handover_performed ho_perf_params;<br>^<br>handover_fsm.c:718:2: error: implicit declaration of function<br>'gsm0808_create_handover_performed' [-Werror=implicit-function-declaration]<br>msg = gsm0808_create_handover_performed(&ho_perf_params);<br>^<br>handover_fsm.c:678:36: error: unused variable 'ho_perf_params'<br>[-Werror=unused-variable]<br>struct gsm0808_handover_performed ho_perf_params;<br>^<br>cc1: all warnings being treated as errors<br>====</p><p style="white-space: pre-wrap; word-wrap: break-word;">otherwise the patch looks good to me</p></blockquote><p style="white-space: pre-wrap; word-wrap: break-word;">I think this is just because the related libosmocore patch is not merged. It should build after #11728 is merged.</p><p><a href="https://gerrit.osmocom.org/11734">View Change</a></p><p>3 comments:</p><ul style="list-style: none; padding: 0;"><li style="margin: 0; padding: 0;"><p><a href="https://gerrit.osmocom.org/#/c/11734/1/src/osmo-bsc/handover_fsm.c">File src/osmo-bsc/handover_fsm.c:</a></p><ul style="list-style: none; padding: 0;"><li style="margin: 0; padding: 0 0 0 16px;"><p style="margin-bottom: 4px;"><a href="https://gerrit.osmocom.org/#/c/11734/1/src/osmo-bsc/handover_fsm.c@791">Patch Set #1, Line 791:</a> <code style="font-family:monospace,monospace">        if (result == HO_RESULT_OK && (ho->scope & HO_INTRA_CELL || ho->scope & HO_INTRA_BSC))</code></p><p><blockquote style="border-left: 1px solid #aaa; margin: 10px 0; padding: 0 10px;">&& (ho->scope & (HO_INTRA_CELL | HO_INTRA_BSC)) […]</blockquote></p><p style="white-space: pre-wrap; word-wrap: break-word;">Done</p></li><li style="margin: 0; padding: 0 0 0 16px;"><p style="margin-bottom: 4px;"><a href="https://gerrit.osmocom.org/#/c/11734/1/src/osmo-bsc/handover_fsm.c@792">Patch Set #1, Line 792:</a> <code style="font-family:monospace,monospace">           send_handover_performed(conn);</code></p><p><blockquote style="border-left: 1px solid #aaa; margin: 10px 0; padding: 0 10px;">as I commented in https://gerrit.osmocom. […]</blockquote></p><p style="white-space: pre-wrap; word-wrap: break-word;">Done</p></li><li style="margin: 0; padding: 0 0 0 16px;"><p style="margin-bottom: 4px;"><a href="https://gerrit.osmocom.org/#/c/11734/1/src/osmo-bsc/handover_fsm.c@792">Patch Set #1, Line 792:</a> <code style="font-family:monospace,monospace">              send_handover_performed(conn);</code></p><p><blockquote style="border-left: 1px solid #aaa; margin: 10px 0; padding: 0 10px;">should we *always* send a handover performed, or only when the codec changed? IIUC the MSC need not  […]</blockquote></p><p style="white-space: pre-wrap; word-wrap: break-word;">See 3GPP TS 48.008, 3.2.1.25. There it reads "This message is sent from the BSS to the MSC in order to indicate that the BSS has performed an internal handover." So I think yes, it is sent always. The message contains the cell id as mandatory element, so maybe this is also related to location based services.</p></li></ul></li></ul><p>To view, visit <a href="https://gerrit.osmocom.org/11734">change 11734</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/11734"/><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: comment </div>
<div style="display:none"> Gerrit-Change-Id: If26e5807280e0f75a423b3b04f8e3c698c82a351 </div>
<div style="display:none"> Gerrit-Change-Number: 11734 </div>
<div style="display:none"> Gerrit-PatchSet: 3 </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: Neels Hofmeyr <nhofmeyr@sysmocom.de> </div>
<div style="display:none"> Gerrit-Reviewer: dexter <pmaier@sysmocom.de> </div>
<div style="display:none"> Gerrit-CC: Pau Espin Pedrol <pespin@sysmocom.de> </div>
<div style="display:none"> Gerrit-Comment-Date: Fri, 23 Nov 2018 11:40:58 +0000 </div>
<div style="display:none"> Gerrit-HasComments: Yes </div>
<div style="display:none"> Gerrit-HasLabels: No </div>