<p style="white-space: pre-wrap; word-wrap: break-word;">unless I'm missing something, this actually removes useful features (like sending a NS-STATUS in response to the unexpected NS-ALIVE) and just moves the loggign around?</p><p>Patch set 1:<span style="border-radius: 3px; display: inline-block; margin: 0 2px; padding: 4px;background-color: #ffd4d4;">Code-Review -1</span></p><p><a href="https://gerrit.osmocom.org/c/libosmocore.git/+/17331">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/libosmocore.git/+/17331/1/src/gb/gprs_ns.c">File src/gb/gprs_ns.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/libosmocore.git/+/17331/1/src/gb/gprs_ns.c@321">Patch Set #1, Line 321:</a> <code style="font-family:monospace,monospace">%d, Data weight %</code></p><p style="white-space: pre-wrap; word-wrap: break-word;">formally correct would be %u as it's unsigned integers. Sure, with uint8_t an int should never overflow, so no practical issue.</p></li><li style="margin: 0; padding: 0 0 0 16px;"><p style="margin-bottom: 4px;"><a href="https://gerrit.osmocom.org/c/libosmocore.git/+/17331/1/src/gb/gprs_ns.c@1678">Patch Set #1, Line 1678:</a> <code style="font-family:monospace,monospace">         return GPRS_NS_CS_REJECTED;</code></p><p><blockquote style="border-left: 1px solid #aaa; margin: 10px 0; padding: 0 10px;">I'm just trying to understand the patch in relation to your issue description. […]</blockquote></p><p style="white-space: pre-wrap; word-wrap: break-word;">I agree with pespin here, from INFO level it should have shown up exactly in line 1655 above "Rejecting NS PDU type ...".  The advantage of the old log line is that you could use the log filters (as it sets the nsvc context).  Also, it actually returns a NS-STATUS informing the peer about the cause of the reject.</p></li></ul></li></ul><p>To view, visit <a href="https://gerrit.osmocom.org/c/libosmocore.git/+/17331">change 17331</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/libosmocore.git/+/17331"/><meta itemprop="name" content="View Change"/></div></div>

<div style="display:none"> Gerrit-Project: libosmocore.git </div>
<div style="display:none"> Gerrit-Branch: master </div>
<div style="display:none"> Gerrit-Change-Id: Ic145bcc54e3f0779ea9e284216831b17ab22eb91 </div>
<div style="display:none"> Gerrit-Change-Number: 17331 </div>
<div style="display:none"> Gerrit-PatchSet: 1 </div>
<div style="display:none"> Gerrit-Owner: ipse <Alexander.Chemeris@gmail.com> </div>
<div style="display:none"> Gerrit-Reviewer: ipse <Alexander.Chemeris@gmail.com> </div>
<div style="display:none"> Gerrit-Reviewer: laforge <laforge@osmocom.org> </div>
<div style="display:none"> Gerrit-Reviewer: lynxis lazus <lynxis@fe80.eu> </div>
<div style="display:none"> Gerrit-CC: pespin <pespin@sysmocom.de> </div>
<div style="display:none"> Gerrit-Comment-Date: Tue, 03 Mar 2020 16:23:21 +0000 </div>
<div style="display:none"> Gerrit-HasComments: Yes </div>
<div style="display:none"> Gerrit-Has-Labels: Yes </div>
<div style="display:none"> Comment-In-Reply-To: pespin <pespin@sysmocom.de> </div>
<div style="display:none"> Comment-In-Reply-To: ipse <Alexander.Chemeris@gmail.com> </div>
<div style="display:none"> Gerrit-MessageType: comment </div>