<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/+/15538">View Change</a></p><p>1 comment:</p><ul style="list-style: none; padding: 0;"><li style="margin: 0; padding: 0;"><p><a href="https://gerrit.osmocom.org/#/c/15538/1/include/osmocom/core/logging.h">File include/osmocom/core/logging.h:</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/15538/1/include/osmocom/core/logging.h@157">Patch Set #1, Line 157:</a> <code style="font-family:monospace,monospace">LOG_CTX_PCU_IF_DATA</code></p><p style="white-space: pre-wrap; word-wrap: break-word;">this doesn't really explain what it is about.  The above elements in the enum are self-explanatory, as it's quite clear what a NSVC, BVC or subscriber is.  But what's PCU_IF_DATA? </p><p style="white-space: pre-wrap; word-wrap: break-word;">Also if this is about filtering by L1 SAPI, then make sure that L1 SAPI is mentioned.  SAPIs exist potentially at any SAP, and there are plenty of SAPs in the protocol stacks: one at every layer boundary.</p><p style="white-space: pre-wrap; word-wrap: break-word;">Furthermore, if we introduce something to libosmocore which is about L1 SAPI filtering, and we do have a L1 SAP (and hence L1 SAPI) in multiple programs, why is something called "PCU" here?</p></li></ul></li></ul><p>To view, visit <a href="https://gerrit.osmocom.org/c/libosmocore/+/15538">change 15538</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/+/15538"/><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: I814cb3328d99faca9220adb5a80ffb934f219d7d </div>
<div style="display:none"> Gerrit-Change-Number: 15538 </div>
<div style="display:none"> Gerrit-PatchSet: 1 </div>
<div style="display:none"> Gerrit-Owner: osmith <osmith@sysmocom.de> </div>
<div style="display:none"> Gerrit-Reviewer: Jenkins Builder </div>
<div style="display:none"> Gerrit-Reviewer: fixeria <axilirator@gmail.com> </div>
<div style="display:none"> Gerrit-Reviewer: laforge <laforge@gnumonks.org> </div>
<div style="display:none"> Gerrit-Reviewer: osmith <osmith@sysmocom.de> </div>
<div style="display:none"> Gerrit-Reviewer: pespin <pespin@sysmocom.de> </div>
<div style="display:none"> Gerrit-Comment-Date: Mon, 16 Sep 2019 15:17:16 +0000 </div>
<div style="display:none"> Gerrit-HasComments: Yes </div>
<div style="display:none"> Gerrit-Has-Labels: Yes </div>
<div style="display:none"> Gerrit-MessageType: comment </div>