<p style="white-space: pre-wrap; word-wrap: break-word;">I'm not quite sure I understand. Do you want me to explain more why this is needed or do you doubt that it is?</p><p style="white-space: pre-wrap; word-wrap: break-word;">While refactoring the other patch I looked through the Iu and Gb FSMs and saw that in the Iu FSM the implicit detach event will always go to state DETACHED while in Gb it will not. Also the Gb FSM state STANDBY already has an out_state_mask which includes ST_MM_IDLE, but st_mm_standby() never transitions to it.</p><p style="white-space: pre-wrap; word-wrap: break-word;">Finally, st_mm_idle_on_enter() will clean up the llme, which is what we want to do during a RAT change.<br>So apart from the last all are more meta-reasons for me.</p><p style="white-space: pre-wrap; word-wrap: break-word;">Another issu with this patch:<br>src/sgsn/sgsn_libgtp.c:668 has an OSMO_ASSERT() on Gb FSM state not being IDLE. Will something break here? This could happen if an MS RAT-changes from 2G to 3G and back to 2G, right?</p><p style="white-space: pre-wrap; word-wrap: break-word;">The TTCN3 tests pass without this patch, so I moved it at the end to get the rest merged for the moment.</p><p><a href="https://gerrit.osmocom.org/c/osmo-sgsn/+/17080">View Change</a></p><ul style="list-style: none; padding: 0;"></ul><p>To view, visit <a href="https://gerrit.osmocom.org/c/osmo-sgsn/+/17080">change 17080</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-sgsn/+/17080"/><meta itemprop="name" content="View Change"/></div></div>

<div style="display:none"> Gerrit-Project: osmo-sgsn </div>
<div style="display:none"> Gerrit-Branch: master </div>
<div style="display:none"> Gerrit-Change-Id: I63d04a2dcdc17b4df6616c515641c435d919c787 </div>
<div style="display:none"> Gerrit-Change-Number: 17080 </div>
<div style="display:none"> Gerrit-PatchSet: 1 </div>
<div style="display:none"> Gerrit-Owner: daniel <dwillmann@sysmocom.de> </div>
<div style="display:none"> Gerrit-Reviewer: Jenkins Builder </div>
<div style="display:none"> Gerrit-Reviewer: daniel <dwillmann@sysmocom.de> </div>
<div style="display:none"> Gerrit-Reviewer: fixeria <axilirator@gmail.com> </div>
<div style="display:none"> Gerrit-Reviewer: laforge <laforge@osmocom.org> </div>
<div style="display:none"> Gerrit-Reviewer: lynxis lazus <lynxis@fe80.eu> </div>
<div style="display:none"> Gerrit-Reviewer: pespin <pespin@sysmocom.de> </div>
<div style="display:none"> Gerrit-Comment-Date: Tue, 04 Feb 2020 09:08:11 +0000 </div>
<div style="display:none"> Gerrit-HasComments: No </div>
<div style="display:none"> Gerrit-Has-Labels: No </div>
<div style="display:none"> Gerrit-MessageType: comment </div>