<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 1:</p><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 1:</p><p style="white-space: pre-wrap; word-wrap: break-word;">looks good. But somehow I cannot find any place where the NSE log context is set on packet receive?</p></blockquote><p style="white-space: pre-wrap; word-wrap: break-word;">Interesting, I added the NSE context everywhere I found the NSVC context being set.<br>I'll check what's up with rx</p></blockquote><p style="white-space: pre-wrap; word-wrap: break-word;">Maybe rx is not handled inside the library but only set by applications? Or even the NSVC case is broken?  In any case, we should set the context as soon as possible once we know the NSVC/NSE to which a message belongs. The NSE should IMHO be known immediately, as one 'bind' (FR AF_PACKET socket, UDP socket) always directly belongs to one NSE.   The NSVC should be known almost at the same time: remote peer address (UDP) or netdev name (FR).</p><p><a href="https://gerrit.osmocom.org/c/libosmocore/+/21465">View Change</a></p><ul style="list-style: none; padding: 0;"></ul><p>To view, visit <a href="https://gerrit.osmocom.org/c/libosmocore/+/21465">change 21465</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/+/21465"/><meta itemprop="name" content="View Change"/></div></div>

<div style="display:none"> Gerrit-Project: libosmocore </div>
<div style="display:none"> Gerrit-Branch: master </div>
<div style="display:none"> Gerrit-Change-Id: I63c0e85f82f5d08c5a6f535da94b8648498439d2 </div>
<div style="display:none"> Gerrit-Change-Number: 21465 </div>
<div style="display:none"> Gerrit-PatchSet: 1 </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-Reviewer: lynxis lazus <lynxis@fe80.eu> </div>
<div style="display:none"> Gerrit-Comment-Date: Wed, 02 Dec 2020 20:49:26 +0000 </div>
<div style="display:none"> Gerrit-HasComments: No </div>
<div style="display:none"> Gerrit-Has-Labels: No </div>
<div style="display:none"> Gerrit-MessageType: comment </div>