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: <a2482fd4-05ae-c1a0-b12f-a6c8189746ea@cumulusnetworks.com>
Date:   Mon, 26 Mar 2018 17:16:12 +0300
From:   Nikolay Aleksandrov <nikolay@...ulusnetworks.com>
To:     Xin Long <lucien.xin@...il.com>,
        network dev <netdev@...r.kernel.org>
Cc:     davem@...emloft.net, Jiri Pirko <jiri@...nulli.us>,
        Jay Vosburgh <j.vosburgh@...il.com>,
        Veaceslav Falico <vfalico@...il.com>,
        Andy Gospodarek <andy@...yhouse.net>
Subject: Re: [PATCH net 0/3] bonding: a bunch of fixes for dev hwaddr sync in
 bond_enslave

On 25/03/18 20:16, Xin Long wrote:
> This patchset is mainly to fix a crash when adding vlan as slave of
> bond which is also the parent link in patch 2/3,  and also fix some
> err process problems in bond_enslave in patch 1/3 and 3/3.
> 
> Xin Long (3):
>    bonding: fix the err path for dev hwaddr sync in bond_enslave
>    bonding: move dev_mc_sync after master_upper_dev_link in bond_enslave
>    bonding: process the err returned by dev_set_allmulti properly in
>      bond_enslave
> 
>   drivers/net/bonding/bond_main.c | 73 +++++++++++++++++++++--------------------
>   1 file changed, 37 insertions(+), 36 deletions(-)
> 

I have no objections to the patches but you've missed to add the bonding maintainers
to the CC list (added now and removed the bouncing emails).

Thanks,
  Nik

Powered by blists - more mailing lists

Powered by Openwall GNU/*/Linux Powered by OpenVZ