Summary:	ASTERISK-05069: [patch] [post 1.2] allow safe_asterisk to set asterisk priority and adjust max open files with ulimit -n
Reporter:	Roy Sigurd Karlsbakk (rkarlsba)	Labels:
Date Opened:	2005-09-13 05:08:52	Date Closed:	2008-01-15 16:25:14.000-0600
Priority:	Major	Regression?	No
Status:	Closed/Complete	Components:	Core/General
Versions:		Frequency of Occurrence
Related Issues:
Environment:		Attachments:	( 0) safe_asterisk_ulimit_and_nice.patch ( 1) safe_asterisk-2006-01-13.patch ( 2) safe_asterisk-2006-01-13-v2.patch ( 3) safe_asterisk-2006-01-13-v3.patch
Description:	set asterisk process priority and ulimit -n filemax values upon startup
Comments:	By: Roy Sigurd Karlsbakk (rkarlsba) 2005-09-13 05:09:30 topic should be [PATCH] ... sorry By: opsys (opsys) 2005-12-30 17:43:16.000-0600 HOUSEKEEPING Does running asterisk with nice effect performance?? By: Roy Sigurd Karlsbakk (rkarlsba) 2005-12-30 19:13:45.000-0600 I do not know if running it with other priority than normal processes 'effects' anything, but it certainly affects performance in a positive way. the -p flag is there to do this in absolute way, but using that on a linux box, your asterisk box might suddenly be hanging all along if somthing goes wrong, not just the asterisk process but all the rest of it........ By: Matt O'Gorman (mogorman) 2006-01-13 00:36:18.000-0600 hey rkarlsba, your patch looked okay but dont know how comfortable i am with something that says FIXME in caps... esp when safe_asterisk works fine now. By: Roy Sigurd Karlsbakk (rkarlsba) 2006-01-13 01:23:21.000-0600 this new patch checks for OS and allows files max to be set manually or to be set automatically if running on linux. it also allows setting file max value in kernel on os x and linux (others may be added easily). the first one uploaded does not allow this and may be deleted happy friday thirteenth :) By: Roy Sigurd Karlsbakk (rkarlsba) 2006-01-13 01:31:02.000-0600 and another this also allows setting the number of seconds for which to wait before restarting asteirsk, and allowing for killing all mpg123 processes when asterisk exits. if running asterisk in nice -19, these processes may hang and spin at 100% cpu which is quite a nuisance at prio 19....... roy By: Matthew Fredrickson (mattf) 2006-01-24 14:46:01.000-0600 Put in trunk By: Digium Subversion (svnbot) 2008-01-15 16:23:46.000-0600 Repository: asterisk Revision: 8582 U   trunk/contrib/scripts/safe_asterisk ------------------------------------------------------------------------ r8582 | mattf | 2008-01-15 16:23:46 -0600 (Tue, 15 Jan 2008) | 2 lines Updates from royk to safe_asterisk (ASTERISK-5069) Thanks! ------------------------------------------------------------------------ http://svn.digium.com/view/asterisk?view=rev&revision=8582 By: Digium Subversion (svnbot) 2008-01-15 16:24:01.000-0600 Repository: asterisk Revision: 8598 _U  team/russell/make_output/ U   team/russell/make_output/asterisk.c U   team/russell/make_output/channel.c U   team/russell/make_output/channels/chan_sip.c U   team/russell/make_output/contrib/scripts/safe_asterisk U   team/russell/make_output/include/asterisk/channel.h ------------------------------------------------------------------------ r8598 | russell | 2008-01-15 16:24:00 -0600 (Tue, 15 Jan 2008) | 63 lines Conflicts on files I haven't touched are starting to get on my nerves. Merged revisions 8571-8572,8574,8582,8587,8589-8593 via svnmerge from https://origsvn.digium.com/svn/asterisk/trunk ................ r8571 | russell | 2006-01-24 15:20:05 -0500 (Tue, 24 Jan 2006) | 2 lines convert ast_channel list to use linked list macros (issue ASTERISK-6178) ................ r8572 | russell | 2006-01-24 15:27:09 -0500 (Tue, 24 Jan 2006) | 2 lines store the list of 'atexit' functions using linked list macros (issue ASTERISK-6169) ................ r8574 | oej | 2006-01-24 15:41:08 -0500 (Tue, 24 Jan 2006) | 2 lines Don't reset scheduled ID until we actually end the scheduled event. ................ r8582 | mattf | 2006-01-24 16:45:42 -0500 (Tue, 24 Jan 2006) | 2 lines Updates from royk to safe_asterisk (ASTERISK-5069) Thanks! ................ r8587 | mattf | 2006-01-24 17:06:37 -0500 (Tue, 24 Jan 2006) | 2 lines Make sure safe_asterisk retains previous script defaults ................ r8589 | kpfleming | 2006-01-24 17:33:58 -0500 (Tue, 24 Jan 2006) | 1 line ................ r8590 | kpfleming | 2006-01-24 17:34:06 -0500 (Tue, 24 Jan 2006) | 1 line ................ r8591 | kpfleming | 2006-01-24 17:38:17 -0500 (Tue, 24 Jan 2006) | 10 lines Merged revisions 8588 via svnmerge from https://origsvn.digium.com/svn/asterisk/branches/1.2 ........ r8588 | kpfleming | 2006-01-24 16:32:09 -0600 (Tue, 24 Jan 2006) | 2 lines ensure that channel cannot become zombie after we check but before we try to start indications ........ ................ r8592 | kpfleming | 2006-01-24 17:40:20 -0500 (Tue, 24 Jan 2006) | 1 line ................ r8593 | kpfleming | 2006-01-24 17:40:57 -0500 (Tue, 24 Jan 2006) | 1 line ................ ------------------------------------------------------------------------ http://svn.digium.com/view/asterisk?view=rev&revision=8598 By: Digium Subversion (svnbot) 2008-01-15 16:25:14.000-0600 Repository: asterisk Revision: 8679 _U  team/oej/astum/ D   team/oej/astum/ChangeLog U   team/oej/astum/apps/app_dial.c U   team/oej/astum/asterisk.c U   team/oej/astum/cdr/cdr_pgsql.c U   team/oej/astum/channel.c U   team/oej/astum/channels/chan_agent.c U   team/oej/astum/channels/chan_features.c U   team/oej/astum/channels/chan_iax2.c U   team/oej/astum/channels/chan_sip.c U   team/oej/astum/configs/sip.conf.sample U   team/oej/astum/contrib/scripts/safe_asterisk U   team/oej/astum/include/asterisk/channel.h U   team/oej/astum/rtp.c U   team/oej/astum/utils/astman.c ------------------------------------------------------------------------ r8679 | oej | 2008-01-15 16:25:13 -0600 (Tue, 15 Jan 2008) | 230 lines Merged revisions 8517,8523-8524,8531,8538-8539,8548,8554,8560-8561,8563,8571-8572,8574,8582,8587,8589-8597,8599,8609-8610,8618,8620,8633,8642-8643,8654,8664-8665,8667,8676,8678 via svnmerge from https://origsvn.digium.com/svn/asterisk/trunk ................ r8517 | oej | 2006-01-24 11:36:45 +0100 (Tue, 24 Jan 2006) | 2 lines Whitespace change, extra <tab> added from my tab storage. ................ r8523 | oej | 2006-01-24 12:42:09 +0100 (Tue, 24 Jan 2006) | 2 lines Declaring conn and result static to avoid collission with realtime driver (issue 6336, pressureman) ................ r8524 | oej | 2006-01-24 12:46:29 +0100 (Tue, 24 Jan 2006) | 3 lines - Adding whitespace that I found unused outside - Adding "if (option_debug)" before outputting to DEBUG channel ................ r8531 | oej | 2006-01-24 13:48:44 +0100 (Tue, 24 Jan 2006) | 2 lines - Report SIP reload in manager (issue 5742 with small changes) ................ r8538 | oej | 2006-01-24 14:21:13 +0100 (Tue, 24 Jan 2006) | 2 lines Importing rev ASTERISK-8302 from 1.2, never send response to ACK (issue ASTERISK-6148) ................ r8539 | oej | 2006-01-24 14:53:45 +0100 (Tue, 24 Jan 2006) | 2 lines Issue ASTERISK-6163, FreeBSD compatibility with compilation of func_odbc.c (reported by nulbyte) ................ r8548 | oej | 2006-01-24 18:47:41 +0100 (Tue, 24 Jan 2006) | 2 lines Reverting change in revision 8539 - fixed wrong problem. Sorry. ................ r8554 | oej | 2006-01-24 19:15:20 +0100 (Tue, 24 Jan 2006) | 2 lines Make it clear that caller ID in sip.conf is used only on incoming calls (inspired by bug ASTERISK-6026) ................ r8560 | oej | 2006-01-24 20:08:44 +0100 (Tue, 24 Jan 2006) | 2 lines Issue ASTERISK-5935: Match realtime non-dynamic peers by IP. (siacali). ................ r8561 | oej | 2006-01-24 20:19:20 +0100 (Tue, 24 Jan 2006) | 2 lines Issue 6114: Don't hangup on bye/also if there's no channel. (gst) ................ r8563 | oej | 2006-01-24 20:29:32 +0100 (Tue, 24 Jan 2006) | 2 lines Blocking fix from 1.2 from being applied again. ................ r8571 | russell | 2006-01-24 21:20:05 +0100 (Tue, 24 Jan 2006) | 2 lines convert ast_channel list to use linked list macros (issue ASTERISK-6178) ................ r8572 | russell | 2006-01-24 21:27:09 +0100 (Tue, 24 Jan 2006) | 2 lines store the list of 'atexit' functions using linked list macros (issue ASTERISK-6169) ................ r8574 | oej | 2006-01-24 21:41:08 +0100 (Tue, 24 Jan 2006) | 2 lines Don't reset scheduled ID until we actually end the scheduled event. ................ r8582 | mattf | 2006-01-24 22:45:42 +0100 (Tue, 24 Jan 2006) | 2 lines Updates from royk to safe_asterisk (ASTERISK-5069) Thanks! ................ r8587 | mattf | 2006-01-24 23:06:37 +0100 (Tue, 24 Jan 2006) | 2 lines Make sure safe_asterisk retains previous script defaults ................ r8589 | kpfleming | 2006-01-24 23:33:58 +0100 (Tue, 24 Jan 2006) | 1 line ................ r8590 | kpfleming | 2006-01-24 23:34:06 +0100 (Tue, 24 Jan 2006) | 1 line ................ r8591 | kpfleming | 2006-01-24 23:38:17 +0100 (Tue, 24 Jan 2006) | 10 lines Merged revisions 8588 via svnmerge from https://origsvn.digium.com/svn/asterisk/branches/1.2 ........ r8588 | kpfleming | 2006-01-24 16:32:09 -0600 (Tue, 24 Jan 2006) | 2 lines ensure that channel cannot become zombie after we check but before we try to start indications ........ ................ r8592 | kpfleming | 2006-01-24 23:40:20 +0100 (Tue, 24 Jan 2006) | 1 line ................ r8593 | kpfleming | 2006-01-24 23:40:57 +0100 (Tue, 24 Jan 2006) | 1 line ................ r8594 | kpfleming | 2006-01-24 23:41:45 +0100 (Tue, 24 Jan 2006) | 1 line ................ r8595 | kpfleming | 2006-01-24 23:42:43 +0100 (Tue, 24 Jan 2006) | 10 lines Merged revisions 8173 via svnmerge from https://origsvn.digium.com/svn/asterisk/branches/1.2 ........ r8173 | russell | 2006-01-17 20:49:21 -0600 (Tue, 17 Jan 2006) | 2 lines remove ChangeLog from the 1.2 branch.  It will only be present in the tags. ........ ................ r8596 | kpfleming | 2006-01-24 23:43:30 +0100 (Tue, 24 Jan 2006) | 1 line ................ r8597 | kpfleming | 2006-01-24 23:43:57 +0100 (Tue, 24 Jan 2006) | 2 lines clean up remaining already-merged revisions ................ r8599 | kpfleming | 2006-01-24 23:45:41 +0100 (Tue, 24 Jan 2006) | 2 lines remove extraneous characters from property ................ r8609 | kpfleming | 2006-01-25 02:52:58 +0100 (Wed, 25 Jan 2006) | 10 lines Merged revisions 8608 via svnmerge from https://origsvn.digium.com/svn/asterisk/branches/1.2 ........ r8608 | kpfleming | 2006-01-24 19:50:52 -0600 (Tue, 24 Jan 2006) | 2 lines ensure hangup cause code is handled properly when channel does not return a frame (issue ASTERISK-6186) ........ ................ r8610 | kpfleming | 2006-01-25 02:53:15 +0100 (Wed, 25 Jan 2006) | 1 line ................ r8618 | russell | 2006-01-25 06:37:29 +0100 (Wed, 25 Jan 2006) | 3 lines don't leak almost 200 bytes for each new channel and store the active channel list using the linked list macros (issue ASTERISK-6170) ................ r8620 | russell | 2006-01-25 06:39:25 +0100 (Wed, 25 Jan 2006) | 1 line ................ r8633 | oej | 2006-01-25 10:50:28 +0100 (Wed, 25 Jan 2006) | 2 lines Issue ASTERISK-6189 - patch by markster, imported from 1.2 ................ r8642 | oej | 2006-01-25 13:01:07 +0100 (Wed, 25 Jan 2006) | 3 lines From now on, apply maxexpiry and minexpiry to all subscriptions. Thanks to fourcheeze in the IRC channel for pointing this out. ................ r8643 | oej | 2006-01-25 13:11:30 +0100 (Wed, 25 Jan 2006) | 3 lines - Remove unused option to transmit_state_notify - Allow for expiry=0 in subscription requests that only wants *one* update and that's it. ................ r8654 | kpfleming | 2006-01-25 15:52:43 +0100 (Wed, 25 Jan 2006) | 3 lines don't queue a congestion frame on a channel that will be immediately hung up anyway clean up/organize code block ................ r8664 | russell | 2006-01-25 19:12:55 +0100 (Wed, 25 Jan 2006) | 2 lines store agent_pvt list using linked list macros (issue ASTERISK-6182) ................ r8665 | russell | 2006-01-25 19:24:32 +0100 (Wed, 25 Jan 2006) | 3 lines store feature_pvt list using linked list macros (issue ASTERISK-6190, with additional changes to prevent a memory leak in unload_module) ................ r8667 | russell | 2006-01-25 19:41:12 +0100 (Wed, 25 Jan 2006) | 1 line ................ r8676 | russell | 2006-01-25 20:06:37 +0100 (Wed, 25 Jan 2006) | 2 lines use arg parsing macros in the AGENT dialplan function (issue ASTERISK-6078, with small mods) ................ r8678 | russell | 2006-01-25 20:16:14 +0100 (Wed, 25 Jan 2006) | 11 lines Merged revisions 8677 via svnmerge from https://origsvn.digium.com/svn/asterisk/branches/1.2 ........ r8677 | russell | 2006-01-25 14:14:43 -0500 (Wed, 25 Jan 2006) | 3 lines don't call ast_update_realtime with uninitialized variables if we get a registration with an expirey of 0 seconds (issue ASTERISK-6016) ........ ................ ------------------------------------------------------------------------ http://svn.digium.com/view/asterisk?view=rev&revision=8679