Summary: | ASTERISK-04805: [patch] Remove realtime priority for AGI scripts | ||
Reporter: | critch (critch) | Labels: | |
Date Opened: | 2005-08-10 13:17:38 | Date Closed: | 2008-01-15 15:44:56.000-0600 |
Priority: | Minor | Regression? | No |
Status: | Closed/Complete | Components: | Resources/res_agi |
Versions: | Frequency of Occurrence | ||
Related Issues: | |||
Environment: | Attachments: | ( 0) 20050810__no_realtime_for_agi.diff.txt | |
Description: | On Wednesday 10 August 2005 12:40, Steven Critchfield wrote: > When asterisk is run in the pseudo realtime mode, any agi app that > runs also inherits this priority and can compete with asterisk to > complete it's task. While some agi apps may not take long to start up > and then spend a large amount of time waiting for asterisk to reply > to events, it is the start up time of an agi app that can cause > problems. > > Specifically we have a complex perl script that was causing up to 1 > second of choppy audio on other zap channels when a new call was > started. It would get worse with the more concurrent calls running as > asterisk had more work to do and the perl script still had a fixed > amount of work to get accomplished before it could go to a mostly > idle state. > > After discussions with Tilghman about the solution, here is a set of > diffs that I think need a moment more of peer review before being > accepted. > > I do have a disclaimer on file with Digium. | ||
Comments: | By: Kevin P. Fleming (kpfleming) 2005-08-22 21:28:31 Committed to CVS HEAD, thanks! By: Digium Subversion (svnbot) 2008-01-15 15:44:56.000-0600 Repository: asterisk Revision: 6368 U trunk/asterisk.c U trunk/include/asterisk.h U trunk/res/res_agi.c ------------------------------------------------------------------------ r6368 | kpfleming | 2008-01-15 15:44:56 -0600 (Tue, 15 Jan 2008) | 2 lines make sure realtime/high scheduling priority is relinquished before executing an AGI script (issue ASTERISK-4805) ------------------------------------------------------------------------ http://svn.digium.com/view/asterisk?view=rev&revision=6368 |