[Asterisk-code-review] loader: support for permanent dlopen() (...asterisk[master])

[Sebastian Kemper]

Sebastian Kemper has posted comments on this change. ( https://gerrit.asterisk.org/c/asterisk/+/11217 )

Change subject: loader: support for permanent dlopen()
......................................................................


Patch Set 3: Code-Review+1

> Patch Set 2: Code-Review-1

Hello Corey,

I put some printfs into the code and run-tested it. Works fine.

PBX UUID: 50cfdc5f-ab3d-494d-94a8-474b812f5c14
Creating module_info_list.
[Apr  4 18:23:47] NOTICE[4030]: loader.c:2384 load_modules: 99 modules will be loaded.
Module res_pjproject not yet in module_info_list, adding it now.
Module res_pjproject had its first dlopen, setting dlopened to 1.
Module res_pjproject dlclosed, running destructor manually.
Module res_pjproject was dlopened already, running constructor manually.
Module res_pjproject already in module_info_list.
Info pointer: 0x7749a030
Module app_echo not yet in module_info_list, adding it now.
Module app_echo had its first dlopen, setting dlopened to 1.

Also manually unloading/loading modules works fine:

Asterisk Ready.
*CLI> module unload chan_sip
Module chan_sip dlclosed, running destructor manually.
Unloaded chan_sip
*CLI> module load chan_sip
Module chan_sip was dlopened already, running constructor manually.
Module chan_sip already in module_info_list.
Info pointer: 0x7701f190
SIP channel loading...
[Apr  4 18:24:00] WARNING[4030]: chan_sip.c:33362 reload_config: Failed to bind to 0.0.0.0:5060: Address in use
Loaded chan_sip
*CLI> Asterisk cleanly ending (0).
Executing last minute cleanups
root at hank2:/tmp#

Kind regards,
Seb


-- 
To view, visit https://gerrit.asterisk.org/c/asterisk/+/11217
To unsubscribe, or for help writing mail filters, visit https://gerrit.asterisk.org/settings

Gerrit-Project: asterisk
Gerrit-Branch: master
Gerrit-Change-Id: I86693a0ecf25d5ba81c73773a03df4abc3426875
Gerrit-Change-Number: 11217
Gerrit-PatchSet: 3
Gerrit-Owner: Sebastian Kemper <sebastian_ml at gmx.net>
Gerrit-Reviewer: Corey Farrell <git at cfware.com>
Gerrit-Reviewer: Friendly Automation
Gerrit-Reviewer: Sebastian Kemper <sebastian_ml at gmx.net>
Gerrit-Comment-Date: Thu, 04 Apr 2019 18:30:26 +0000
Gerrit-HasComments: No
Gerrit-Has-Labels: Yes
Gerrit-MessageType: comment
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.digium.com/pipermail/asterisk-code-review/attachments/20190404/27d4893e/attachment-0001.html>