Commit 25f150f4 authored by Roi Dayan's avatar Roi Dayan Committed by Saeed Mahameed
Browse files

net/mlx5e: Return -EOPNOTSUPP if more relevant when parsing tc actions 



Instead of returning -EINVAL.

Signed-off-by: default avatarRoi Dayan <roid@nvidia.com>
Reviewed-by: default avatarMaor Dickman <maord@nvidia.com>
Signed-off-by: default avatarSaeed Mahameed <saeedm@nvidia.com>
parent 97a8d29a

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

+3 −3
Original line number Diff line number Diff line
@@ -3400,7 +3400,7 @@ static int parse_tc_nic_actions(struct mlx5e_priv *priv, 
						   "device is not on same HW, can't offload");

				netdev_warn(priv->netdev, "device %s not on same HW, can't offload\n",

					    peer_dev->name);

				return -EINVAL;

				return -EOPNOTSUPP;

			}

			}

			break;
@@ -3410,7 +3410,7 @@ static int parse_tc_nic_actions(struct mlx5e_priv *priv, 
			if (mark & ~MLX5E_TC_FLOW_ID_MASK) {

				NL_SET_ERR_MSG_MOD(extack,

						   "Bad flow mark - only 16 bit is supported");

				return -EINVAL;

				return -EOPNOTSUPP;

			}



			nic_attr->flow_tag = mark;
@@ -3921,7 +3921,7 @@ static int parse_tc_fdb_actions(struct mlx5e_priv *priv, 
					    "devices %s %s not on same switch HW, can't offload forwarding\n",

					    priv->netdev->name,

					    out_dev->name);

				return -EINVAL;

				return -EOPNOTSUPP;

			}

			}

			break;