Commit 86f9453c authored by Bodong Wang's avatar Bodong Wang Committed by Saeed Mahameed
Browse files

net/mlx5: E-Switch, Remove redundant check of eswitch manager cap 



esw_vport_create_legacy_acl_tables bails out immediately for eswitch
manager, hence remove all the check of esw manager cap after.

Signed-off-by: default avatarBodong Wang <bodong@mellanox.com>
Reviewed-by: default avatarParav Pandit <parav@mellanox.com>
Signed-off-by: default avatarSaeed Mahameed <saeedm@mellanox.com>
parent 1d343579

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

+2 −4
Original line number Diff line number Diff line
@@ -1707,8 +1707,7 @@ static int esw_vport_create_legacy_acl_tables(struct mlx5_eswitch *esw, 
	if (mlx5_esw_is_manager_vport(esw, vport->vport))

		return 0;



	if (!mlx5_esw_is_manager_vport(esw, vport->vport) &&

	    MLX5_CAP_ESW_INGRESS_ACL(esw->dev, flow_counter)) {

	if (MLX5_CAP_ESW_INGRESS_ACL(esw->dev, flow_counter)) {

		vport->ingress.legacy.drop_counter = mlx5_fc_create(esw->dev, false);

		if (IS_ERR(vport->ingress.legacy.drop_counter)) {

			esw_warn(esw->dev,
@@ -1722,8 +1721,7 @@ static int esw_vport_create_legacy_acl_tables(struct mlx5_eswitch *esw, 
	if (ret)

		goto ingress_err;



	if (!mlx5_esw_is_manager_vport(esw, vport->vport) &&

	    MLX5_CAP_ESW_EGRESS_ACL(esw->dev, flow_counter)) {

	if (MLX5_CAP_ESW_EGRESS_ACL(esw->dev, flow_counter)) {

		vport->egress.legacy.drop_counter = mlx5_fc_create(esw->dev, false);

		if (IS_ERR(vport->egress.legacy.drop_counter)) {

			esw_warn(esw->dev,

CRA Git | Maintained and supported by SUSTech CRA and CCSE