pespin has submitted this change. (
https://gerrit.osmocom.org/c/osmo-ggsn/+/27415 )
Change subject: gtp: Small log improvements in gtp_create_pdp_ind()
......................................................................
gtp: Small log improvements in gtp_create_pdp_ind()
Drop unneeded log line, rewrite line to better fit code path.
Change-Id: Id254e04d539cc055fee8c16fb66cd897b041557e
---
M gtp/gtp.c
1 file changed, 1 insertion(+), 3 deletions(-)
Approvals:
Jenkins Builder: Verified
laforge: Looks good to me, approved
diff --git a/gtp/gtp.c b/gtp/gtp.c
index f847e43..aeb9603 100644
--- a/gtp/gtp.c
+++ b/gtp/gtp.c
@@ -1754,8 +1754,6 @@
in_addr2gsna(&pdp->gsnlc, &gsn->gsnc);
in_addr2gsna(&pdp->gsnlu, &gsn->gsnu);
- DEBUGP(DLGTP, "gtp_create_pdp_ind: Before gtp_pdp_tidget\n");
-
if (!gtp_pdp_getimsi(gsn, &pdp_old, pdp->imsi, pdp->nsapi)) {
/* Found old pdp with same tid. Now the voodoo begins! */
/* 09.60 / 29.060 allows create on existing context to "steal"
*/
@@ -1777,7 +1775,7 @@
* QoS: MS will get originally negotiated QoS.
* End user address (EUA). MS will get old EUA anyway.
* Protocol configuration option (PCO): Only
application can verify */
- DEBUGP(DLGTP, "gtp_create_pdp_ind: Old context
found\n");
+ DEBUGP(DLGTP, "gtp_create_pdp_ind: Reusing old
context\n");
/* Copy remote flow label */
pdp_old->flru = pdp->flru;
--
To view, visit https://gerrit.osmocom.org/c/osmo-ggsn/+/27415
To unsubscribe, or for help writing mail filters, visit
https://gerrit.osmocom.org/settings
Gerrit-Project: osmo-ggsn
Gerrit-Branch: master
Gerrit-Change-Id: Id254e04d539cc055fee8c16fb66cd897b041557e
Gerrit-Change-Number: 27415
Gerrit-PatchSet: 1
Gerrit-Owner: pespin <pes...@sysmocom.de>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: laforge <lafo...@osmocom.org>
Gerrit-Reviewer: pespin <pes...@sysmocom.de>
Gerrit-MessageType: merged
