]> git.ipfire.org Git - thirdparty/asterisk.git/commit
internal_timing: Remove the option and always make it enabled if a timing module...
authorRichard Mudgett <rmudgett@digium.com>
Fri, 4 Apr 2014 19:02:57 +0000 (19:02 +0000)
committerRichard Mudgett <rmudgett@digium.com>
Fri, 4 Apr 2014 19:02:57 +0000 (19:02 +0000)
commit84d7ae1894222aeaac3976a4a7e619b32480f474
treef456c3b7e5f7dac1d9da9fb94820f6541b784558
parent3ad7a09002355a6b7fd0ed093eb16ced35c34031
internal_timing: Remove the option and always make it enabled if a timing module is loaded.

The masquerade supertest frequently fails because either the local channel
chain doesn't completely optimize out or the DTMF handshake doesn't
completely get accross.  Local channel optimization requires frames
flowing to trigger when optimization can happen.  When optimization
happens the media frame that triggered the optimization is dropped.
Sending DTMF requires frames to flow in the other direction for timing
purposes while sending nothing.  If internal timing is not enabled when
MOH is playing, Asterisk switches to received timing when an audio frame
is received.  With optimization dropping media frames and MOH not sending
frames unless it receives frames, occasionaly there are no more frames
being passed and the test fails.

* The asterisk command line -I option and the asterisk.conf
internal_timing option are removed.  Asterisk now always uses internal
timing when needed if any timing module is loaded.  The issue
ASTERISK-14861 did this quite awhile ago in v1.4 but effectively is broken
if other internal timing modules besides DAHDI are used.  The
ast_read_generator_actions() now only does received timing if it has no
choice for frame generators like MOH, silence, and playback streaming.

* Cleaned up some code dealing with frame generators in
ast_deactivate_generator(), generator_write_format_change(),
ast_activate_generator(), and ast_channel_stop_silence_generator().

ASTERISK-22846 #close
Reported by: Matt Jordan

Review: https://reviewboard.asterisk.org/r/3414/
........

Merged revisions 411715 from http://svn.asterisk.org/svn/asterisk/branches/1.8
........

Merged revisions 411716 from http://svn.asterisk.org/svn/asterisk/branches/11

git-svn-id: https://origsvn.digium.com/svn/asterisk/branches/12@411717 65c4cc65-6c06-0410-ace0-fbb531ad65f3
UPGRADE.txt
channels/chan_sip.c
configs/asterisk.conf.sample
include/asterisk/options.h
main/asterisk.c
main/channel.c