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:12Date Closed:2008-01-15 15:41:38.000-0600
Versions:Frequency of
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).


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)