ziti-tunnel-sdk-c icon indicating copy to clipboard operation
ziti-tunnel-sdk-c copied to clipboard

Feature/geneve

Open dariuszSki opened this issue 3 years ago • 4 comments

@ekoby and @scareything just wanted to create this PR , so you can officially review the code. I worked with Shawn quite a bit, so he is familiar with it a bit more already. Please review and provide comments. I have tested the modified code and seems to be working fine. The throughput is about 60% of the ZET in my testing using the same network and ERs plus load balance for the geneve case. Obvious question is if you want to support multiple drivers. Let me know what you think and how you would see this enabled if accepted. Thanks!

dariuszSki avatar Jan 07 '22 14:01 dariuszSki

@dariuszSki - you found another way of doing this, right? should we keep this PR out here or can we close it?

dovholuknf avatar Aug 09 '22 22:08 dovholuknf

@dariuszSki Question for you above from Clint.

qrkourier avatar Feb 02 '23 16:02 qrkourier

But my original question was not answered? Support of more than one driver mode?

dariuszSki avatar Feb 02 '23 16:02 dariuszSki

The only reason not to merge this PR as it exists is because it changes the ZET initialization on Linux to use the geneve driver instead of the tun driver.

Adding a special command line option to select between geneve or tun would be a simple but inelegant solution to that problem. Alternatively, we've been talking and thinking about netif_driver extensions for a while now, and I think this would be a good use of the extension mechanism once it exists. Issues #596 and #597 have been created to track the extension mechanism itself and the implementation of this PR as an extension.

scareything avatar Feb 02 '23 20:02 scareything