pespin submitted this change.

View Change

Approvals: Jenkins Builder: Verified laforge: Looks good to me, approved
gtp: Specify retrans queue name & seqnum in log lines

Change-Id: I4f193d7a482ace33afd8526b5f50d2d03467d5fa
---
M gtp/gtp.c
1 file changed, 4 insertions(+), 3 deletions(-)

diff --git a/gtp/gtp.c b/gtp/gtp.c
index aeb9603..608e5f9 100644
--- a/gtp/gtp.c
+++ b/gtp/gtp.c
@@ -630,8 +630,8 @@
/* Use new queue structure */
if (queue_newmsg(gsn->queue_req, &qmsg, &addr, gsn->seq_next)) {
gsn->err_queuefull++;
- LOGP(DLGTP, LOGL_ERROR,
- "Retransmit queue is full\n");
+ LOGP(DLGTP, LOGL_ERROR, "Retransmit req queue is full (seq=%" PRIu16 ")\n",
+ gsn->seq_next);
} else {
memcpy(&qmsg->p, packet, sizeof(union gtp_packet));
qmsg->l = len;
@@ -758,7 +758,8 @@
/* Use new queue structure */
if (queue_newmsg(gsn->queue_resp, &qmsg, peer, seq)) {
gsn->err_queuefull++;
- LOGP(DLGTP, LOGL_ERROR, "Retransmit queue is full\n");
+ LOGP(DLGTP, LOGL_ERROR, "Retransmit resp queue is full (seq=%" PRIu16 ")\n",
+ seq);
} else {
memcpy(&qmsg->p, packet, sizeof(union gtp_packet));
qmsg->l = len;

To view, visit change 27416. To unsubscribe, or for help writing mail filters, visit settings.

Gerrit-Project: osmo-ggsn
Gerrit-Branch: master
Gerrit-Change-Id: I4f193d7a482ace33afd8526b5f50d2d03467d5fa
Gerrit-Change-Number: 27416
Gerrit-PatchSet: 4
Gerrit-Owner: pespin <pespin@sysmocom.de>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: laforge <laforge@osmocom.org>
Gerrit-Reviewer: pespin <pespin@sysmocom.de>
Gerrit-MessageType: merged