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: 34477072

This is an out of tree patch to fix VF link propagation
problem reported on UEK7. 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]>
(cherry picked from commit a272d4ffdc6e68fd6726b71b849eae2f2af012a7)
Signed-off-by: Jack Vogel <[email protected]>

Author: dickmanmaor

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

@@ -431,20 +431,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);
@@ -454,15 +448,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

@@ -3170,9 +3170,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);