Details

    • Type: New Feature New Feature
    • Status: Closed
    • Severity: Major Major
    • Resolution: Fixed
    • Affects Version/s: None
    • Target Release Version/s: None
    • Component/s: Core/General
    • Labels:
      None
    • Mantis ID:
      2532
    • Regression:
      No

      Description

      Steve Kann and I have started discussing and documenting the design of a new jitter buffer.

      I will start implementation as soon as there is reasonable agreement on the design.

      For the moment, we are using the voip-info Wiki. Please see http://www.voip-info.org/tiki-index.php?page=Asterisk+new+jitterbuffer

      Please put your comments and contributions on that page, without removing existing text, please.

      Regards,
      Steve

        Activity

        Hide
        stevekstevek added a comment -

        "Asterisk Stable v1.0.7 and later versions will be able to handle this without errors.
        Trunking IAX2 calls from Asterisk to versions of Asterisk prior to v1.0.7,
        you need to disable the jitterbuffer in the configuration file in order to avoid
        strange problems with the trunk."

        This is not correct. If you send trunktimestamps to asterisk 1.0.6 or earlier, it will misinterpret the trunk frames, and bad things[tm] will happen. If you send trunktimestamps to 1.0.7 or later, it will print warning messages, and discard these frames. The calls still won't work. [My earlier methods of implementing this would have still worked in this case, but ignored the trunktimestamps, for all previous versions, but they were much more complicated].

        So, the rule still applies that if you send the trunktimestamps to earlier versions of asterisk, it will never work, but with 1.0.7+, the consequences are less severe, and you'll get some warning about the error.

        Show
        stevekstevek added a comment - "Asterisk Stable v1.0.7 and later versions will be able to handle this without errors. Trunking IAX2 calls from Asterisk to versions of Asterisk prior to v1.0.7, you need to disable the jitterbuffer in the configuration file in order to avoid strange problems with the trunk." This is not correct. If you send trunktimestamps to asterisk 1.0.6 or earlier, it will misinterpret the trunk frames, and bad things [tm] will happen. If you send trunktimestamps to 1.0.7 or later, it will print warning messages, and discard these frames. The calls still won't work. [My earlier methods of implementing this would have still worked in this case, but ignored the trunktimestamps, for all previous versions, but they were much more complicated] . So, the rule still applies that if you send the trunktimestamps to earlier versions of asterisk, it will never work, but with 1.0.7+, the consequences are less severe, and you'll get some warning about the error.
        Hide
        Olle Johansson added a comment -

        A bit updated README after discussion with Stevek, but not necessarily done and approved...

        Show
        Olle Johansson added a comment - A bit updated README after discussion with Stevek, but not necessarily done and approved...
        Hide
        Mark Spencer added a comment -

        Added to CVS, thanks!

        Show
        Mark Spencer added a comment - Added to CVS, thanks!
        Hide
        Digium Subversion added a comment -

        Repository: asterisk
        Revision: 5192

        U trunk/Makefile
        U trunk/channels/chan_h323.c
        U trunk/channels/chan_iax2.c
        U trunk/channels/chan_mgcp.c
        U trunk/channels/chan_sip.c
        U trunk/channels/chan_skinny.c
        U trunk/channels/iax2.h
        U trunk/codecs/codec_adpcm.c
        U trunk/codecs/codec_alaw.c
        U trunk/codecs/codec_g726.c
        U trunk/codecs/codec_gsm.c
        U trunk/codecs/codec_ilbc.c
        U trunk/codecs/codec_lpc10.c
        U trunk/codecs/codec_speex.c
        U trunk/codecs/codec_ulaw.c
        U trunk/configs/codecs.conf.sample
        U trunk/configs/iax.conf.sample
        U trunk/include/asterisk/channel.h
        U trunk/include/asterisk/translate.h
        A trunk/jitterbuf.c
        A trunk/jitterbuf.h
        A trunk/plc.c

        ------------------------------------------------------------------------
        r5192 | markster | 2008-01-15 15:27:41 -0600 (Tue, 15 Jan 2008) | 2 lines

        Add PLC and jitter buffer and iax2 meta trunk with timestamps (bug ASTERISK-2491, ASTERISK-3327)

        ------------------------------------------------------------------------

        http://svn.digium.com/view/asterisk?view=rev&revision=5192

        Show
        Digium Subversion added a comment - Repository: asterisk Revision: 5192 U trunk/Makefile U trunk/channels/chan_h323.c U trunk/channels/chan_iax2.c U trunk/channels/chan_mgcp.c U trunk/channels/chan_sip.c U trunk/channels/chan_skinny.c U trunk/channels/iax2.h U trunk/codecs/codec_adpcm.c U trunk/codecs/codec_alaw.c U trunk/codecs/codec_g726.c U trunk/codecs/codec_gsm.c U trunk/codecs/codec_ilbc.c U trunk/codecs/codec_lpc10.c U trunk/codecs/codec_speex.c U trunk/codecs/codec_ulaw.c U trunk/configs/codecs.conf.sample U trunk/configs/iax.conf.sample U trunk/include/asterisk/channel.h U trunk/include/asterisk/translate.h A trunk/jitterbuf.c A trunk/jitterbuf.h A trunk/plc.c ------------------------------------------------------------------------ r5192 | markster | 2008-01-15 15:27:41 -0600 (Tue, 15 Jan 2008) | 2 lines Add PLC and jitter buffer and iax2 meta trunk with timestamps (bug ASTERISK-2491 , ASTERISK-3327 ) ------------------------------------------------------------------------ http://svn.digium.com/view/asterisk?view=rev&revision=5192
        Hide
        Digium Subversion added a comment -

        Repository: asterisk
        Revision: 5211

        U trunk/include/asterisk/plc.h
        U trunk/plc.c

        ------------------------------------------------------------------------
        r5211 | markster | 2008-01-15 15:27:59 -0600 (Tue, 15 Jan 2008) | 2 lines

        Fix PLC for BSD (bug ASTERISK-2491)

        ------------------------------------------------------------------------

        http://svn.digium.com/view/asterisk?view=rev&revision=5211

        Show
        Digium Subversion added a comment - Repository: asterisk Revision: 5211 U trunk/include/asterisk/plc.h U trunk/plc.c ------------------------------------------------------------------------ r5211 | markster | 2008-01-15 15:27:59 -0600 (Tue, 15 Jan 2008) | 2 lines Fix PLC for BSD (bug ASTERISK-2491 ) ------------------------------------------------------------------------ http://svn.digium.com/view/asterisk?view=rev&revision=5211

          People

          • Watchers:
            0 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved:

              Development