<p>pespin has uploaded this change for <strong>review</strong>.</p><p><a href="https://gerrit.osmocom.org/c/osmo-sgsn/+/23743">View Change</a></p><pre style="font-family: monospace,monospace; white-space: pre-wrap;">mm_state_iu_fsm: T3314 expiry must lead to PMM IDLE, not PMM DETACHED<br><br>This Iu timer is Osmocom specific, but is made to resemble T3314 times<br>from GERAN (also named READY timer). The READY timer mission is to make<br>the MM state transition from READY to STANDBY, which in PMM (UTRAN)<br>matches the transition from CONNECTED to IDLE.<br>Instead, the patch introducing the timer was making it transition to<br>DETACHED directly, but this was clearly not the intention:<br>* Detaching a UE after 44 seconds (default value for T3314) is overkill.<br>* The comment describing it says: "Iu User inactivity timer. On expiry<br>  release Iu connection". The release of Iu connection happens during<br>  the CONNECTED->IDLE transition (that's basically the difference between<br>  both states).<br><br>The transition CONNECTED->IDLE is done by means of calling<br>sgsn_ranap_iu_release_free(), which will eventually answer with a event<br>RANAP_IU_EVENT_IU_RELEASE from lower layers when the Release Complete is<br>received. At that point, osmo-sgsn code frees the connection and<br>transitions to IDLE state. This way we maintain the state according to<br>the connection existance.<br><br>Related: SYS#5389<br>Fixes: 3bad31bcb42c3449e6847ea7495578817029a2e1<br>Change-Id: I7279102ad51b0c39eb6d04c129986984112d15cc<br>---<br>M src/sgsn/gprs_mm_state_iu_fsm.c<br>1 file changed, 0 insertions(+), 1 deletion(-)<br><br></pre><pre style="font-family: monospace,monospace; white-space: pre-wrap;">git pull ssh://gerrit.osmocom.org:29418/osmo-sgsn refs/changes/43/23743/1</pre><pre style="font-family: monospace,monospace; white-space: pre-wrap;"><span>diff --git a/src/sgsn/gprs_mm_state_iu_fsm.c b/src/sgsn/gprs_mm_state_iu_fsm.c</span><br><span>index 69b4ef0..7f2d3b4 100644</span><br><span>--- a/src/sgsn/gprs_mm_state_iu_fsm.c</span><br><span>+++ b/src/sgsn/gprs_mm_state_iu_fsm.c</span><br><span>@@ -64,7 +64,6 @@</span><br><span>          break;</span><br><span>       case E_PMM_USER_INACTIVITY:</span><br><span>          sgsn_ranap_iu_release_free(ctx, &user_inactive_cause);</span><br><span style="color: hsl(0, 100%, 40%);">-              mm_state_iu_fsm_state_chg(fi, ST_PMM_DETACHED);</span><br><span>              break;</span><br><span>       case E_PMM_RA_UPDATE:</span><br><span>                break;</span><br><span></span><br></pre><p>To view, visit <a href="https://gerrit.osmocom.org/c/osmo-sgsn/+/23743">change 23743</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/+/23743"/><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: I7279102ad51b0c39eb6d04c129986984112d15cc </div>
<div style="display:none"> Gerrit-Change-Number: 23743 </div>
<div style="display:none"> Gerrit-PatchSet: 1 </div>
<div style="display:none"> Gerrit-Owner: pespin <pespin@sysmocom.de> </div>
<div style="display:none"> Gerrit-MessageType: newchange </div>