netdev - Re: [PATCH iproute2-next 1/7] iproute_lwtunnel: add options support for geneve metadata

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] [thread-next>] [day] [month] [year] [list]

Message-ID: <4e62548a-6d2d-558e-aa8e-999648a89b7a@gmail.com>
Date:   Thu, 6 Feb 2020 09:07:55 -0700
From:   David Ahern <dsahern@...il.com>
To:     Xin Long <lucien.xin@...il.com>,
        network dev <netdev@...r.kernel.org>,
        stephen@...workplumber.org
Cc:     Simon Horman <simon.horman@...ronome.com>,
        Pieter Jansen van Vuuren 
        <pieter.jansenvanvuuren@...ronome.com>
Subject: Re: [PATCH iproute2-next 1/7] iproute_lwtunnel: add options support
 for geneve metadata

On 2/2/20 10:39 PM, Xin Long wrote:
> This patch is to add LWTUNNEL_IP(6)_OPTS and LWTUNNEL_IP_OPTS_GENEVE's
> parse and print to implement geneve options support in iproute_lwtunnel.
> 
> Options are expressed as class:type:data and multiple options may be
> listed using a comma delimiter.
> 
> With this patch, users can add and dump geneve options like:
> 
>   # ip net d a; ip net d b; ip net a a; ip net a b
>   # ip -n a l a eth0 type veth peer name eth0 netns b
>   # ip -n a l s eth0 up; ip -n b link set eth0 up
>   # ip -n a a a 10.1.0.1/24 dev eth0; ip -n b a a 10.1.0.2/24 dev eth0
>   # ip -n b l a geneve1 type geneve id 1 remote 10.1.0.1 ttl 64
>   # ip -n b a a 1.1.1.1/24 dev geneve1; ip -n b l s geneve1 up
>   # ip -n b r a 2.1.1.0/24 dev geneve1
>   # ip -n a l a geneve1 type geneve external
>   # ip -n a a a 2.1.1.1/24 dev geneve1; ip -n a l s geneve1 up
>   # ip -n a r a 1.1.1.0/24 encap ip id 1 geneve_opts \
>     1:1:1212121234567890,1:1:1212121234567890,1:1:1212121234567890 \
>     dst 10.1.0.2 dev geneve1
>   # ip -n a r s; echo ''; ip net exec a ping 1.1.1.1 -c 1
> 

Thanks for the command list and example. It would be a lot easier to
read if commands were 1 per line and used the long form.

Same for patches 2 and 3