[test-results] [Bamboo] Asterisk - Trunk > Ubuntu Lucid (10.04) > #221 was SUCCESSFUL (with 194 tests). Change made by jpeeler.

[Bamboo]

-----------------------------------------------------------------------
Asterisk - Trunk > Ubuntu Lucid (10.04) > #221 was successful.
-----------------------------------------------------------------------
Code has been updated by jpeeler.
All 2 jobs passed with 194 tests in total.

http://bamboo.asterisk.org/browse/ASTTRUNK-LUCID-221/


--------------
Code Changes
--------------
jpeeler (302270):

>Merged revisions 302266 via svnmerge from 
>https://origsvn.digium.com/svn/asterisk/branches/1.8
>
>................
>  r302266 | jpeeler | 2011-01-18 14:19:57 -0600 (Tue, 18 Jan 2011) | 34 lines
>  
>  Merged revisions 302265 via svnmerge from 
>  https://origsvn.digium.com/svn/asterisk/branches/1.6.2
>  
>  ........
>    r302265 | jpeeler | 2011-01-18 14:13:52 -0600 (Tue, 18 Jan 2011) | 27 lines
>    
>    Convert device state callbacks to ao2 objects to fix a deadlock in chan_sip.
>    
>    Lock scenario presented here:
>    Thread 1
>     holds ast_rdlock_contexts &conlock
>     holds handle_statechange hints
>     holds handle_statechange hint
>      waiting for cb_extensionstate
>       Locked Here: chan_sip.c line 7428 (find_call)
>    Thread 2
>     holds handle_request_do &netlock
>     holds find_call sip_pvt_ptr
>      waiting for ast_rdlock_contexts &conlock
>       Locked Here: pbx.c line 9911 (ast_rdlock_contexts)
>    
>    Chan_sip has an established locking order of locking the sip_pvt and then
>    getting the context lock. So the as stated by the summary, the operations in
>    thread 2 have been modified to no longer require the context lock.
>    
>    (closes issue #18310)
>    Reported by: one47
>    Patches: 
>          statecbs_ao2.mk2.patch uploaded by one47 (license 23),
>          modified by me
>    
>    Review: https://reviewboard.asterisk.org/r/1072/
>  ........
>................
>


--------------
Tests
--------------
Fixed Tests (1)
   - AsteriskTestSuite: S/fax/local channel t38 queryoption

--
This message is automatically generated by Atlassian Bamboo
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.digium.com/pipermail/test-results/attachments/20110118/aa05ad49/attachment-0001.htm>