On Nov 28, 2011, at 2:31 AM, Karol Golab wrote:
Hi!
I've noticed some strange code in src/switch_event.c - could someone
smarter (& knowing the mentioned code) please take a look?
The problem is in the order of calls to locks BLOCK & RWLOCK - it
differs between functions:
switch_event_bind_removable
switch_mutex_lock(BLOCK);
switch_thread_rwlock_wrlock(RWLOCK);
switch_event_unbind_callback
switch_thread_rwlock_wrlock(RWLOCK);
switch_mutex_lock(BLOCK);
switch_event_unbind
switch_thread_rwlock_wrlock(RWLOCK);
switch_mutex_lock(BLOCK);
As I understand this may lead to an outright deadlock between bind
and unbind.
I stumbled upon this code while debugging what seems like deadlocks
in our test FS installation put under some heavy load. The load is about
25 new event subscribers per second (destroyed few seconds later) and
about 700 events per second.
Regards,
Karol