[v2] net/mlx5: fix RSS level validation

Message ID 1529566255-14001-1-git-send-email-rasland@mellanox.com (mailing list archive)
State Superseded, archived
Headers
Series [v2] net/mlx5: fix RSS level validation |

Checks

Context Check Description
ci/checkpatch success coding style OK
ci/Intel-compilation success Compilation OK

Commit Message

Raslan Darawsheh June 21, 2018, 7:30 a.m. UTC
  When setting the level in rss action it's checking for the value
stored in the parser which is set to 0 by default.

This change the check to be for the requested action instead.

Fixes: d4a40518 ("net/mlx5: support tunnel RSS level")

Cc: stable@dpdk.org
Signed-off-by: Raslan Darawsheh <rasland@mellanox.com>
Acked-by: Shahaf Shuler <shahafs@mellanox.com>

---
changes in v2:
	commit massage reword.

---
---
 drivers/net/mlx5/mlx5_flow.c | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)
  

Patch

diff --git a/drivers/net/mlx5/mlx5_flow.c b/drivers/net/mlx5/mlx5_flow.c
index 994be05..40df633 100644
--- a/drivers/net/mlx5/mlx5_flow.c
+++ b/drivers/net/mlx5/mlx5_flow.c
@@ -738,7 +738,7 @@  mlx5_flow_convert_actions(struct rte_eth_dev *dev,
 				return -rte_errno;
 			}
 #ifndef HAVE_IBV_DEVICE_TUNNEL_SUPPORT
-			if (parser->rss_conf.level > 1) {
+			if (rss->level > 1) {
 				rte_flow_error_set(error, EINVAL,
 						   RTE_FLOW_ERROR_TYPE_ACTION,
 						   actions,
@@ -747,7 +747,7 @@  mlx5_flow_convert_actions(struct rte_eth_dev *dev,
 				return -rte_errno;
 			}
 #endif
-			if (parser->rss_conf.level > 2) {
+			if (rss->level > 2) {
 				rte_flow_error_set(error, EINVAL,
 						   RTE_FLOW_ERROR_TYPE_ACTION,
 						   actions,