Acked-by: Gert Doering <g...@greenie.muc.de>

I have listened to the discussion, and I think we all agree that we
need to revisit this "DCO counter" business:
  - definition of c2 structure elements
  - do we need extra fields for "dco counters"?
    - do we need more counters?  Windows currently has 
      "TransportBytes*" and "TunBytes*", while FreeBSD only has "bytes"
      (which are what, exactly?)
      -> *I* definitely need a better understanding of these
  - when and where do we query the kernel for counters (timer, signal, ...?)

This said, we want the functionality now :-) - so, I have tested this
(on FreeBSD), and it does what it says, and the code looks reasonable
according to "it will not access undefined pointers, will not leak
memory" etc.

Your patch has been applied to the master and release/2.6 branch.

commit d5238627e4fab93a6c09816c60eb90e237b626c3 (master)
commit d598871ca9fb7b4814ee8d8edfb26d20479bb6ed (release/2.6)
Author: Lev Stipakov
Date:   Wed Mar 22 13:32:49 2023 +0200

     Print DCO client stats on SIGUSR2

     Signed-off-by: Lev Stipakov <l...@openvpn.net>
     Acked-by: Gert Doering <g...@greenie.muc.de>
     Message-Id: <20230322113249.2039-1-lstipa...@gmail.com>
     URL: 
https://www.mail-archive.com/openvpn-devel@lists.sourceforge.net/msg26471.html
     Signed-off-by: Gert Doering <g...@greenie.muc.de>


--
kind regards,

Gert Doering



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

Reply via email to