From: Ye Xiaolong <xiaolong.ye@intel.com>
To: Simei Su <simei.su@intel.com>
Cc: qi.z.zhang@intel.com, qiming.yang@intel.com,
beilei.xing@intel.com, dev@dpdk.org, ferruh.yigit@intel.com
Subject: Re: [dpdk-dev] [PATCH v2] net/ice: enable advanced RSS
Date: Fri, 13 Sep 2019 00:19:10 +0800 [thread overview]
Message-ID: <20190912161910.GB71525@intel.com> (raw)
In-Reply-To: <1568041014-341418-1-git-send-email-simei.su@intel.com>
On 09/09, Simei Su wrote:
>This patch supports the following features:
> (1)inner header hash for tunnel packets, including comms package.
> (2)symmetric hash by rte_flow RSS action.
> (3)input set change by rte_flow RSS action.
>
>Signed-off-by: Simei Su <simei.su@intel.com>
>---
>This patch depends on the following patches on patchwork:
> (1)https://patchwork.dpdk.org/patch/58546/
> [2/4] net/ice: rework for generic flow enabling
> (2)https://patchwork.dpdk.org/patch/57137/
> [v2,1/2] ethdev: add symmetric toeplitz hash support
> (3)https://patchwork.dpdk.org/patch/57138/
> [v2,2/2] app/testpmd: add symmetric toeplitz hash support
> (4)https://patchwork.dpdk.org/patch/57601/
> [1/2] ethdev: extend RSS offload types
> (5)https://patchwork.dpdk.org/patch/57602/
> [2/2] app/testpmd: add RSS offload types extending support
>
> drivers/net/ice/ice_ethdev.c | 7 +
> drivers/net/ice/ice_hash.c | 548 +++++++++++++++++++++++++++++++++++++++++++
> 2 files changed, 555 insertions(+)
> create mode 100644 drivers/net/ice/ice_hash.c
>
>diff --git a/drivers/net/ice/ice_ethdev.c b/drivers/net/ice/ice_ethdev.c
>index f5cc647..3766a32 100644
>--- a/drivers/net/ice/ice_ethdev.c
>+++ b/drivers/net/ice/ice_ethdev.c
>@@ -1874,6 +1874,7 @@ static int ice_init_rss(struct ice_pf *pf)
> uint16_t i, nb_q;
> int ret = 0;
> bool is_safe_mode = pf->adapter->is_safe_mode;
>+ uint32_t reg;
>
> rss_conf = &dev->data->dev_conf.rx_adv_conf.rss_conf;
> nb_q = dev->data->nb_rx_queues;
>@@ -1917,6 +1918,12 @@ static int ice_init_rss(struct ice_pf *pf)
> if (ret)
> return -EINVAL;
>
>+ /* Enable registers for symmetric_toeplitz function. */
>+ reg = ICE_READ_REG(hw, VSIQF_HASH_CTL(vsi->vsi_id));
>+ reg = (reg & (~VSIQF_HASH_CTL_HASH_SCHEME_M)) |
>+ (1 << VSIQF_HASH_CTL_HASH_SCHEME_S);
>+ ICE_WRITE_REG(hw, VSIQF_HASH_CTL(vsi->vsi_id), reg);
>+
> /* configure RSS for IPv4 with input set IPv4 src/dst */
> ret = ice_add_rss_cfg(hw, vsi->idx, ICE_FLOW_HASH_IPV4,
> ICE_FLOW_SEG_HDR_IPV4, 0);
>diff --git a/drivers/net/ice/ice_hash.c b/drivers/net/ice/ice_hash.c
>new file mode 100644
>index 0000000..fbd7cd7
>--- /dev/null
>+++ b/drivers/net/ice/ice_hash.c
>@@ -0,0 +1,548 @@
>+/* SPDX-License-Identifier: BSD-3-Clause
>+ * Copyright(c) 2019 Intel Corporation
>+ */
>+
>+#include <sys/queue.h>
>+#include <stdio.h>
>+#include <errno.h>
>+#include <stdint.h>
>+#include <string.h>
>+#include <unistd.h>
>+#include <stdarg.h>
>+
>+#include <rte_debug.h>
>+#include <rte_ether.h>
>+#include <rte_ethdev_driver.h>
>+#include <rte_log.h>
>+#include <rte_malloc.h>
>+#include <rte_eth_ctrl.h>
>+#include <rte_tailq.h>
>+#include <rte_flow_driver.h>
>+
>+#include "ice_logs.h"
>+#include "base/ice_type.h"
>+#include "base/ice_flow.h"
>+#include "ice_ethdev.h"
>+#include "ice_generic_flow.h"
>+
>+#define ICE_ACTION_RSS_MAX_QUEUE_NUM 32
>+
>+static struct ice_pattern_match_item ice_hash_pattern_list_os[];
>+static struct ice_pattern_match_item ice_hash_pattern_list_comms[];
Above two forward declaration seems unnecessary.
>+
>+struct rss_type_match_hdr {
>+ uint32_t hdr_mask;
>+ uint64_t eth_rss_hint;
>+};
>+
>+struct ice_hash_match_type {
>+ uint64_t hash_type;
>+ uint64_t hash_flds;
>+};
>+
>+struct rss_meta {
>+ uint32_t pkt_hdr;
>+ uint64_t hash_flds;
>+ uint8_t hash_function;
>+};
>+
>+static int
>+ice_hash_init(struct ice_adapter *ad);
>+
>+static int
>+ice_hash_create(struct ice_adapter *ad,
>+ struct rte_flow *flow,
>+ void *meta,
>+ struct rte_flow_error *error);
>+
>+static int
>+ice_hash_destroy(struct ice_adapter *ad,
>+ struct rte_flow *flow,
>+ struct rte_flow_error *error);
>+
>+static void
>+ice_hash_uninit(struct ice_adapter *ad);
>+
>+static void
>+ice_hash_free(struct rte_flow *flow);
>+
>+static int
>+ice_hash_parse_pattern_action(struct ice_adapter *ad,
>+ struct ice_pattern_match_item *array,
>+ uint32_t array_len,
>+ const struct rte_flow_item pattern[],
>+ const struct rte_flow_action actions[],
>+ void **meta,
>+ struct rte_flow_error *error);
>+
>+/* The first member is protocol header, the second member is ETH_RSS_*. */
>+const struct rss_type_match_hdr hint_0 = {
>+ ICE_FLOW_SEG_HDR_NONE, 0};
>+const struct rss_type_match_hdr hint_1 = {
>+ ICE_FLOW_SEG_HDR_IPV4, ETH_RSS_IPV4};
>+const struct rss_type_match_hdr hint_2 = {
>+ ICE_FLOW_SEG_HDR_IPV4|ICE_FLOW_SEG_HDR_UDP, ETH_RSS_NONFRAG_IPV4_UDP};
>+const struct rss_type_match_hdr hint_3 = {
>+ ICE_FLOW_SEG_HDR_IPV4|ICE_FLOW_SEG_HDR_TCP, ETH_RSS_NONFRAG_IPV4_TCP};
>+const struct rss_type_match_hdr hint_4 = {
>+ ICE_FLOW_SEG_HDR_IPV4|ICE_FLOW_SEG_HDR_SCTP, ETH_RSS_NONFRAG_IPV4_SCTP};
>+const struct rss_type_match_hdr hint_5 = {
>+ ICE_FLOW_SEG_HDR_IPV6, ETH_RSS_IPV6};
>+const struct rss_type_match_hdr hint_6 = {
>+ ICE_FLOW_SEG_HDR_IPV6|ICE_FLOW_SEG_HDR_UDP, ETH_RSS_NONFRAG_IPV6_UDP};
>+const struct rss_type_match_hdr hint_7 = {
>+ ICE_FLOW_SEG_HDR_IPV6|ICE_FLOW_SEG_HDR_TCP, ETH_RSS_NONFRAG_IPV6_TCP};
>+const struct rss_type_match_hdr hint_8 = {
>+ ICE_FLOW_SEG_HDR_IPV6|ICE_FLOW_SEG_HDR_SCTP, ETH_RSS_NONFRAG_IPV6_SCTP};
>+const struct rss_type_match_hdr hint_9 = {
>+ ICE_FLOW_SEG_HDR_GTPU_IP, ETH_RSS_IPV4};
>+const struct rss_type_match_hdr hint_10 = {
>+ ICE_FLOW_SEG_HDR_PPPOE, ETH_RSS_IPV4};
>+const struct rss_type_match_hdr hint_11 = {
>+ ICE_FLOW_SEG_HDR_PPPOE, ETH_RSS_NONFRAG_IPV4_UDP};
>+const struct rss_type_match_hdr hint_12 = {
>+ ICE_FLOW_SEG_HDR_PPPOE, ETH_RSS_NONFRAG_IPV4_TCP};
>+const struct rss_type_match_hdr hint_13 = {
>+ ICE_FLOW_SEG_HDR_PPPOE, ETH_RSS_NONFRAG_IPV4_SCTP};
>+
>+/* Supported pattern for os default package. */
>+static struct ice_pattern_match_item ice_hash_pattern_list_os[] = {
>+ {pattern_eth_ipv4, ICE_INSET_NONE, (uint64_t)(&hint_1)},
>+ {pattern_eth_ipv4_udp, ICE_INSET_NONE, (uint64_t)(&hint_2)},
>+ {pattern_eth_ipv4_tcp, ICE_INSET_NONE, (uint64_t)(&hint_3)},
>+ {pattern_eth_ipv4_sctp, ICE_INSET_NONE, (uint64_t)(&hint_4)},
>+ {pattern_eth_ipv6, ICE_INSET_NONE, (uint64_t)(&hint_5)},
>+ {pattern_eth_ipv6_udp, ICE_INSET_NONE, (uint64_t)(&hint_6)},
>+ {pattern_eth_ipv6_tcp, ICE_INSET_NONE, (uint64_t)(&hint_7)},
>+ {pattern_eth_ipv6_sctp, ICE_INSET_NONE, (uint64_t)(&hint_8)},
>+ {pattern_empty, ICE_INSET_NONE, (uint64_t)(&hint_0)},
>+};
>+
>+/* Supported pattern for comms package. */
>+static struct ice_pattern_match_item ice_hash_pattern_list_comms[] = {
>+ {pattern_eth_ipv4, ICE_INSET_NONE, (uint64_t)(&hint_1)},
>+ {pattern_eth_ipv4_udp, ICE_INSET_NONE, (uint64_t)(&hint_2)},
>+ {pattern_eth_ipv4_tcp, ICE_INSET_NONE, (uint64_t)(&hint_3)},
>+ {pattern_eth_ipv4_sctp, ICE_INSET_NONE, (uint64_t)(&hint_4)},
>+ {pattern_eth_ipv6, ICE_INSET_NONE, (uint64_t)(&hint_5)},
>+ {pattern_eth_ipv6_udp, ICE_INSET_NONE, (uint64_t)(&hint_6)},
>+ {pattern_eth_ipv6_tcp, ICE_INSET_NONE, (uint64_t)(&hint_7)},
>+ {pattern_eth_ipv6_sctp, ICE_INSET_NONE, (uint64_t)(&hint_8)},
>+ {pattern_empty, ICE_INSET_NONE, (uint64_t)(&hint_0)},
>+ {pattern_eth_ipv4_gtpu_ipv4, ICE_INSET_NONE, (uint64_t)(&hint_9)},
>+ {pattern_eth_ipv4_gtpu_ipv4_udp, ICE_INSET_NONE, (uint64_t)(&hint_9)},
>+ {pattern_eth_ipv4_gtpu_ipv4_tcp, ICE_INSET_NONE, (uint64_t)(&hint_9)},
>+ {pattern_eth_pppoes_ipv4, ICE_INSET_NONE, (uint64_t)(&hint_10)},
>+ {pattern_eth_pppoes_ipv4_udp, ICE_INSET_NONE, (uint64_t)(&hint_11)},
>+ {pattern_eth_pppoes_ipv4_tcp, ICE_INSET_NONE, (uint64_t)(&hint_12)},
>+ {pattern_eth_pppoes_ipv4_sctp, ICE_INSET_NONE, (uint64_t)(&hint_13)},
>+};
>+
>+/**
>+ * The first member is input set combination,
>+ * the second member is hash fields.
>+ */
>+struct ice_hash_match_type ice_hash_type_list[] = {
>+ {ETH_RSS_IPV4|ETH_RSS_L3_SRC_ONLY, BIT_ULL(ICE_FLOW_FIELD_IDX_IPV4_SA)},
>+ {ETH_RSS_IPV4|ETH_RSS_L3_DST_ONLY, BIT_ULL(ICE_FLOW_FIELD_IDX_IPV4_DA)},
>+ {ETH_RSS_IPV4, ICE_FLOW_HASH_IPV4},
>+ {ETH_RSS_NONFRAG_IPV4_UDP|ETH_RSS_L3_SRC_ONLY|ETH_RSS_L4_SRC_ONLY, BIT_ULL(ICE_FLOW_FIELD_IDX_IPV4_SA)|BIT_ULL(ICE_FLOW_FIELD_IDX_UDP_SRC_PORT)},
>+ {ETH_RSS_NONFRAG_IPV4_UDP|ETH_RSS_L3_SRC_ONLY|ETH_RSS_L4_DST_ONLY, BIT_ULL(ICE_FLOW_FIELD_IDX_IPV4_SA)|BIT_ULL(ICE_FLOW_FIELD_IDX_UDP_DST_PORT)},
>+ {ETH_RSS_NONFRAG_IPV4_UDP|ETH_RSS_L3_SRC_ONLY, BIT_ULL(ICE_FLOW_FIELD_IDX_IPV4_SA)|ICE_FLOW_HASH_UDP_PORT},
>+ {ETH_RSS_NONFRAG_IPV4_UDP|ETH_RSS_L3_DST_ONLY|ETH_RSS_L4_SRC_ONLY, BIT_ULL(ICE_FLOW_FIELD_IDX_IPV4_DA)|BIT_ULL(ICE_FLOW_FIELD_IDX_UDP_SRC_PORT)},
>+ {ETH_RSS_NONFRAG_IPV4_UDP|ETH_RSS_L3_DST_ONLY|ETH_RSS_L4_DST_ONLY, BIT_ULL(ICE_FLOW_FIELD_IDX_IPV4_DA)|BIT_ULL(ICE_FLOW_FIELD_IDX_UDP_DST_PORT)},
>+ {ETH_RSS_NONFRAG_IPV4_UDP|ETH_RSS_L3_DST_ONLY, BIT_ULL(ICE_FLOW_FIELD_IDX_IPV4_DA)|ICE_FLOW_HASH_UDP_PORT},
>+ {ETH_RSS_NONFRAG_IPV4_UDP|ETH_RSS_L4_SRC_ONLY, ICE_FLOW_HASH_IPV4|BIT_ULL(ICE_FLOW_FIELD_IDX_UDP_SRC_PORT)},
>+ {ETH_RSS_NONFRAG_IPV4_UDP|ETH_RSS_L4_DST_ONLY, ICE_FLOW_HASH_IPV4|BIT_ULL(ICE_FLOW_FIELD_IDX_UDP_DST_PORT)},
>+ {ETH_RSS_NONFRAG_IPV4_UDP, ICE_HASH_UDP_IPV4},
>+ {ETH_RSS_NONFRAG_IPV4_TCP|ETH_RSS_L3_SRC_ONLY|ETH_RSS_L4_SRC_ONLY, BIT_ULL(ICE_FLOW_FIELD_IDX_IPV4_SA)|BIT_ULL(ICE_FLOW_FIELD_IDX_TCP_SRC_PORT)},
>+ {ETH_RSS_NONFRAG_IPV4_TCP|ETH_RSS_L3_SRC_ONLY|ETH_RSS_L4_DST_ONLY, BIT_ULL(ICE_FLOW_FIELD_IDX_IPV4_SA)|BIT_ULL(ICE_FLOW_FIELD_IDX_TCP_DST_PORT)},
>+ {ETH_RSS_NONFRAG_IPV4_TCP|ETH_RSS_L3_SRC_ONLY, BIT_ULL(ICE_FLOW_FIELD_IDX_IPV4_SA)|ICE_FLOW_HASH_TCP_PORT},
>+ {ETH_RSS_NONFRAG_IPV4_TCP|ETH_RSS_L3_DST_ONLY|ETH_RSS_L4_SRC_ONLY, BIT_ULL(ICE_FLOW_FIELD_IDX_IPV4_DA)|BIT_ULL(ICE_FLOW_FIELD_IDX_TCP_SRC_PORT)},
>+ {ETH_RSS_NONFRAG_IPV4_TCP|ETH_RSS_L3_DST_ONLY|ETH_RSS_L4_DST_ONLY, BIT_ULL(ICE_FLOW_FIELD_IDX_IPV4_DA)|BIT_ULL(ICE_FLOW_FIELD_IDX_TCP_DST_PORT)},
>+ {ETH_RSS_NONFRAG_IPV4_TCP|ETH_RSS_L3_DST_ONLY, BIT_ULL(ICE_FLOW_FIELD_IDX_IPV4_DA)|ICE_FLOW_HASH_TCP_PORT},
>+ {ETH_RSS_NONFRAG_IPV4_TCP|ETH_RSS_L4_SRC_ONLY, ICE_FLOW_HASH_IPV4|BIT_ULL(ICE_FLOW_FIELD_IDX_TCP_SRC_PORT)},
>+ {ETH_RSS_NONFRAG_IPV4_TCP|ETH_RSS_L4_DST_ONLY, ICE_FLOW_HASH_IPV4|BIT_ULL(ICE_FLOW_FIELD_IDX_TCP_DST_PORT)},
>+ {ETH_RSS_NONFRAG_IPV4_TCP, ICE_HASH_TCP_IPV4},
>+ {ETH_RSS_NONFRAG_IPV4_SCTP|ETH_RSS_L3_SRC_ONLY|ETH_RSS_L4_SRC_ONLY, BIT_ULL(ICE_FLOW_FIELD_IDX_IPV4_SA)|BIT_ULL(ICE_FLOW_FIELD_IDX_SCTP_SRC_PORT)},
>+ {ETH_RSS_NONFRAG_IPV4_SCTP|ETH_RSS_L3_SRC_ONLY|ETH_RSS_L4_DST_ONLY, BIT_ULL(ICE_FLOW_FIELD_IDX_IPV4_SA)|BIT_ULL(ICE_FLOW_FIELD_IDX_SCTP_DST_PORT)},
>+ {ETH_RSS_NONFRAG_IPV4_SCTP|ETH_RSS_L3_SRC_ONLY, BIT_ULL(ICE_FLOW_FIELD_IDX_IPV4_SA)|ICE_FLOW_HASH_SCTP_PORT},
>+ {ETH_RSS_NONFRAG_IPV4_SCTP|ETH_RSS_L3_DST_ONLY|ETH_RSS_L4_SRC_ONLY, BIT_ULL(ICE_FLOW_FIELD_IDX_IPV4_DA)|BIT_ULL(ICE_FLOW_FIELD_IDX_SCTP_SRC_PORT)},
>+ {ETH_RSS_NONFRAG_IPV4_SCTP|ETH_RSS_L3_DST_ONLY|ETH_RSS_L4_DST_ONLY, BIT_ULL(ICE_FLOW_FIELD_IDX_IPV4_DA)|BIT_ULL(ICE_FLOW_FIELD_IDX_SCTP_DST_PORT)},
>+ {ETH_RSS_NONFRAG_IPV4_SCTP|ETH_RSS_L3_DST_ONLY, BIT_ULL(ICE_FLOW_FIELD_IDX_IPV4_DA)|ICE_FLOW_HASH_SCTP_PORT},
>+ {ETH_RSS_NONFRAG_IPV4_SCTP|ETH_RSS_L4_SRC_ONLY, ICE_FLOW_HASH_IPV4|BIT_ULL(ICE_FLOW_FIELD_IDX_SCTP_SRC_PORT)},
>+ {ETH_RSS_NONFRAG_IPV4_SCTP|ETH_RSS_L4_DST_ONLY, ICE_FLOW_HASH_IPV4|BIT_ULL(ICE_FLOW_FIELD_IDX_SCTP_DST_PORT)},
>+ {ETH_RSS_NONFRAG_IPV4_SCTP, ICE_HASH_SCTP_IPV4},
>+ {ETH_RSS_IPV6|ETH_RSS_L3_SRC_ONLY, BIT_ULL(ICE_FLOW_FIELD_IDX_IPV6_SA)},
>+ {ETH_RSS_IPV6|ETH_RSS_L3_DST_ONLY, BIT_ULL(ICE_FLOW_FIELD_IDX_IPV6_DA)},
>+ {ETH_RSS_IPV6, ICE_FLOW_HASH_IPV6},
>+ {ETH_RSS_NONFRAG_IPV6_UDP|ETH_RSS_L3_SRC_ONLY|ETH_RSS_L4_SRC_ONLY, BIT_ULL(ICE_FLOW_FIELD_IDX_IPV6_SA)|BIT_ULL(ICE_FLOW_FIELD_IDX_UDP_SRC_PORT)},
>+ {ETH_RSS_NONFRAG_IPV6_UDP|ETH_RSS_L3_SRC_ONLY|ETH_RSS_L4_DST_ONLY, BIT_ULL(ICE_FLOW_FIELD_IDX_IPV6_SA)|BIT_ULL(ICE_FLOW_FIELD_IDX_UDP_DST_PORT)},
>+ {ETH_RSS_NONFRAG_IPV6_UDP|ETH_RSS_L3_SRC_ONLY, BIT_ULL(ICE_FLOW_FIELD_IDX_IPV6_SA)|ICE_FLOW_HASH_UDP_PORT},
>+ {ETH_RSS_NONFRAG_IPV6_UDP|ETH_RSS_L3_DST_ONLY|ETH_RSS_L4_SRC_ONLY, BIT_ULL(ICE_FLOW_FIELD_IDX_IPV6_DA)|BIT_ULL(ICE_FLOW_FIELD_IDX_UDP_SRC_PORT)},
>+ {ETH_RSS_NONFRAG_IPV6_UDP|ETH_RSS_L3_DST_ONLY|ETH_RSS_L4_DST_ONLY, BIT_ULL(ICE_FLOW_FIELD_IDX_IPV6_DA)|BIT_ULL(ICE_FLOW_FIELD_IDX_UDP_DST_PORT)},
>+ {ETH_RSS_NONFRAG_IPV6_UDP|ETH_RSS_L3_DST_ONLY, BIT_ULL(ICE_FLOW_FIELD_IDX_IPV6_DA)|ICE_FLOW_HASH_UDP_PORT},
>+ {ETH_RSS_NONFRAG_IPV6_UDP|ETH_RSS_L4_SRC_ONLY, ICE_FLOW_HASH_IPV6|BIT_ULL(ICE_FLOW_FIELD_IDX_UDP_SRC_PORT)},
>+ {ETH_RSS_NONFRAG_IPV6_UDP|ETH_RSS_L4_DST_ONLY, ICE_FLOW_HASH_IPV6|BIT_ULL(ICE_FLOW_FIELD_IDX_UDP_DST_PORT)},
>+ {ETH_RSS_NONFRAG_IPV6_UDP, ICE_HASH_UDP_IPV6},
>+ {ETH_RSS_NONFRAG_IPV6_TCP|ETH_RSS_L3_SRC_ONLY|ETH_RSS_L4_SRC_ONLY, BIT_ULL(ICE_FLOW_FIELD_IDX_IPV6_SA)|BIT_ULL(ICE_FLOW_FIELD_IDX_TCP_SRC_PORT)},
>+ {ETH_RSS_NONFRAG_IPV6_TCP|ETH_RSS_L3_SRC_ONLY|ETH_RSS_L4_DST_ONLY, BIT_ULL(ICE_FLOW_FIELD_IDX_IPV6_SA)|BIT_ULL(ICE_FLOW_FIELD_IDX_TCP_DST_PORT)},
>+ {ETH_RSS_NONFRAG_IPV6_TCP|ETH_RSS_L3_SRC_ONLY, BIT_ULL(ICE_FLOW_FIELD_IDX_IPV6_SA)|ICE_FLOW_HASH_TCP_PORT},
>+ {ETH_RSS_NONFRAG_IPV6_TCP|ETH_RSS_L3_DST_ONLY|ETH_RSS_L4_SRC_ONLY, BIT_ULL(ICE_FLOW_FIELD_IDX_IPV6_DA)|BIT_ULL(ICE_FLOW_FIELD_IDX_TCP_SRC_PORT)},
>+ {ETH_RSS_NONFRAG_IPV6_TCP|ETH_RSS_L3_DST_ONLY|ETH_RSS_L4_DST_ONLY, BIT_ULL(ICE_FLOW_FIELD_IDX_IPV6_DA)|BIT_ULL(ICE_FLOW_FIELD_IDX_TCP_DST_PORT)},
>+ {ETH_RSS_NONFRAG_IPV6_TCP|ETH_RSS_L3_DST_ONLY, BIT_ULL(ICE_FLOW_FIELD_IDX_IPV6_DA)|ICE_FLOW_HASH_TCP_PORT},
>+ {ETH_RSS_NONFRAG_IPV6_TCP|ETH_RSS_L4_SRC_ONLY, ICE_FLOW_HASH_IPV6|BIT_ULL(ICE_FLOW_FIELD_IDX_TCP_SRC_PORT)},
>+ {ETH_RSS_NONFRAG_IPV6_TCP|ETH_RSS_L4_DST_ONLY, ICE_FLOW_HASH_IPV6|BIT_ULL(ICE_FLOW_FIELD_IDX_TCP_DST_PORT)},
>+ {ETH_RSS_NONFRAG_IPV6_TCP, ICE_HASH_TCP_IPV6},
>+ {ETH_RSS_NONFRAG_IPV6_SCTP|ETH_RSS_L3_SRC_ONLY|ETH_RSS_L4_SRC_ONLY, BIT_ULL(ICE_FLOW_FIELD_IDX_IPV6_SA)|BIT_ULL(ICE_FLOW_FIELD_IDX_SCTP_SRC_PORT)},
>+ {ETH_RSS_NONFRAG_IPV6_SCTP|ETH_RSS_L3_SRC_ONLY|ETH_RSS_L4_DST_ONLY, BIT_ULL(ICE_FLOW_FIELD_IDX_IPV6_SA)|BIT_ULL(ICE_FLOW_FIELD_IDX_SCTP_DST_PORT)},
>+ {ETH_RSS_NONFRAG_IPV6_SCTP|ETH_RSS_L3_SRC_ONLY, BIT_ULL(ICE_FLOW_FIELD_IDX_IPV6_SA)|ICE_FLOW_HASH_SCTP_PORT},
>+ {ETH_RSS_NONFRAG_IPV6_SCTP|ETH_RSS_L3_DST_ONLY|ETH_RSS_L4_SRC_ONLY, BIT_ULL(ICE_FLOW_FIELD_IDX_IPV6_DA)|BIT_ULL(ICE_FLOW_FIELD_IDX_SCTP_SRC_PORT)},
>+ {ETH_RSS_NONFRAG_IPV6_SCTP|ETH_RSS_L3_DST_ONLY|ETH_RSS_L4_DST_ONLY, BIT_ULL(ICE_FLOW_FIELD_IDX_IPV6_DA)|BIT_ULL(ICE_FLOW_FIELD_IDX_SCTP_DST_PORT)},
>+ {ETH_RSS_NONFRAG_IPV6_SCTP|ETH_RSS_L3_DST_ONLY, BIT_ULL(ICE_FLOW_FIELD_IDX_IPV6_DA)|ICE_FLOW_HASH_SCTP_PORT},
>+ {ETH_RSS_NONFRAG_IPV6_SCTP|ETH_RSS_L4_SRC_ONLY, ICE_FLOW_HASH_IPV6|BIT_ULL(ICE_FLOW_FIELD_IDX_SCTP_SRC_PORT)},
>+ {ETH_RSS_NONFRAG_IPV6_SCTP|ETH_RSS_L4_DST_ONLY, ICE_FLOW_HASH_IPV6|BIT_ULL(ICE_FLOW_FIELD_IDX_SCTP_DST_PORT)},
>+ {ETH_RSS_NONFRAG_IPV6_SCTP, ICE_HASH_SCTP_IPV6},
>+};
>+
>+static struct ice_flow_engine ice_hash_engine = {
>+ .init = ice_hash_init,
>+ .create = ice_hash_create,
>+ .destroy = ice_hash_destroy,
>+ .uninit = ice_hash_uninit,
>+ .free = ice_hash_free,
>+ .type = ICE_FLOW_ENGINE_HASH,
>+};
>+
>+/* Register parser for os package. */
>+static struct ice_flow_parser ice_hash_parser_os = {
>+ .engine = &ice_hash_engine,
>+ .array = ice_hash_pattern_list_os,
>+ .array_len = RTE_DIM(ice_hash_pattern_list_os),
>+ .parse_pattern_action = ice_hash_parse_pattern_action,
>+ .stage = ICE_FLOW_STAGE_RSS,
>+};
>+
>+/* Register parser for comms package. */
>+static struct ice_flow_parser ice_hash_parser_comms = {
>+ .engine = &ice_hash_engine,
>+ .array = ice_hash_pattern_list_comms,
>+ .array_len = RTE_DIM(ice_hash_pattern_list_comms),
>+ .parse_pattern_action = ice_hash_parse_pattern_action,
>+ .stage = ICE_FLOW_STAGE_RSS,
>+};
>+
>+RTE_INIT(ice_hash_init_log)
I don't think this function has something to do with 'log', better to use name
like ice_hash_engine_init.
>+{
>+ struct ice_flow_engine *engine = &ice_hash_engine;
>+ ice_register_flow_engine(engine);
>+}
>+
>+static int
>+ice_hash_init(struct ice_adapter *ad)
>+{
>+ int ret = 0;
Unnecessary initialization for the ret.
>+
>+ if (ad->active_pkg_type == ICE_PKG_TYPE_OS_DEFAULT) {
>+ struct ice_flow_parser *parser = &ice_hash_parser_os;
>+ ret = ice_register_parser(parser, ad);
>+ } else if (ad->active_pkg_type == ICE_PKG_TYPE_COMMS) {
>+ struct ice_flow_parser *parser = &ice_hash_parser_comms;
>+ ret = ice_register_parser(parser, ad);
>+ }
>+
What about:
struct ice_flow_parser *parser;
if (ad->active_pkg_type == ICE_PKG_TYPE_OS_DEFAULT) {
parser = &ice_hash_parser_os;
} else if (ad->active_pkg_type == ICE_PKG_TYPE_COMMS) {
parser = &ice_hash_parser_comms;
}
return ice_register_parser(parser, ad);
>+ return ret;
>+}
>+
>+static int
>+ice_hash_check_inset(const struct rte_flow_item pattern[],
>+ struct rte_flow_error *error)
>+{
>+ const struct rte_flow_item *item = pattern;
>+
>+ for (item = pattern; item->type != RTE_FLOW_ITEM_TYPE_END; item++) {
>+ if (item->last) {
>+ rte_flow_error_set(error, EINVAL,
>+ RTE_FLOW_ERROR_TYPE_ITEM, item,
>+ "Not support range");
>+ return -rte_errno;
>+ }
>+
>+ /* Ignore spec and mask. */
>+ if (item->spec || item->mask) {
>+ rte_flow_error_set(error, EINVAL,
>+ RTE_FLOW_ERROR_TYPE_ITEM, item,
>+ "Invalid mask.");
"Invalid spec/mask."
>+ return -rte_errno;
>+ }
>+ }
>+
>+ return 0;
>+}
>+
>+static int
>+ice_hash_parse_action(struct ice_pattern_match_item *pattern_match_item,
>+ const struct rte_flow_action actions[],
>+ void **meta,
>+ struct rte_flow_error *error)
>+{
>+ const struct rte_flow_action *action;
>+ enum rte_flow_action_type action_type;
>+ const struct rte_flow_action_rss *rss;
>+ struct rss_type_match_hdr *m = (struct rss_type_match_hdr *)
>+ (pattern_match_item->meta);
>+ uint32_t type_list_len = RTE_DIM(ice_hash_type_list);
>+ struct ice_hash_match_type *type_match_item;
>+
>+ /* Supported action is RSS. */
>+ for (action = actions; action->type !=
>+ RTE_FLOW_ACTION_TYPE_END; action++) {
>+ action_type = action->type;
>+ switch (action_type) {
>+ case RTE_FLOW_ACTION_TYPE_RSS:
>+ rss = action->conf;
>+ uint16_t i;
>+
>+ /* Check if pattern is empty. */
>+ if (((pattern_match_item->pattern_list) !=
>+ pattern_empty) && (rss->func ==
>+ RTE_ETH_HASH_FUNCTION_SIMPLE_XOR))
>+ return rte_flow_error_set(error, ENOTSUP,
>+ RTE_FLOW_ERROR_TYPE_ACTION, action,
>+ "Not supported flow");
>+
>+ /* Check if rss types match pattern. */
>+ if (rss->func != RTE_ETH_HASH_FUNCTION_SIMPLE_XOR) {
>+ if (((rss->types & ETH_RSS_IPV4) != m->eth_rss_hint) ||
>+ ((rss->types & ETH_RSS_NONFRAG_IPV4_UDP) != m->eth_rss_hint) ||
>+ ((rss->types & ETH_RSS_NONFRAG_IPV4_TCP) != m->eth_rss_hint) ||
>+ ((rss->types & ETH_RSS_NONFRAG_IPV4_SCTP) != m->eth_rss_hint) ||
>+ ((rss->types & ETH_RSS_IPV6) != m->eth_rss_hint) ||
>+ ((rss->types & ETH_RSS_NONFRAG_IPV6_UDP) != m->eth_rss_hint) ||
>+ ((rss->types & ETH_RSS_NONFRAG_IPV6_TCP) != m->eth_rss_hint) ||
>+ ((rss->types & ETH_RSS_NONFRAG_IPV6_SCTP) != m->eth_rss_hint))
>+ return rte_flow_error_set(error,
>+ ENOTSUP, RTE_FLOW_ERROR_TYPE_ACTION,
>+ action, "Not supported RSS types");
>+ }
>+
>+ if (rss->level)
>+ return rte_flow_error_set(error, ENOTSUP,
>+ RTE_FLOW_ERROR_TYPE_ACTION, action,
>+ "a nonzero RSS encapsulation level is not supported");
>+
>+ if (rss->key_len == 0)
>+ return rte_flow_error_set(error, ENOTSUP,
>+ RTE_FLOW_ERROR_TYPE_ACTION, action,
>+ "RSS hash key_len mustn't be 0");
>+
>+ if ((rss->queue_num) > ICE_ACTION_RSS_MAX_QUEUE_NUM)
>+ return rte_flow_error_set(error, ENOTSUP,
>+ RTE_FLOW_ERROR_TYPE_ACTION, action,
>+ "too many queues for RSS context");
>+
>+ /* Check hash function and save it to rss_meta. */
>+ if (rss->func ==
>+ RTE_ETH_HASH_FUNCTION_SIMPLE_XOR)
>+ ((struct rss_meta *)*meta)->hash_function =
>+ RTE_ETH_HASH_FUNCTION_SIMPLE_XOR;
>+
>+ if (rss->func ==
>+ RTE_ETH_HASH_FUNCTION_SYMMETRIC_TOEPLITZ)
>+ ((struct rss_meta *)*meta)->hash_function =
>+ RTE_ETH_HASH_FUNCTION_SYMMETRIC_TOEPLITZ;
>+
>+ type_match_item = rte_zmalloc("ice_type_match_item",
>+ sizeof(struct ice_hash_match_type), 0);
>+ if (!type_match_item) {
>+ rte_flow_error_set(error, EINVAL,
>+ RTE_FLOW_ERROR_TYPE_HANDLE, NULL,
>+ "No memory for type_match_item");
>+ return -ENOMEM;
>+ }
>+
>+ /* Find matched hash fields according to hash type. */
>+ for (i = 0; i < type_list_len; i++) {
>+ if (rss->types ==
>+ ice_hash_type_list[i].hash_type) {
>+ type_match_item->hash_type =
>+ ice_hash_type_list[i].hash_type;
>+ type_match_item->hash_flds =
>+ ice_hash_type_list[i].hash_flds;
>+ }
>+ }
>+
>+ /* Save hash fileds to rss_meta. */
>+ ((struct rss_meta *)*meta)->hash_flds =
>+ type_match_item->hash_flds;
>+
>+ rte_free(type_match_item);
>+ break;
>+
>+ case RTE_FLOW_ACTION_TYPE_END:
>+ return 0;
>+ break;
>+
>+ default:
>+ rte_flow_error_set(error, EINVAL,
>+ RTE_FLOW_ERROR_TYPE_ACTION, action,
>+ "Invalid action.");
>+ return -rte_errno;
>+ }
>+ }
>+
>+ return 0;
>+}
>+
>+static int
>+ice_hash_parse_pattern_action(__rte_unused struct ice_adapter *ad,
>+ struct ice_pattern_match_item *array,
>+ uint32_t array_len,
>+ const struct rte_flow_item pattern[],
>+ const struct rte_flow_action actions[],
>+ void **meta,
>+ struct rte_flow_error *error)
>+{
>+ int ret = 0;
>+ struct ice_pattern_match_item *pattern_match_item = NULL;
>+ struct rss_meta *rss_meta_ptr = NULL;
Unnecessary initializations.
>+
>+ rss_meta_ptr = rte_zmalloc(NULL, sizeof(*rss_meta_ptr), 0);
>+ if (!rss_meta_ptr) {
>+ rte_flow_error_set(error, EINVAL,
>+ RTE_FLOW_ERROR_TYPE_HANDLE, NULL,
>+ "No memory for rss_meta_ptr");
>+ return -ENOMEM;
>+ }
>+
>+ /* Check rss supported pattern and find matched pattern. */
>+ pattern_match_item = ice_search_pattern_match_item(pattern,
>+ array, array_len, error);
>+ if (!pattern_match_item)
>+ return -rte_errno;
>+
>+ ret = ice_hash_check_inset(pattern, error);
>+ if (ret)
>+ return -rte_errno;
>+
>+ /* Save protocol header to rss_meta. */
>+ *meta = rss_meta_ptr;
>+ ((struct rss_meta *)*meta)->pkt_hdr = ((struct rss_type_match_hdr *)
>+ (pattern_match_item->meta))->hdr_mask;
>+
>+ /* Check rss action. */
>+ ret = ice_hash_parse_action(pattern_match_item, actions, meta, error);
>+ if (ret)
>+ return -rte_errno;
>+
>+ rte_free(pattern_match_item);
>+
>+ return 0;
>+}
>+
>+static int
>+ice_hash_create(struct ice_adapter *ad,
>+ struct rte_flow *flow,
>+ void *meta,
>+ __rte_unused struct rte_flow_error *error)
>+{
>+ struct ice_pf *pf = &ad->pf;
>+ struct ice_hw *hw = ICE_PF_TO_HW(pf);
>+ struct ice_vsi *vsi = pf->main_vsi;
>+ int ret = 0;
>+ uint32_t reg;
>+ struct ice_rss_cfg *filter_ptr;
>+
>+ uint32_t headermask = ((struct rss_meta *)meta)->pkt_hdr;
>+ uint64_t hash_field = ((struct rss_meta *)meta)->hash_flds;
>+ uint8_t hash_function = ((struct rss_meta *)meta)->hash_function;
>+
>+ filter_ptr = rte_zmalloc("ice_rss_filter",
>+ sizeof(struct ice_rss_cfg), 0);
>+ if (!filter_ptr) {
>+ rte_flow_error_set(error, EINVAL,
>+ RTE_FLOW_ERROR_TYPE_HANDLE, NULL,
>+ "No memory for filter_ptr");
>+ return -ENOMEM;
>+ }
>+
>+ if (hash_function == RTE_ETH_HASH_FUNCTION_SIMPLE_XOR) {
>+ /* Enable registers for simple_xor hash function. */
>+ reg = ICE_READ_REG(hw, VSIQF_HASH_CTL(vsi->vsi_id));
>+ reg = (reg & (~VSIQF_HASH_CTL_HASH_SCHEME_M)) |
>+ (2 << VSIQF_HASH_CTL_HASH_SCHEME_S);
>+ ICE_WRITE_REG(hw, VSIQF_HASH_CTL(vsi->vsi_id), reg);
>+
>+ filter_ptr->symm = hash_function;
>+
>+ goto out;
>+ } else if (hash_function == RTE_ETH_HASH_FUNCTION_SYMMETRIC_TOEPLITZ) {
>+ ret = ice_add_rss_cfg(hw, vsi->idx, hash_field, headermask, 1);
>+ if (ret)
>+ rte_flow_error_set(error, EINVAL,
>+ RTE_FLOW_ERROR_TYPE_HANDLE, NULL,
>+ "rss flow create fail");
Do we need to error out here?
>+ } else {
>+ ret = ice_add_rss_cfg(hw, vsi->idx, hash_field, headermask, 0);
>+ if (ret)
>+ rte_flow_error_set(error, EINVAL,
>+ RTE_FLOW_ERROR_TYPE_HANDLE, NULL,
>+ "rss flow create fail");
Ditto.
>+ }
>+
>+ filter_ptr->packet_hdr = headermask;
>+ filter_ptr->hashed_flds = hash_field;
>+
>+out:
>+ flow->rule = filter_ptr;
>+ rte_free(meta);
>+ return 0;
>+}
>+
>+static int
>+ice_hash_destroy(struct ice_adapter *ad,
>+ struct rte_flow *flow,
>+ __rte_unused struct rte_flow_error *error)
It's actually used in this function, so just remove the __rte_unused tag.
>+{
>+ struct ice_pf *pf = ICE_DEV_PRIVATE_TO_PF(ad);
>+ struct ice_hw *hw = ICE_PF_TO_HW(pf);
>+ struct ice_vsi *vsi = pf->main_vsi;
>+ int ret = 0;
Unnecessary initialization.
>+ uint32_t reg;
>+ struct ice_rss_cfg *filter_ptr;
>+
>+ filter_ptr = (struct ice_rss_cfg *)flow->rule;
>+
>+ if (filter_ptr->symm == RTE_ETH_HASH_FUNCTION_SIMPLE_XOR) {
>+ /* Return to symmetric_toeplitz state. */
>+ reg = ICE_READ_REG(hw, VSIQF_HASH_CTL(vsi->vsi_id));
>+ reg = (reg & (~VSIQF_HASH_CTL_HASH_SCHEME_M)) |
>+ (1 << VSIQF_HASH_CTL_HASH_SCHEME_S);
>+ ICE_WRITE_REG(hw, VSIQF_HASH_CTL(vsi->vsi_id), reg);
>+ } else {
>+ ret = ice_rem_vsi_rss_cfg(hw, vsi->idx);
>+ if (ret)
>+ rte_flow_error_set(error, EINVAL,
>+ RTE_FLOW_ERROR_TYPE_HANDLE, NULL,
>+ "rss flow destroy fail");
If ice_rem_vsi_rss_cfg fails here, do we still need to proceed to free the flow
other than return error here?
>+ }
>+
>+ rte_free(filter_ptr);
>+
>+ return 0;
>+}
>+
>+static void
>+ice_hash_uninit(struct ice_adapter *ad)
>+{
>+ if (ad->active_pkg_type == ICE_PKG_TYPE_OS_DEFAULT)
>+ ice_unregister_parser(&ice_hash_parser_os, ad);
>+ else if (ad->active_pkg_type == ICE_PKG_TYPE_COMMS)
>+ ice_unregister_parser(&ice_hash_parser_comms, ad);
>+}
>+
>+static void
>+ice_hash_free(struct rte_flow *flow)
>+{
>+ rte_free(flow->rule);
>+}
>--
>1.8.3.1
>
next prev parent reply other threads:[~2019-09-12 16:21 UTC|newest]
Thread overview: 26+ messages / expand[flat|nested] mbox.gz Atom feed top
[not found] <1567866291-268287-1-git-send-email-simei.su@intel.com>
2019-09-09 14:56 ` Simei Su
2019-09-12 16:19 ` Ye Xiaolong [this message]
2019-09-16 1:47 ` Su, Simei
2019-09-19 5:15 ` [dpdk-dev] [PATCH v3] " Simei Su
2019-09-27 9:53 ` [dpdk-dev] [PATCH v4 0/2] " Simei Su
2019-09-27 9:53 ` [dpdk-dev] [PATCH v4 1/2] net/ice: add RSS configuration for gtpu/pppoe Simei Su
2019-09-27 9:53 ` [dpdk-dev] [PATCH v4 2/2] net/ice: enable advanced RSS Simei Su
2019-09-29 10:30 ` [dpdk-dev] [PATCH v5 0/2] " Simei Su
2019-09-29 10:30 ` [dpdk-dev] [PATCH v5 1/2] net/ice: add RSS configuration for gtpu/pppoe Simei Su
2019-09-29 10:30 ` [dpdk-dev] [PATCH v5 2/2] net/ice: enable advanced RSS Simei Su
2019-09-30 0:28 ` [dpdk-dev] [PATCH v5 0/2] " Zhang, Qi Z
2019-10-16 22:16 ` [dpdk-dev] [PATCH v6 " Simei Su
2019-10-16 22:16 ` [dpdk-dev] [PATCH v6 1/2] net/ice: add RSS configuration for gtpu/pppoe Simei Su
2019-10-16 22:16 ` [dpdk-dev] [PATCH v6 2/2] net/ice: enable advanced RSS Simei Su
2019-10-20 2:14 ` [dpdk-dev] [PATCH v7 0/2] " Simei Su
2019-10-20 2:14 ` [dpdk-dev] [PATCH v7 1/2] net/ice: add RSS configuration for gtpu/pppoe Simei Su
2019-10-20 2:14 ` [dpdk-dev] [PATCH v7 2/2] net/ice: enable advanced RSS Simei Su
2019-10-21 12:06 ` [dpdk-dev] [PATCH v8 0/3] " Simei Su
2019-10-21 12:06 ` [dpdk-dev] [PATCH v8 1/3] net/ice: add RSS configuration for gtpu/pppoe Simei Su
2019-10-21 12:06 ` [dpdk-dev] [PATCH v8 2/3] net/ice: change one member type of the pattern structure Simei Su
2019-10-21 14:16 ` Ye Xiaolong
2019-10-21 17:30 ` Ferruh Yigit
2019-10-21 12:06 ` [dpdk-dev] [PATCH v8 3/3] net/ice: enable advanced RSS Simei Su
2019-10-21 17:30 ` Ferruh Yigit
2019-10-22 2:04 ` Ye Xiaolong
2019-10-21 14:24 ` [dpdk-dev] [PATCH v8 0/3] " Ye Xiaolong
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=20190912161910.GB71525@intel.com \
--to=xiaolong.ye@intel.com \
--cc=beilei.xing@intel.com \
--cc=dev@dpdk.org \
--cc=ferruh.yigit@intel.com \
--cc=qi.z.zhang@intel.com \
--cc=qiming.yang@intel.com \
--cc=simei.su@intel.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).