net/mlx5: E-Switch, move devcom init from TC to eswitch layer

Move the E-swtich devcom component management from TC layer to ESW
layer.

This refactoring places devcom lifecycle management at the appropriate
layer and prepares for SD LAG which needs devcom registration
independent of the TC/representor initialization.

Signed-off-by: Shay Drory <shayd@nvidia.com>
Reviewed-by: Mark Bloch <mbloch@nvidia.com>
Signed-off-by: Tariq Toukan <tariqt@nvidia.com>
Link: https://patch.msgid.link/20260531113954.395443-4-tariqt@nvidia.com
Signed-off-by: Jakub Kicinski <kuba@kernel.org>

diff --git a/drivers/net/ethernet/mellanox/mlx5/core/en_tc.c b/drivers/net/ethernet/mellanox/mlx5/core/en_tc.c
index a9001d1c902ff14f44e7b71ddab873066ce47028..3846c16c3138af258f247426b4fbce1eae57dc34 100644 (file)
--- a/drivers/net/ethernet/mellanox/mlx5/core/en_tc.c
+++ b/drivers/net/ethernet/mellanox/mlx5/core/en_tc.c
@@ -5394,8 +5394,6 @@ int mlx5e_tc_esw_init(struct mlx5_rep_uplink_priv *uplink_priv)
 {
        const size_t sz_enc_opts = sizeof(struct tunnel_match_enc_opts);
        u8 mapping_id[MLX5_SW_IMAGE_GUID_MAX_BYTES];
-       struct mlx5_devcom_match_attr attr = {};
-       struct netdev_phys_item_id ppid;
        struct mlx5e_rep_priv *rpriv;
        struct mapping_ctx *mapping;
        struct mlx5_eswitch *esw;
@@ -5456,14 +5454,6 @@ int mlx5e_tc_esw_init(struct mlx5_rep_uplink_priv *uplink_priv)
                goto err_action_counter;
        }
 
-       err = netif_get_port_parent_id(priv->netdev, &ppid, false);
-       if (!err) {
-               memcpy(&attr.key.buf, &ppid.id, ppid.id_len);
-               attr.flags = MLX5_DEVCOM_MATCH_FLAGS_NS;
-               attr.net = mlx5_core_net(esw->dev);
-               mlx5_esw_offloads_devcom_init(esw, &attr);
-       }
-
        return 0;
 
 err_action_counter:
@@ -5484,16 +5474,6 @@ err_tun_mapping:
 
 void mlx5e_tc_esw_cleanup(struct mlx5_rep_uplink_priv *uplink_priv)
 {
-       struct mlx5e_rep_priv *rpriv;
-       struct mlx5_eswitch *esw;
-       struct mlx5e_priv *priv;
-
-       rpriv = container_of(uplink_priv, struct mlx5e_rep_priv, uplink_priv);
-       priv = netdev_priv(rpriv->netdev);
-       esw = priv->mdev->priv.eswitch;
-
-       mlx5_esw_offloads_devcom_cleanup(esw);
-
        mlx5e_tc_tun_cleanup(uplink_priv->encap);
 
        mapping_destroy(uplink_priv->tunnel_enc_opts_mapping);

diff --git a/drivers/net/ethernet/mellanox/mlx5/core/eswitch_offloads.c b/drivers/net/ethernet/mellanox/mlx5/core/eswitch_offloads.c
index 189be11c4c39d796c8401ec948a01349291a646e..d9683d3ea0e7a461f86c2970cb0ba97e76915462 100644 (file)
--- a/drivers/net/ethernet/mellanox/mlx5/core/eswitch_offloads.c
+++ b/drivers/net/ethernet/mellanox/mlx5/core/eswitch_offloads.c
@@ -3866,6 +3866,7 @@ bool mlx5_esw_offloads_controller_valid(const struct mlx5_eswitch *esw, u32 cont
 int esw_offloads_enable(struct mlx5_eswitch *esw)
 {
        u8 mapping_id[MLX5_SW_IMAGE_GUID_MAX_BYTES];
+       struct mlx5_devcom_match_attr attr = {};
        struct mapping_ctx *reg_c0_obj_pool;
        struct mlx5_vport *vport;
        unsigned long i;
@@ -3926,6 +3927,10 @@ int esw_offloads_enable(struct mlx5_eswitch *esw)
        if (err)
                goto err_vports;
 
+       memcpy(attr.key.buf, mapping_id, id_len);
+       attr.flags = MLX5_DEVCOM_MATCH_FLAGS_NS;
+       attr.net = mlx5_core_net(esw->dev);
+       mlx5_esw_offloads_devcom_init(esw, &attr);
        return 0;
 
 err_vports:
@@ -3970,6 +3975,7 @@ static int esw_offloads_stop(struct mlx5_eswitch *esw,
 
 void esw_offloads_disable(struct mlx5_eswitch *esw)
 {
+       mlx5_esw_offloads_devcom_cleanup(esw);
        mlx5_eswitch_disable_pf_vf_vports(esw);
        mlx5_esw_offloads_rep_unload(esw, MLX5_VPORT_UPLINK);
        esw_set_passing_vport_metadata(esw, false);