<p><a href="https://gerrit.osmocom.org/c/osmo-sgsn/+/21456">View Change</a></p><p>14 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/+/21456/3/src/gbproxy/gb_proxy.c">File src/gbproxy/gb_proxy.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/+/21456/3/src/gbproxy/gb_proxy.c@a462">Patch Set #3, Line 462:</a> <code style="font-family:monospace,monospace">                                 "NSEI=%d(BSS) IMSI acquisition was in progress "</code></p><p><blockquote style="border-left: 1px solid #aaa; margin: 10px 0; padding: 0 10px;">NSEI can now go?</blockquote></p><p style="white-space: pre-wrap; word-wrap: break-word;">Done</p></li><li style="margin: 0; padding: 0 0 0 16px;"><p style="margin-bottom: 4px;"><a href="https://gerrit.osmocom.org/c/osmo-sgsn/+/21456/3/src/gbproxy/gb_proxy.c@a468">Patch Set #3, Line 468:</a> <code style="font-family:monospace,monospace">                                   "NSEI=%d(BSS) IMSI not yet acquired, "</code></p><p><blockquote style="border-left: 1px solid #aaa; margin: 10px 0; padding: 0 10px;">NSEI can now go?</blockquote></p><p style="white-space: pre-wrap; word-wrap: break-word;">Done</p></li><li style="margin: 0; padding: 0 0 0 16px;"><p style="margin-bottom: 4px;"><a href="https://gerrit.osmocom.org/c/osmo-sgsn/+/21456/3/src/gbproxy/gb_proxy.c@a487">Patch Set #3, Line 487:</a> <code style="font-family:monospace,monospace">                     "NSEI=%d(BSS) IMSI acquisition succeeded, "</code></p><p><blockquote style="border-left: 1px solid #aaa; margin: 10px 0; padding: 0 10px;">NSEI can now go?</blockquote></p><p style="white-space: pre-wrap; word-wrap: break-word;">Done</p></li><li style="margin: 0; padding: 0 0 0 16px;"><p style="margin-bottom: 4px;"><a href="https://gerrit.osmocom.org/c/osmo-sgsn/+/21456/3/src/gbproxy/gb_proxy.c@a516">Patch Set #3, Line 516:</a> <code style="font-family:monospace,monospace">                        "NSEI=%d(BSS) Dropping stored msgb from list "</code></p><p><blockquote style="border-left: 1px solid #aaa; margin: 10px 0; padding: 0 10px;">NSEI can now go?</blockquote></p><p style="white-space: pre-wrap; word-wrap: break-word;">Done</p></li><li style="margin: 0; padding: 0 0 0 16px;"><p style="margin-bottom: 4px;"><a href="https://gerrit.osmocom.org/c/osmo-sgsn/+/21456/3/src/gbproxy/gb_proxy.c@a526">Patch Set #3, Line 526:</a> <code style="font-family:monospace,monospace">             "NSEI=%d(BSS) IMSI acquisition in progress, "</code></p><p><blockquote style="border-left: 1px solid #aaa; margin: 10px 0; padding: 0 10px;">NSEI can now go?</blockquote></p><p style="white-space: pre-wrap; word-wrap: break-word;">Done</p></li><li style="margin: 0; padding: 0 0 0 16px;"><p style="margin-bottom: 4px;"><a href="https://gerrit.osmocom.org/c/osmo-sgsn/+/21456/3/src/gbproxy/gb_proxy.c@a537">Patch Set #3, Line 537:</a> <code style="font-family:monospace,monospace">              "NSEI=%d(BSS) IMSI is required but not available, "</code></p><p><blockquote style="border-left: 1px solid #aaa; margin: 10px 0; padding: 0 10px;">NSEI can now go?</blockquote></p><p style="white-space: pre-wrap; word-wrap: break-word;">Done</p></li><li style="margin: 0; padding: 0 0 0 16px;"><p style="margin-bottom: 4px;"><a href="https://gerrit.osmocom.org/c/osmo-sgsn/+/21456/3/src/gbproxy/gb_proxy.c@a607">Patch Set #3, Line 607:</a> <code style="font-family:monospace,monospace">                "NSEI=%u(BSS) patching: failed to parse invalid %s message\n",</code></p><p><blockquote style="border-left: 1px solid #aaa; margin: 10px 0; padding: 0 10px;">NSEI can now go?</blockquote></p><p style="white-space: pre-wrap; word-wrap: break-word;">peer can actually be NULL here, I'll revert and just adjust the formatting</p></li><li style="margin: 0; padding: 0 0 0 16px;"><p style="margin-bottom: 4px;"><a href="https://gerrit.osmocom.org/c/osmo-sgsn/+/21456/3/src/gbproxy/gb_proxy.c@a724">Patch Set #3, Line 724:</a> <code style="font-family:monospace,monospace">                   "NSEI=%u(SGSN) patching: failed to parse invalid %s message\n",</code></p><p><blockquote style="border-left: 1px solid #aaa; margin: 10px 0; padding: 0 10px;">NSEI can now go?</blockquote></p><p style="white-space: pre-wrap; word-wrap: break-word;">peer can actually be NULL here, I'll revert and just adjust the formatting</p></li><li style="margin: 0; padding: 0 0 0 16px;"><p style="margin-bottom: 4px;"><a href="https://gerrit.osmocom.org/c/osmo-sgsn/+/21456/3/src/gbproxy/gb_proxy.c@a728">Patch Set #3, Line 728:</a> <code style="font-family:monospace,monospace">                  "NSEI=%u(SGSN) invalid message was: %s\n",</code></p><p><blockquote style="border-left: 1px solid #aaa; margin: 10px 0; padding: 0 10px;">NSEI can now go?</blockquote></p><p style="white-space: pre-wrap; word-wrap: break-word;">Same</p></li><li style="margin: 0; padding: 0 0 0 16px;"><p style="margin-bottom: 4px;"><a href="https://gerrit.osmocom.org/c/osmo-sgsn/+/21456/3/src/gbproxy/gb_proxy.c@a990">Patch Set #3, Line 990:</a> <code style="font-family:monospace,monospace">                 "blocked BVCI=%u via NSEI=%u\n",</code></p><p><blockquote style="border-left: 1px solid #aaa; margin: 10px 0; padding: 0 10px;">Both NSEI and BVCI can go?</blockquote></p><p style="white-space: pre-wrap; word-wrap: break-word;">The nsei here is the one from the SGSN, so should stay</p></li><li style="margin: 0; padding: 0 0 0 16px;"><p style="margin-bottom: 4px;"><a href="https://gerrit.osmocom.org/c/osmo-sgsn/+/21456/3/src/gbproxy/gb_proxy.c@a1063">Patch Set #3, Line 1063:</a> <code style="font-family:monospace,monospace">                        "RAI snooping: RAI %s behind BVCI=%u\n",</code></p><p><blockquote style="border-left: 1px solid #aaa; margin: 10px 0; padding: 0 10px;">BVCI can go?</blockquote></p><p style="white-space: pre-wrap; word-wrap: break-word;">Ack</p></li><li style="margin: 0; padding: 0 0 0 16px;"><p style="margin-bottom: 4px;"><a href="https://gerrit.osmocom.org/c/osmo-sgsn/+/21456/3/src/gbproxy/gb_proxy.c@91">Patch Set #3, Line 91:</a> <code style="font-family:monospace,monospace">expected NSEI=%u</code></p><p><blockquote style="border-left: 1px solid #aaa; margin: 10px 0; padding: 0 10px;">expected that part can go as it's now logged by the macro, right?</blockquote></p><p style="white-space: pre-wrap; word-wrap: break-word;">The BVCI as well, yeah.</p></li><li style="margin: 0; padding: 0 0 0 16px;"><p style="margin-bottom: 4px;"><a href="https://gerrit.osmocom.org/c/osmo-sgsn/+/21456/3/src/gbproxy/gb_proxy.c@455">Patch Set #3, Line 455:</a> <code style="font-family:monospace,monospace">"NSEI=%d(</code></p><p><blockquote style="border-left: 1px solid #aaa; margin: 10px 0; padding: 0 10px;">NSEI can now go?</blockquote></p><p style="white-space: pre-wrap; word-wrap: break-word;">Done</p></li><li style="margin: 0; padding: 0 0 0 16px;"><p style="margin-bottom: 4px;"><a href="https://gerrit.osmocom.org/c/osmo-sgsn/+/21456/3/src/gbproxy/gb_proxy.c@612">Patch Set #3, Line 612:</a> <code style="font-family:monospace,monospace">         LOGPBVC(peer, LOGL_NOTICE,</code></p><p><blockquote style="border-left: 1px solid #aaa; margin: 10px 0; padding: 0 10px;">NSEI can now go?</blockquote></p><p style="white-space: pre-wrap; word-wrap: break-word;">Same</p></li></ul></li></ul><p>To view, visit <a href="https://gerrit.osmocom.org/c/osmo-sgsn/+/21456">change 21456</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/+/21456"/><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: If445f18bc8390c338b5aeb9085951ea392e68b3a </div>
<div style="display:none"> Gerrit-Change-Number: 21456 </div>
<div style="display:none"> Gerrit-PatchSet: 4 </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-CC: laforge <laforge@osmocom.org> </div>
<div style="display:none"> Gerrit-CC: pespin <pespin@sysmocom.de> </div>
<div style="display:none"> Gerrit-Comment-Date: Wed, 02 Dec 2020 16:48:13 +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: laforge <laforge@osmocom.org> </div>
<div style="display:none"> Gerrit-MessageType: comment </div>