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

[Asterisk Bug Tracker]

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 13:47 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.
====================================================================== 

---------------------------------------------------------------------- 
 (0118174) aragon (reporter) - 2010-02-17 13:47
 https://issues.asterisk.org/view.php?id=14998#c118174 
---------------------------------------------------------------------- 
wedhorn: tilghman has led me to believe that this patch could fix my
locking issues in https://issues.asterisk.org/view.php?id=16811
What started out as that bug report led to a recommendation of a possible
fix at https://reviewboard.asterisk.org/r/219/ but that patch was abandoned
in favour of your code. My fingers are crossed, and hopefully I'm not
heading down a bad path... it would be nice for a core developer to put in
his extra 2 cents here. 

Issue History 
Date Modified    Username       Field                    Change               
====================================================================== 
2010-02-17 13:47 aragon         Note Added: 0118174                          
======================================================================