netdev - Re: [PATCH net-next 0/4] net: sched: support vxlan and erspan options

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: <CADvbK_c8tbqwwa5ZRkak7ZbY5WijH94Jn3yn1HnvM-Tmxe7AQQ@mail.gmail.com>
Date:   Wed, 20 Nov 2019 00:23:03 +0800
From:   Xin Long <lucien.xin@...il.com>
To:     Roman Mashak <mrv@...atatu.com>
Cc:     network dev <netdev@...r.kernel.org>,
        David Miller <davem@...emloft.net>,
        Simon Horman <simon.horman@...ronome.com>
Subject: Re: [PATCH net-next 0/4] net: sched: support vxlan and erspan options

On Wed, Nov 20, 2019 at 12:06 AM Roman Mashak <mrv@...atatu.com> wrote:
>
> Xin Long <lucien.xin@...il.com> writes:
>
> > On Tue, Nov 19, 2019 at 10:18 PM Roman Mashak <mrv@...atatu.com> wrote:
> >>
> >> On Tue, Nov 19, 2019 at 4:32 AM Xin Long <lucien.xin@...il.com> wrote:
> >> >
> >> > This patchset is to add vxlan and erspan options support in
> >> > cls_flower and act_tunnel_key. The form is pretty much like
> >> > geneve_opts in:
> >> >
> >> >   https://patchwork.ozlabs.org/patch/935272/
> >> >   https://patchwork.ozlabs.org/patch/954564/
> >> >
> >> > but only one option is allowed for vxlan and erspan.
> >>
> >> [...]
> >>
> >> Are you considering to add tdc tests for the new features in separate patch?
> > You mean in selftests?
> > I will post iproute2 side patch to support these first, then
> > considering to add selftests.
> > (this patch series is the kernel side only)
>
> Yes, I mean tools/testing/selftests/tc-testing/tc-tests/actions/tunnel_key.json
>
Sure, and it can only be done after tc userspace supports it.
Thanks for pointing this .json file out.