On Fri, 2012-12-14 at 23:17 +0000, Rob Day wrote:
> The actual patch I'll be committing is below.
Apologies, that patch had an error in it (and the definition of MAX was
redundant):
@@ -61,7 +61,9 @@ unsigned int opentask::wake() {
return 0;
} else {
/* We need to compute when the next call is going to be opened. */
- return (unsigned long) (last_rate_change_time +
((calls_since_last_rate_change + 1) / (rate/rate_period_ms)));
+ return (unsigned long)
+ MAX(last_rate_change_time + (calls_since_last_rate_change /
+ MAX(rate/MAX(rate_period_ms, 1), 1)), 1);
}
}
I'm now less convinced that I diagnosed the problem with rates > 1000
correctly; I'll do some further investigation of it on a Fedora 12
system, where it reproduces more consistently.
Best,
Rob
------------------------------------------------------------------------------
LogMeIn Rescue: Anywhere, Anytime Remote support for IT. Free Trial
Remotely access PCs and mobile devices and provide instant support
Improve your efficiency, and focus on delivering more value-add services
Discover what IT Professionals Know. Rescue delivers
http://p.sf.net/sfu/logmein_12329d2d
_______________________________________________
Sipp-users mailing list
Sipp-users@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/sipp-users
