pespin has submitted this change. ( https://gerrit.osmocom.org/c/osmo-ggsn/+/27416 ) Change subject: gtp: Specify retrans queue name & seqnum in log lines ...................................................................... gtp: Specify retrans queue name & seqnum in log lines Change-Id: I4f193d7a482ace33afd8526b5f50d2d03467d5fa --- M gtp/gtp.c 1 file changed, 4 insertions(+), 3 deletions(-) Approvals: Jenkins Builder: Verified laforge: Looks good to me, approved 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 https://gerrit.osmocom.org/c/osmo-ggsn/+/27416 To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings Gerrit-Project: osmo-ggsn Gerrit-Branch: master Gerrit-Change-Id: I4f193d7a482ace33afd8526b5f50d2d03467d5fa Gerrit-Change-Number: 27416 Gerrit-PatchSet: 4 Gerrit-Owner: pespin <pespin(a)sysmocom.de> Gerrit-Reviewer: Jenkins Builder Gerrit-Reviewer: laforge <laforge(a)osmocom.org> Gerrit-Reviewer: pespin <pespin(a)sysmocom.de> Gerrit-MessageType: merged