netdev - Re: [net-next v7 1/3] net: bonding: add broadcast

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: <9829a19e-0f19-46e4-a007-0e943f1fcdb8@redhat.com>
Date: Thu, 26 Jun 2025 12:49:18 +0200
From: Paolo Abeni <pabeni@...hat.com>
To: Tonghao Zhang <tonghao@...aicloud.com>, netdev@...r.kernel.org
Cc: Jay Vosburgh <jv@...sburgh.net>, "David S. Miller" <davem@...emloft.net>,
 Eric Dumazet <edumazet@...gle.com>, Jakub Kicinski <kuba@...nel.org>,
 Simon Horman <horms@...nel.org>, Jonathan Corbet <corbet@....net>,
 Andrew Lunn <andrew+netdev@...n.ch>, Steven Rostedt <rostedt@...dmis.org>,
 Masami Hiramatsu <mhiramat@...nel.org>,
 Mathieu Desnoyers <mathieu.desnoyers@...icios.com>,
 Nikolay Aleksandrov <razor@...ckwall.org>,
 Zengbing Tu <tuzengbing@...iglobal.com>
Subject: Re: [net-next v7 1/3] net: bonding: add broadcast_neighbor option for
 802.3ad

On 6/24/25 4:18 AM, Tonghao Zhang wrote:
> @@ -893,6 +909,8 @@ static int bond_option_mode_set(struct bonding *bond,
>  	/* don't cache arp_validate between modes */
>  	bond->params.arp_validate = BOND_ARP_VALIDATE_NONE;
>  	bond->params.mode = newval->value;
> +	/* cleanup broadcast_neighbor when changing mode */
> +	bond->params.broadcast_neighbor = 0;

This does not look enough: it looks like 'bond_bcast_neigh_enabled'
accounting is ignored, and should instead adjusted accordingly.

Side notes: please include per patch changelog to help reviewers, and
when replying to reviewers comment, please properly trim the unrelevant
message part and separate your text from the quoted one.

Thanks,

Paolo