[MERGED] openbsc[master]: dyn TS: debug log 'switchover complete' only when there was ...

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
Sat Aug 27 02:23:48 UTC 2016


Harald Welte has submitted this change and it was merged.

Change subject: dyn TS: debug log 'switchover complete' only when there was a switchover
......................................................................


dyn TS: debug log 'switchover complete' only when there was a switchover

Change-Id: I7ddcb41edce1cd7b22fe91e33bdcaedb21856222
---
M openbsc/src/libbsc/abis_rsl.c
1 file changed, 3 insertions(+), 2 deletions(-)

Approvals:
  Harald Welte: Looks good to me, approved
  Jenkins Builder: Verified



diff --git a/openbsc/src/libbsc/abis_rsl.c b/openbsc/src/libbsc/abis_rsl.c
index 3cf1b25..0acff21 100644
--- a/openbsc/src/libbsc/abis_rsl.c
+++ b/openbsc/src/libbsc/abis_rsl.c
@@ -2538,8 +2538,9 @@
 	pchan_was = ts->dyn.pchan_is;
 	ts->dyn.pchan_is = ts->dyn.pchan_want = pchan_act;
 
-	LOGP(DRSL, LOGL_INFO, "%s switchover from %s complete.\n",
-	     gsm_ts_and_pchan_name(ts), gsm_pchan_name(pchan_was));
+	if (pchan_was != ts->dyn.pchan_is)
+		LOGP(DRSL, LOGL_INFO, "%s switchover from %s complete.\n",
+		     gsm_ts_and_pchan_name(ts), gsm_pchan_name(pchan_was));
 }
 
 /* Entry-point where L2 RSL from BTS enters */

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

Gerrit-MessageType: merged
Gerrit-Change-Id: I7ddcb41edce1cd7b22fe91e33bdcaedb21856222
Gerrit-PatchSet: 2
Gerrit-Project: openbsc
Gerrit-Branch: master
Gerrit-Owner: Neels Hofmeyr <nhofmeyr at sysmocom.de>
Gerrit-Reviewer: Harald Welte <laforge at gnumonks.org>
Gerrit-Reviewer: Jenkins Builder



More information about the gerrit-log mailing list