> On Jan. 10, 2014, 3:05 a.m., Guenther Kelleter wrote: > > I'm missing an update to app_dumpchan which was fixed in svn revision > > 397961 "Verbose logging discrepancies" for AST-1193. > > > > these calls to the very expensive functions > > serialize_showchan(chan, info, sizeof(info)); > > pbx_builtin_serialize_variables(chan, &vars); > > by dumpchan_exec() should only be made when the sys verbose level is at > > least dumpchan's argument, i.e. last hunk of revision 397961 for > > app_dumpchan.c should be reverted appropriately.
Fixed. - rmudgett ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviewboard.asterisk.org/r/3114/#review10561 ----------------------------------------------------------- On Jan. 9, 2014, 1:30 p.m., rmudgett wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviewboard.asterisk.org/r/3114/ > ----------------------------------------------------------- > > (Updated Jan. 9, 2014, 1:30 p.m.) > > > Review request for Asterisk Developers. > > > Bugs: AST-1252 > https://issues.asterisk.org/jira/browse/AST-1252 > > > Repository: Asterisk > > > Description > ------- > > The per console verbose level feature as previously implemented caused a > large performance penalty. The fix required some minor incompatibilities if > the new rasterisk is used to connect to an earlier version. If the new > rasterisk connects to an older Asterisk version then the root console verbose > level is always affected by the "core set verbose" command of the remote > console even though it may appear to only affect the current console. If an > older version of rasterisk connects to the new version then the "core set > verbose" command will have no effect. > > * Fixed the verbose performance by not generating a verbose message if > nothing is going to use it and then filtered any generated verbose messages > before actually sending them to the remote consoles. > > * Split the "core set debug" and "core set verbose" CLI commands to remove > the per module verbose support that cannot work with the per console verbose > level. > > * Added a silent option to the "core set verbose" command. > > * Fixed "core set debug off" tab completion. > > * Made "core show settings" list the current console verbosity in addition to > the root console verbosity. > > * Changed the default verbose level of the 'verbose' setting in the > logger.conf [logfiles] section. The default is now to once again follow the > current root console level. As a result, using the AMI Command action with > "core set verbose" could again set the root console verbose level and affect > the verbose level logged. > > > Diffs > ----- > > /branches/11/main/pbx.c 405252 > /branches/11/main/manager.c 405252 > /branches/11/main/logger.c 405252 > /branches/11/main/cli.c 405252 > /branches/11/main/asterisk.c 405252 > /branches/11/include/asterisk/logger.h 405252 > /branches/11/funcs/func_timeout.c 405252 > /branches/11/configs/logger.conf.sample 405252 > /branches/11/UPGRADE.txt 405252 > > Diff: https://reviewboard.asterisk.org/r/3114/diff/ > > > Testing > ------- > > Setup an extension that had a simple dialplan extension that decremented a > counter from 500000 and nothing logging or viewing verbose messages. > On my machine, Asterisk v11 without the patch took 63 seconds while with the > patch it took 25 seconds. > > Tested the "core set verbose" CLI command tab completion. > > Tested that different verbose levels on multiple rasterisk connections got > the expected verbose messages. > > > Thanks, > > rmudgett > >
-- _____________________________________________________________________ -- Bandwidth and Colocation Provided by http://www.api-digital.com -- asterisk-dev mailing list To UNSUBSCRIBE or update options visit: http://lists.digium.com/mailman/listinfo/asterisk-dev