[asterisk-dev] [Code Review] 4604: loader/main: Don't set ast_fully_booted until deferred reloads are processed

George Joseph reviewboard at asterisk.org
Thu Apr 9 00:51:49 CDT 2015


-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviewboard.asterisk.org/r/4604/
-----------------------------------------------------------

(Updated April 8, 2015, 11:51 p.m.)


Review request for Asterisk Developers and Corey Farrell.


Changes
-------

Changed per Corey.


Repository: Asterisk


Description
-------

Until we have a true module management facility it's sometimes necessary for one module to force a reload on another before its own load is complete.  If Asterisk isn't fully booted yet, these reloads are deferred.  The problem is that asterisk reports fully booted before processing the deferred reloads which means Asterisk really isn't quite ready when it says it is.

This patch moves the report of fully booted after the processing of the deferred reloads is complete.


Diffs (updated)
-----

  branches/13/main/loader.c 434447 
  branches/13/main/asterisk.c 434447 

Diff: https://reviewboard.asterisk.org/r/4604/diff/


Testing
-------

Since the pjsip stack has the most number of related modules, I'm running the channels/pjsip testsuite to make sure there aren't any issues.  So far...none.


Thanks,

George Joseph

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.digium.com/pipermail/asterisk-dev/attachments/20150409/12afd77d/attachment.html>


More information about the asterisk-dev mailing list