if (!gst_structure_get_uint (s, "ssrc", &ssrc))
ssrc = -1;
- GST_DEBUG_OBJECT (rtx,
- "request seqnum: %" G_GUINT32_FORMAT ", ssrc: %" G_GUINT32_FORMAT,
+ GST_DEBUG_OBJECT (rtx, "got rtx request for seqnum: %u, ssrc: %X",
seqnum, ssrc);
GST_OBJECT_LOCK (rtx);
if (g_hash_table_lookup_extended (rtx->ssrc2_ssrc1_map,
GUINT_TO_POINTER (ssrc), NULL, &ssrc2)
&& GPOINTER_TO_UINT (ssrc2) != GPOINTER_TO_UINT (ssrc)) {
- GST_DEBUG_OBJECT (rtx, "Retransmited stream %" G_GUINT32_FORMAT
- " already associated to its master", GPOINTER_TO_UINT (ssrc2));
+ GST_TRACE_OBJECT (rtx, "Retransmited stream %X already associated "
+ "to its master, %X", GPOINTER_TO_UINT (ssrc2), ssrc);
} else {
SsrcAssoc *assoc;
if (g_hash_table_lookup_extended (rtx->seqnum_ssrc1_map,
GUINT_TO_POINTER (seqnum), NULL, (gpointer *) & assoc)) {
if (assoc->ssrc == ssrc) {
+ /* same seqnum, same ssrc */
+
/* do nothing because we have already considered this request
* The jitter may be too impatient of the rtx packet has been
* lost too.
* It does not mean we reject the event, we still want to forward
* the request to the gstrtpsession to be translater into a FB NACK
*/
- GST_DEBUG_OBJECT (rtx, "Duplicated request seqnum: %"
- G_GUINT32_FORMAT ", ssrc1: %" G_GUINT32_FORMAT, seqnum, ssrc);
+ GST_LOG_OBJECT (rtx, "Duplicate request: seqnum: %u, ssrc: %X",
+ seqnum, ssrc);
} else {
+ /* same seqnum, different ssrc */
/* If the association attempt is larger than ASSOC_TIMEOUT,
* then we give up on it, and try this one.
GUINT_TO_POINTER (seqnum));
goto retransmit;
} else {
- GST_DEBUG_OBJECT (rtx,
- "reject request for seqnum %" G_GUINT32_FORMAT
- " of master stream %" G_GUINT32_FORMAT, seqnum, ssrc);
+ GST_INFO_OBJECT (rtx, "rejecting request for seqnum %u"
+ " of master stream %X; there is already a pending request "
+ "for the same seqnum on ssrc %X that has not expired",
+ seqnum, ssrc, assoc->ssrc);
/* do not forward the event as we are rejecting this request */
GST_OBJECT_UNLOCK (rtx);
}
}
- GST_DEBUG_OBJECT (rtx,
- "packet number %" G_GUINT32_FORMAT " of master stream %"
- G_GUINT32_FORMAT " needs to be retransmitted", seqnum, ssrc);
+ GST_DEBUG_OBJECT (rtx, "packet number %u of master stream %X"
+ " needs to be retransmitted", seqnum, ssrc);
GST_OBJECT_UNLOCK (rtx);
}
GPOINTER_TO_UINT (g_hash_table_lookup (rtx->rtx_pt_map,
GUINT_TO_POINTER (payload_type)));
+ GST_DEBUG_OBJECT (rtx, "Got rtx packet: rtx seqnum %u, rtx ssrc %X, "
+ "rtx pt %u, orig seqnum %u, orig pt %u", seqnum, ssrc, payload_type,
+ orign_seqnum, origin_payload_type);
+
/* first we check if we already have associated this retransmission stream
* to a master stream */
if (g_hash_table_lookup_extended (rtx->ssrc2_ssrc1_map,
GUINT_TO_POINTER (ssrc), NULL, &ssrc1)) {
- GST_DEBUG_OBJECT (rtx,
- "packet is from retransmission stream %" G_GUINT32_FORMAT
- " already associated to master stream %" G_GUINT32_FORMAT, ssrc,
- GPOINTER_TO_UINT (ssrc1));
+ GST_TRACE_OBJECT (rtx,
+ "packet is from retransmission stream %X already associated to "
+ "master stream %X", ssrc, GPOINTER_TO_UINT (ssrc1));
ssrc2 = ssrc;
} else {
SsrcAssoc *assoc;
* history */
if (g_hash_table_lookup_extended (rtx->seqnum_ssrc1_map,
GUINT_TO_POINTER (orign_seqnum), NULL, (gpointer *) & assoc)) {
- GST_DEBUG_OBJECT (rtx,
- "associate retransmitted stream %" G_GUINT32_FORMAT
- " to master stream %" G_GUINT32_FORMAT " thanks to packet %"
- G_GUINT16_FORMAT "", ssrc, assoc->ssrc, orign_seqnum);
+ GST_LOG_OBJECT (rtx,
+ "associating retransmitted stream %X to master stream %X thanks "
+ "to rtx packet %u (orig seqnum %u)", ssrc, assoc->ssrc, seqnum,
+ orign_seqnum);
ssrc1 = GUINT_TO_POINTER (assoc->ssrc);
ssrc2 = ssrc;
/* just put a guard */
if (GPOINTER_TO_UINT (ssrc1) == ssrc2)
GST_WARNING_OBJECT (rtx, "RTX receiver ssrc2_ssrc1_map bad state, "
- "ssrc %" G_GUINT32_FORMAT " are the same\n", ssrc);
+ "master and rtx SSRCs are the same (%X)\n", ssrc);
/* free the spot so that this seqnum can be used to do another
* association */
} else {
/* we are not able to associate this rtx packet with a master stream */
- GST_DEBUG_OBJECT (rtx,
- "drop rtx packet because its orign_seqnum %" G_GUINT16_FORMAT
- " is not in pending retransmission requests", orign_seqnum);
+ GST_INFO_OBJECT (rtx,
+ "dropping rtx packet %u because its orig seqnum (%u) is not in our"
+ " pending retransmission requests", seqnum, orign_seqnum);
drop = TRUE;
}
}
/* push the packet */
if (is_rtx) {
gst_buffer_unref (buffer);
- GST_LOG_OBJECT (rtx, "push packet seqnum:%" G_GUINT16_FORMAT
- " from a restransmission stream ssrc2:%" G_GUINT32_FORMAT " (src %"
- G_GUINT32_FORMAT ")", orign_seqnum, ssrc2, GPOINTER_TO_UINT (ssrc1));
+ GST_LOG_OBJECT (rtx, "pushing packet seqnum:%u from restransmission "
+ "stream ssrc: %X (master ssrc %X)", orign_seqnum, ssrc2,
+ GPOINTER_TO_UINT (ssrc1));
ret = gst_pad_push (rtx->srcpad, new_buffer);
} else {
- GST_LOG_OBJECT (rtx, "push packet seqnum:%" G_GUINT16_FORMAT
- " from a master stream ssrc: %" G_GUINT32_FORMAT, seqnum, ssrc);
+ GST_TRACE_OBJECT (rtx, "pushing packet seqnum:%u from master stream "
+ "ssrc: %X", seqnum, ssrc);
ret = gst_pad_push (rtx->srcpad, buffer);
}
fmtp = GPOINTER_TO_UINT (g_hash_table_lookup (rtx->rtx_pt_map,
GUINT_TO_POINTER (gst_rtp_buffer_get_payload_type (&rtp))));
- GST_DEBUG_OBJECT (rtx,
- "retransmit seqnum: %" G_GUINT16_FORMAT ", ssrc: %" G_GUINT32_FORMAT,
+ GST_DEBUG_OBJECT (rtx, "creating rtx buffer, orig seqnum: %u, "
+ "rtx seqnum: %u, rtx ssrc: %X", gst_rtp_buffer_get_seq (&rtp),
seqnum, ssrc);
/* gst_rtp_buffer_map does not map the payload so do it now */
if (!gst_structure_get_uint (s, "ssrc", &ssrc))
ssrc = -1;
- GST_DEBUG_OBJECT (rtx,
- "request seqnum: %" G_GUINT32_FORMAT ", ssrc: %" G_GUINT32_FORMAT,
+ GST_DEBUG_OBJECT (rtx, "got rtx request for seqnum: %u, ssrc: %X",
seqnum, ssrc);
GST_OBJECT_LOCK (rtx);
(GCompareDataFunc) buffer_queue_items_cmp, NULL);
if (iter) {
BufferQueueItem *item = g_sequence_get (iter);
- GST_DEBUG_OBJECT (rtx, "found %" G_GUINT16_FORMAT, item->seqnum);
+ GST_LOG_OBJECT (rtx, "found %u", item->seqnum);
rtx_buf = gst_rtp_rtx_buffer_new (rtx, item->buffer);
}
}
if (!gst_structure_get_uint (s, "ssrc", &ssrc))
ssrc = -1;
- GST_DEBUG_OBJECT (rtx, "collision ssrc: %" G_GUINT32_FORMAT, ssrc);
+ GST_DEBUG_OBJECT (rtx, "got ssrc collision, ssrc: %X", ssrc);
GST_OBJECT_LOCK (rtx);
GST_WARNING_OBJECT (rtx, "Payload %d not in rtx-pt-map", payload);
GST_DEBUG_OBJECT (rtx,
- "got caps for payload: %d->%d, ssrc: %u->%" G_GUINT32_FORMAT ": %"
- GST_PTR_FORMAT, payload, GPOINTER_TO_INT (rtx_payload), ssrc,
- data->rtx_ssrc, caps);
+ "got caps for payload: %d->%d, ssrc: %u->%u : %" GST_PTR_FORMAT,
+ payload, GPOINTER_TO_INT (rtx_payload), ssrc, data->rtx_ssrc, caps);
gst_structure_get_int (s, "clock-rate", &data->clock_rate);
rtptime = gst_rtp_buffer_get_timestamp (&rtp);
gst_rtp_buffer_unmap (&rtp);
- GST_TRACE_OBJECT (rtx,
- "Processing buffer seqnum: %" G_GUINT16_FORMAT ", ssrc: %"
- G_GUINT32_FORMAT, seqnum, ssrc);
+ GST_TRACE_OBJECT (rtx, "Processing buffer seqnum: %u, ssrc: %X", seqnum,
+ ssrc);
/* do not store the buffer if it's payload type is unknown */
if (g_hash_table_contains (rtx->rtx_pt_map, GUINT_TO_POINTER (payload_type))) {