Change in osmo-bts[master]: l1sap/gsmtap: Don't log UI fill frames [zero information field]

laforge gerrit-no-reply at
Tue Aug 24 15:50:49 UTC 2021

laforge has uploaded this change for review. ( )

Change subject: l1sap/gsmtap: Don't log UI fill frames [zero information field]

l1sap/gsmtap: Don't log UI fill frames [zero information field]

When generating GSMTAP, we are already suppressing generation for
fill frames on PCH and AGCH.  Let's expand that also to dedicated
channels SDCCH, TCH/F and TCH/H.  The fill frames just clog up the
logs without providing much benefit.

SACCH is still being logged, as it contains SI or MEAS REP, so every
active channel still sees GSMTAP traffic every 480ms (SACCH interval).

Change-Id: I0dd37d8848eb0379f84d15f5f006142b16823c6a
M src/common/l1sap.c
1 file changed, 4 insertions(+), 0 deletions(-)

  git pull ssh:// refs/changes/37/25237/1

diff --git a/src/common/l1sap.c b/src/common/l1sap.c
index c028a2c..3b225c2 100644
--- a/src/common/l1sap.c
+++ b/src/common/l1sap.c
@@ -474,6 +474,9 @@
 	switch (chan_type) {
 		if (!memcmp(data, fill_frame, GSM_MACBLOCK_LEN))
 			return true;
@@ -481,6 +484,7 @@
 		if (!memcmp(data, paging_fill, GSM_MACBLOCK_LEN))
 			return true;
+	/* FIXME: implement the same for GSMTAP_CHANNEL_PDTCH from/to PCU */
 	/* don't use 'default' case here as the above only conditionally return true */
 	return false;

To view, visit
To unsubscribe, or for help writing mail filters, visit

Gerrit-Project: osmo-bts
Gerrit-Branch: master
Gerrit-Change-Id: I0dd37d8848eb0379f84d15f5f006142b16823c6a
Gerrit-Change-Number: 25237
Gerrit-PatchSet: 1
Gerrit-Owner: laforge <laforge at>
Gerrit-MessageType: newchange
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <>

More information about the gerrit-log mailing list