search

RestComm / media-core

RMS - Restcomm Media Server for Real Time Cloud Communications
http://www.restcomm.com/
GNU Affero General Public License v3.0
160 stars 123 forks source link

JitterBuffer: Buffer overflow #225

Open thanhthuy40 opened 7 years ago

thanhthuy40 commented 7 years ago

I received many warning related JitterBuffer: Buffer overflow

I don't know why queue size is 10? Please let me know that and how to fix this issue.

hrosa commented 7 years ago

hi @thanhthuy40

Having a jitter buffer dropping packets is acceptable. That being said, it only becomes a problem if too many overflows begin to create gaps or glitches in your audio stream.

There is already an open issue https://github.com/RestComm/mediaserver/issues/174 to refactor the jitter buffer and make it more resilient to this kind of problems.

I don't know why queue size is 10?

You mean initial size of the underlying list? It really doesn't matter. First, the jitter buffer size wakes up the consumer when the queue contains around 50-60 ms of audio. Considering that each RTP packet usually contains 20ms, then 10 * 20 = 200ms of audio. This is more than enough for the jitter buffer. Second, the list is resizable.