]> git.ipfire.org Git - thirdparty/asterisk.git/commit
cel_pgsql: deadlock on unload and core_event_dispatcher
authorKevin Harwell <kharwell@digium.com>
Tue, 31 Dec 2013 21:38:48 +0000 (21:38 +0000)
committerKevin Harwell <kharwell@digium.com>
Tue, 31 Dec 2013 21:38:48 +0000 (21:38 +0000)
commitcbe6ec4a636d4c5ccffe8cab84b95b99de2f0cb7
tree6eb9d10066695adf7c09c828fba1735cb1890a42
parent1b885946c1397708cbfb59348ab87978eac7fad0
cel_pgsql: deadlock on unload and core_event_dispatcher

A deadlock can happen between a thread unloading or reloading the cel_pgsql
module and the core_event_dispatcher taskprocessor thread. Description of
what is happening:

Thread 1 (for example, a netconsole thread):

    a "module reload cel_pgsql" is launched
    the thread enter the "my_unload_module" function (cel_pgsql.c)
    the thread acquire the write lock on psql_columns
    the thread enter the "ast_event_unsubscribe" function (event.c)
    the thread try to acquire the write lock on ast_event_subs[sub->type]

Thread 2 (core_event_dispatcher taskprocessor thread):

    the taskprocessor pop a CEL event
    the thread enter the "handle_event" function (event.c)
    the thread acquire the read lock on ast_event_subs[sub->type]
    the thread callback the "pgsql_log" function (cel_pgsql.c), since it's a subscriber of CEL events
    the thread try to acquire a read lock on psql_columns

(closes issue ASTERISK-22854)
Reported by: Etienne Lessard
Patches:
     cel_pgsql_fix_deadlock_event.patch uploaded by hexanol (license 6394)
........

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

Merged revisions 404604 from http://svn.asterisk.org/svn/asterisk/branches/11

git-svn-id: https://origsvn.digium.com/svn/asterisk/branches/12@404605 65c4cc65-6c06-0410-ace0-fbb531ad65f3
cel/cel_pgsql.c