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;