<p>Neels Hofmeyr has uploaded this change for <strong>review</strong>.</p><p><a href="https://gerrit.osmocom.org/13191">View Change</a></p><pre style="font-family: monospace,monospace; white-space: pre-wrap;">msc: for mo and mt call, interleave MGCP and Assignment<br><br>Upcoming osmo-msc changes move away from the current ordering of MGCP and<br>Assignment messages. Allow these async dialogs to appear in any order.<br><br>Change-Id: Ia06af1e347601949f4ddb19f963daa400766d9e7<br>---<br>M msc/BSC_ConnectionHandler.ttcn<br>1 file changed, 43 insertions(+), 35 deletions(-)<br><br></pre><pre style="font-family: monospace,monospace; white-space: pre-wrap;">git pull ssh://gerrit.osmocom.org:29418/osmo-ttcn3-hacks refs/changes/91/13191/1</pre><pre style="font-family: monospace,monospace; white-space: pre-wrap;"><span>diff --git a/msc/BSC_ConnectionHandler.ttcn b/msc/BSC_ConnectionHandler.ttcn</span><br><span>index e6cf2b0..90cdf25 100644</span><br><span>--- a/msc/BSC_ConnectionHandler.ttcn</span><br><span>+++ b/msc/BSC_ConnectionHandler.ttcn</span><br><span>@@ -513,31 +513,37 @@</span><br><span>              f_mgcp_par_append(mgcp_resp.params, ts_MgcpParSpecEP(cpars.mgcp_ep));</span><br><span>                MGCP.send(mgcp_resp);</span><br><span>                }</span><br><span style="color: hsl(0, 100%, 40%);">-       /* Second MGCP CRCX (this time for MSS/CN side) */</span><br><span style="color: hsl(0, 100%, 40%);">-      MGCP.receive(tr_CRCX(cpars.mgcp_ep)) -> value mgcp_cmd {</span><br><span style="color: hsl(0, 100%, 40%);">-             var SDP_Message sdp := valueof(ts_SDP(cpars.mgw_rtp_ip_mss, cpars.mgw_rtp_ip_mss,</span><br><span style="color: hsl(0, 100%, 40%);">-                                                       hex2str(cpars.mgcp_call_id), "42",</span><br><span style="color: hsl(0, 100%, 40%);">-                                                    cpars.mgw_rtp_port_mss,</span><br><span style="color: hsl(0, 100%, 40%);">-                                                 { int2str(cpars.rtp_payload_type) },</span><br><span style="color: hsl(0, 100%, 40%);">-                                                    { valueof(ts_SDP_rtpmap(cpars.rtp_payload_type,</span><br><span style="color: hsl(0, 100%, 40%);">-                                                                         cpars.rtp_sdp_format)),</span><br><span style="color: hsl(0, 100%, 40%);">-                                                   valueof(ts_SDP_ptime(20)) }));</span><br><span style="color: hsl(0, 100%, 40%);">-                MGCP.send(ts_CRCX_ACK(mgcp_cmd.line.trans_id, cpars.mgcp_connection_id_mss, sdp));</span><br><span style="color: hsl(0, 100%, 40%);">-              /* MSC acknowledges the MNCC_CREATE to the MNCC handler */</span><br><span style="color: hsl(0, 100%, 40%);">-              MNCC.receive(tr_MNCC_RTP_CREATE(cpars.mncc_callref));</span><br><span style="color: hsl(0, 100%, 40%);">-   }</span><br><span> </span><br><span style="color: hsl(0, 100%, 40%);">-   /* expect the MSC to trigger a BSSMAP ASSIGNMENT */</span><br><span>  var BSSMAP_IE_AoIP_TransportLayerAddress tla_ass :=</span><br><span>          valueof(ts_BSSMAP_IE_AoIP_TLA4(f_inet_addr(cpars.mgw_rtp_ip_bss),cpars.mgw_rtp_port_bss));</span><br><span style="color: hsl(0, 100%, 40%);">-      BSSAP.receive(tr_BSSMAP_AssignmentReq(omit, tla_ass)) {</span><br><span style="color: hsl(0, 100%, 40%);">-         var BSSMAP_IE_AoIP_TransportLayerAddress tla;</span><br><span style="color: hsl(0, 100%, 40%);">-           var BSSMAP_IE_SpeechCodec codec;</span><br><span style="color: hsl(0, 100%, 40%);">-                tla := valueof(ts_BSSMAP_IE_AoIP_TLA4(f_inet_addr(cpars.bss_rtp_ip), cpars.bss_rtp_port));</span><br><span style="color: hsl(0, 100%, 40%);">-              codec := valueof(ts_BSSMAP_IE_SpeechCodec({ts_CodecFR}));</span><br><span style="color: hsl(0, 100%, 40%);">-               BSSAP.send(ts_BSSMAP_AssignmentComplete(omit, tla, codec));</span><br><span style="color: hsl(120, 100%, 40%);">+</span><br><span style="color: hsl(120, 100%, 40%);">+ interleave {</span><br><span style="color: hsl(120, 100%, 40%);">+  /* Second MGCP CRCX (this time for MSS/CN side) */</span><br><span style="color: hsl(120, 100%, 40%);">+    [] MGCP.receive(tr_CRCX(cpars.mgcp_ep)) -> value mgcp_cmd {</span><br><span style="color: hsl(120, 100%, 40%);">+                        var SDP_Message sdp := valueof(ts_SDP(cpars.mgw_rtp_ip_mss, cpars.mgw_rtp_ip_mss,</span><br><span style="color: hsl(120, 100%, 40%);">+                                                             hex2str(cpars.mgcp_call_id), "42",</span><br><span style="color: hsl(120, 100%, 40%);">+                                                          cpars.mgw_rtp_port_mss,</span><br><span style="color: hsl(120, 100%, 40%);">+                                                               { int2str(cpars.rtp_payload_type) },</span><br><span style="color: hsl(120, 100%, 40%);">+                                                          { valueof(ts_SDP_rtpmap(cpars.rtp_payload_type,</span><br><span style="color: hsl(120, 100%, 40%);">+                                                                                       cpars.rtp_sdp_format)),</span><br><span style="color: hsl(120, 100%, 40%);">+                                                                 valueof(ts_SDP_ptime(20)) }));</span><br><span style="color: hsl(120, 100%, 40%);">+                      MGCP.send(ts_CRCX_ACK(mgcp_cmd.line.trans_id, cpars.mgcp_connection_id_mss, sdp));</span><br><span style="color: hsl(120, 100%, 40%);">+                    /* MSC acknowledges the MNCC_CREATE to the MNCC handler */</span><br><span style="color: hsl(120, 100%, 40%);">+                    MNCC.receive(tr_MNCC_RTP_CREATE(cpars.mncc_callref));</span><br><span>                }</span><br><span> </span><br><span style="color: hsl(120, 100%, 40%);">+ /* expect the MSC to trigger a BSSMAP ASSIGNMENT */</span><br><span style="color: hsl(120, 100%, 40%);">+   [] BSSAP.receive(tr_BSSMAP_AssignmentReq(omit, tla_ass)) {</span><br><span style="color: hsl(120, 100%, 40%);">+                    var BSSMAP_IE_AoIP_TransportLayerAddress tla;</span><br><span style="color: hsl(120, 100%, 40%);">+                 var BSSMAP_IE_SpeechCodec codec;</span><br><span style="color: hsl(120, 100%, 40%);">+</span><br><span style="color: hsl(120, 100%, 40%);">+                    tla := valueof(ts_BSSMAP_IE_AoIP_TLA4(f_inet_addr(cpars.bss_rtp_ip), cpars.bss_rtp_port));</span><br><span style="color: hsl(120, 100%, 40%);">+                    codec := valueof(ts_BSSMAP_IE_SpeechCodec({ts_CodecFR}));</span><br><span style="color: hsl(120, 100%, 40%);">+</span><br><span style="color: hsl(120, 100%, 40%);">+                   BSSAP.send(ts_BSSMAP_AssignmentComplete(omit, tla, codec));</span><br><span style="color: hsl(120, 100%, 40%);">+           }</span><br><span style="color: hsl(120, 100%, 40%);">+     }</span><br><span style="color: hsl(120, 100%, 40%);">+</span><br><span>  /* MS -> MSC: ALERTING */</span><br><span>         BSSAP.send(ts_PDU_DTAP_MO(ts_ML3_MO_CC_CONNECT(cpars.transaction_id)));</span><br><span>      MNCC.receive(tr_MNCC_SETUP_cnf(cpars.mncc_callref));</span><br><span>@@ -613,25 +619,27 @@</span><br><span>                 MGCP.send(mgcp_resp);</span><br><span>                }</span><br><span>    }</span><br><span style="color: hsl(0, 100%, 40%);">-       /* Second MGCP CRCX (this time for MSS/CN side) */</span><br><span style="color: hsl(0, 100%, 40%);">-      MGCP.receive(tr_CRCX(cpars.mgcp_ep)) -> value mgcp_cmd {</span><br><span style="color: hsl(0, 100%, 40%);">-             var SDP_Message sdp := valueof(ts_SDP(cpars.mgw_rtp_ip_mss, cpars.mgw_rtp_ip_mss,</span><br><span style="color: hsl(0, 100%, 40%);">-                                                       hex2str(cpars.mgcp_call_id), "42",</span><br><span style="color: hsl(0, 100%, 40%);">-                                                    cpars.mgw_rtp_port_mss,</span><br><span style="color: hsl(0, 100%, 40%);">-                                                 { int2str(cpars.rtp_payload_type) },</span><br><span style="color: hsl(0, 100%, 40%);">-                                                    { valueof(ts_SDP_rtpmap(cpars.rtp_payload_type,</span><br><span style="color: hsl(0, 100%, 40%);">-                                                                         cpars.rtp_sdp_format)),</span><br><span style="color: hsl(0, 100%, 40%);">-                                                   valueof(ts_SDP_ptime(20)) }));</span><br><span style="color: hsl(0, 100%, 40%);">-                MGCP.send(ts_CRCX_ACK(mgcp_cmd.line.trans_id, cpars.mgcp_connection_id_mss, sdp));</span><br><span style="color: hsl(0, 100%, 40%);">-      }</span><br><span style="color: hsl(0, 100%, 40%);">-</span><br><span style="color: hsl(0, 100%, 40%);">-       /* Alerting */</span><br><span style="color: hsl(0, 100%, 40%);">-  MNCC.send(ts_MNCC_ALERT_req(cpars.mncc_callref));</span><br><span> </span><br><span>        var BSSMAP_IE_AoIP_TransportLayerAddress tla_ass :=</span><br><span>          valueof(ts_BSSMAP_IE_AoIP_TLA4(f_inet_addr(cpars.mgw_rtp_ip_bss),cpars.mgw_rtp_port_bss));</span><br><span>   interleave {</span><br><span style="color: hsl(0, 100%, 40%);">-    [] BSSAP.receive(tr_PDU_DTAP_MT(tr_ML3_MT_CC_ALERTING(cpars.transaction_id))) {}</span><br><span style="color: hsl(120, 100%, 40%);">+      /* Second MGCP CRCX (this time for MSS/CN side) */</span><br><span style="color: hsl(120, 100%, 40%);">+    [] MGCP.receive(tr_CRCX(cpars.mgcp_ep)) -> value mgcp_cmd {</span><br><span style="color: hsl(120, 100%, 40%);">+                        var SDP_Message sdp := valueof(ts_SDP(cpars.mgw_rtp_ip_mss, cpars.mgw_rtp_ip_mss,</span><br><span style="color: hsl(120, 100%, 40%);">+                                                             hex2str(cpars.mgcp_call_id), "42",</span><br><span style="color: hsl(120, 100%, 40%);">+                                                          cpars.mgw_rtp_port_mss,</span><br><span style="color: hsl(120, 100%, 40%);">+                                                               { int2str(cpars.rtp_payload_type) },</span><br><span style="color: hsl(120, 100%, 40%);">+                                                          { valueof(ts_SDP_rtpmap(cpars.rtp_payload_type,</span><br><span style="color: hsl(120, 100%, 40%);">+                                                                                       cpars.rtp_sdp_format)),</span><br><span style="color: hsl(120, 100%, 40%);">+                                                                 valueof(ts_SDP_ptime(20)) }));</span><br><span style="color: hsl(120, 100%, 40%);">+                      MGCP.send(ts_CRCX_ACK(mgcp_cmd.line.trans_id, cpars.mgcp_connection_id_mss, sdp));</span><br><span style="color: hsl(120, 100%, 40%);">+</span><br><span style="color: hsl(120, 100%, 40%);">+                  /* Alerting */</span><br><span style="color: hsl(120, 100%, 40%);">+                        MNCC.send(ts_MNCC_ALERT_req(cpars.mncc_callref));</span><br><span style="color: hsl(120, 100%, 40%);">+             }</span><br><span style="color: hsl(120, 100%, 40%);">+</span><br><span style="color: hsl(120, 100%, 40%);">+   [] BSSAP.receive(tr_PDU_DTAP_MT(tr_ML3_MT_CC_ALERTING(cpars.transaction_id))) {</span><br><span style="color: hsl(120, 100%, 40%);">+               }</span><br><span>    /* expect AoIP IP/Port to match what we returned in CRCX_ACK above */</span><br><span>        [] BSSAP.receive(tr_BSSMAP_AssignmentReq(omit, tla_ass)) {</span><br><span>           var BSSMAP_IE_AoIP_TransportLayerAddress tla;</span><br><span></span><br></pre><p>To view, visit <a href="https://gerrit.osmocom.org/13191">change 13191</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/13191"/><meta itemprop="name" content="View Change"/></div></div>

<div style="display:none"> Gerrit-Project: osmo-ttcn3-hacks </div>
<div style="display:none"> Gerrit-Branch: master </div>
<div style="display:none"> Gerrit-MessageType: newchange </div>
<div style="display:none"> Gerrit-Change-Id: Ia06af1e347601949f4ddb19f963daa400766d9e7 </div>
<div style="display:none"> Gerrit-Change-Number: 13191 </div>
<div style="display:none"> Gerrit-PatchSet: 1 </div>
<div style="display:none"> Gerrit-Owner: Neels Hofmeyr <nhofmeyr@sysmocom.de> </div>