Your patch has been applied to the master branch. I have "forward-logged" Steffan's ACK on the v6 patch as the v8 is close enough, except for the "struct win_tun" change (which look reasonable), plus Simon's ACK.
What I do not like is code that uses malloc() and then calls ZeroMemory() right after. calloc() exists. Standard-based, less lines of code. No reason to NAK this, but maybe this can be fixed in a followup patch (and put on our styleguide somewhere?)... Compile-tested on Ubuntu 16.04 / mingw (which worked fine). No own code review or any sort of testing done. commit d5fc4bd41616ee01816df873cb6b6567872a29e2 Author: Lev Stipakov Date: Tue Dec 17 14:44:10 2019 +0200 wintun: ring buffers based I/O Signed-off-by: Lev Stipakov <l...@openvpn.net> Acked-by: Steffan Karger <steffan.kar...@fox-it.com> Acked-by: Simon Rozman <si...@rozman.si> Message-Id: <20191217124410.81-1-lstipa...@gmail.com> URL: https://www.mail-archive.com/openvpn-devel@lists.sourceforge.net/msg19243.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