[PATCH] openbsc[master]: libbsc: add debug log message to S_L_INP_* callbacks

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/.

lynxis lazus gerrit-no-reply at lists.osmocom.org
Mon Mar 13 09:38:13 UTC 2017


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

libbsc: add debug log message to S_L_INP_* callbacks

Improve debug log output of input callbacks by adding a line containing
the signal event name.

Change-Id: Ifca46dd8b356d0de31cccbd79e406079d3a0d7d2
---
M openbsc/src/libbsc/bsc_init.c
M openbsc/src/libbsc/bts_ericsson_rbs2000.c
2 files changed, 7 insertions(+), 0 deletions(-)


  git pull ssh://gerrit.osmocom.org:29418/openbsc refs/changes/41/2041/1

diff --git a/openbsc/src/libbsc/bsc_init.c b/openbsc/src/libbsc/bsc_init.c
index b17ff79..35a456a 100644
--- a/openbsc/src/libbsc/bsc_init.c
+++ b/openbsc/src/libbsc/bsc_init.c
@@ -316,6 +316,7 @@
 
 	switch (signal) {
 	case S_L_INP_TEI_UP:
+		LOGP(DLMI, LOGL_DEBUG, "inp_sig_cb() signal: S_L_INP_TEI_UP\n");
 		if (isd->link_type == E1INP_SIGN_OML) {
 			/* TODO: this is required for the Nokia BTS, hopping is configured
 			   during OML, other MA is not set.  */
@@ -338,6 +339,7 @@
 			bootstrap_rsl(trx);
 		break;
 	case S_L_INP_TEI_DN:
+		LOGP(DLMI, LOGL_DEBUG, "inp_sig_cb() signal: S_L_INP_TEI_DN\n");
 		LOGP(DLMI, LOGL_ERROR, "Lost some E1 TEI link: %d %p\n", isd->link_type, trx);
 
 		if (isd->link_type == E1INP_SIGN_OML)
diff --git a/openbsc/src/libbsc/bts_ericsson_rbs2000.c b/openbsc/src/libbsc/bts_ericsson_rbs2000.c
index 1abf4a3..191ce2b 100644
--- a/openbsc/src/libbsc/bts_ericsson_rbs2000.c
+++ b/openbsc/src/libbsc/bts_ericsson_rbs2000.c
@@ -116,6 +116,7 @@
 
 	switch (signal) {
 	case S_L_INP_TEI_UP:
+		LOGP(DNM, LOGL_DEBUG, "inp_sig_cb() signal: S_L_INP_TEI_UP\n");
 		switch (isd->link_type) {
 		case E1INP_SIGN_OML:
 			if (isd->trx->bts->type != GSM_BTS_TYPE_RBS2000)
@@ -128,6 +129,7 @@
 		}
 		break;
 	case S_L_INP_TEI_DN:
+		LOGP(DNM, LOGL_DEBUG, "inp_sig_cb() signal: S_L_INP_TEI_DN\n");
 		if (isd->trx->bts->type != GSM_BTS_TYPE_RBS2000)
 			break;
 		LOGP(DNM, LOGL_NOTICE, "Line-%u TS-%u TEI-%u SAPI-%u: Link "
@@ -139,13 +141,16 @@
 		lapd_sap_start(e1i_ts->lapd, isd->tei, isd->sapi);
 		break;
 	case S_L_INP_LINE_INIT:
+		LOGP(DNM, LOGL_DEBUG, "inp_sig_cb() signal: S_L_INP_LINE_INIT\n");
 	case S_L_INP_LINE_NOALARM:
+		LOGP(DNM, LOGL_DEBUG, "inp_sig_cb() signal: S_L_INP_LINE_NOALARM\n");
 		if (strcasecmp(isd->line->driver->name, "DAHDI")
 		 && strcasecmp(isd->line->driver->name, "MISDN_LAPD"))
 			break;
 		start_sabm_in_line(isd->line, 1);
 		break;
 	case S_L_INP_LINE_ALARM:
+		LOGP(DNM, LOGL_DEBUG, "inp_sig_cb() signal: S_L_INP_LINE_ALARM\n");
 		if (strcasecmp(isd->line->driver->name, "DAHDI")
 		 && strcasecmp(isd->line->driver->name, "MISDN_LAPD"))
 			break;

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

Gerrit-MessageType: newchange
Gerrit-Change-Id: Ifca46dd8b356d0de31cccbd79e406079d3a0d7d2
Gerrit-PatchSet: 1
Gerrit-Project: openbsc
Gerrit-Branch: master
Gerrit-Owner: lynxis lazus <lynxis at fe80.eu>
Gerrit-Reviewer: dexter <pmaier at sysmocom.de>



More information about the gerrit-log mailing list