]> git.ipfire.org Git - thirdparty/kernel/linux.git/commitdiff
net/mlx5: MPFS, add support for dynamic enable/disable
authorSaeed Mahameed <saeedm@nvidia.com>
Sat, 8 Nov 2025 07:04:03 +0000 (23:04 -0800)
committerPaolo Abeni <pabeni@redhat.com>
Tue, 11 Nov 2025 12:17:54 +0000 (13:17 +0100)
MPFS (Multi PF Switch) is enabled by default in Multi-Host environments,
the driver keeps a list of desired unicast mac addresses of all vports
(vfs/Sfs) and applied to HW via L2_table FW command.

Add API to dynamically apply the list of MACs to HW when needed for next
patches, to utilize this new API in devlink eswitch active/in-active uAPI.

Signed-off-by: Saeed Mahameed <saeedm@nvidia.com>
Signed-off-by: Adithya Jayachandran <ajayachandra@nvidia.com>
Reviewed-by: Jiri Pirko <jiri@nvidia.com>
Link: https://patch.msgid.link/20251108070404.1551708-3-saeed@kernel.org
Signed-off-by: Paolo Abeni <pabeni@redhat.com>
drivers/net/ethernet/mellanox/mlx5/core/lib/mpfs.c
drivers/net/ethernet/mellanox/mlx5/core/lib/mpfs.h

index 4450091e181a104bd8bc122f845735a470b68aa0..99fb7a53add08ba6af042c645606fb9c001e669e 100644 (file)
@@ -65,13 +65,14 @@ static int del_l2table_entry_cmd(struct mlx5_core_dev *dev, u32 index)
 /* UC L2 table hash node */
 struct l2table_node {
        struct l2addr_node node;
-       u32                index; /* index in HW l2 table */
+       int                index; /* index in HW l2 table */
        int                ref_count;
 };
 
 struct mlx5_mpfs {
        struct hlist_head    hash[MLX5_L2_ADDR_HASH_SIZE];
        struct mutex         lock; /* Synchronize l2 table access */
+       bool                 enabled;
        u32                  size;
        unsigned long        *bitmap;
 };
@@ -114,6 +115,8 @@ int mlx5_mpfs_init(struct mlx5_core_dev *dev)
                return -ENOMEM;
        }
 
+       mpfs->enabled = true;
+
        dev->priv.mpfs = mpfs;
        return 0;
 }
@@ -135,7 +138,7 @@ int mlx5_mpfs_add_mac(struct mlx5_core_dev *dev, u8 *mac)
        struct mlx5_mpfs *mpfs = dev->priv.mpfs;
        struct l2table_node *l2addr;
        int err = 0;
-       u32 index;
+       int index;
 
        if (!mpfs)
                return 0;
@@ -148,30 +151,34 @@ int mlx5_mpfs_add_mac(struct mlx5_core_dev *dev, u8 *mac)
                goto out;
        }
 
-       err = alloc_l2table_index(mpfs, &index);
-       if (err)
-               goto out;
-
        l2addr = l2addr_hash_add(mpfs->hash, mac, struct l2table_node, GFP_KERNEL);
        if (!l2addr) {
                err = -ENOMEM;
-               goto hash_add_err;
+               goto out;
        }
 
-       err = set_l2table_entry_cmd(dev, index, mac);
-       if (err)
-               goto set_table_entry_err;
+       index = -1;
+
+       if (mpfs->enabled) {
+               err = alloc_l2table_index(mpfs, &index);
+               if (err)
+                       goto hash_del;
+               err = set_l2table_entry_cmd(dev, index, mac);
+               if (err)
+                       goto free_l2table_index;
+               mlx5_core_dbg(dev, "MPFS entry %pM, set @index (%d)\n",
+                             l2addr->node.addr, l2addr->index);
+       }
 
        l2addr->index = index;
        l2addr->ref_count = 1;
 
        mlx5_core_dbg(dev, "MPFS mac added %pM, index (%d)\n", mac, index);
        goto out;
-
-set_table_entry_err:
-       l2addr_hash_del(l2addr);
-hash_add_err:
+free_l2table_index:
        free_l2table_index(mpfs, index);
+hash_del:
+       l2addr_hash_del(l2addr);
 out:
        mutex_unlock(&mpfs->lock);
        return err;
@@ -183,7 +190,7 @@ int mlx5_mpfs_del_mac(struct mlx5_core_dev *dev, u8 *mac)
        struct mlx5_mpfs *mpfs = dev->priv.mpfs;
        struct l2table_node *l2addr;
        int err = 0;
-       u32 index;
+       int index;
 
        if (!mpfs)
                return 0;
@@ -200,12 +207,87 @@ int mlx5_mpfs_del_mac(struct mlx5_core_dev *dev, u8 *mac)
                goto unlock;
 
        index = l2addr->index;
