mirror of
https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git
synced 2026-05-16 14:51:51 -04:00
mlx5: convert to ndo_set_rx_mode_async
Convert mlx5 from ndo_set_rx_mode to ndo_set_rx_mode_async. The
driver's mlx5e_set_rx_mode now receives uc/mc snapshots and calls
mlx5e_fs_set_rx_mode_work directly instead of queueing work.
mlx5e_sync_netdev_addr and mlx5e_handle_netdev_addr now take
explicit uc/mc list parameters and iterate with
netdev_hw_addr_list_for_each instead of netdev_for_each_{uc,mc}_addr.
Fallback to netdev's uc/mc in a few places and grab addr lock.
Cc: Saeed Mahameed <saeedm@nvidia.com>
Cc: Tariq Toukan <tariqt@nvidia.com>
Cc: Cosmin Ratiu <cratiu@nvidia.com>
Reviewed-by: Aleksandr Loktionov <aleksandr.loktionov@intel.com>
Signed-off-by: Stanislav Fomichev <sdf@fomichev.me>
Link: https://patch.msgid.link/20260416185712.2155425-7-sdf@fomichev.me
Signed-off-by: Paolo Abeni <pabeni@redhat.com>
This commit is contained in:
committed by
Paolo Abeni
parent
60dd9781e9
commit
5cf06fbdaf
@@ -201,7 +201,10 @@ int mlx5e_add_vlan_trap(struct mlx5e_flow_steering *fs, int trap_id, int tir_nu
|
||||
void mlx5e_remove_vlan_trap(struct mlx5e_flow_steering *fs);
|
||||
int mlx5e_add_mac_trap(struct mlx5e_flow_steering *fs, int trap_id, int tir_num);
|
||||
void mlx5e_remove_mac_trap(struct mlx5e_flow_steering *fs);
|
||||
void mlx5e_fs_set_rx_mode_work(struct mlx5e_flow_steering *fs, struct net_device *netdev);
|
||||
void mlx5e_fs_set_rx_mode_work(struct mlx5e_flow_steering *fs,
|
||||
struct net_device *netdev,
|
||||
struct netdev_hw_addr_list *uc,
|
||||
struct netdev_hw_addr_list *mc);
|
||||
int mlx5e_fs_vlan_rx_add_vid(struct mlx5e_flow_steering *fs,
|
||||
struct net_device *netdev,
|
||||
__be16 proto, u16 vid);
|
||||
|
||||
@@ -609,20 +609,26 @@ static void mlx5e_execute_l2_action(struct mlx5e_flow_steering *fs,
|
||||
}
|
||||
|
||||
static void mlx5e_sync_netdev_addr(struct mlx5e_flow_steering *fs,
|
||||
struct net_device *netdev)
|
||||
struct net_device *netdev,
|
||||
struct netdev_hw_addr_list *uc,
|
||||
struct netdev_hw_addr_list *mc)
|
||||
{
|
||||
struct netdev_hw_addr *ha;
|
||||
|
||||
netif_addr_lock_bh(netdev);
|
||||
if (!uc || !mc) {
|
||||
netif_addr_lock_bh(netdev);
|
||||
mlx5e_sync_netdev_addr(fs, netdev, &netdev->uc, &netdev->mc);
|
||||
netif_addr_unlock_bh(netdev);
|
||||
return;
|
||||
}
|
||||
|
||||
mlx5e_add_l2_to_hash(fs->l2.netdev_uc, netdev->dev_addr);
|
||||
netdev_for_each_uc_addr(ha, netdev)
|
||||
|
||||
netdev_hw_addr_list_for_each(ha, uc)
|
||||
mlx5e_add_l2_to_hash(fs->l2.netdev_uc, ha->addr);
|
||||
|
||||
netdev_for_each_mc_addr(ha, netdev)
|
||||
netdev_hw_addr_list_for_each(ha, mc)
|
||||
mlx5e_add_l2_to_hash(fs->l2.netdev_mc, ha->addr);
|
||||
|
||||
netif_addr_unlock_bh(netdev);
|
||||
}
|
||||
|
||||
static void mlx5e_fill_addr_array(struct mlx5e_flow_steering *fs, int list_type,
|
||||
@@ -724,7 +730,9 @@ static void mlx5e_apply_netdev_addr(struct mlx5e_flow_steering *fs)
|
||||
}
|
||||
|
||||
static void mlx5e_handle_netdev_addr(struct mlx5e_flow_steering *fs,
|
||||
struct net_device *netdev)
|
||||
struct net_device *netdev,
|
||||
struct netdev_hw_addr_list *uc,
|
||||
struct netdev_hw_addr_list *mc)
|
||||
{
|
||||
struct mlx5e_l2_hash_node *hn;
|
||||
struct hlist_node *tmp;
|
||||
@@ -736,7 +744,7 @@ static void mlx5e_handle_netdev_addr(struct mlx5e_flow_steering *fs,
|
||||
hn->action = MLX5E_ACTION_DEL;
|
||||
|
||||
if (fs->state_destroy)
|
||||
mlx5e_sync_netdev_addr(fs, netdev);
|
||||
mlx5e_sync_netdev_addr(fs, netdev, uc, mc);
|
||||
|
||||
mlx5e_apply_netdev_addr(fs);
|
||||
}
|
||||
@@ -820,13 +828,15 @@ static void mlx5e_destroy_promisc_table(struct mlx5e_flow_steering *fs)
|
||||
}
|
||||
|
||||
void mlx5e_fs_set_rx_mode_work(struct mlx5e_flow_steering *fs,
|
||||
struct net_device *netdev)
|
||||
struct net_device *netdev,
|
||||
struct netdev_hw_addr_list *uc,
|
||||
struct netdev_hw_addr_list *mc)
|
||||
{
|
||||
struct mlx5e_priv *priv = netdev_priv(netdev);
|
||||
struct mlx5e_l2_table *ea = &fs->l2;
|
||||
|
||||
if (mlx5e_is_uplink_rep(priv)) {
|
||||
mlx5e_handle_netdev_addr(fs, netdev);
|
||||
mlx5e_handle_netdev_addr(fs, netdev, uc, mc);
|
||||
goto update_vport_context;
|
||||
}
|
||||
|
||||
@@ -856,7 +866,7 @@ void mlx5e_fs_set_rx_mode_work(struct mlx5e_flow_steering *fs,
|
||||
if (enable_broadcast)
|
||||
mlx5e_add_l2_flow_rule(fs, &ea->broadcast, MLX5E_FULLMATCH);
|
||||
|
||||
mlx5e_handle_netdev_addr(fs, netdev);
|
||||
mlx5e_handle_netdev_addr(fs, netdev, uc, mc);
|
||||
|
||||
if (disable_broadcast)
|
||||
mlx5e_del_l2_flow_rule(fs, &ea->broadcast);
|
||||
|
||||
@@ -4145,11 +4145,13 @@ static void mlx5e_nic_set_rx_mode(struct mlx5e_priv *priv)
|
||||
queue_work(priv->wq, &priv->set_rx_mode_work);
|
||||
}
|
||||
|
||||
static void mlx5e_set_rx_mode(struct net_device *dev)
|
||||
static void mlx5e_set_rx_mode(struct net_device *dev,
|
||||
struct netdev_hw_addr_list *uc,
|
||||
struct netdev_hw_addr_list *mc)
|
||||
{
|
||||
struct mlx5e_priv *priv = netdev_priv(dev);
|
||||
|
||||
mlx5e_nic_set_rx_mode(priv);
|
||||
mlx5e_fs_set_rx_mode_work(priv->fs, dev, uc, mc);
|
||||
}
|
||||
|
||||
static int mlx5e_set_mac(struct net_device *netdev, void *addr)
|
||||
@@ -5324,7 +5326,7 @@ const struct net_device_ops mlx5e_netdev_ops = {
|
||||
.ndo_setup_tc = mlx5e_setup_tc,
|
||||
.ndo_select_queue = mlx5e_select_queue,
|
||||
.ndo_get_stats64 = mlx5e_get_stats,
|
||||
.ndo_set_rx_mode = mlx5e_set_rx_mode,
|
||||
.ndo_set_rx_mode_async = mlx5e_set_rx_mode,
|
||||
.ndo_set_mac_address = mlx5e_set_mac,
|
||||
.ndo_vlan_rx_add_vid = mlx5e_vlan_rx_add_vid,
|
||||
.ndo_vlan_rx_kill_vid = mlx5e_vlan_rx_kill_vid,
|
||||
@@ -6309,8 +6311,11 @@ void mlx5e_set_rx_mode_work(struct work_struct *work)
|
||||
{
|
||||
struct mlx5e_priv *priv = container_of(work, struct mlx5e_priv,
|
||||
set_rx_mode_work);
|
||||
struct net_device *dev = priv->netdev;
|
||||
|
||||
return mlx5e_fs_set_rx_mode_work(priv->fs, priv->netdev);
|
||||
netdev_lock_ops(dev);
|
||||
mlx5e_fs_set_rx_mode_work(priv->fs, dev, NULL, NULL);
|
||||
netdev_unlock_ops(dev);
|
||||
}
|
||||
|
||||
/* mlx5e generic netdev management API (move to en_common.c) */
|
||||
|
||||
Reference in New Issue
Block a user