On Wed, May 04, 2022 at 12:14:01AM +0200, Alexander Bluhm wrote:
> Hi,
>
> We have one comment that locking for ratecheck(9) is missing. In
> all other places locking status of the struct timeval *lasttime
> is unclear.
>
> The easiest fix is a global mutex for all lasttime in ratecheck().
>
On Wed, May 04, 2022 at 04:42:21AM -0500, Scott Cheloha wrote:
> > On May 3, 2022, at 17:16, Alexander Bluhm wrote:
> >
> > ???Hi,
> >
> > We have one comment that locking for ratecheck(9) is missing. In
> > all other places locking status of the struct timeval *lasttime
> > is unclear.
> >
>
> On May 3, 2022, at 17:16, Alexander Bluhm wrote:
>
> Hi,
>
> We have one comment that locking for ratecheck(9) is missing. In
> all other places locking status of the struct timeval *lasttime
> is unclear.
>
> The easiest fix is a global mutex for all lasttime in ratecheck().
> This covers
Hi,
We have one comment that locking for ratecheck(9) is missing. In
all other places locking status of the struct timeval *lasttime
is unclear.
The easiest fix is a global mutex for all lasttime in ratecheck().
This covers the usual usecase of the function.
Same for ppsratecheck(9), lasttime
