diff options
| author | Or Gerlitz <ogerlitz@mellanox.com> | 2018-12-09 16:15:23 +0100 |
|---|---|---|
| committer | Saeed Mahameed <saeedm@mellanox.com> | 2018-12-13 10:24:44 +0100 |
| commit | d14f6f2a84326dd2751962167b2293fc91751ff8 (patch) | |
| tree | 07c9ea3ea2568c68296960d9706453fa8afdffbb /drivers | |
| parent | net/mlx5e: Err if asked to mirror a goto chain tc eswitch rule (diff) | |
| download | linux-d14f6f2a84326dd2751962167b2293fc91751ff8.tar.xz linux-d14f6f2a84326dd2751962167b2293fc91751ff8.zip | |
net/mlx5e: Avoid overriding the user provided priority for offloaded tc rules
Just a leftover which was wrongly left there, remove it while spawning
a message to suggest firmware upgrade.
Fixes: bf07aa730a04 ('net/mlx5e: Support offloading tc priorities and chains for eswitch flows')
Signed-off-by: Or Gerlitz <ogerlitz@mellanox.com>
Reviewed-by: Paul Blakey <paulb@mellanox.com>
Signed-off-by: Saeed Mahameed <saeedm@mellanox.com>
Diffstat (limited to 'drivers')
| -rw-r--r-- | drivers/net/ethernet/mellanox/mlx5/core/en_tc.c | 9 |
1 files changed, 4 insertions, 5 deletions
diff --git a/drivers/net/ethernet/mellanox/mlx5/core/en_tc.c b/drivers/net/ethernet/mellanox/mlx5/core/en_tc.c index 1f01c905a042..e6170ddf2cc4 100644 --- a/drivers/net/ethernet/mellanox/mlx5/core/en_tc.c +++ b/drivers/net/ethernet/mellanox/mlx5/core/en_tc.c @@ -907,11 +907,10 @@ mlx5e_tc_add_fdb_flow(struct mlx5e_priv *priv, struct mlx5e_priv *out_priv; int err = 0, encap_err = 0; - /* if prios are not supported, keep the old behaviour of using same prio - * for all offloaded rules. - */ - if (!mlx5_eswitch_prios_supported(esw)) - attr->prio = 1; + if (!mlx5_eswitch_prios_supported(esw) && attr->prio != 1) { + NL_SET_ERR_MSG(extack, "E-switch priorities unsupported, upgrade FW"); + return -EOPNOTSUPP; + } if (attr->chain > max_chain) { NL_SET_ERR_MSG(extack, "Requested chain is out of supported range"); |