netdev - RE: [PATCH] net: bond: skip vlan header when do layer 3+4 hash policy

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: <4F88C5DDA1E80143B232E89585ACE27D019F2C52@DGGEMA502-MBS.china.huawei.com>
Date:   Mon, 2 Apr 2018 10:53:35 +0000
From:   "liujian (CE)" <liujian56@...wei.com>
To:     Nikolay Aleksandrov <nikolay@...ulusnetworks.com>,
        "davem@...emloft.net" <davem@...emloft.net>,
        "j.vosburgh@...il.com" <j.vosburgh@...il.com>,
        "vfalico@...il.com" <vfalico@...il.com>,
        "andy@...yhouse.net" <andy@...yhouse.net>
CC:     "netdev@...r.kernel.org" <netdev@...r.kernel.org>,
        "weiyongjun (A)" <weiyongjun1@...wei.com>
Subject: RE: [PATCH] net: bond: skip vlan header when do layer 3+4 hash
 policy

> -----Original Message-----
> From: Nikolay Aleksandrov [mailto:nikolay@...ulusnetworks.com]
> Sent: Saturday, March 31, 2018 5:23 PM
> To: liujian (CE) <liujian56@...wei.com>; davem@...emloft.net;
> j.vosburgh@...il.com; vfalico@...il.com; andy@...yhouse.net
> Cc: netdev@...r.kernel.org; weiyongjun (A) <weiyongjun1@...wei.com>
> Subject: Re: [PATCH] net: bond: skip vlan header when do layer 3+4 hash policy
> 
> On 31/03/18 12:14, liujian56@...wei.com wrote:
> > From: liujian <liujian56@...wei.com>
> >
> > When the hash policy is BOND_XMIT_POLICY_LAYER34 mode and skb
> protocol
> > is 802.1q VLAN, the policy will be degenerated to LAYER2 mode; Now,
> > change it to get the next layer protocol to ensure that it worked in
> > BOND_XMIT_POLICY_LAYER34 mode.
> >
> > Signed-off-by: liujian <liujian56@...wei.com>
> > ---
> >   drivers/net/bonding/bond_main.c | 11 ++++++++---
> >   1 file changed, 8 insertions(+), 3 deletions(-)
> 
> 
> Nak
> Use BOND_XMIT_POLICY_ENCAP34 (encap3+4), that was one of the main
> reasons it was added.

Got it, thank you~