[PATCH] osmo-sgsn[master]: Use extended logging for PDP contexts

[Max]

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

Use extended logging for PDP contexts

Log additional details (IMSI, NSAPI etc) on PDP context
creation/deletion by using appropriate LOGP() wrappers.

Change-Id: I91a126952ae75ae9ba1f42aac4d26413384bba5d
Related: SYS#3610
---
M src/gprs/sgsn_libgtp.c
1 file changed, 3 insertions(+), 2 deletions(-)


  git pull ssh://gerrit.osmocom.org:29418/osmo-sgsn refs/changes/73/4073/1

diff --git a/src/gprs/sgsn_libgtp.c b/src/gprs/sgsn_libgtp.c
index 9a00266..9b8fe16 100644
--- a/src/gprs/sgsn_libgtp.c
+++ b/src/gprs/sgsn_libgtp.c
@@ -145,7 +145,6 @@
 	const uint8_t *qos;
 	int rc;
 
-	LOGP(DGPRS, LOGL_ERROR, "Create PDP Context\n");
 	pctx = sgsn_pdp_ctx_alloc(mmctx, nsapi);
 	if (!pctx) {
 		LOGP(DGPRS, LOGL_ERROR, "Couldn't allocate PDP Ctx\n");
@@ -174,6 +173,7 @@
 	pdp->selmode = 0xFC | 0x00;
 
 	/* IMSI, TEID/TEIC, FLLU/FLLC, TID, NSAPI set in pdp_newpdp */
+	LOGPDPCTXP(LOGL_NOTICE, pctx, "Create PDP Context\n");
 
 	/* Put the MSISDN in case we have it */
 	if (mmctx->subscr && mmctx->subscr->sgsn_data->msisdn_len) {
@@ -635,7 +635,8 @@
 /* Called whenever a PDP context is deleted for any reason */
 static int cb_delete_context(struct pdp_t *pdp)
 {
-	LOGP(DGPRS, LOGL_INFO, "PDP Context was deleted\n");
+	LOGPDPX(DGPRS, LOGL_INFO, pdp, "Context was deleted\n")   ;
+
 	return 0;
 }
 

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

Gerrit-MessageType: newchange
Gerrit-Change-Id: I91a126952ae75ae9ba1f42aac4d26413384bba5d
Gerrit-PatchSet: 1
Gerrit-Project: osmo-sgsn
Gerrit-Branch: master
Gerrit-Owner: Max <msuraev at sysmocom.de>