Thread (94 messages) 94 messages, 15 authors, 2015-12-09

RE: [PATCH v1 1/6] net: Generalize udp based tunnel offload

From: Singhai, Anjali <hidden>
Date: 2015-11-24 00:32:26

-----Original Message-----
From: Jesse Gross [mailto:jesse@kernel.org]
Sent: Monday, November 23, 2015 2:50 PM
To: Tom Herbert
Cc: Singhai, Anjali; Linux Kernel Network Developers; Patil, Kiran
Subject: Re: [PATCH v1 1/6] net: Generalize udp based tunnel offload

On Mon, Nov 23, 2015 at 1:53 PM, Tom Herbert [off-list ref]
wrote:
quoted
quoted
diff --git a/include/net/udp_tunnel.h b/include/net/udp_tunnel.h
index cb2f89f..72415aa 100644
--- a/include/net/udp_tunnel.h
+++ b/include/net/udp_tunnel.h
@@ -9,6 +9,12 @@
 #include <net/addrconf.h>
 #endif

+enum udp_tunnel_type {
+       UDP_TUNNEL_UNSPEC,
+       UDP_TUNNEL_VXLAN,
+       UDP_TUNNEL_GENEVE,
+};
+
Sorry, I still don't like this. Grant it least it gets rid of of VXLAN
specific ops, but the problem is there no such things as a common set
of encapsulations in the kernel (e.g. foo-over-udp adds a bunch of
encapsulations not represented here), no defined common set of device
functionality that needs this, and this precludes the use of the RX
accelerations to be available from a userpsace  implementation.
Regardless, I think this is at least a good cleanup of what is already
there compared to having VXLAN-specific NDOs. We can always add
additional things in the future.
Agreed with Jesse that this will help not hurt,  when we are ready to cross the bridge for removing RX side Protocol ossification. 
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help