Adding the maintainers,
Tuesday, June 19, 2018 4:09 PM, Raslan Darawsheh:
>Subject: [PATCH] net/mlx5: fix Rss level validation
Rss -> RSS
>
>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 insted.
this -> This
instead -> instead
>
>Fixes: d4a40518 ("net/mlx5: support tunnel RSS level")
>
>Cc: sta...@dpdk.org
>Signed-off-by: Raslan Darawsheh <rasl...@mellanox.com>
>---
> drivers/net/mlx5/mlx5_flow.c | 4 ++--
> 1 file changed, 2 insertions(+), 2 deletions(-)
>
>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,
>--
The above changes (if they are the only one) can be address by me when I apply
the patch.
Will wait few days for the maintainers comments.
Acked-by: Shahaf Shuler <shah...@mellanox.com>
>2.7.4
