<p>laforge <strong>submitted</strong> this change.</p><p><a href="https://gerrit.osmocom.org/c/osmo-bts/+/25237">View Change</a></p><div style="white-space:pre-wrap">Approvals:
  laforge: Looks good to me, approved
  dexter: Looks good to me, but someone else must approve
  pespin: Looks good to me, but someone else must approve
  Hoernchen: Looks good to me, but someone else must approve
  Jenkins Builder: Verified

</div><pre style="font-family: monospace,monospace; white-space: pre-wrap;">l1sap/gsmtap: Don't log UI fill frames [zero information field]<br><br>When generating GSMTAP, we are already suppressing generation for<br>fill frames on PCH and AGCH.  Let's expand that also to dedicated<br>channels SDCCH, TCH/F and TCH/H.  The fill frames just clog up the<br>logs without providing much benefit.<br><br>SACCH is still being logged, as it contains SI or MEAS REP, so every<br>active channel still sees GSMTAP traffic every 480ms (SACCH interval).<br><br>Change-Id: I0dd37d8848eb0379f84d15f5f006142b16823c6a<br>---<br>M src/common/l1sap.c<br>1 file changed, 4 insertions(+), 0 deletions(-)<br><br></pre><pre style="font-family: monospace,monospace; white-space: pre-wrap;"><span>diff --git a/src/common/l1sap.c b/src/common/l1sap.c</span><br><span>index c028a2c..3b225c2 100644</span><br><span>--- a/src/common/l1sap.c</span><br><span>+++ b/src/common/l1sap.c</span><br><span>@@ -474,6 +474,9 @@</span><br><span> </span><br><span>        switch (chan_type) {</span><br><span>         case GSMTAP_CHANNEL_AGCH:</span><br><span style="color: hsl(120, 100%, 40%);">+     case GSMTAP_CHANNEL_SDCCH:</span><br><span style="color: hsl(120, 100%, 40%);">+    case GSMTAP_CHANNEL_TCH_F:</span><br><span style="color: hsl(120, 100%, 40%);">+    case GSMTAP_CHANNEL_TCH_H:</span><br><span>           if (!memcmp(data, fill_frame, GSM_MACBLOCK_LEN))</span><br><span>                     return true;</span><br><span>                 break;</span><br><span>@@ -481,6 +484,7 @@</span><br><span>                 if (!memcmp(data, paging_fill, GSM_MACBLOCK_LEN))</span><br><span>                    return true;</span><br><span>                 break;</span><br><span style="color: hsl(120, 100%, 40%);">+        /* FIXME: implement the same for GSMTAP_CHANNEL_PDTCH from/to PCU */</span><br><span>         /* don't use 'default' case here as the above only conditionally return true */</span><br><span>  }</span><br><span>    return false;</span><br><span></span><br></pre><p>To view, visit <a href="https://gerrit.osmocom.org/c/osmo-bts/+/25237">change 25237</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-bts/+/25237"/><meta itemprop="name" content="View Change"/></div></div>

<div style="display:none"> Gerrit-Project: osmo-bts </div>
<div style="display:none"> Gerrit-Branch: master </div>
<div style="display:none"> Gerrit-Change-Id: I0dd37d8848eb0379f84d15f5f006142b16823c6a </div>
<div style="display:none"> Gerrit-Change-Number: 25237 </div>
<div style="display:none"> Gerrit-PatchSet: 2 </div>
<div style="display:none"> Gerrit-Owner: laforge <laforge@osmocom.org> </div>
<div style="display:none"> Gerrit-Reviewer: Hoernchen <ewild@sysmocom.de> </div>
<div style="display:none"> Gerrit-Reviewer: Jenkins Builder </div>
<div style="display:none"> Gerrit-Reviewer: dexter <pmaier@sysmocom.de> </div>
<div style="display:none"> Gerrit-Reviewer: laforge <laforge@osmocom.org> </div>
<div style="display:none"> Gerrit-Reviewer: pespin <pespin@sysmocom.de> </div>
<div style="display:none"> Gerrit-MessageType: merged </div>