netdev - Re: [PATCH net-next 1/3] net/mlx5: Enable 8 ports LAG

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: Mon, 13 May 2024 09:59:27 +0100
From: Simon Horman <horms@...nel.org>
To: Tariq Toukan <tariqt@...dia.com>
Cc: "David S. Miller" <davem@...emloft.net>,
	Jakub Kicinski <kuba@...nel.org>, Paolo Abeni <pabeni@...hat.com>,
	Eric Dumazet <edumazet@...gle.com>, netdev@...r.kernel.org,
	Saeed Mahameed <saeedm@...dia.com>, Gal Pressman <gal@...dia.com>,
	Leon Romanovsky <leonro@...dia.com>, Shay Drory <shayd@...dia.com>,
	Mark Bloch <mbloch@...dia.com>
Subject: Re: [PATCH net-next 1/3] net/mlx5: Enable 8 ports LAG

On Sun, May 12, 2024 at 03:43:03PM +0300, Tariq Toukan wrote:
> From: Shay Drory <shayd@...dia.com>
> 
> This patch adds to mlx5 drivers support for 8 ports HCAs.
> Starting with ConnectX-8 HCAs with 8 ports are possible.
> 
> As most driver parts aren't affected by such configuration most driver
> code is unchanged.
> 
> Specially the only affected areas are:
> - Lag
> - Multiport E-Switch
> - Single FDB E-Switch
> 
> All of the above are already factored in generic way, and LAG and VF LAG
> are tested, so all that left is to change a #define and remove checks
> which are no longer needed.
> However, Multiport E-Switch is not tested yet, so it is left untouched.
> 
> This patch will allow to create hardware LAG/VF LAG when all 8 ports are
> added to the same bond device.
> 
> for example, In order to activate the hardware lag a user can execute
> the following:
> 
> ip link add bond0 type bond
> ip link set bond0 type bond miimon 100 mode 2
> ip link set eth2 master bond0
> ip link set eth3 master bond0
> ip link set eth4 master bond0
> ip link set eth5 master bond0
> ip link set eth6 master bond0
> ip link set eth7 master bond0
> ip link set eth8 master bond0
> ip link set eth9 master bond0
> 
> Where eth2, eth3, eth4, eth5, eth6, eth7, eth8 and eth9 are the PFs of
> the same HCA.
> 
> Signed-off-by: Shay Drory <shayd@...dia.com>
> Reviewed-by: Mark Bloch <mbloch@...dia.com>
> Signed-off-by: Tariq Toukan <tariqt@...dia.com>

Reviewed-by: Simon Horman <horms@...nel.org>