Re: [PATCH] iptunnel: make some user care TUNNEL_FLAGS available in uapi

[Jakub Kicinski <jakub.kicinski@netronome.com>]

On Tue, 18 Dec 2018 17:53:08 +0800, wenxu@ucloud.cn wrote:
> From: wenxu <wenxu@ucloud.cn>
> 
> ip l add dev tun type gretap external
> ip r a 10.0.0.1 encap ip dst 192.168.152.171 id 1000 dev gretap
> 
> For gretap Key example when the command set the id but don't set the
> TUNNEL_KEY flags. There is no key field in the send packet
> 
> In the lwtunnel situation, some TUNNEL_FLAGS should can be set by
> userspace
> 
> Signed-off-by: wenxu <wenxu@ucloud.cn>
> ---
>  include/net/ip_tunnels.h       | 4 ----
>  include/uapi/linux/if_tunnel.h | 6 ++++++
>  2 files changed, 6 insertions(+), 4 deletions(-)
> 
> diff --git a/include/net/ip_tunnels.h b/include/net/ip_tunnels.h
> index b0d022f..897db0a 100644
> --- a/include/net/ip_tunnels.h
> +++ b/include/net/ip_tunnels.h
> @@ -144,15 +144,11 @@ struct ip_tunnel {
>  	bool			ignore_df;
>  };
>  
> -#define TUNNEL_CSUM		__cpu_to_be16(0x01)
>  #define TUNNEL_ROUTING		__cpu_to_be16(0x02)
> -#define TUNNEL_KEY		__cpu_to_be16(0x04)
> -#define TUNNEL_SEQ		__cpu_to_be16(0x08)
>  #define TUNNEL_STRICT		__cpu_to_be16(0x10)
>  #define TUNNEL_REC		__cpu_to_be16(0x20)
>  #define TUNNEL_VERSION		__cpu_to_be16(0x40)
>  #define TUNNEL_NO_KEY		__cpu_to_be16(0x80)
> -#define TUNNEL_DONT_FRAGMENT    __cpu_to_be16(0x0100)
>  #define TUNNEL_OAM		__cpu_to_be16(0x0200)
>  #define TUNNEL_CRIT_OPT		__cpu_to_be16(0x0400)
>  #define TUNNEL_GENEVE_OPT	__cpu_to_be16(0x0800)

If these are UAPI, should they all be moved?  Perhaps w/ renaming?

> diff --git a/include/uapi/linux/if_tunnel.h b/include/uapi/linux/if_tunnel.h
> index 1b3d148..06c0f14 100644
> --- a/include/uapi/linux/if_tunnel.h
> +++ b/include/uapi/linux/if_tunnel.h
> @@ -160,4 +160,10 @@ enum {
>  };
>  
>  #define IFLA_VTI_MAX	(__IFLA_VTI_MAX - 1)
> +
> +#define TUNNEL_CSUM		__cpu_to_be16(0x01)
> +#define TUNNEL_KEY		__cpu_to_be16(0x04)
> +#define TUNNEL_SEQ		__cpu_to_be16(0x08)
> +#define TUNNEL_DONT_FRAGMENT    __cpu_to_be16(0x0100)
> +
>  #endif /* _UAPI_IF_TUNNEL_H_ */