[dpdk-dev] [PATCH] net/i40e: fix hash enable issue in RSS flow

[Shougang Wang]

This patch fixes the issue that failed to create an RSS rule with type
L2-payload.

Fixes: feaae285b342 ("net/i40e: support hash configuration in RSS flow")

Signed-off-by: Shougang Wang <shougangx.w...@intel.com>
---
 drivers/net/i40e/i40e_flow.c | 14 ++++++--------
 1 file changed, 6 insertions(+), 8 deletions(-)

diff --git a/drivers/net/i40e/i40e_flow.c b/drivers/net/i40e/i40e_flow.c
index 7e64ae53a..f5f2f0d5d 100644
--- a/drivers/net/i40e/i40e_flow.c
+++ b/drivers/net/i40e/i40e_flow.c
@@ -4511,6 +4511,7 @@ i40e_flow_parse_rss_pattern(__rte_unused struct 
rte_eth_dev *dev,
                { pattern_fdir_ipv6_tcp, ETH_RSS_NONFRAG_IPV6_TCP },
                { pattern_fdir_ipv6_udp, ETH_RSS_NONFRAG_IPV6_UDP },
                { pattern_fdir_ipv6_sctp, ETH_RSS_NONFRAG_IPV6_SCTP },
+               { pattern_ethertype, ETH_RSS_L2_PAYLOAD },
                { pattern_fdir_ipv6_esp, ETH_RSS_ESP },
                { pattern_fdir_ipv6_udp_esp, ETH_RSS_ESP },
        };
@@ -4544,8 +4545,7 @@ i40e_flow_parse_rss_pattern(__rte_unused struct 
rte_eth_dev *dev,
                if (i40e_match_pattern(i40e_rss_pctype_patterns[i].item_array,
                                        items)) {
                        p_info->types = i40e_rss_pctype_patterns[i].type;
-                       rte_free(items);
-                       return 0;
+                       break;
                }
        }
 
@@ -4580,11 +4580,9 @@ i40e_flow_parse_rss_pattern(__rte_unused struct 
rte_eth_dev *dev,
                        }
                        break;
                default:
-                       rte_flow_error_set(error, EINVAL,
-                                       RTE_FLOW_ERROR_TYPE_ITEM,
-                                       item,
-                                       "Not support range");
-                       return -rte_errno;
+                       p_info->action_flag = 0;
+                       memset(info, 0, sizeof(struct i40e_queue_regions));
+                       return 0;
                }
        }
 
@@ -4640,7 +4638,7 @@ i40e_flow_parse_rss_action(struct rte_eth_dev *dev,
                return -rte_errno;
        }
 
-       if (p_info.action_flag) {
+       if (p_info.action_flag && rss->queue_num) {
                for (n = 0; n < 64; n++) {
                        if (rss->types & (hf_bit << n)) {
                                conf_info->region[0].hw_flowtype[0] = n;
-- 
2.17.1