Am 07.08.25 um 20:29 schrieb Jon Chiappetta via Openvpn-devel:
Thanks to Gert's help on this, I was able to finally configure and
compile and run and test the bulk mode changes against the latest git
source code to ensure everything still works correctly.
I also fixed up some other issues like properly freeing the extra buffer
allocations and removing the unneeded batched data prefixes and
converting a remaining while loop to a max limited for loop and properly
resetting the outgoing tun buffer pointer at the end of the write method
when finished.
It would still good to explain what you are trying to achieve here and
what the idea behind the patch is to be able to review and understand
your patch.
The patch itself basically has no comments at all, so it is very hard to
decipher for me from the patch what it is trying to to do. Eg there is a
variable flag_ciph that fiddles with encryption of packets.
You are talking and describing this bulk mode as if it was obvious but
it is not. The description on your blog says:
> [...] read 8192 bytes off of the client’s TCP sockets directly and
> proxy them in one write call over TCP directly to the VPN server
> without needing a tunnel interface with a small sized MTU which
> bottlenecks reads+writes to <1500 bytes per function call.
It also not helping as you talking about TCP write/reads, where I can
see some improvement by cutting down the number of reads/writes. But the
second part then talks about not using a tunnel with a small sized MTU.
But if you use a larger sized TUN interface with a larger MTU, then you
already have larger reads/writes to the TCP socket.
Also your speedtest showing 562 is meaningless without having any
comparison without your patch.
Arne
_______________________________________________
Openvpn-devel mailing list
Openvpn-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/openvpn-devel
