Fix timing source dependency issues with MOH
authorMatthew Jordan <mjordan@digium.com>
Tue, 27 Dec 2011 20:55:15 +0000 (20:55 +0000)
committerMatthew Jordan <mjordan@digium.com>
Tue, 27 Dec 2011 20:55:15 +0000 (20:55 +0000)
commitd9651f2be9e5f19d5a5408eb818df7202af3bf1d
tree4a98128d7a9e37fc509a58eb531a0d3217a1e8b4
parent8017be6fa9e1c9e3fa0a98523c6e82d5eb239cdd
Fix timing source dependency issues with MOH

Prior to this patch, res_musiconhold existed at the same module priority level
as the timing sources that it depends on.  This would cause a problem when
music on hold was reloaded, as the timing source could be changed after
res_musiconhold was processed.  This patch adds a new module priority level,
AST_MODPRI_TIMING, that the various timing modules are now loaded at.  This
now occurs before loading other resource modules, such that the timing source
is guaranteed to be set prior to resolving the timing source dependencies.

(closes issue ASTERISK-17474)
Reporter: Luke H
Tested by: Luke H, Vladimir Mikhelson, zzsurf, Wes Van Tlghem, elguero, Thomas Arimont
Patches:
 asterisk-17474-dahdi_timing-infinite-wait-fix_v3_branch-1.8.diff uploaded by elguero (License #5026)
 asterisk-17474-dahdi_timing-infinite-wait-fix_v3_branch-10.diff uploaded by elguero (License #5026)
 asterisk-17474-dahdi_timing-infinite-wait-fix_v3.diff uploaded by elguero (License #5026)

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

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

Merged revisions 349195 from http://svn.asterisk.org/svn/asterisk/branches/10

git-svn-id: https://origsvn.digium.com/svn/asterisk/trunk@349196 65c4cc65-6c06-0410-ace0-fbb531ad65f3
include/asterisk/module.h
res/res_musiconhold.c
res/res_timing_dahdi.c
res/res_timing_pthread.c
res/res_timing_timerfd.c