[PATCH] libosmo-abis[master]: lapd: Improve log output

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

dexter gerrit-no-reply at lists.osmocom.org
Wed Mar 8 13:49:22 UTC 2017


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

lapd: Improve log output

Add pointer address output to the sap alloc/free functions in
order to track the objects later in the debug output (we have three
of them)

Change-Id: I9b961d5e36967cfd953551b24696849541257a1c
---
M src/input/lapd.c
1 file changed, 8 insertions(+), 2 deletions(-)


  git pull ssh://gerrit.osmocom.org:29418/libosmo-abis refs/changes/11/2011/1

diff --git a/src/input/lapd.c b/src/input/lapd.c
index 370697d..3e0f6f7 100644
--- a/src/input/lapd.c
+++ b/src/input/lapd.c
@@ -229,8 +229,9 @@
 	if (!sap)
 		return NULL;
 
-	LOGP(DLLAPD, LOGL_NOTICE, "LAPD Allocating SAP for SAPI=%u / TEI=%u\n",
-		sapi, teip->tei);
+	LOGP(DLLAPD, LOGL_NOTICE,
+	     "LAPD Allocating SAP for SAPI=%u / TEI=%u (dl=%p, sap=%p)\n",
+	     sapi, teip->tei, &sap->dl, sap);
 
 	sap->sapi = sapi;
 	sap->tei = teip;
@@ -266,6 +267,10 @@
 /* Free SAP instance, including the datalink */
 static void lapd_sap_free(struct lapd_sap *sap)
 {
+	LOGP(DLLAPD, LOGL_NOTICE,
+	     "LAPD Freeing SAP for SAPI=%u / TEI=%u (dl=%p, sap=%p)\n",
+	     sap->sapi, sap->tei->tei, &sap->dl, sap);
+
 	/* free datalink structures and timers */
 	lapd_dl_exit(&sap->dl);
 
@@ -703,3 +708,4 @@
 
 	talloc_free(li);
 }
+

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

Gerrit-MessageType: newchange
Gerrit-Change-Id: I9b961d5e36967cfd953551b24696849541257a1c
Gerrit-PatchSet: 1
Gerrit-Project: libosmo-abis
Gerrit-Branch: master
Gerrit-Owner: dexter <pmaier at sysmocom.de>



More information about the gerrit-log mailing list