netdev - Re: [PATCH net-next v3 2/3] openvswitch: Use regular GRE net

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [day] [month] [year] [list]

Date:	Tue, 4 Aug 2015 21:07:52 -0700
From:	Alexei Starovoitov <alexei.starovoitov@...il.com>
To:	Pravin Shelar <pshelar@...ira.com>
Cc:	netdev <netdev@...r.kernel.org>
Subject: Re: [PATCH net-next v3 2/3] openvswitch: Use regular GRE net_device
 instead of vport

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 majordomo@...r.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html