netdev - Re: [patch net-next] net: 8021q: move vlan offload registrations into vlan

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]

Date:   Wed, 14 Nov 2018 22:39:58 -0800
From:   David Ahern <dsahern@...il.com>
To:     Jiri Pirko <jiri@...nulli.us>, netdev@...r.kernel.org
Cc:     davem@...emloft.net, michaelsh86@...il.com, ktkhai@...tuozzo.com,
        sbrivio@...hat.com, sd@...asysnail.net, xiyou.wangcong@...il.com,
        dcaratti@...hat.com, weiyongjun1@...wei.com,
        ivan.khoronzhuk@...aro.org, grygorii.strashko@...com,
        tariqt@...lanox.com, makita.toshiaki@....ntt.co.jp,
        mirq-linux@...e.qmqm.pl, mlxsw@...lanox.com
Subject: Re: [patch net-next] net: 8021q: move vlan offload registrations into
 vlan_core

On 11/13/18 2:22 PM, Jiri Pirko wrote:
> From: Jiri Pirko <jiri@...lanox.com>
> 
> Currently, the vlan packet offloads are registered only upon 8021q module
> load. However, even without this module loaded, the offloads could be
> utilized, for example by openvswitch datapath. As reported by Michael,
> that causes 2x to 5x performance improvement, depending on a testcase.
> 
> So move the vlan offload registrations into vlan_core and make this
> available even without 8021q module loaded.
> 
> Reported-by: Michael Shteinbok <michaelsh86@...il.com>
> Signed-off-by: Jiri Pirko <jiri@...lanox.com>
> Tested-by: Michael Shteinbok <michaelsh86@...il.com>
> ---
>  net/8021q/vlan.c      | 96 -----------------------------------------
>  net/8021q/vlan_core.c | 99 +++++++++++++++++++++++++++++++++++++++++++
>  2 files changed, 99 insertions(+), 96 deletions(-)
> 

Reviewed-by: David Ahern <dsahern@...il.com>