[asterisk-dev] [Code Review] 2540: Fix shutdown assertions in stasis-core

svnbot reviewboard at asterisk.org
Fri May 17 16:10:44 CDT 2013


-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviewboard.asterisk.org/r/2540/
-----------------------------------------------------------

(Updated May 17, 2013, 4:10 p.m.)


Status
------

This change has been marked as submitted.


Review request for Asterisk Developers, Matt Jordan and opticron.


Changes
-------

Committed in revision 389011


Repository: Asterisk


Description
-------

In r388005, macros were introduced to consistently define message
types. This added an assert if a message type was used either before
it was initialized or after it had been cleaned up. It turns out that
this assertion fires during shutdown.

This actually exposed a hidden shutdown ordering problem. Since
unsubscribing is asynchronous, it's possible that the message types
used by the subscription could be freed before the final message of
the subscription was processed.

This patch adds stasis_subscription_join(), which blocks until the
last message has been processed by the subscription. Since joining was
most commonly done right after an unsubscribe, a
stasis_unsubscribe_and_join() convenience function was also added.

Similar functions were also added to the stasis_caching_topic and
stasis_message_router, since they wrap subscriptions and have similar
problems.

Other code in trunk was refactored to join() where appropriate, or at
least verify that the subscription was complete before being
destroyed.

Review: https://reviewboard.asterisk.org/r/2540


Diffs
-----

  /trunk/main/devicestate.c 388689 
  /trunk/main/app.c 388689 
  /trunk/include/asterisk/stasis_message_router.h 388689 
  /trunk/include/asterisk/stasis.h 388689 
  /trunk/channels/chan_sip.c 388689 
  /trunk/funcs/func_presencestate.c 388689 
  /trunk/channels/chan_iax2.c 388689 
  /trunk/apps/app_voicemail.c 388689 
  /trunk/apps/app_queue.c 388689 
  /trunk/main/endpoints.c 388689 
  /trunk/main/manager.c 388689 
  /trunk/main/manager_channels.c 388689 
  /trunk/main/pbx.c 388689 
  /trunk/main/stasis.c 388689 
  /trunk/main/stasis_cache.c 388689 
  /trunk/main/stasis_channels.c 388689 
  /trunk/main/stasis_endpoints.c 388689 
  /trunk/main/stasis_message_router.c 388689 
  /trunk/res/res_chan_stats.c 388689 
  /trunk/res/res_jabber.c 388689 
  /trunk/res/res_stasis.c 388689 

Diff: https://reviewboard.asterisk.org/r/2540/diff/


Testing
-------

Ran unit tests; core stop now and core stop gracefully.


Thanks,

David Lee

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.digium.com/pipermail/asterisk-dev/attachments/20130517/5961b778/attachment-0001.htm>


More information about the asterisk-dev mailing list