diff options
-rw-r--r-- | include/asterisk/_private.h | 12 | ||||
-rw-r--r-- | main/asterisk.c | 17 | ||||
-rw-r--r-- | main/loader.c | 81 |
3 files changed, 101 insertions, 9 deletions
diff --git a/include/asterisk/_private.h b/include/asterisk/_private.h index c709b1f15..574ca82b7 100644 --- a/include/asterisk/_private.h +++ b/include/asterisk/_private.h @@ -59,6 +59,18 @@ int ast_indications_reload(void);/*!< Provided by indications.c */ */ int ast_module_reload(const char *name); +/*! + * \brief Process reload requests received during startup. + * + * This function requests that the loader execute the pending reload requests + * that were queued during server startup. + * + * \note This function will do nothing if the server has not completely started + * up. Once called, the reload queue is emptied, and further invocations + * will have no affect. + */ +void ast_process_pending_reloads(void); + /*! \brief Load XML documentation. Provided by xmldoc.c * \retval 1 on error. * \retval 0 on success. diff --git a/main/asterisk.c b/main/asterisk.c index 0f53abecf..3867707f3 100644 --- a/main/asterisk.c +++ b/main/asterisk.c @@ -3593,6 +3593,11 @@ int main(int argc, char *argv[]) ast_channels_init(); + if (init_manager()) { + printf("%s", term_quit()); + exit(1); + } + if (ast_cdr_engine_init()) { printf("%s", term_quit()); exit(1); @@ -3652,15 +3657,6 @@ int main(int argc, char *argv[]) /* loads the cli_permissoins.conf file needed to implement cli restrictions. */ ast_cli_perms_init(0); - /* AMI is initialized after loading modules because of a potential - * conflict between issuing a module reload from manager and - * registering manager actions. This will cause reversed locking - * order between the module list and manager actions list. */ - if (init_manager()) { - printf("%s", term_quit()); - exit(1); - } - dnsmgr_start_refresh(); /* We might have the option of showing a console, but for now just @@ -3676,6 +3672,9 @@ int main(int argc, char *argv[]) sig_alert_pipe[0] = sig_alert_pipe[1] = -1; ast_set_flag(&ast_options, AST_OPT_FLAG_FULLY_BOOTED); + + ast_process_pending_reloads(); + pthread_sigmask(SIG_UNBLOCK, &sigs, NULL); #ifdef __AST_DEBUG_MALLOC diff --git a/main/loader.c b/main/loader.c index 5f2fe8678..987cf8233 100644 --- a/main/loader.c +++ b/main/loader.c @@ -112,6 +112,15 @@ static AST_LIST_HEAD_STATIC(updaters, loadupdate); AST_MUTEX_DEFINE_STATIC(reloadlock); +struct reload_queue_item { + AST_LIST_ENTRY(reload_queue_item) entry; + char module[0]; +}; + +static int do_full_reload = 0; + +static AST_LIST_HEAD_STATIC(reload_queue, reload_queue_item); + /* when dynamic modules are being loaded, ast_module_register() will need to know what filename the module was loaded from while it is being registered @@ -546,12 +555,84 @@ char *ast_module_helper(const char *line, const char *word, int pos, int state, return ret; } +void ast_process_pending_reloads(void) +{ + struct reload_queue_item *item; + + if (!ast_fully_booted) { + return; + } + + AST_LIST_LOCK(&reload_queue); + + if (do_full_reload) { + do_full_reload = 0; + AST_LIST_UNLOCK(&reload_queue); + ast_log(LOG_NOTICE, "Executing deferred reload request.\n"); + ast_module_reload(NULL); + return; + } + + while ((item = AST_LIST_REMOVE_HEAD(&reload_queue, entry))) { + ast_log(LOG_NOTICE, "Executing deferred reload request for module '%s'.\n", item->module); + ast_module_reload(item->module); + ast_free(item); + } + + AST_LIST_UNLOCK(&reload_queue); +} + +static void queue_reload_request(const char *module) +{ + struct reload_queue_item *item; + + AST_LIST_LOCK(&reload_queue); + + if (do_full_reload) { + AST_LIST_UNLOCK(&reload_queue); + return; + } + + if (ast_strlen_zero(module)) { + /* A full reload request (when module is NULL) wipes out any previous + reload requests and causes the queue to ignore future ones */ + while ((item = AST_LIST_REMOVE_HEAD(&reload_queue, entry))) { + ast_free(item); + } + do_full_reload = 1; + } else { + /* No reason to add the same module twice */ + AST_LIST_TRAVERSE(&reload_queue, item, entry) { + if (!strcasecmp(item->module, module)) { + AST_LIST_UNLOCK(&reload_queue); + return; + } + } + item = ast_calloc(1, sizeof(*item) + strlen(module) + 1); + if (!item) { + ast_log(LOG_ERROR, "Failed to allocate reload queue item.\n"); + AST_LIST_UNLOCK(&reload_queue); + return; + } + strcpy(item->module, module); + AST_LIST_INSERT_TAIL(&reload_queue, item, entry); + } + AST_LIST_UNLOCK(&reload_queue); +} + int ast_module_reload(const char *name) { struct ast_module *cur; int res = 0; /* return value. 0 = not found, others, see below */ int i; + /* If we aren't fully booted, we just pretend we reloaded but we queue this + up to run once we are booted up. */ + if (!ast_fully_booted) { + queue_reload_request(name); + return 0; + } + if (ast_mutex_trylock(&reloadlock)) { ast_verbose("The previous reload command didn't finish yet\n"); return -1; /* reload already in progress */ |