> On April 8, 2015, 9:40 p.m., Corey Farrell wrote: > > branches/13/main/loader.c, lines 1382-1386 > > <https://reviewboard.asterisk.org/r/4604/diff/1/?file=73808#file73808line1382> > > > > Please remove this, it's simpler to just do it in > > ast_process_pending_reloads. There are a couple initializations that > > happen after final modules load, I don't think we want to start allowing > > reloads earlier than right before setting ast_fullybooted.
A specific example of where this could be a problem is the reload class "dnsmgr". With modules_loaded set here we could allow dnsmgr to be reloaded before it's initialized. - Corey ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviewboard.asterisk.org/r/4604/#review15138 ----------------------------------------------------------- On April 8, 2015, 4:14 p.m., George Joseph wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviewboard.asterisk.org/r/4604/ > ----------------------------------------------------------- > > (Updated April 8, 2015, 4:14 p.m.) > > > Review request for Asterisk Developers and Corey Farrell. > > > 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 > ----- > > 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 > >
-- _____________________________________________________________________ -- Bandwidth and Colocation Provided by http://www.api-digital.com -- asterisk-dev mailing list To UNSUBSCRIBE or update options visit: http://lists.digium.com/mailman/listinfo/asterisk-dev
