Re: [PATCH net-next 13/15] net/mlx5: E-switch, drop master egress rule under a SW vport LAG

[sashiko-bot@kernel.org]

> From: Shay Drory <shayd@nvidia.com>
> 
> VPORT_LAG steers all traffic of a LAG slave to the LAG master. The
> eswitch master egress rule does the opposite direction: it bounces
> traffic that reached the master's manager vport into the slave eswitch.
> With a FW vport LAG the FW steers that bounced packet to the LAG master,
> so the two coexist.
> 
> The software vport LAG emulation cannot. It is a catch-all rule in the
> slave eswitch's egress ACL that forwards to the master's manager vport,
> while the master egress rule redirects the packet after the FDB - past
> the point where the slave egress rule is evaluated - so the bounced
> packet never reaches it.
> 
> The two therefore cannot coexist: drop the master egress rule when the
> [ ... ]
> 
> Signed-off-by: Shay Drory <shayd@nvidia.com>
> Reviewed-by: Mark Bloch <mbloch@nvidia.com>
> Reviewed-by: Moshe Shemesh <moshe@nvidia.com>
> Signed-off-by: Tariq Toukan <tariqt@nvidia.com>

Sashiko has reviewed this patch and found no issues.

-- 
Sashiko AI review · https://sashiko.dev/#/patchset/20261103171009.563034-1-tariqt@nvidia.com?part=13