Hi,

On Thu, Dec 15, 2022 at 02:48:49PM +0200, Lev Stipakov wrote:
> > So it seems as the timer activation got delayed somehow
> > -> this is something we need to come back to, when Linux/FreeBSD DCO
> > stats get implemented.  Starting with "management-hold", then "bytecount",
> > then "hold release" gave me counters right away.  Mysteries...
> 
> This is because we do not trigger this new timer (for example by
> calling reset_coarse_timers)
> when we init it in man_bytecount(). We do not have "struct context"
> there to do it.

Ah.  I guessed something like this - thanks for explaining why this would
not actually be easy to achive.

> Follow-up calls to reset_coarse_timers() or process_coarse_timers()
> will do the trick. However, in DCO case nothing might happen in userspace
> for a while, hence delay. GUI does "management-hold", sends "bytecount 5"
> and after that "hold-release".

This works (tested!), so the point above is not a serious problem.

gert
-- 
"If was one thing all people took for granted, was conviction that if you 
 feed honest figures into a computer, honest figures come out. Never doubted 
 it myself till I met a computer with a sense of humor."
                             Robert A. Heinlein, The Moon is a Harsh Mistress

Gert Doering - Munich, Germany                             g...@greenie.muc.de

Attachment: signature.asc
Description: PGP signature

_______________________________________________
Openvpn-devel mailing list
Openvpn-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/openvpn-devel

Reply via email to