netdev - Re: [PATCH net-next 0/4] nfp: flower: add Geneve tunnel support

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]

Message-Id: <20171219.145849.807022103361406249.davem@davemloft.net>
Date:   Tue, 19 Dec 2017 14:58:49 -0500 (EST)
From:   David Miller <davem@...emloft.net>
To:     simon.horman@...ronome.com
Cc:     jakub.kicinski@...ronome.com, netdev@...r.kernel.org,
        oss-drivers@...ronome.com, john.hurley@...ronome.com
Subject: Re: [PATCH net-next 0/4] nfp: flower: add Geneve tunnel support

From: Simon Horman <simon.horman@...ronome.com>
Date: Tue, 19 Dec 2017 17:58:25 +0100

> John Hurley says:
> 
> This patchset adds support for offloading the encap and decap of Geneve
> tunnels to the NFP. In both cases, specifying well known port 6081 is a
> requirement for rule offload.
> 
> Geneve firmware support has been recently added, so the patchset includes
> the reading of a fw symbol that defines a bitmap of newly supported
> features. Geneve will only be offloaded if the fw supports it. The new
> symbol is added in fw r5646.
> 
> Geneve option fields are not supported as either a match or an action due
> there current exclussion from TC flower. Because Geneve (as both a match
> and action) behaves the same as other udp tunnels such as VXLAN, generic
> functions are created that handle both Geneve and VXLAN. It is anticapated
> that these functions will be modified to support options in future
> patches.
> 
> The removal of an unused variable 'tun_dst_mask' is included as a separate
> patch here. This does not affect functionality.
> 
> Also included are modifications to the test framework to check that the
> new encap and decap features are functioning correctly.

Series applied, thank you.