<p><a href="https://gerrit.osmocom.org/c/osmo-msc/+/24832">View Change</a></p><p>1 comment:</p><ul style="list-style: none; padding: 0;"><li style="margin: 0; padding: 0;"><p><a href="https://gerrit.osmocom.org/c/osmo-msc/+/24832/2/src/libvlr/vlr_sgs_fsm.c">File src/libvlr/vlr_sgs_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/osmo-msc/+/24832/2/src/libvlr/vlr_sgs_fsm.c@64">Patch Set #2, Line 64:</a> <code style="font-family:monospace,monospace">      vlr_subscr_set_last_used_eutran_plmn_id(vsub, NULL);</code></p><p><blockquote style="border-left: 1px solid #aaa; margin: 10px 0; padding: 0 10px;">looking at the semantics and reading the related issue it seems that immediate removal is not necess […]</blockquote></p><p style="white-space: pre-wrap; word-wrap: break-word;">I did run the existing TTCN3 tests and actually have to change them to validate that they work as expected when applying this patch. Some of those tests validate the case where a GERAN LU is done after having done stuff over SGs, to make sure later GERAN Lu erases correctly previous state.</p><p style="white-space: pre-wrap; word-wrap: break-word;">I think it makes sense to link the use of LastUsedEutranPLMN to the availability of the MS in the SGs interface.<br>If the MS came from EUTRAN (eg CSFB), the MSC passes the info to the BSC in order to accomplish fast return. However, if the MSC did GERAN LU, the MSC should avoid sending it to avoid the MS doing fast return, since it didn't come from EUTRAN in first place, so it's a good rationale to let it stay in GERAN.</p><p style="white-space: pre-wrap; word-wrap: break-word;">The easiest and most elegant way is actually to simply drop the information when the MS stops being available on EUTRAN, as done here.<br>I verified in general the required paths call this to_null() function to clean up the FSm and go to NULL state (MS not handled in EUTRAN), so we are fine here.</p></li></ul></li></ul><p>To view, visit <a href="https://gerrit.osmocom.org/c/osmo-msc/+/24832">change 24832</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-msc/+/24832"/><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-Change-Id: I0dcb1850ab4f6f7d42bfcb19665ddeae2d3b057a </div>
<div style="display:none"> Gerrit-Change-Number: 24832 </div>
<div style="display:none"> Gerrit-PatchSet: 2 </div>
<div style="display:none"> Gerrit-Owner: pespin <pespin@sysmocom.de> </div>
<div style="display:none"> Gerrit-Reviewer: Jenkins Builder </div>
<div style="display:none"> Gerrit-Reviewer: dexter <pmaier@sysmocom.de> </div>
<div style="display:none"> Gerrit-Reviewer: fixeria <vyanitskiy@sysmocom.de> </div>
<div style="display:none"> Gerrit-CC: neels <nhofmeyr@sysmocom.de> </div>
<div style="display:none"> Gerrit-Comment-Date: Tue, 06 Jul 2021 07:51:08 +0000 </div>
<div style="display:none"> Gerrit-HasComments: Yes </div>
<div style="display:none"> Gerrit-Has-Labels: No </div>
<div style="display:none"> Comment-In-Reply-To: neels <nhofmeyr@sysmocom.de> </div>
<div style="display:none"> Gerrit-MessageType: comment </div>