<p><a href="https://gerrit.osmocom.org/c/osmo-sgsn/+/21882">View Change</a></p><p>3 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/+/21882/2/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/+/21882/2/src/gbproxy/gb_proxy.c@224">Patch Set #2, Line 224:</a> <code style="font-family:monospace,monospace">              LOGPS</code></p><p><blockquote style="border-left: 1px solid #aaa; margin: 10px 0; padding: 0 10px;">do we want this as NOTICE? Even the existing "INFO" might be high fro something that has the potenti […]</blockquote></p><p style="white-space: pre-wrap; word-wrap: break-word;">I don't think this happens for regular user-plane traffic (not having a TLLI there should be a serious issue).</p><p style="white-space: pre-wrap; word-wrap: break-word;">In any case, I'll submit those as separate patch</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/+/21882/2/src/gbproxy/gb_proxy.c@233">Patch Set #2, Line 233:</a> <code style="font-family:monospace,monospace">                sgsn = llist_first_entry(&cfg->sgsns, struct gbproxy_sgsn, list);</code></p><p><blockquote style="border-left: 1px solid #aaa; margin: 10px 0; padding: 0 10px;">This all looks like a separate patch imho.</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/+/21882/2/src/gbproxy/gb_proxy.c@300">Patch Set #2, Line 300:</a> <code style="font-family:monospace,monospace">            DEBUGP(DGPRS, "CELL(%05u) using SGSN %s at NSE(%05u)\n", cell->bvci, sgsn->name, sgsn->nse->nsei);</code></p><p><blockquote style="border-left: 1px solid #aaa; margin: 10px 0; padding: 0 10px;">why not using LOGPSGSN here?</blockquote></p><p style="white-space: pre-wrap; word-wrap: break-word;">Because DEBUGP can be disabled during compilation and this might be called a lot.</p></li></ul></li></ul><p>To view, visit <a href="https://gerrit.osmocom.org/c/osmo-sgsn/+/21882">change 21882</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/+/21882"/><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: I2a3410dd9bebb242957e13a63ed70e447204203c </div>
<div style="display:none"> Gerrit-Change-Number: 21882 </div>
<div style="display:none"> Gerrit-PatchSet: 3 </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: laforge <laforge@osmocom.org> </div>
<div style="display:none"> Gerrit-CC: pespin <pespin@sysmocom.de> </div>
<div style="display:none"> Gerrit-Comment-Date: Mon, 04 Jan 2021 17:38:36 +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"> Comment-In-Reply-To: pespin <pespin@sysmocom.de> </div>
<div style="display:none"> Gerrit-MessageType: comment </div>