net/mlx5: E-Switch, change VFs default admin state to auto in switchdev

This change change VFs default admin state to auto in switchdev causing the VF
link to follow the uplink link instead of the VF representor.

Signed-off-by: Maor Dickman <[email protected]>

Orabug: 34477073

This is an out of tree patch to fix VF link propagation
problem reported on UEK6u3. There is a requirement to propagate
link up/down events to all the VFs of a given PF. The driver
feature has been removed by vendor as regular driver update process.
Thus, to fulfil the requirement, a change has been proposed by nic
vendor where the representor link state is set to Up without confirming
with the device firmware. The change set the vport link-state to auto.
This change is been maintained as an out of tree patch.

Tested-by: Gerald Gibson <[email protected]>
Reviewed-by: Qing Huang <[email protected]>
Signed-off-by: Devesh Sharma <[email protected]>
Signed-off-by: Brian Maly <[email protected]>

Author: dickmanmaor

drivers/net/ethernet/mellanox/mlx5/core/en_rep.c

@@ -396,20 +396,14 @@ void mlx5e_remove_sqs_fwd_rules(struct mlx5e_priv *priv)
 static int mlx5e_rep_open(struct net_device *dev)
 {
 	struct mlx5e_priv *priv = netdev_priv(dev);
-	struct mlx5e_rep_priv *rpriv = priv->ppriv;
-	struct mlx5_eswitch_rep *rep = rpriv->rep;
 	int err;
 
 	mutex_lock(&priv->state_lock);
 	err = mlx5e_open_locked(dev);
 	if (err)
 		goto unlock;
 
-	if (!mlx5_modify_vport_admin_state(priv->mdev,
-					   MLX5_VPORT_STATE_OP_MOD_ESW_VPORT,
-					   rep->vport, 1,
-					   MLX5_VPORT_ADMIN_STATE_UP))
-		netif_carrier_on(dev);
+	netif_carrier_on(dev);
 
 unlock:
 	mutex_unlock(&priv->state_lock);
@@ -419,15 +413,9 @@ static int mlx5e_rep_open(struct net_device *dev)
 static int mlx5e_rep_close(struct net_device *dev)
 {
 	struct mlx5e_priv *priv = netdev_priv(dev);
-	struct mlx5e_rep_priv *rpriv = priv->ppriv;
-	struct mlx5_eswitch_rep *rep = rpriv->rep;
 	int ret;
 
 	mutex_lock(&priv->state_lock);
-	mlx5_modify_vport_admin_state(priv->mdev,
-				      MLX5_VPORT_STATE_OP_MOD_ESW_VPORT,
-				      rep->vport, 1,
-				      MLX5_VPORT_ADMIN_STATE_DOWN);
 	ret = mlx5e_close_locked(dev);
 	mutex_unlock(&priv->state_lock);
 	return ret;

drivers/net/ethernet/mellanox/mlx5/core/eswitch_offloads.c

@@ -2347,9 +2347,8 @@ int esw_offloads_enable(struct mlx5_eswitch *esw)
 	if (err)
 		goto err_steering_init;
 
-	/* Representor will control the vport link state */
 	mlx5_esw_for_each_vf_vport(esw, i, vport, esw->esw_funcs.num_vfs)
-		vport->info.link_state = MLX5_VPORT_ADMIN_STATE_DOWN;
+		vport->info.link_state = MLX5_VPORT_ADMIN_STATE_AUTO;
 
 	/* Uplink vport rep must load first. */
 	err = esw_offloads_load_rep(esw, MLX5_VPORT_UPLINK);