[test-results] [Bamboo] Asterisk Testing > Asterisk 11 Branch > #435 was SUCCESSFUL (with 446 tests). Change made by Matthew Jordan.

Bamboo bamboo at asterisk.org
Sat Apr 20 16:59:34 CDT 2013


-----------------------------------------------------------------------
Asterisk Testing > Asterisk 11 Branch > #435 was successful (rerun 3 times).
-----------------------------------------------------------------------
Code has been updated by Matthew Jordan.
446 tests in total.

http://bamboo.asterisk.org/browse/TESTING-AST11BRANCH-435/


--------------
Code Changes
--------------
Matthew Jordan (386159):

>Prevent res_timing_pthread from blocking callers
>
>There were several reports of deadlock when using
>res_timing_pthread. Backtraces indicated that one thread was blocked
>waiting for the write to the pipe to complete and this thread held
>the container lock for the timers.  Therefore any thread that wanted
>to create a new timer or read an existing timer would block waiting
>for either the timer lock or the container lock and deadlock ensued.
>
>This patch changes the way the pipe is used to eliminate this source
>of deadlocks:
>
>1) The pipe is placed in non-blocking mode so that it would never
>block even if the following changes someone fail...
>
>2) Instead of writing bytes into the pipe for each "tick" that's
>fired the pipe now has two states--signaled and unsignaled. If
>signaled, the pipe is hot and any pollers of the read side
>filedescriptor will be woken up. If unsigned the pipe is idle. This
>eliminates even the chance of filling up the pipe and reduces the
>potential overhead of calling unnecessary writes.
>
>3) Since we're tracking the signaled / unsignaled state, we can
>eliminate the exta poll system call for every firing because we know
>that there is data to be read.
>
>(closes issue ASTERISK-21389)
>Reported by: Matt Jordan
>Tested by: Shaun Ruffell, Matt Jordan, Tony Lewis
>patches:
>  0001-res_timing_pthread-Reduce-probability-of-deadlocking.patch uploaded by sruffell (License 5417)
>
>(closes issue ASTERISK-19754)
>Reported by: Nikola Ciprich
>
>(closes issue ASTERISK-20577)
>Reported by: Kien Kennedy
>
>(closes issue ASTERISK-17436)
>Reported by: Henry Fernandes
>
>(closes issue ASTERISK-17467)
>Reported by: isrl
>
>(closes issue ASTERISK-17458)
>Reported by: isrl
>
>Review: https://reviewboard.asterisk.org/r/2441/
>........
>
>Merged revisions 386109 from http://svn.asterisk.org/svn/asterisk/branches/1.8
>



--------------
Tests
--------------
Fixed Tests (2)
   - AsteriskTestSuite: S/channels/ s i p/info dtmf
   - AsteriskTestSuite: S/channels/ s i p/sip tls call

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


More information about the Test-results mailing list