[PATCH] osmo-bts[master]: virt: Don't print NOTICE log message if ARFCN doesn't match

This is merely a historical archive of years 2008-2021, before the migration to mailman3.

A maintained and still updated list archive can be found at https://lists.osmocom.org/hyperkitty/list/gerrit-log@lists.osmocom.org/.

Harald Welte gerrit-no-reply at lists.osmocom.org
Wed Jul 19 14:02:47 UTC 2017


Review at  https://gerrit.osmocom.org/3296

virt: Don't print NOTICE log message if ARFCN doesn't match

In a larger simulated network with multiple BTSs it is normal that one
BTS will see GSMTAP frames for an ARFCN that is not an ARFCN used by the
local BTS.  This is just normal operation.

Change-Id: Ic68cace9648ccb17500c94b6ede8814674aa9c29
---
M src/osmo-bts-virtual/l1_if.c
1 file changed, 1 insertion(+), 4 deletions(-)


  git pull ssh://gerrit.osmocom.org:29418/osmo-bts refs/changes/96/3296/1

diff --git a/src/osmo-bts-virtual/l1_if.c b/src/osmo-bts-virtual/l1_if.c
index 9212121..3ef8c72 100644
--- a/src/osmo-bts-virtual/l1_if.c
+++ b/src/osmo-bts-virtual/l1_if.c
@@ -102,11 +102,8 @@
 
 	/* Generally ignore all msgs that are either not received with the right ARFCN... */
 	pinst = phy_instance_by_arfcn(plink, arfcn & GSMTAP_ARFCN_MASK);
-	if (!pinst) {
-		LOGP(DL1P, LOGL_NOTICE, "Ignoring incoming msg - msg ARFCN=%d not part of BTS\n",
-		     arfcn & GSMTAP_ARFCN_MASK);
+	if (!pinst)
 		goto nomessage;
-	}
 
 	/* switch case with removed ACCH flag */
 	switch ((gsmtap_chantype & ~GSMTAP_CHANNEL_ACCH) & 0xff) {

-- 
To view, visit https://gerrit.osmocom.org/3296
To unsubscribe, visit https://gerrit.osmocom.org/settings

Gerrit-MessageType: newchange
Gerrit-Change-Id: Ic68cace9648ccb17500c94b6ede8814674aa9c29
Gerrit-PatchSet: 1
Gerrit-Project: osmo-bts
Gerrit-Branch: master
Gerrit-Owner: Harald Welte <laforge at gnumonks.org>



More information about the gerrit-log mailing list