[asterisk-bugs] [LibPRI 0016909]: [patch] pritimer's in chan_dahdi.conf not honoured

[Asterisk Bug Tracker]

A NOTE has been added to this issue. 
====================================================================== 
https://issues.asterisk.org/view.php?id=16909 
====================================================================== 
Reported By:                alecdavis
Assigned To:                
====================================================================== 
Project:                    LibPRI
Issue ID:                   16909
Category:                   Channels/chan_dahdi
Reproducibility:            always
Severity:                   minor
Priority:                   normal
Status:                     new
Asterisk Version:           SVN 
JIRA:                        
SVN Branch (only for SVN checkouts, not tarball releases): 1.6.1 
SVN Revision (number only!): 248954 
Disclaimer on File?:        N/A 
Request Review:              
====================================================================== 
Date Submitted:             2010-02-26 02:53 CST
Last Modified:              2010-03-01 19:42 CST
====================================================================== 
Summary:                    [patch] pritimer's in chan_dahdi.conf not honoured
Description: 
configuring pritimer=>k,3 in chan_dahdi.conf is overwritten by libpri's
pri_default_timers, which defaults 'k' timer back to 7.

required to prevent pri lockups under heavy conditions where switch
doesn't like so many unacknowledge i-frames.
======================================================================
Relationships       ID      Summary
----------------------------------------------------------------------
related to          0002518 Provide configurable timers for PRI's
related to          0016790 [patch] !! Got reject for frame 41, ret...
====================================================================== 

---------------------------------------------------------------------- 
 (0118729) rmudgett (administrator) - 2010-03-01 19:42
 https://issues.asterisk.org/view.php?id=16909#c118729 
---------------------------------------------------------------------- 
The asterisk patch looks good to go.  Configuring the timers got broke when
sig_pri.c was split out of chan_dahdi.c so this patch is only needed in
trunk.

The K value used by Q.921 needs to be protected from changing at
inopportune times since it is the window size.  Simply deleting the k
element from the pri structure and replacing references to it by
timers[PRI_TIMER_K] is not safe.

The best time to update the K value is when the D channel is idle and the
I-frame queue is empty. 

Issue History 
Date Modified    Username       Field                    Change               
====================================================================== 
2010-03-01 19:42 rmudgett       Note Added: 0118729                          
======================================================================