On Tue, Aug 04, 2015 at 07:50:40PM -0700, Pravin Shelar wrote: > > I am planning on adding the ip_gre netlink interface later on. OVS > userspace needs same interface to directly create the GRE device > without vport-gre module. vport-gre module is compatibility code. If > you want I can do it in same patch.
I think it has to be part of the same patch set especially if ovs userspace is going to create gre netdev directly. Adding IFLA_GRE_COLLECT_METADATA flag will let anyone use this gre tunnel with normal routing that already understands tunnel metadata and if routing is working then bpf helpers will work with it seamlessly as well. Have another question: + info->key.tun_flags = tpi->flags & (TUNNEL_CSUM | TUNNEL_KEY); why keep CSUM flag ? Current ip_tunnel_key doesn't have csum field, so gre csum gets lost after decap anyway? Also I think FLOWBASED is a bad name both here and in vxlan. Tunnel that understands metadata on tx/rx has nothing to do with flows. I'm thinking to combine IFLA_VXLAN_FLOWBASED + IFLA_VXLAN_COLLECT_METADATA into one IFLA_VXLAN_COLLECT_METADATA while it's not too late. -- To unsubscribe from this list: send the line "unsubscribe netdev" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
