From: Bodong Wang <bodong@mellanox.com>
Date: Thu, 17 Oct 2019 14:55:52 -0500
Subject: net/mlx5: E-Switch, Remove redundant warning when QoS enable failed
Patch-mainline: v5.7-rc1
Git-commit: a9814d7fde59332d54b244d55a7322b272a27430
References: jsc#SLE-15172
esw_vport_enable_qos can return error in cases below:
1. QoS is already enabled. Warnning is useless in this case.
2. Create scheduling element cmd failed. There is already a warning.
Remove the redundant warnning if esw_vport_enable_qos returns err.
Signed-off-by: Bodong Wang <bodong@mellanox.com>
Reviewed-by: Parav Pandit <parav@mellanox.com>
Signed-off-by: Saeed Mahameed <saeedm@mellanox.com>
Acked-by: Thomas Bogendoerfer <tbogendoerfer@suse.de>
---
drivers/net/ethernet/mellanox/mlx5/core/eswitch.c | 4 +---
1 file changed, 1 insertion(+), 3 deletions(-)
--- a/drivers/net/ethernet/mellanox/mlx5/core/eswitch.c
+++ b/drivers/net/ethernet/mellanox/mlx5/core/eswitch.c
@@ -1801,9 +1801,7 @@ static int esw_enable_vport(struct mlx5_
goto done;
/* Attach vport to the eswitch rate limiter */
- if (esw_vport_enable_qos(esw, vport, vport->info.max_rate,
- vport->qos.bw_share))
- esw_warn(esw->dev, "Failed to attach vport %d to eswitch rate limiter", vport_num);
+ esw_vport_enable_qos(esw, vport, vport->info.max_rate, vport->qos.bw_share);
/* Sync with current vport context */
vport->enabled_events = enabled_events;