From: <wt...@ke...> - 2007-04-29 14:46:45
|
CVS Root: /cvs/gstreamer Module: gst-plugins-bad Changes by: wtay Date: Sun Apr 29 2007 14:46:39 UTC Log message: * gst/rtpmanager/gstrtpsession.c: (gst_rtp_session_send_rtcp): Remove debug. * gst/rtpmanager/rtpsession.c: (rtp_session_process_sr), (rtp_session_process_sdes), (calculate_rtcp_interval), (rtp_session_next_timeout), (session_report_blocks): * gst/rtpmanager/rtpstats.c: (rtp_stats_calculate_rtcp_interval): Improve debugging Fix interval for BYE/RTCP packets. Modified files: . : ChangeLog gst/rtpmanager : gstrtpsession.c rtpsession.c rtpstats.c Links: http://freedesktop.org/cgi-bin/viewcvs.cgi/gstreamer/gst-plugins-bad/ChangeLog.diff?r1=1.2521&r2=1.2522 http://freedesktop.org/cgi-bin/viewcvs.cgi/gstreamer/gst-plugins-bad/gst/rtpmanager/gstrtpsession.c.diff?r1=1.8&r2=1.9 http://freedesktop.org/cgi-bin/viewcvs.cgi/gstreamer/gst-plugins-bad/gst/rtpmanager/rtpsession.c.diff?r1=1.6&r2=1.7 http://freedesktop.org/cgi-bin/viewcvs.cgi/gstreamer/gst-plugins-bad/gst/rtpmanager/rtpstats.c.diff?r1=1.3&r2=1.4 ====Begin Diffs==== Index: ChangeLog =================================================================== RCS file: /cvs/gstreamer/gst-plugins-bad/ChangeLog,v retrieving revision 1.2521 retrieving revision 1.2522 diff -u -d -r1.2521 -r1.2522 --- ChangeLog 29 Apr 2007 14:01:05 -0000 1.2521 +++ ChangeLog 29 Apr 2007 14:46:27 -0000 1.2522 @@ -1,3 +1,15 @@ +2007-04-29 Wim Taymans <wi...@fl...> + + * gst/rtpmanager/gstrtpsession.c: (gst_rtp_session_send_rtcp): + Remove debug. + * gst/rtpmanager/rtpsession.c: (rtp_session_process_sr), + (rtp_session_process_sdes), (calculate_rtcp_interval), + (rtp_session_next_timeout), (session_report_blocks): + * gst/rtpmanager/rtpstats.c: (rtp_stats_calculate_rtcp_interval): + Improve debugging + Fix interval for BYE/RTCP packets. 2007-04-29 Thomas Vander Stichele <thomas at apestaart dot org> * docs/plugins/gst-plugins-bad-plugins.args: Index: gstrtpsession.c RCS file: /cvs/gstreamer/gst-plugins-bad/gst/rtpmanager/gstrtpsession.c,v retrieving revision 1.8 retrieving revision 1.9 diff -u -d -r1.8 -r1.9 --- gstrtpsession.c 27 Apr 2007 15:09:12 -0000 1.8 +++ gstrtpsession.c 29 Apr 2007 14:46:27 -0000 1.9 @@ -497,8 +497,6 @@ GST_DEBUG_OBJECT (rtpsession, "sending RTCP"); - gst_util_dump_mem (GST_BUFFER_DATA (buffer), GST_BUFFER_SIZE (buffer)); - if (rtpsession->send_rtcp_src) { result = gst_pad_push (rtpsession->send_rtcp_src, buffer); } else { Index: rtpsession.c RCS file: /cvs/gstreamer/gst-plugins-bad/gst/rtpmanager/rtpsession.c,v retrieving revision 1.6 retrieving revision 1.7 diff -u -d -r1.6 -r1.7 --- rtpsession.c 27 Apr 2007 15:09:12 -0000 1.6 +++ rtpsession.c 29 Apr 2007 14:46:27 -0000 1.7 @@ -1051,7 +1051,8 @@ gst_rtcp_packet_sr_get_sender_info (packet, &senderssrc, &ntptime, &rtptime, &packet_count, &octet_count); - GST_DEBUG ("got SR packet: SSRC %08x", senderssrc); + GST_DEBUG ("got SR packet: SSRC %08x, time %" GST_TIME_FORMAT, + senderssrc, GST_TIME_ARGS (arrival->time)); source = obtain_source (sess, senderssrc, &created, arrival, FALSE); @@ -1158,7 +1159,8 @@ gst_rtcp_packet_sdes_get_entry (packet, &type, &len, &data); - GST_DEBUG ("entry %d, type %d, len %d, data %s", j, type, len, data); + GST_DEBUG ("entry %d, type %d, len %d, data %.*s", j, type, len, len, + data); more_entries = gst_rtcp_packet_sdes_next_entry (packet); j++; @@ -1395,14 +1397,14 @@ GstClockTime result; if (sess->source->received_bye) { + result = rtp_stats_calculate_bye_interval (&sess->stats); + } else { result = rtp_stats_calculate_rtcp_interval (&sess->stats, RTP_SOURCE_IS_SENDER (sess->source), first); - } else { - result = rtp_stats_calculate_bye_interval (&sess->stats); } - GST_DEBUG ("next deterministic interval: %" GST_TIME_FORMAT, - GST_TIME_ARGS (result)); + GST_DEBUG ("next deterministic interval: %" GST_TIME_FORMAT ", first %d", + GST_TIME_ARGS (result), first); if (!deterministic) result = rtp_stats_add_rtcp_jitter (&sess->stats, result); @@ -1495,7 +1497,7 @@ result = GST_CLOCK_TIME_NONE; else if (sess->stats.active_sources >= 50) /* reconsider BYE if members >= 50 */ - result = time + calculate_rtcp_interval (sess, FALSE, TRUE);; + result = time + calculate_rtcp_interval (sess, FALSE, TRUE); if (sess->first_rtcp) /* we are called for the first time */ @@ -1597,10 +1599,14 @@ extended_max, stats->jitter >> 4); if (rtp_source_get_last_sr (source, &ntptime, NULL, NULL, NULL, &time)) { + GstClockTime diff; /* LSR is middle bits of the last ntptime */ LSR = (ntptime >> 16) & 0xffffffff; + diff = data->time - time; + GST_DEBUG ("last SR time diff %" GST_TIME_FORMAT, GST_TIME_ARGS (diff)); /* DLSR, delay since last SR is expressed in 1/65536 second units */ - DLSR = gst_util_uint64_scale_int (data->time - time, 65536, GST_SECOND); + DLSR = gst_util_uint64_scale_int (diff, 65536, GST_SECOND); } else { /* No valid SR received, LSR/DLSR are set to 0 then */ LSR = 0; Index: rtpstats.c RCS file: /cvs/gstreamer/gst-plugins-bad/gst/rtpmanager/rtpstats.c,v retrieving revision 1.3 retrieving revision 1.4 diff -u -d -r1.3 -r1.4 --- rtpstats.c 27 Apr 2007 15:09:12 -0000 1.3 +++ rtpstats.c 29 Apr 2007 14:46:27 -0000 1.4 @@ -56,7 +56,6 @@ gdouble interval; gdouble rtcp_min_time; /* Very first call at application start-up uses half the min * delay for quicker notification while still allowing some time * before reporting for randomization and to learn about other |