<p><a href="https://gerrit.osmocom.org/11642">View Change</a></p><p>10 comments:</p><ul style="list-style: none; padding: 0;"><li style="margin: 0; padding: 0;"><p><a href="https://gerrit.osmocom.org/#/c/11642/29/src/libmsc/sgs_iface.c">File src/libmsc/sgs_iface.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/11642/29/src/libmsc/sgs_iface.c@597">Patch Set #29, Line 597:</a> <code style="font-family:monospace,monospace">    if (type_ie[0] == SGSAP_ID_NONEPS_T_EXPLICIT_UE_NONEPS)</code></p><p style="white-space: pre-wrap; word-wrap: break-word;">(a switch() would be nicer here)</p></li><li style="margin: 0; padding: 0 0 0 16px;"><p style="margin-bottom: 4px;"><a href="https://gerrit.osmocom.org/#/c/11642/29/src/libmsc/sgs_iface.c@624">Patch Set #29, Line 624:</a> <code style="font-family:monospace,monospace">   if (type_ie[0] == SGSAP_ID_EPS_T_NETWORK_INITIATED)</code></p><p style="white-space: pre-wrap; word-wrap: break-word;">(a switch() would be nicer here)</p></li><li style="margin: 0; padding: 0 0 0 16px;"><p style="margin-bottom: 4px;"><a href="https://gerrit.osmocom.org/#/c/11642/29/src/libmsc/sgs_iface.c@650">Patch Set #29, Line 650:</a> <code style="font-family:monospace,monospace">       /* Subscriber must be known by the HLR */</code></p><p style="white-space: pre-wrap; word-wrap: break-word;">VLR != HLR</p></li><li style="margin: 0; padding: 0 0 0 16px;"><p style="margin-bottom: 4px;"><a href="https://gerrit.osmocom.org/#/c/11642/29/src/libmsc/sgs_iface.c@655">Patch Set #29, Line 655:</a> <code style="font-family:monospace,monospace">       /* Inform the HLR */</code></p><p style="white-space: pre-wrap; word-wrap: break-word;">AFAICT this is not sending anything to the HLR</p></li><li style="margin: 0; padding: 0 0 0 16px;"><p style="margin-bottom: 4px;"><a href="https://gerrit.osmocom.org/#/c/11642/29/src/libmsc/sgs_iface.c@708">Patch Set #29, Line 708:</a> <code style="font-family:monospace,monospace">         * is comparable to a paging response. The SGsAP SERVICE REQUEST must</code></p><p style="white-space: pre-wrap; word-wrap: break-word;">so an SGsAP Service Request is a reply to an earlier message from the 2G CN?</p></li><li style="margin: 0; padding: 0 0 0 16px;"><p style="margin-bottom: 4px;"><a href="https://gerrit.osmocom.org/#/c/11642/29/src/libmsc/sgs_iface.c@772">Patch Set #29, Line 772:</a> <code style="font-family:monospace,monospace"> conn = connection_for_subscr(vsub);</code></p><p style="white-space: pre-wrap; word-wrap: break-word;">is it possible that we obtain a conn that was established on GSM and not via SGs?</p><p style="white-space: pre-wrap; word-wrap: break-word;">A subscriber could be active on 2G, lose that connection, and while that is in timeout, some LTE action starts. Not sure where that should happen, but would it make sense to make sure that when establishing SGs for a subscriber, that any previous 2G/3G connections are discarded?</p></li><li style="margin: 0; padding: 0 0 0 16px;"><p style="margin-bottom: 4px;"><a href="https://gerrit.osmocom.org/#/c/11642/29/src/libmsc/sgs_iface.c@1140">Patch Set #29, Line 1140:</a> <code style="font-family:monospace,monospace">  /*! This function expects a pointer to the related gsm subscriber</code></p><p style="white-space: pre-wrap; word-wrap: break-word;">inline comment rather without doxygen marker</p></li><li style="margin: 0; padding: 0 0 0 16px;"><p style="margin-bottom: 4px;"><a href="https://gerrit.osmocom.org/#/c/11642/29/src/libmsc/sgs_iface.c@1212">Patch Set #29, Line 1212:</a> <code style="font-family:monospace,monospace">   osmo_fsm_register(&sgs_vlr_reset_fsm);</code></p><p style="white-space: pre-wrap; word-wrap: break-word;">register sgs_ue_fsm?</p></li></ul></li><li style="margin: 0; padding: 0;"><p><a href="https://gerrit.osmocom.org/#/c/11642/29/src/libvlr/vlr_sgs.c">File src/libvlr/vlr_sgs.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/11642/29/src/libvlr/vlr_sgs.c@205">Patch Set #29, Line 205:</a> <code style="font-family:monospace,monospace">/*! Perform an SGs TMSI reallocation complete.</code></p><p style="white-space: pre-wrap; word-wrap: break-word;">wrong comment</p></li><li style="margin: 0; padding: 0 0 0 16px;"><p style="margin-bottom: 4px;"><a href="https://gerrit.osmocom.org/#/c/11642/29/src/libvlr/vlr_sgs.c@226">Patch Set #29, Line 226:</a> <code style="font-family:monospace,monospace">/*! Notify that an SGs paging has been rejected by the MME.</code></p><p style="white-space: pre-wrap; word-wrap: break-word;">wrong comment</p></li></ul></li></ul><p>To view, visit <a href="https://gerrit.osmocom.org/11642">change 11642</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/11642"/><meta itemprop="name" content="View Change"/></div></div>

<div style="display:none"> Gerrit-Project: osmo-msc </div>
<div style="display:none"> Gerrit-Branch: master </div>
<div style="display:none"> Gerrit-MessageType: comment </div>
<div style="display:none"> Gerrit-Change-Id: I73359925fc1ca72b33a1466e6ac41307f2f0b11d </div>
<div style="display:none"> Gerrit-Change-Number: 11642 </div>
<div style="display:none"> Gerrit-PatchSet: 29 </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-CC: Stefan Sperling <stsp@stsp.name> </div>
<div style="display:none"> Gerrit-CC: Vadim Yanitskiy <axilirator@gmail.com> </div>
<div style="display:none"> Gerrit-Comment-Date: Wed, 09 Jan 2019 16:13:26 +0000 </div>
<div style="display:none"> Gerrit-HasComments: Yes </div>
<div style="display:none"> Gerrit-HasLabels: No </div>