Re: [PATCH net] net/mlx5: Bridge, fix bogus MDB offload errors

[sashiko-bot@kernel.org]

> From: Gal Pressman <gal@nvidia.com>
> 
> MDB add operations are queued on the switchdev deferred queue
> (SWITCHDEV_F_DEFER), while vlan object deletions are handled
> synchronously.
> A 'bridge vlan del' can overtake an MDB add that is still queued, and
> the vlan is already gone by the time the driver is called.
> 
> The two operations are independent in practice: in my test, the MDB add
> is triggered by assigning the VF an IPv6 address (hence joining a
> solicited-node multicast group), while another process deletes the vlan
> from its representor.
> 
> Racing 'bridge mdb add' against concurrent 'bridge vlan del' could
> result in:
> [ ... ]
> 
> Fixes: 70f0302b3f20 ("net/mlx5: Bridge, implement mdb offload")
> Signed-off-by: Gal Pressman <gal@nvidia.com>
> Reviewed-by: Cosmin Ratiu <cratiu@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/20261021122115.3737284-1-tariqt@nvidia.com?part=1