<p>lynxis lazus has uploaded this change for <strong>review</strong>.</p><p><a href="https://gerrit.osmocom.org/c/osmo-sgsn/+/15172">View Change</a></p><pre style="font-family: monospace,monospace; white-space: pre-wrap;">gprs_gmm: only update gb/iu cell information when Iu/Gb present<br><br>This function should not do the transistion of Iu/Gb. Only copy simple data<br><br>Change-Id: Id52761f3ea655a2d0cc663a950154c6b7c1b5f7a<br>---<br>M src/gprs/gprs_gmm.c<br>1 file changed, 13 insertions(+), 8 deletions(-)<br><br></pre><pre style="font-family: monospace,monospace; white-space: pre-wrap;">git pull ssh://gerrit.osmocom.org:29418/osmo-sgsn refs/changes/72/15172/1</pre><pre style="font-family: monospace,monospace; white-space: pre-wrap;"><span>diff --git a/src/gprs/gprs_gmm.c b/src/gprs/gprs_gmm.c</span><br><span>index c8bc0f7..8d8c877 100644</span><br><span>--- a/src/gprs/gprs_gmm.c</span><br><span>+++ b/src/gprs/gprs_gmm.c</span><br><span>@@ -298,15 +298,20 @@</span><br><span> /* Store BVCI/NSEI in MM context */</span><br><span> static void msgid2mmctx(struct sgsn_mm_ctx *mm, const struct msgb *msg)</span><br><span> {</span><br><span style="color: hsl(0, 100%, 40%);">-  mm->gb.bvci = msgb_bvci(msg);</span><br><span style="color: hsl(0, 100%, 40%);">-        mm->gb.nsei = msgb_nsei(msg);</span><br><span style="color: hsl(0, 100%, 40%);">-        /* In case a Iu connection is reconnected we need to update the ue ctx */</span><br><span style="color: hsl(0, 100%, 40%);">-       mm->iu.ue_ctx = MSG_IU_UE_CTX(msg);</span><br><span style="color: hsl(0, 100%, 40%);">-  if (mm->ran_type == MM_CTX_T_UTRAN_Iu</span><br><span style="color: hsl(0, 100%, 40%);">-            && mm->iu.ue_ctx) {</span><br><span style="color: hsl(120, 100%, 40%);">+    /* check for Iu or Gb */</span><br><span style="color: hsl(120, 100%, 40%);">+      if (!MSG_IU_UE_CTX(msg)) {</span><br><span style="color: hsl(120, 100%, 40%);">+            mm->gb.bvci = msgb_bvci(msg);</span><br><span style="color: hsl(120, 100%, 40%);">+              mm->gb.nsei = msgb_nsei(msg);</span><br><span style="color: hsl(120, 100%, 40%);">+      } else {</span><br><span> #ifdef BUILD_IU</span><br><span style="color: hsl(0, 100%, 40%);">-             mm->iu.ue_ctx->rab_assign_addr_enc =</span><br><span style="color: hsl(0, 100%, 40%);">-                      sgsn->cfg.iu.rab_assign_addr_enc;</span><br><span style="color: hsl(120, 100%, 40%);">+          /* In case a Iu connection is reconnected we need to update the ue ctx */</span><br><span style="color: hsl(120, 100%, 40%);">+             /* FIXME: the old ue_ctx have to be freed/disconnected */</span><br><span style="color: hsl(120, 100%, 40%);">+             mm->iu.ue_ctx = msg->dst;</span><br><span style="color: hsl(120, 100%, 40%);">+               if (mm->ran_type == MM_CTX_T_UTRAN_Iu</span><br><span style="color: hsl(120, 100%, 40%);">+                              && mm->iu.ue_ctx) {</span><br><span style="color: hsl(120, 100%, 40%);">+                        mm->iu.ue_ctx->rab_assign_addr_enc =</span><br><span style="color: hsl(120, 100%, 40%);">+                                    sgsn->cfg.iu.rab_assign_addr_enc;</span><br><span style="color: hsl(120, 100%, 40%);">+          }</span><br><span> #endif</span><br><span>  }</span><br><span> }</span><br><span></span><br></pre><p>To view, visit <a href="https://gerrit.osmocom.org/c/osmo-sgsn/+/15172">change 15172</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/+/15172"/><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: Id52761f3ea655a2d0cc663a950154c6b7c1b5f7a </div>
<div style="display:none"> Gerrit-Change-Number: 15172 </div>
<div style="display:none"> Gerrit-PatchSet: 1 </div>
<div style="display:none"> Gerrit-Owner: lynxis lazus <lynxis@fe80.eu> </div>
<div style="display:none"> Gerrit-MessageType: newchange </div>