Summary: | ASTERISK-04583: [patch] make use of ast_sched_dump and add missing Makefile directives | ||
Reporter: | drmac (drmac) | Labels: | |
Date Opened: | 2005-07-13 14:19:12 | Date Closed: | 2008-01-15 15:41:38.000-0600 |
Priority: | Major | Regression? | No |
Status: | Closed/Complete | Components: | Core/General |
Versions: | Frequency of Occurrence | ||
Related Issues: | |||
Environment: | Attachments: | ( 0) debug_scheduler.patch | |
Description: | While browsing thru sched.c I noticed a function, ast_sched_dump. I grep'd thru the entire code and never found another instance of it. So I figured, what the heck, lets implement its use incase someone really needs to see what's going on in each sched_context. Also in sched.c is an #ifdef check for DEBUG_SCHEDULER but nothing in the Makefile to show its there. Patch adds a call to the sched_dump when something is added/deleted to/from the sched_context in question and adds 2 compiler defines (commented out as to not break anything). ****** ADDITIONAL INFORMATION ****** Further improvements to this might be a container for all sched_contexts so that a CLI command could be written that goes thru the container and prints info on each context on user demand, rather than always the same place (re: add/delete). It would be nice to be able to "see" the name of the callback function and data instead of a bunch of hex...but I don't know how to do that since the callback datatype is of void* | ||
Comments: | By: Kevin P. Fleming (kpfleming) 2005-07-15 18:14:01 Committed to CVS HEAD, but with the defines disabled by default. Thanks! By: Digium Subversion (svnbot) 2008-01-15 15:41:38.000-0600 Repository: asterisk Revision: 6145 U trunk/Makefile U trunk/sched.c ------------------------------------------------------------------------ r6145 | kpfleming | 2008-01-15 15:41:38 -0600 (Tue, 15 Jan 2008) | 2 lines clean up scheduler debugging and expose defines in the Makefile (bug ASTERISK-4583) ------------------------------------------------------------------------ http://svn.digium.com/view/asterisk?view=rev&revision=6145 |