[test-results] [Bamboo] Asterisk - Trunk > Ubuntu Lucid (10.04) > #211 has FAILED (2 tests failed). Change made by jpeeler.

Bamboo bamboo at asterisk.org
Fri Jan 14 12:30:29 CST 2011


-----------------------------------------------------------------------
Asterisk - Trunk > Ubuntu Lucid (10.04) > #211 failed.
-----------------------------------------------------------------------
Code has been updated by jpeeler.
1/2 jobs failed with 2 failing tests.

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


--------------
Failing Jobs
--------------
  - amd64 (Default Stage): 2 of 97 tests failed.


--------------
Code Changes
--------------
jpeeler (301791):

>Merged revisions 301790 via svnmerge from 
>https://origsvn.digium.com/svn/asterisk/branches/1.8
>
>........
>  r301790 | jpeeler | 2011-01-14 11:32:52 -0600 (Fri, 14 Jan 2011) | 42 lines
>  
>  Resolve deadlock involving REFER.
>  
>  Two fixes:
>  1) One must always have the private unlocked before calling
>  pbx_builtin_setvar_helper to not invalidate locking order since it locks the
>  channel.
>  2) Unlock the channel before calling pbx_find_extension, which starts and stops
>  autoservice during the lookup. The problem scenario as illustrated by the
>  reporter:
>  
>  Thread: do_monitor
>  -----------------------
>  handle_request_do
>   handle_incoming
>    handle_request_refer
>     ast_parking_ext_valid
>      pbx_find_extension
>       ast_autoservice_stop
>        while (chan_list_state == as_chan_list_state) { usleep(1000); }
>  
>  Thread: autoservice_run
>  -----------------------
>  autoservice_run
>   chan = ast_waitfor_n
>    ast_waitfor_nandfds
>     ast_waitfor_nandfds_classic / simple / complex (depending on your system)
>      ast_channel_lock(c[x]);
>  
>  handle_request_do and schedule_process_request_queue locks the owner
>  if it exists. The autoservice thread is waiting for the channel lock, which
>  wasn't ever released since the do_monitor thread was waiting for autoservice
>  operations to complete. Solved by unlocking the channel but keeping a reference
>  to guarantee safety.
>  
>  (closes issue #18403)
>  Reported by: jthurman
>  Patches: 
>        20110103-blind_deadlock.diff uploaded by jthurman (license 614)
>        issue18403.patch uploaded by jpeeler (license 325)
>  Tested by: jthurman
>........
>


--------------
Tests
--------------
New Test Failures (2)
   - AsteriskTestSuite: S/feature blonde transfer
   - AsteriskTestSuite: S/feature attended transfer

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


More information about the Test-results mailing list