Subject: Re: [PATCH] add GENEVE netdev tunnel driver

From: Stephen Hemminger <[email-masked]>
Date: Fri, 8 May 2015 12:32:48 -0700

On Fri,  8 May 2015 13:20:52 -0400
"John W. Linville" <[email-masked]> wrote:

> This 5-patch kernel series adds a netdev implementation of a GENEVE
> tunnel driver, and the single iproute2 patch enables creation and
> such for those netdevs.  This makes use of the existing GENEVE
> infrastructure already used by the OVS code.  The net/ipv4/geneve.c
> file is renamed as net/ipv4/geneve_core.c as part of these changes.
> 
>  drivers/net/Kconfig            |   14 +
>  drivers/net/Makefile           |    1 
>  drivers/net/geneve.c           |  550 +++++++++++++++++++++++++++++++++++++++++
>  include/net/geneve.h           |    5 
>  include/uapi/linux/if_link.h   |    9 
>  net/ipv4/Kconfig               |    4 
>  net/ipv4/Makefile              |    2 
>  net/ipv4/geneve.c              |    6 
>  net/ipv4/geneve_core.c         |    4 
>  net/openvswitch/Kconfig        |    2 
>  net/openvswitch/vport-geneve.c |    5 
>  11 files changed, 585 insertions(+), 17 deletions(-)
> 
> The overall structure of the GENEVE netdev driver is strongly
> influenced by the VXLAN netdev driver.  This is not surprising, as the
> two drivers are intended to serve similar purposes.  As development of
> the GENEVE driver continues, it is likely that those similarities will
> grow stronger.  This will include both simple configuration options
> (e.g. TOS and TTL settings) and new control plane support.

Look good. Thanks.

Acked-by: Stephen Hemminger <[email-masked]>