GNOME Bugzilla – Bug 491323
Extend RTP MPEG-TS payloader to handle large RTP payload packets
Last modified: 2007-11-15 18:19:22 UTC
Currently, the RTP MPEG-TS payloader component 'rtpmp2tpay' packs only one MPT frame of 188 bytes in the RTP packet payload. The idea is to increase the payload size of to 1316 bytes. As an idea, the number of MPT frames in the payload could be a configurable parameter from 1 to 7 (the maximum allowed by the MTU).
Created attachment 98090 [details] [review] patch to increase the RTP payload to 1316 bytes First patch attempt to provide functionality.
- packetsize in the caps for video/mpegts means the size of the transport stream packet, which can be 188 or 194 (with timestamp). A packetsize of >1000 is not allowed. - you should probably just fill up to the MTU (see rtpmpapay for an example)
Created attachment 98107 [details] [review] Updated patch with wtay suggestions. Left the payload size as 188. Removed hardcoded variables. Changed "filled buffer" condition.
Patch by: Jordi Jaen Pallares <jordijp at gmail dot com> * gst/rtp/gstrtpmp2tpay.c: (gst_rtp_mp2t_pay_class_init), (gst_rtp_mp2t_pay_init), (gst_rtp_mp2t_pay_finalize), (gst_rtp_mp2t_pay_flush), (gst_rtp_mp2t_pay_handle_buffer): * gst/rtp/gstrtpmp2tpay.h: Fill the MTU with as many packets as possible. Fixes #491323.