<div dir="ltr">I like the "two sets of counters" idea.</div><br><div class="gmail_quote"><div dir="ltr">On Wed, Jun 6, 2018 at 10:18 PM Hal Murray via devel <<a href="mailto:devel@ntpsec.org">devel@ntpsec.org</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><br>
[if (0) on debugging msyslog]<br>
> What I do is delete them unless I think they might have continuing value, in<br>
> which case I put them under DPRINTF.<br>
<br>
I agree on the "continuing value" part.<br>
<br>
For what I'm after, msyslog is more convenient that DPRINTF.  I'm willing to <br>
do an edit/build/restart cycle to get the printout I need.  For now, I'll try <br>
my "if (0)" hack.  If anybody thinks that's too ugly we can discuss <br>
alternatives.<br>
<br>
>> extern  int     clocktime       (int, int, int, int, int, time_t, <br>
uint32_t, uint32_t *, uint32_t *);<br>
> Fair point.  Are you advocating outting the formal names back in?<br>
<br>
I don't have any strong opinions in that area.  That's a particularly ugly <br>
example.  I ran into a few ugly ones when working on the CMAC authentication. <br>
 It seems like something we could do better.<br>
<br>
[*CAST stuff in ntpd code]<br>
> I agree with moving them to ntpclients.  Anything  that reeduces the amount<br>
> of ceuft ubthe ntpd code is good. <br>
<br>
I didn't find any usage in ntpclients so there probably isn't any need to <br>
move them.  We should concentrate on removing code from ntpd.<br>
<br>
If we find things like "if (xxx->flags & MCAST)" but there is no way that the <br>
flag ever gets set, then we can just remove the code.<br>
<br>
[counters that get reset every hour after writing to log file]<br>
> Reasonable. Would you make ntpq parse the logfile?<br>
<br>
No, that's ugly and only works if you are on the system with the log files.<br>
<br>
I was thinking of having ntpd maintain 2 sets of counters.  The new set is <br>
parallel to the current set.  It gets updated when the current set is <br>
cleared.  When ntpq asks, ntpd would return both values.<br>
<br>
[ntpq -p showing authenticated slots]<br>
> No need. Just boldface the ones with authentication.<br>
<br>
I like it.  What happens if I cut/paste into email?<br>
<br>
<br>
-- <br>
These are my opinions.  I hate spam.<br>
<br>
<br>
<br>
_______________________________________________<br>
devel mailing list<br>
<a href="mailto:devel@ntpsec.org" target="_blank">devel@ntpsec.org</a><br>
<a href="http://lists.ntpsec.org/mailman/listinfo/devel" rel="noreferrer" target="_blank">http://lists.ntpsec.org/mailman/listinfo/devel</a><br>
</blockquote></div>-- <br><div dir="ltr" class="gmail_signature" data-smartmail="gmail_signature"><div dir="ltr"><p dir="ltr">Mark Atwood<br>
<a href="http://about.me/markatwood">http://about.me/markatwood</a><br>
+1-206-604-2198</p>
</div></div>