----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviewboard.asterisk.org/r/3865/#review12886 -----------------------------------------------------------
team/group/ari-greedy-atxfer/res/stasis/app.c <https://reviewboard.asterisk.org/r/3865/#comment23255> Use of OBJ_NOLOCK is required _only_ if the container has a rd/wr lick. Otherwise it is an optimization at best. You are telling the ao2 function that you have alread obtained a lock on the container. The app->forwards container has a recursive mutex so OBJ_NOLOCK really should not be used at all. In the case of this patch have you obtained it? I don't think so. - rmudgett On July 28, 2014, 11:41 a.m., opticron wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviewboard.asterisk.org/r/3865/ > ----------------------------------------------------------- > > (Updated July 28, 2014, 11:41 a.m.) > > > Review request for Asterisk Developers, Matt Jordan and Mark Michelson. > > > Bugs: ASTERISK-23941 > https://issues.asterisk.org/jira/browse/ASTERISK-23941 > > > Repository: Asterisk > > > Description > ------- > > This adds handling for a channel being pushed into Stasis() via masquerade. > It notifies the Stasis() application using the previously established > StasisStart with the "replace_channel" key populated with a channel snapshot > of the channel that is being replaced. It also sets up the new channel topic > forwards required to get information from the new channel and tears down the > old channel topic forwards. > > This patch also introduces the concept of chan_breakdown datastore callbacks > which are called for the channel being masqueraded into during a masquerade. > > > Diffs > ----- > > team/group/ari-greedy-atxfer/res/stasis/app.c 419681 > team/group/ari-greedy-atxfer/res/stasis/app.h 419681 > team/group/ari-greedy-atxfer/res/res_stasis.c 419681 > team/group/ari-greedy-atxfer/main/channel.c 419681 > team/group/ari-greedy-atxfer/include/asterisk/datastore.h 419681 > > Diff: https://reviewboard.asterisk.org/r/3865/diff/ > > > Testing > ------- > > Verified that the correct messages were being received by the Stasis() > application monitoring the channel. > > > Thanks, > > opticron > >
-- _____________________________________________________________________ -- Bandwidth and Colocation Provided by http://www.api-digital.com -- asterisk-dev mailing list To UNSUBSCRIBE or update options visit: http://lists.digium.com/mailman/listinfo/asterisk-dev