<blockquote style="border-left: 1px solid #aaa; margin: 10px 0; padding: 0 10px;"><p style="white-space: pre-wrap; word-wrap: break-word;">Patch Set 9:</p><p style="white-space: pre-wrap; word-wrap: break-word;">This seems to be the fix for OS#3727 (crash when moving from 3g to 2g or vice versa. I rebased it to get rid of the merge conflict, but I don't know what else this patch is missing. To me it looks like it does what it's supposed to. Should it still be WIP?</p></blockquote><p style="white-space: pre-wrap; word-wrap: break-word;">I moved it to WIP because I didn't had yet the time working on it and to prevent notifications on it. Thanks for rebasing it! The TTCN testcase should succeed if it's done. The testcases are outcomment to not run while jenkins. (SGSN_Iu)</p><p><a href="https://gerrit.osmocom.org/c/osmo-sgsn/+/15487">View Change</a></p><p>2 comments:</p><ul style="list-style: none; padding: 0;"><li style="margin: 0; padding: 0;"><p><a href="https://gerrit.osmocom.org/c/osmo-sgsn/+/15487/2/src/sgsn/gprs_gmm.c">File src/sgsn/gprs_gmm.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-sgsn/+/15487/2/src/sgsn/gprs_gmm.c@1105">Patch Set #2, Line 1105:</a> <code style="font-family:monospace,monospace">      if (ctx->ran_type == new_ran)</code></p><p><blockquote style="border-left: 1px solid #aaa; margin: 10px 0; padding: 0 10px;">I'd prefer getting rid of this check here and doing it in the caller instead, I think it's easier to […]</blockquote></p><p style="white-space: pre-wrap; word-wrap: break-word;">i think it's better this way.</p></li></ul></li><li style="margin: 0; padding: 0;"><p><a href="https://gerrit.osmocom.org/c/osmo-sgsn/+/15487/2/src/sgsn/gprs_mm_state_gb_fsm.c">File src/sgsn/gprs_mm_state_gb_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-sgsn/+/15487/2/src/sgsn/gprs_mm_state_gb_fsm.c@49">Patch Set #2, Line 49:</a> <code style="font-family:monospace,monospace">        case E_MM_RAN_CHANGE:</code></p><p><blockquote style="border-left: 1px solid #aaa; margin: 10px 0; padding: 0 10px;">Are you sure this cannot happen in MM_STANDBY? Just a guess: I have the feeling you want to have it  […]</blockquote></p><p style="white-space: pre-wrap; word-wrap: break-word;">right. it should also happen MM_STANDBY</p></li></ul></li></ul><p>To view, visit <a href="https://gerrit.osmocom.org/c/osmo-sgsn/+/15487">change 15487</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/+/15487"/><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: I3fc614da6ba137e871ee0fe86ca22b6a4a354dd2 </div>
<div style="display:none"> Gerrit-Change-Number: 15487 </div>
<div style="display:none"> Gerrit-PatchSet: 9 </div>
<div style="display:none"> Gerrit-Owner: lynxis lazus <lynxis@fe80.eu> </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: lynxis lazus <lynxis@fe80.eu> </div>
<div style="display:none"> Gerrit-CC: laforge <laforge@osmocom.org> </div>
<div style="display:none"> Gerrit-CC: pespin <pespin@sysmocom.de> </div>
<div style="display:none"> Gerrit-Comment-Date: Tue, 21 Jan 2020 18:16:20 +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: pespin <pespin@sysmocom.de> </div>
<div style="display:none"> Gerrit-MessageType: comment </div>