[test-results] [Bamboo] Asterisk - Team Branches > Pimp My SIP > #273 has FAILED (1 tests failed, no failures were new). Change made by root.

[Bamboo]

-----------------------------------------------------------------------
Asterisk - Team Branches > Pimp My SIP > #273 failed.
-----------------------------------------------------------------------
Code has been updated by root.
1/2 jobs failed, with 1 failing test, no failures were new.

http://bamboo.asterisk.org/browse/ASTTEAM-PIMPMYSIP-273/


--------------
Failing Jobs
--------------
  - Asterisk 1.8 CentOS 6 32-Bit (CentOS 6): 1 of 273 tests failed.



--------------
Code Changes
--------------
root (386163):

>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
>........
>
>Merged revisions 386159 from http://svn.asterisk.org/svn/asterisk/branches/11
>........
>
>Merged revisions 386160 from file:///srv/subversion/repos/asterisk/trunk
>



--------------
Tests
--------------
Existing Test Failures (1)
   - AsteriskTestSuite: S/channels/gulp/incoming calls without auth

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