[asterisk-bugs] [Asterisk 0010590]: [patch] RTP statistics returned by ast_rtp_get_quality reflects the last RTCP packet not a call overall

[noreply at bugs.digium.com]

A NOTE has been added to this issue. 
====================================================================== 
http://bugs.digium.com/view.php?id=10590 
====================================================================== 
Reported By:                gasparz
Assigned To:                
====================================================================== 
Project:                    Asterisk
Issue ID:                   10590
Category:                   Core/NewFeature
Reproducibility:            always
Severity:                   minor
Priority:                   normal
Status:                     new
Asterisk Version:           SVN 
SVN Branch (only for SVN checkouts, not tarball releases):  trunk 
SVN Revision (number only!):  
Disclaimer on File?:        N/A 
Request Review:              
====================================================================== 
Date Submitted:             08-29-2007 06:22 CDT
Last Modified:              05-17-2008 22:08 CDT
====================================================================== 
Summary:                    [patch] RTP statistics returned by
ast_rtp_get_quality reflects the last RTCP packet not a call overall
Description: 
The result of the function ast_rtp_get_quality is the value set by the LAST
RTCP frame. This value is inserted to the RTPAUDIOQOS channel variable and
should be a some kind of quality metric of the call.
So if we take only one sample: one (the last) RTCP frame we won't get a
relevant value for a parameter. The min/max values calculated (ex:
maxrtt,minrtt and all the others) are unused and would be very usefull to
aproximate call quality.
An average value could be easily computed for the parameters (ex: rtt) and
would give a mutch better picture of RTP quality.

Ex: 
Last rtt value: 80ms
Min rtt value: 30ms
Max rtt value: 500ms (perhaps congestion)
Avg rtt value: 35ms

My proposal is to create a new function:ast_rtp_get_quality_ext that would
return a detailed parsable string with the 4 values for each parameter.
This could be saved in another channel variable RTPAUDIOQOSEXT. This way we
would be backwards compatible with the current version.

Another less elegant way is to modify the curent function and add the
informations (min/max/avg) for each parameter at the end of the string
returned by ast_rtp_get_quality.




====================================================================== 

---------------------------------------------------------------------- 
 chappell - 05-17-08 22:08  
---------------------------------------------------------------------- 
Tried latest patch.  If the bridged channel is an IAX2 channel, Asterisk
crashes on hangup. 

Issue History 
Date Modified   Username       Field                    Change               
====================================================================== 
05-17-08 22:08  chappell       Note Added: 0086983                          
======================================================================