[test-results] [Bamboo] Asterisk Testing > Asterisk Trunk > #1192 has FAILED (7 tests failed, 4 failures were new). Change made by Matthew Jordan.

[Bamboo]

-----------------------------------------------------------------------
Asterisk Testing > Asterisk Trunk > #1192 failed (rerun once).
-----------------------------------------------------------------------
Code has been updated by Matthew Jordan.
7/634 tests failed, 4 failures were new.

http://bamboo.asterisk.org/browse/TESTING-ASTERISKTRUNK-1192/


--------------
Failing Jobs
--------------
  - Asterisk CentOS 6 32-Bit (CentOS 6): 7 of 509 tests failed.



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

>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
>



--------------
Tests
--------------
New Test Failures (4)
   - AsteriskTestSuite: S/channels/ s i p/sip custom presence/multiple state change
   - AsteriskTestSuite: S/channels/ s i p/sip custom presence/nominal state change
   - AsteriskTestSuite: S/masquerade
   - AsteriskTestSuite: S/bridge/dial l s options
Existing Test Failures (3)
   - AsteriskTestSuite: S/channels/ s i p/sip tls call
   - AsteriskTestSuite: S/bridge/transfer failure
   - AsteriskTestSuite: S/apps/confbridge/confbridge marked
Fixed Tests (1)
   - AsteriskTestSuite: S/bridge/connected line update

--
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/440740e5/attachment-0001.htm>