-       del_l2table_entry_cmd(dev, index);
+       if (index >= 0) {
+               del_l2table_entry_cmd(dev, index);
+               free_l2table_index(mpfs, index);
+               mlx5_core_dbg(dev, "MPFS entry %pM, deleted @index (%d)\n",
+                             mac, index);
+       }
        l2addr_hash_del(l2addr);
-       free_l2table_index(mpfs, index);
        mlx5_core_dbg(dev, "MPFS mac deleted %pM, index (%d)\n", mac, index);
 unlock:
        mutex_unlock(&mpfs->lock);
        return err;
 }
 EXPORT_SYMBOL(mlx5_mpfs_del_mac);
+
+int mlx5_mpfs_enable(struct mlx5_core_dev *dev)
+{
+       struct mlx5_mpfs *mpfs = dev->priv.mpfs;
+       struct l2table_node *l2addr;
+       struct hlist_node *n;
+       int err = 0, i;
+
+       if (!mpfs)
+               return -ENODEV;
+
+       mutex_lock(&mpfs->lock);
+       if (mpfs->enabled)
+               goto out;
+       mpfs->enabled = true;
+       mlx5_core_dbg(dev, "MPFS enabling mpfs\n");
+
+       mlx5_mpfs_foreach(l2addr, n, mpfs, i) {
+               u32 index;
+
+               err = alloc_l2table_index(mpfs, &index);
+               if (err) {
+                       mlx5_core_err(dev, "Failed to allocated MPFS index for %pM, err(%d)\n",
+                                     l2addr->node.addr, err);
+                       goto out;
+               }
+
+               err = set_l2table_entry_cmd(dev, index, l2addr->node.addr);
+               if (err) {
+                       mlx5_core_err(dev, "Failed to set MPFS l2table entry for %pM index=%d, err(%d)\n",
+                                     l2addr->node.addr, index, err);
+                       free_l2table_index(mpfs, index);
+                       goto out;
+               }
+
+               l2addr->index = index;
+               mlx5_core_dbg(dev, "MPFS entry %pM, set @index (%d)\n",
+                             l2addr->node.addr, l2addr->index);
+       }
+out:
+       mutex_unlock(&mpfs->lock);
+       return err;
+}
+
+void mlx5_mpfs_disable(struct mlx5_core_dev *dev)
+{
+       struct mlx5_mpfs *mpfs = dev->priv.mpfs;
+       struct l2table_node *l2addr;
+       struct hlist_node *n;
+       int i;
+
+       if (!mpfs)
+               return;
+
+       mutex_lock(&mpfs->lock);
+       if (!mpfs->enabled)
+               goto unlock;
+       mlx5_mpfs_foreach(l2addr, n, mpfs, i) {
+               if (l2addr->index < 0)
+                       continue;
+               del_l2table_entry_cmd(dev, l2addr->index);
+               free_l2table_index(mpfs, l2addr->index);
+               mlx5_core_dbg(dev, "MPFS entry %pM, deleted @index (%d)\n",
+                             l2addr->node.addr, l2addr->index);
+               l2addr->index = -1;
+       }
+       mpfs->enabled = false;
+       mlx5_core_dbg(dev, "MPFS disabled\n");
+unlock:
+       mutex_unlock(&mpfs->lock);
+}
index 4a293542a7aa1a0b6f9fa20bc8c2d50e15e7c2c0..9c63838ce1f3297ef11d74bffa57be6a8bcc5fef 100644 (file)
@@ -45,6 +45,10 @@ struct l2addr_node {
        u8                addr[ETH_ALEN];
 };
 
+#define mlx5_mpfs_foreach(hs, tmp, mpfs, i) \
+       for (i = 0; i < MLX5_L2_ADDR_HASH_SIZE; i++) \
+               hlist_for_each_entry_safe(hs, tmp, &(mpfs)->hash[i], node.hlist)
+
 #define for_each_l2hash_node(hn, tmp, hash, i) \
        for (i = 0; i < MLX5_L2_ADDR_HASH_SIZE; i++) \
                hlist_for_each_entry_safe(hn, tmp, &(hash)[i], hlist)
@@ -82,11 +86,16 @@ struct l2addr_node {
 })
 
 #ifdef CONFIG_MLX5_MPFS
+struct mlx5_core_dev;
 int  mlx5_mpfs_init(struct mlx5_core_dev *dev);
 void mlx5_mpfs_cleanup(struct mlx5_core_dev *dev);
+int mlx5_mpfs_enable(struct mlx5_core_dev *dev);
+void mlx5_mpfs_disable(struct mlx5_core_dev *dev);
 #else /* #ifndef CONFIG_MLX5_MPFS */
 static inline int  mlx5_mpfs_init(struct mlx5_core_dev *dev) { return 0; }
 static inline void mlx5_mpfs_cleanup(struct mlx5_core_dev *dev) {}
+static inline int mlx5_mpfs_enable(struct mlx5_core_dev *dev) { return 0; }
+static inline void mlx5_mpfs_disable(struct mlx5_core_dev *dev) {}
 #endif
 
 #endif