<p>laforge has uploaded this change for <strong>review</strong>.</p><p><a href="https://gerrit.osmocom.org/c/osmo-bts/+/25237">View Change</a></p><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;">git pull ssh://gerrit.osmocom.org:29418/osmo-bts refs/changes/37/25237/1</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: 1 </div>
<div style="display:none"> Gerrit-Owner: laforge <laforge@osmocom.org> </div>
<div style="display:none"> Gerrit-MessageType: newchange </div>