[asterisk-bugs] [Asterisk 0014998]: [patch] Change readq locking

Asterisk Bug Tracker noreply at bugs.digium.com
Wed Feb 17 16:32:10 CST 2010


A NOTE has been added to this issue. 
====================================================================== 
https://issues.asterisk.org/view.php?id=14998 
====================================================================== 
Reported By:                wedhorn
Assigned To:                
====================================================================== 
Project:                    Asterisk
Issue ID:                   14998
Category:                   Core/Channels
Reproducibility:            N/A
Severity:                   tweak
Priority:                   normal
Status:                     ready for review
Asterisk Version:           SVN 
JIRA:                        
Regression:                 No 
Reviewboard Link:            
SVN Branch (only for SVN checkouts, not tarball releases):  trunk 
SVN Revision (number only!): 191173 
Request Review:              
====================================================================== 
Date Submitted:             2009-04-29 15:32 CDT
Last Modified:              2010-02-17 16:32 CST
====================================================================== 
Summary:                    [patch] Change readq locking
Description: 
Revision of chan->readq locking to utilise a list lock rather than the chan
lock. The main advantage of this patch is that channels need not lock
before calling ast_queue_frame or related functions purely to avoid a
deadlock.

Obvious changes to chan_sip and chan_iax have been included. I'm sure
there are others.
====================================================================== 

---------------------------------------------------------------------- 
 (0118194) wedhorn (reporter) - 2010-02-17 16:32
 https://issues.asterisk.org/view.php?id=14998#c118194 
---------------------------------------------------------------------- 
aragon, looks like there is no way this is going to be applied to anything
but trunk (see
http://lists.digium.com/pipermail/asterisk-dev/2010-February/042425.html).

The patch may take a bit longer as I'll look at integrating an atomic
update of softhangup so we can remove all chan locking around readq
changes. 

Issue History 
Date Modified    Username       Field                    Change               
====================================================================== 
2010-02-17 16:32 wedhorn        Note Added: 0118194                          
======================================================================




More information about the asterisk-bugs mailing list