ixgbe: code refactoring
[dpdk.git] / lib / librte_pmd_ixgbe / ixgbe_fdir.c
index bf9fe69..afc53cb 100644 (file)
 #define SIG_BUCKET_64KB_HASH_MASK       0x1FFF  /* 13 bits */
 #define SIG_BUCKET_128KB_HASH_MASK      0x3FFF  /* 14 bits */
 #define SIG_BUCKET_256KB_HASH_MASK      0x7FFF  /* 15 bits */
+#define IXGBE_DEFAULT_FLEXBYTES_OFFSET  12 /* default flexbytes offset in bytes */
+#define IXGBE_FDIR_MAX_FLEX_LEN         2 /* len in bytes of flexbytes */
+#define IXGBE_MAX_FLX_SOURCE_OFF        62
+#define IXGBE_FDIRCTRL_FLEX_MASK        (0x1F << IXGBE_FDIRCTRL_FLEX_SHIFT)
+#define IXGBE_FDIRCMD_CMD_INTERVAL_US   10
+
+#define IXGBE_FDIR_FLOW_TYPES ( \
+       (1 << RTE_ETH_FLOW_NONFRAG_IPV4_UDP) | \
+       (1 << RTE_ETH_FLOW_NONFRAG_IPV4_TCP) | \
+       (1 << RTE_ETH_FLOW_NONFRAG_IPV4_SCTP) | \
+       (1 << RTE_ETH_FLOW_NONFRAG_IPV4_OTHER) | \
+       (1 << RTE_ETH_FLOW_NONFRAG_IPV6_UDP) | \
+       (1 << RTE_ETH_FLOW_NONFRAG_IPV6_TCP) | \
+       (1 << RTE_ETH_FLOW_NONFRAG_IPV6_SCTP) | \
+       (1 << RTE_ETH_FLOW_NONFRAG_IPV6_OTHER))
+
+#define IPV6_ADDR_TO_MASK(ipaddr, ipv6m) do { \
+       uint8_t ipv6_addr[16]; \
+       uint8_t i; \
+       rte_memcpy(ipv6_addr, (ipaddr), sizeof(ipv6_addr));\
+       (ipv6m) = 0; \
+       for (i = 0; i < sizeof(ipv6_addr); i++) { \
+               if (ipv6_addr[i] == UINT8_MAX) \
+                       (ipv6m) |= 1 << i; \
+               else if (ipv6_addr[i] != 0) { \
+                       PMD_DRV_LOG(ERR, " invalid IPv6 address mask."); \
+                       return -EINVAL; \
+               } \
+       } \
+} while (0)
+
+#define IPV6_MASK_TO_ADDR(ipv6m, ipaddr) do { \
+       uint8_t ipv6_addr[16]; \
+       uint8_t i; \
+       for (i = 0; i < sizeof(ipv6_addr); i++) { \
+               if ((ipv6m) & (1 << i)) \
+                       ipv6_addr[i] = UINT8_MAX; \
+               else \
+                       ipv6_addr[i] = 0; \
+       } \
+       rte_memcpy((ipaddr), ipv6_addr, sizeof(ipv6_addr));\
+} while (0)
+
+static int fdir_erase_filter_82599(struct ixgbe_hw *hw, uint32_t fdirhash);
+static int fdir_set_input_mask_82599(struct rte_eth_dev *dev,
+               const struct rte_eth_fdir_masks *input_mask);
+static int ixgbe_set_fdir_flex_conf(struct rte_eth_dev *dev,
+               const struct rte_eth_fdir_flex_conf *conf);
+static int fdir_enable_82599(struct ixgbe_hw *hw, uint32_t fdirctrl);
+static int ixgbe_fdir_filter_to_atr_input(
+               const struct rte_eth_fdir_filter *fdir_filter,
+               union ixgbe_atr_input *input);
+static uint32_t ixgbe_atr_compute_hash_82599(union ixgbe_atr_input *atr_input,
+                                uint32_t key);
+static uint32_t atr_compute_sig_hash_82599(union ixgbe_atr_input *input,
+               enum rte_fdir_pballoc_type pballoc);
+static uint32_t atr_compute_perfect_hash_82599(union ixgbe_atr_input *input,
+               enum rte_fdir_pballoc_type pballoc);
+static int fdir_write_perfect_filter_82599(struct ixgbe_hw *hw,
+                       union ixgbe_atr_input *input, uint8_t queue,
+                       uint32_t fdircmd, uint32_t fdirhash);
+static int fdir_add_signature_filter_82599(struct ixgbe_hw *hw,
+               union ixgbe_atr_input *input, u8 queue, uint32_t fdircmd,
+               uint32_t fdirhash);
+static int ixgbe_add_del_fdir_filter(struct rte_eth_dev *dev,
+                             const struct rte_eth_fdir_filter *fdir_filter,
+                             bool del,
+                             bool update);
+static int ixgbe_fdir_flush(struct rte_eth_dev *dev);
+static void ixgbe_fdir_info_get(struct rte_eth_dev *dev,
+                       struct rte_eth_fdir_info *fdir_info);
+static void ixgbe_fdir_stats_get(struct rte_eth_dev *dev,
+                       struct rte_eth_fdir_stats *fdir_stats);
 
 /**
  * This function is based on ixgbe_fdir_enable_82599() in ixgbe/ixgbe_82599.c.
  *  @hw: pointer to hardware structure
  *  @fdirctrl: value to write to flow director control register
  **/
-static void fdir_enable_82599(struct ixgbe_hw *hw, u32 fdirctrl)
+static int
+fdir_enable_82599(struct ixgbe_hw *hw, uint32_t fdirctrl)
 {
        int i;
 
@@ -111,16 +185,20 @@ static void fdir_enable_82599(struct ixgbe_hw *hw, u32 fdirctrl)
                msec_delay(1);
        }
 
-       if (i >= IXGBE_FDIR_INIT_DONE_POLL)
-               PMD_INIT_LOG(WARNING, "Flow Director poll time exceeded!\n");
+       if (i >= IXGBE_FDIR_INIT_DONE_POLL) {
+               PMD_INIT_LOG(ERR, "Flow Director poll time exceeded "
+                       "during enabling!");
+               return -ETIMEDOUT;
+       }
+       return 0;
 }
 
 /*
  * Set appropriate bits in fdirctrl for: variable reporting levels, moving
  * flexbytes matching field, and drop queue (only for perfect matching mode).
  */
-static int
-configure_fdir_flags(struct rte_fdir_conf *conf, uint32_t *fdirctrl)
+static inline int
+configure_fdir_flags(const struct rte_fdir_conf *conf, uint32_t *fdirctrl)
 {
        *fdirctrl = 0;
 
@@ -162,16 +240,197 @@ configure_fdir_flags(struct rte_fdir_conf *conf, uint32_t *fdirctrl)
                return -EINVAL;
        };
 
-       *fdirctrl |= (conf->flexbytes_offset << IXGBE_FDIRCTRL_FLEX_SHIFT);
+       *fdirctrl |= (IXGBE_DEFAULT_FLEXBYTES_OFFSET / sizeof(uint16_t)) <<
+                    IXGBE_FDIRCTRL_FLEX_SHIFT;
 
        if (conf->mode == RTE_FDIR_MODE_PERFECT) {
                *fdirctrl |= IXGBE_FDIRCTRL_PERFECT_MATCH;
                *fdirctrl |= (conf->drop_queue << IXGBE_FDIRCTRL_DROP_Q_SHIFT);
        }
+       /*
+        * Continue setup of fdirctrl register bits:
+        *  Set the maximum length per hash bucket to 0xA filters
+        *  Send interrupt when 64 filters are left
+        */
+       *fdirctrl |= (0xA << IXGBE_FDIRCTRL_MAX_LENGTH_SHIFT) |
+                   (4 << IXGBE_FDIRCTRL_FULL_THRESH_SHIFT);
 
        return 0;
 }
 
+/**
+ * Reverse the bits in FDIR registers that store 2 x 16 bit masks.
+ *
+ *  @hi_dword: Bits 31:16 mask to be bit swapped.
+ *  @lo_dword: Bits 15:0  mask to be bit swapped.
+ *
+ *  Flow director uses several registers to store 2 x 16 bit masks with the
+ *  bits reversed such as FDIRTCPM, FDIRUDPM. The LS bit of the
+ *  mask affects the MS bit/byte of the target. This function reverses the
+ *  bits in these masks.
+ *  **/
+static inline uint32_t
+reverse_fdir_bitmasks(uint16_t hi_dword, uint16_t lo_dword)
+{
+       uint32_t mask = hi_dword << 16;
+       mask |= lo_dword;
+       mask = ((mask & 0x55555555) << 1) | ((mask & 0xAAAAAAAA) >> 1);
+       mask = ((mask & 0x33333333) << 2) | ((mask & 0xCCCCCCCC) >> 2);
+       mask = ((mask & 0x0F0F0F0F) << 4) | ((mask & 0xF0F0F0F0) >> 4);
+       return ((mask & 0x00FF00FF) << 8) | ((mask & 0xFF00FF00) >> 8);
+}
+
+/*
+ * This is based on ixgbe_fdir_set_input_mask_82599() in ixgbe/ixgbe_82599.c,
+ * but makes use of the rte_fdir_masks structure to see which bits to set.
+ */
+static int
+fdir_set_input_mask_82599(struct rte_eth_dev *dev,
+               const struct rte_eth_fdir_masks *input_mask)
+{
+       struct ixgbe_hw *hw = IXGBE_DEV_PRIVATE_TO_HW(dev->data->dev_private);
+       struct ixgbe_hw_fdir_info *info =
+                       IXGBE_DEV_PRIVATE_TO_FDIR_INFO(dev->data->dev_private);
+       /*
+        * mask VM pool and DIPv6 since there are currently not supported
+        * mask FLEX byte, it will be set in flex_conf
+        */
+       uint32_t fdirm = IXGBE_FDIRM_POOL | IXGBE_FDIRM_DIPv6 | IXGBE_FDIRM_FLEX;
+       uint32_t fdirtcpm;  /* TCP source and destination port masks. */
+       uint32_t fdiripv6m; /* IPv6 source and destination masks. */
+       uint16_t dst_ipv6m = 0;
+       uint16_t src_ipv6m = 0;
+
+       PMD_INIT_FUNC_TRACE();
+
+       /*
+        * Program the relevant mask registers.  If src/dst_port or src/dst_addr
+        * are zero, then assume a full mask for that field. Also assume that
+        * a VLAN of 0 is unspecified, so mask that out as well.  L4type
+        * cannot be masked out in this implementation.
+        */
+       if (input_mask->dst_port_mask == 0 && input_mask->src_port_mask == 0)
+               /* use the L4 protocol mask for raw IPv4/IPv6 traffic */
+               fdirm |= IXGBE_FDIRM_L4P;
+
+       if (input_mask->vlan_tci_mask == 0x0FFF)
+               /* mask VLAN Priority */
+               fdirm |= IXGBE_FDIRM_VLANP;
+       else if (input_mask->vlan_tci_mask == 0xE000)
+               /* mask VLAN ID */
+               fdirm |= IXGBE_FDIRM_VLANID;
+       else if (input_mask->vlan_tci_mask == 0)
+               /* mask VLAN ID and Priority */
+               fdirm |= IXGBE_FDIRM_VLANID | IXGBE_FDIRM_VLANP;
+       else if (input_mask->vlan_tci_mask != 0xEFFF) {
+               PMD_INIT_LOG(ERR, "invalid vlan_tci_mask");
+               return -EINVAL;
+       }
+       info->mask.vlan_tci_mask = input_mask->vlan_tci_mask;
+
+       IXGBE_WRITE_REG(hw, IXGBE_FDIRM, fdirm);
+
+       /* store the TCP/UDP port masks, bit reversed from port layout */
+       fdirtcpm = reverse_fdir_bitmasks(input_mask->dst_port_mask,
+                                        input_mask->src_port_mask);
+
+       /* write both the same so that UDP and TCP use the same mask */
+       IXGBE_WRITE_REG(hw, IXGBE_FDIRTCPM, ~fdirtcpm);
+       IXGBE_WRITE_REG(hw, IXGBE_FDIRUDPM, ~fdirtcpm);
+       info->mask.src_port_mask = input_mask->src_port_mask;
+       info->mask.dst_port_mask = input_mask->dst_port_mask;
+
+       /* Store source and destination IPv4 masks (big-endian) */
+       IXGBE_WRITE_REG(hw, IXGBE_FDIRSIP4M, ~(input_mask->ipv4_mask.src_ip));
+       IXGBE_WRITE_REG(hw, IXGBE_FDIRDIP4M, ~(input_mask->ipv4_mask.dst_ip));
+       info->mask.src_ipv4_mask = input_mask->ipv4_mask.src_ip;
+       info->mask.dst_ipv4_mask = input_mask->ipv4_mask.dst_ip;
+
+       if (dev->data->dev_conf.fdir_conf.mode == RTE_FDIR_MODE_SIGNATURE) {
+               /*
+                * IPv6 mask is only meaningful in signature mode
+                * Store source and destination IPv6 masks (bit reversed)
+                */
+               IPV6_ADDR_TO_MASK(input_mask->ipv6_mask.src_ip, src_ipv6m);
+               IPV6_ADDR_TO_MASK(input_mask->ipv6_mask.dst_ip, dst_ipv6m);
+               fdiripv6m = (dst_ipv6m << 16) | src_ipv6m;
+
+               IXGBE_WRITE_REG(hw, IXGBE_FDIRIP6M, ~fdiripv6m);
+               info->mask.src_ipv6_mask = src_ipv6m;
+               info->mask.dst_ipv6_mask = dst_ipv6m;
+       }
+
+       return IXGBE_SUCCESS;
+}
+
+/*
+ * ixgbe_check_fdir_flex_conf -check if the flex payload and mask configuration
+ * arguments are valid
+ */
+static int
+ixgbe_set_fdir_flex_conf(struct rte_eth_dev *dev,
+               const struct rte_eth_fdir_flex_conf *conf)
+{
+       struct ixgbe_hw *hw = IXGBE_DEV_PRIVATE_TO_HW(dev->data->dev_private);
+       struct ixgbe_hw_fdir_info *info =
+                       IXGBE_DEV_PRIVATE_TO_FDIR_INFO(dev->data->dev_private);
+       const struct rte_eth_flex_payload_cfg *flex_cfg;
+       const struct rte_eth_fdir_flex_mask *flex_mask;
+       uint32_t fdirctrl, fdirm;
+       uint16_t flexbytes = 0;
+       uint16_t i;
+
+       fdirctrl = IXGBE_READ_REG(hw, IXGBE_FDIRCTRL);
+       fdirm = IXGBE_READ_REG(hw, IXGBE_FDIRM);
+
+       if (conf == NULL) {
+               PMD_DRV_LOG(INFO, "NULL pointer.");
+               return -EINVAL;
+       }
+
+       for (i = 0; i < conf->nb_payloads; i++) {
+               flex_cfg = &conf->flex_set[i];
+               if (flex_cfg->type != RTE_ETH_RAW_PAYLOAD) {
+                       PMD_DRV_LOG(ERR, "unsupported payload type.");
+                       return -EINVAL;
+               }
+               if (((flex_cfg->src_offset[0] & 0x1) == 0) &&
+                   (flex_cfg->src_offset[1] == flex_cfg->src_offset[0] + 1) &&
+                   (flex_cfg->src_offset[0] <= IXGBE_MAX_FLX_SOURCE_OFF)) {
+                       fdirctrl &= ~IXGBE_FDIRCTRL_FLEX_MASK;
+                       fdirctrl |= (flex_cfg->src_offset[0] / sizeof(uint16_t)) <<
+                                       IXGBE_FDIRCTRL_FLEX_SHIFT;
+               } else {
+                       PMD_DRV_LOG(ERR, "invalid flexbytes arguments.");
+                       return -EINVAL;
+               }
+       }
+
+       for (i = 0; i < conf->nb_flexmasks; i++) {
+               flex_mask = &conf->flex_mask[i];
+               if (flex_mask->flow_type != RTE_ETH_FLOW_UNKNOWN) {
+                       PMD_DRV_LOG(ERR, "flexmask should be set globally.");
+                       return -EINVAL;
+               }
+               flexbytes = (uint16_t)(((flex_mask->mask[0] << 8) & 0xFF00) |
+                                       ((flex_mask->mask[1]) & 0xFF));
+               if (flexbytes == UINT16_MAX)
+                       fdirm &= ~IXGBE_FDIRM_FLEX;
+               else if (flexbytes != 0) {
+                       /* IXGBE_FDIRM_FLEX is set by default when set mask */
+                       PMD_DRV_LOG(ERR, " invalid flexbytes mask arguments.");
+                       return -EINVAL;
+               }
+       }
+       IXGBE_WRITE_REG(hw, IXGBE_FDIRCTRL, fdirctrl);
+       IXGBE_WRITE_REG(hw, IXGBE_FDIRM, fdirm);
+       info->mask.flex_bytes_mask = flexbytes ? UINT16_MAX : 0;
+       info->flex_bytes_offset = (uint8_t)((fdirctrl &
+                                           IXGBE_FDIRCTRL_FLEX_MASK) >>
+                                           IXGBE_FDIRCTRL_FLEX_SHIFT);
+       return 0;
+}
+
 int
 ixgbe_fdir_configure(struct rte_eth_dev *dev)
 {
@@ -182,7 +441,10 @@ ixgbe_fdir_configure(struct rte_eth_dev *dev)
 
        PMD_INIT_FUNC_TRACE();
 
-       if (hw->mac.type != ixgbe_mac_82599EB && hw->mac.type !=ixgbe_mac_X540)
+       if (hw->mac.type != ixgbe_mac_82599EB &&
+               hw->mac.type != ixgbe_mac_X540 &&
+               hw->mac.type != ixgbe_mac_X550 &&
+               hw->mac.type != ixgbe_mac_X550EM_x)
                return -ENOSYS;
 
        err = configure_fdir_flags(&dev->data->dev_conf.fdir_conf, &fdirctrl);
@@ -207,7 +469,106 @@ ixgbe_fdir_configure(struct rte_eth_dev *dev)
        for (i = 1; i < 8; i++)
                IXGBE_WRITE_REG(hw, IXGBE_RXPBSIZE(i), 0);
 
-       fdir_enable_82599(hw, fdirctrl);
+       err = fdir_set_input_mask_82599(dev, &dev->data->dev_conf.fdir_conf.mask);
+       if (err < 0) {
+               PMD_INIT_LOG(ERR, " Error on setting FD mask");
+               return err;
+       }
+       err = ixgbe_set_fdir_flex_conf(dev,
+               &dev->data->dev_conf.fdir_conf.flex_conf);
+       if (err < 0) {
+               PMD_INIT_LOG(ERR, " Error on setting FD flexible arguments.");
+               return err;
+       }
+
+       err = fdir_enable_82599(hw, fdirctrl);
+       if (err < 0) {
+               PMD_INIT_LOG(ERR, " Error on enabling FD.");
+               return err;
+       }
+       return 0;
+}
+
+/*
+ * Convert DPDK rte_eth_fdir_filter struct to ixgbe_atr_input union that is used
+ * by the IXGBE driver code.
+ */
+static int
+ixgbe_fdir_filter_to_atr_input(const struct rte_eth_fdir_filter *fdir_filter,
+               union ixgbe_atr_input *input)
+{
+       input->formatted.vlan_id = fdir_filter->input.flow_ext.vlan_tci;
+       input->formatted.flex_bytes = (uint16_t)(
+               (fdir_filter->input.flow_ext.flexbytes[1] << 8 & 0xFF00) |
+               (fdir_filter->input.flow_ext.flexbytes[0] & 0xFF));
+
+       switch (fdir_filter->input.flow_type) {
+       case RTE_ETH_FLOW_NONFRAG_IPV4_UDP:
+               input->formatted.flow_type = IXGBE_ATR_FLOW_TYPE_UDPV4;
+               break;
+       case RTE_ETH_FLOW_NONFRAG_IPV4_TCP:
+               input->formatted.flow_type = IXGBE_ATR_FLOW_TYPE_TCPV4;
+               break;
+       case RTE_ETH_FLOW_NONFRAG_IPV4_SCTP:
+               input->formatted.flow_type = IXGBE_ATR_FLOW_TYPE_SCTPV4;
+               break;
+       case RTE_ETH_FLOW_NONFRAG_IPV4_OTHER:
+               input->formatted.flow_type = IXGBE_ATR_FLOW_TYPE_IPV4;
+               break;
+       case RTE_ETH_FLOW_NONFRAG_IPV6_UDP:
+               input->formatted.flow_type = IXGBE_ATR_FLOW_TYPE_UDPV6;
+               break;
+       case RTE_ETH_FLOW_NONFRAG_IPV6_TCP:
+               input->formatted.flow_type = IXGBE_ATR_FLOW_TYPE_TCPV6;
+               break;
+       case RTE_ETH_FLOW_NONFRAG_IPV6_SCTP:
+               input->formatted.flow_type = IXGBE_ATR_FLOW_TYPE_SCTPV6;
+               break;
+       case RTE_ETH_FLOW_NONFRAG_IPV6_OTHER:
+               input->formatted.flow_type = IXGBE_ATR_FLOW_TYPE_IPV6;
+               break;
+       default:
+               PMD_DRV_LOG(ERR, " Error on flow_type input");
+               return -EINVAL;
+       }
+
+       switch (fdir_filter->input.flow_type) {
+       case RTE_ETH_FLOW_NONFRAG_IPV4_UDP:
+       case RTE_ETH_FLOW_NONFRAG_IPV4_TCP:
+               input->formatted.src_port =
+                       fdir_filter->input.flow.udp4_flow.src_port;
+               input->formatted.dst_port =
+                       fdir_filter->input.flow.udp4_flow.dst_port;
+       /*for SCTP flow type, port and verify_tag are meaningless in ixgbe.*/
+       case RTE_ETH_FLOW_NONFRAG_IPV4_SCTP:
+       case RTE_ETH_FLOW_NONFRAG_IPV4_OTHER:
+               input->formatted.src_ip[0] =
+                       fdir_filter->input.flow.ip4_flow.src_ip;
+               input->formatted.dst_ip[0] =
+                       fdir_filter->input.flow.ip4_flow.dst_ip;
+               break;
+
+       case RTE_ETH_FLOW_NONFRAG_IPV6_UDP:
+       case RTE_ETH_FLOW_NONFRAG_IPV6_TCP:
+               input->formatted.src_port =
+                       fdir_filter->input.flow.udp6_flow.src_port;
+               input->formatted.dst_port =
+                       fdir_filter->input.flow.udp6_flow.dst_port;
+       /*for SCTP flow type, port and verify_tag are meaningless in ixgbe.*/
+       case RTE_ETH_FLOW_NONFRAG_IPV6_SCTP:
+       case RTE_ETH_FLOW_NONFRAG_IPV6_OTHER:
+               rte_memcpy(input->formatted.src_ip,
+                          fdir_filter->input.flow.ipv6_flow.src_ip,
+                          sizeof(input->formatted.src_ip));
+               rte_memcpy(input->formatted.dst_ip,
+                          fdir_filter->input.flow.ipv6_flow.dst_ip,
+                          sizeof(input->formatted.dst_ip));
+               break;
+       default:
+               PMD_DRV_LOG(ERR, " Error on flow_type input");
+               return -EINVAL;
+       }
+
        return 0;
 }
 
@@ -227,9 +588,9 @@ ixgbe_fdir_configure(struct rte_eth_dev *dev)
  *  @stream: input bitstream to compute the hash on
  *  @key: 32-bit hash key
  **/
-static u32
+static uint32_t
 ixgbe_atr_compute_hash_82599(union ixgbe_atr_input *atr_input,
-                                u32 key)
+                                uint32_t key)
 {
        /*
         * The algorithm is as follows:
@@ -280,9 +641,8 @@ ixgbe_atr_compute_hash_82599(union ixgbe_atr_input *atr_input,
        flow_vm_vlan = IXGBE_NTOHL(atr_input->dword_stream[0]);
 
        /* generate common hash dword */
-       for (i = 10; i; i -= 2)
-               common_hash_dword ^= atr_input->dword_stream[i] ^
-                                    atr_input->dword_stream[i - 1];
+       for (i = 1; i <= 13; i++)
+               common_hash_dword ^= atr_input->dword_stream[i];
 
        hi_hash_dword = IXGBE_NTOHL(common_hash_dword);
 
@@ -313,6 +673,43 @@ ixgbe_atr_compute_hash_82599(union ixgbe_atr_input *atr_input,
        return hash_result;
 }
 
+static uint32_t
+atr_compute_perfect_hash_82599(union ixgbe_atr_input *input,
+               enum rte_fdir_pballoc_type pballoc)
+{
+       if (pballoc == RTE_FDIR_PBALLOC_256K)
+               return ixgbe_atr_compute_hash_82599(input,
+                               IXGBE_ATR_BUCKET_HASH_KEY) &
+                               PERFECT_BUCKET_256KB_HASH_MASK;
+       else if (pballoc == RTE_FDIR_PBALLOC_128K)
+               return ixgbe_atr_compute_hash_82599(input,
+                               IXGBE_ATR_BUCKET_HASH_KEY) &
+                               PERFECT_BUCKET_128KB_HASH_MASK;
+       else
+               return ixgbe_atr_compute_hash_82599(input,
+                               IXGBE_ATR_BUCKET_HASH_KEY) &
+                               PERFECT_BUCKET_64KB_HASH_MASK;
+}
+
+/**
+ * ixgbe_fdir_check_cmd_complete - poll to check whether FDIRCMD is complete
+ * @hw: pointer to hardware structure
+ */
+static inline int
+ixgbe_fdir_check_cmd_complete(struct ixgbe_hw *hw, uint32_t *fdircmd)
+{
+       int i;
+
+       for (i = 0; i < IXGBE_FDIRCMD_CMD_POLL; i++) {
+               *fdircmd = IXGBE_READ_REG(hw, IXGBE_FDIRCMD);
+               if (!(*fdircmd & IXGBE_FDIRCMD_CMD_MASK))
+                       return 0;
+               rte_delay_us(IXGBE_FDIRCMD_CMD_INTERVAL_US);
+       }
+
+       return -ETIMEDOUT;
+}
+
 /*
  * Calculate the hash value needed for signature-match filters. In the FreeBSD
  * driver, this is done by the optimised function
@@ -344,6 +741,63 @@ atr_compute_sig_hash_82599(union ixgbe_atr_input *input,
        return (sig_hash << IXGBE_FDIRHASH_SIG_SW_INDEX_SHIFT) | bucket_hash;
 }
 
+/*
+ * This is based on ixgbe_fdir_write_perfect_filter_82599() in
+ * ixgbe/ixgbe_82599.c, with the ability to set extra flags in FDIRCMD register
+ * added, and IPv6 support also added. The hash value is also pre-calculated
+ * as the pballoc value is needed to do it.
+ */
+static int
+fdir_write_perfect_filter_82599(struct ixgbe_hw *hw,
+                       union ixgbe_atr_input *input, uint8_t queue,
+                       uint32_t fdircmd, uint32_t fdirhash)
+{
+       uint32_t fdirport, fdirvlan;
+       int err = 0;
+
+       /* record the IPv4 address (big-endian) */
+       IXGBE_WRITE_REG(hw, IXGBE_FDIRIPSA, input->formatted.src_ip[0]);
+       IXGBE_WRITE_REG(hw, IXGBE_FDIRIPDA, input->formatted.dst_ip[0]);
+
+       /* record source and destination port (little-endian)*/
+       fdirport = IXGBE_NTOHS(input->formatted.dst_port);
+       fdirport <<= IXGBE_FDIRPORT_DESTINATION_SHIFT;
+       fdirport |= IXGBE_NTOHS(input->formatted.src_port);
+       IXGBE_WRITE_REG(hw, IXGBE_FDIRPORT, fdirport);
+
+       /* record vlan (little-endian) and flex_bytes(big-endian) */
+       fdirvlan = input->formatted.flex_bytes;
+       fdirvlan <<= IXGBE_FDIRVLAN_FLEX_SHIFT;
+       fdirvlan |= IXGBE_NTOHS(input->formatted.vlan_id);
+       IXGBE_WRITE_REG(hw, IXGBE_FDIRVLAN, fdirvlan);
+
+       /* configure FDIRHASH register */
+       IXGBE_WRITE_REG(hw, IXGBE_FDIRHASH, fdirhash);
+
+       /*
+        * flush all previous writes to make certain registers are
+        * programmed prior to issuing the command
+        */
+       IXGBE_WRITE_FLUSH(hw);
+
+       /* configure FDIRCMD register */
+       fdircmd |= IXGBE_FDIRCMD_CMD_ADD_FLOW |
+                 IXGBE_FDIRCMD_LAST | IXGBE_FDIRCMD_QUEUE_EN;
+       fdircmd |= input->formatted.flow_type << IXGBE_FDIRCMD_FLOW_TYPE_SHIFT;
+       fdircmd |= (uint32_t)queue << IXGBE_FDIRCMD_RX_QUEUE_SHIFT;
+       fdircmd |= (uint32_t)input->formatted.vm_pool << IXGBE_FDIRCMD_VT_POOL_SHIFT;
+
+       IXGBE_WRITE_REG(hw, IXGBE_FDIRCMD, fdircmd);
+
+       PMD_DRV_LOG(DEBUG, "Rx Queue=%x hash=%x", queue, fdirhash);
+
+       err = ixgbe_fdir_check_cmd_complete(hw, &fdircmd);
+       if (err < 0)
+               PMD_DRV_LOG(ERR, "Timeout writing flow director filter.");
+
+       return err;
+}
+
 /**
  * This function is based on ixgbe_atr_add_signature_filter_82599() in
  * ixgbe/ixgbe_82599.c, but uses a pre-calculated hash value. It also supports
@@ -359,12 +813,12 @@ atr_compute_sig_hash_82599(union ixgbe_atr_input *input,
  *  @fdircmd: any extra flags to set in fdircmd register
  *  @fdirhash: pre-calculated hash value for the filter
  **/
-static void
+static int
 fdir_add_signature_filter_82599(struct ixgbe_hw *hw,
-               union ixgbe_atr_input *input, u8 queue, u32 fdircmd,
-               u32 fdirhash)
+               union ixgbe_atr_input *input, u8 queue, uint32_t fdircmd,
+               uint32_t fdirhash)
 {
-       u64  fdirhashcmd;
+       int err = 0;
 
        PMD_INIT_FUNC_TRACE();
 
@@ -372,126 +826,18 @@ fdir_add_signature_filter_82599(struct ixgbe_hw *hw,
        fdircmd |= IXGBE_FDIRCMD_CMD_ADD_FLOW |
                  IXGBE_FDIRCMD_LAST | IXGBE_FDIRCMD_QUEUE_EN;
        fdircmd |= input->formatted.flow_type << IXGBE_FDIRCMD_FLOW_TYPE_SHIFT;
-       fdircmd |= (u32)queue << IXGBE_FDIRCMD_RX_QUEUE_SHIFT;
-
-       /*
-        * The lower 32-bits of fdirhashcmd is for FDIRHASH, the upper 32-bits
-        * is for FDIRCMD.  Then do a 64-bit register write from FDIRHASH.
-        */
-       fdirhashcmd = (u64)fdircmd << 32;
-       fdirhashcmd |= fdirhash;
-       IXGBE_WRITE_REG64(hw, IXGBE_FDIRHASH, fdirhashcmd);
-
-       PMD_INIT_LOG(DEBUG, "Tx Queue=%x hash=%x\n", queue, (u32)fdirhashcmd);
-}
-
-/*
- * Convert DPDK rte_fdir_filter struct to ixgbe_atr_input union that is used
- * by the IXGBE driver code.
- */
-static int
-fdir_filter_to_atr_input(struct rte_fdir_filter *fdir_filter,
-               union ixgbe_atr_input *input)
-{
-       if ((fdir_filter->l4type == RTE_FDIR_L4TYPE_SCTP ||
-                       fdir_filter->l4type == RTE_FDIR_L4TYPE_NONE) &&
-                       (fdir_filter->port_src || fdir_filter->port_dst)) {
-               PMD_INIT_LOG(ERR, "Invalid fdir_filter");
-               return -EINVAL;
-       }
-
-       memset(input, 0, sizeof(*input));
-
-       input->formatted.vlan_id = fdir_filter->vlan_id;
-       input->formatted.src_port = fdir_filter->port_src;
-       input->formatted.dst_port = fdir_filter->port_dst;
-       input->formatted.flex_bytes = fdir_filter->flex_bytes;
+       fdircmd |= (uint32_t)queue << IXGBE_FDIRCMD_RX_QUEUE_SHIFT;
 
-       switch (fdir_filter->l4type) {
-       case RTE_FDIR_L4TYPE_TCP:
-               input->formatted.flow_type = IXGBE_ATR_FLOW_TYPE_TCPV4;
-               break;
-       case RTE_FDIR_L4TYPE_UDP:
-               input->formatted.flow_type = IXGBE_ATR_FLOW_TYPE_UDPV4;
-               break;
-       case RTE_FDIR_L4TYPE_SCTP:
-               input->formatted.flow_type = IXGBE_ATR_FLOW_TYPE_SCTPV4;
-               break;
-       case RTE_FDIR_L4TYPE_NONE:
-               input->formatted.flow_type = IXGBE_ATR_FLOW_TYPE_IPV4;
-               break;
-       default:
-               PMD_INIT_LOG(ERR, " Error on l4type input");
-               return -EINVAL;
-       }
-
-       if (fdir_filter->iptype == RTE_FDIR_IPTYPE_IPV6) {
-               input->formatted.flow_type |= IXGBE_ATR_L4TYPE_IPV6_MASK;
-
-               input->formatted.src_ip[0] = fdir_filter->ip_src.ipv6_addr[0];
-               input->formatted.src_ip[1] = fdir_filter->ip_src.ipv6_addr[1];
-               input->formatted.src_ip[2] = fdir_filter->ip_src.ipv6_addr[2];
-               input->formatted.src_ip[3] = fdir_filter->ip_src.ipv6_addr[3];
-
-               input->formatted.dst_ip[0] = fdir_filter->ip_dst.ipv6_addr[0];
-               input->formatted.dst_ip[1] = fdir_filter->ip_dst.ipv6_addr[1];
-               input->formatted.dst_ip[2] = fdir_filter->ip_dst.ipv6_addr[2];
-               input->formatted.dst_ip[3] = fdir_filter->ip_dst.ipv6_addr[3];
-
-       } else {
-               input->formatted.src_ip[0] = fdir_filter->ip_src.ipv4_addr;
-               input->formatted.dst_ip[0] = fdir_filter->ip_dst.ipv4_addr;
-       }
-
-       return 0;
-}
-
-/*
- * Adds or updates a signature filter.
- *
- * dev: ethernet device to add filter to
- * fdir_filter: filter details
- * queue: queue index to direct traffic to
- * update: 0 to add a new filter, otherwise update existing.
- */
-static int
-fdir_add_update_signature_filter(struct rte_eth_dev *dev,
-               struct rte_fdir_filter *fdir_filter, uint8_t queue, int update)
-{
-       struct ixgbe_hw *hw= IXGBE_DEV_PRIVATE_TO_HW(dev->data->dev_private);
-       uint32_t fdircmd_flags = (update) ? IXGBE_FDIRCMD_FILTER_UPDATE : 0;
-       uint32_t fdirhash;
-       union ixgbe_atr_input input;
-       int err;
-
-       if (hw->mac.type != ixgbe_mac_82599EB && hw->mac.type !=ixgbe_mac_X540)
-               return -ENOSYS;
-
-       err = fdir_filter_to_atr_input(fdir_filter, &input);
-       if (err)
-               return err;
+       IXGBE_WRITE_REG(hw, IXGBE_FDIRHASH, fdirhash);
+       IXGBE_WRITE_REG(hw, IXGBE_FDIRCMD, fdircmd);
 
-       fdirhash = atr_compute_sig_hash_82599(&input,
-                       dev->data->dev_conf.fdir_conf.pballoc);
-       fdir_add_signature_filter_82599(hw, &input, queue, fdircmd_flags,
-                       fdirhash);
-       return 0;
-}
+       PMD_DRV_LOG(DEBUG, "Rx Queue=%x hash=%x", queue, fdirhash);
 
-int
-ixgbe_fdir_add_signature_filter(struct rte_eth_dev *dev,
-               struct rte_fdir_filter *fdir_filter, uint8_t queue)
-{
-       PMD_INIT_FUNC_TRACE();
-       return fdir_add_update_signature_filter(dev, fdir_filter, queue, 0);
-}
+       err = ixgbe_fdir_check_cmd_complete(hw, &fdircmd);
+       if (err < 0)
+               PMD_DRV_LOG(ERR, "Timeout writing flow director filter.");
 
-int
-ixgbe_fdir_update_signature_filter(struct rte_eth_dev *dev,
-               struct rte_fdir_filter *fdir_filter, uint8_t queue)
-{
-       PMD_INIT_FUNC_TRACE();
-       return fdir_add_update_signature_filter(dev, fdir_filter, queue, 1);
+       return err;
 }
 
 /*
@@ -499,13 +845,11 @@ ixgbe_fdir_update_signature_filter(struct rte_eth_dev *dev,
  * ixgbe/ixgbe_82599.c. It is modified to take in the hash as a parameter so
  * that it can be used for removing signature and perfect filters.
  */
-static s32
-fdir_erase_filter_82599(struct ixgbe_hw *hw,
-       __rte_unused union ixgbe_atr_input *input, uint32_t fdirhash)
+static int
+fdir_erase_filter_82599(struct ixgbe_hw *hw, uint32_t fdirhash)
 {
-       u32 fdircmd = 0;
-       u32 retry_count;
-       s32 err = 0;
+       uint32_t fdircmd = 0;
+       int err = 0;
 
        IXGBE_WRITE_REG(hw, IXGBE_FDIRHASH, fdirhash);
 
@@ -515,18 +859,10 @@ fdir_erase_filter_82599(struct ixgbe_hw *hw,
        /* Query if filter is present */
        IXGBE_WRITE_REG(hw, IXGBE_FDIRCMD, IXGBE_FDIRCMD_CMD_QUERY_REM_FILT);
 
-       for (retry_count = 10; retry_count; retry_count--) {
-               /* allow 10us for query to process */
-               usec_delay(10);
-               /* verify query completed successfully */
-               fdircmd = IXGBE_READ_REG(hw, IXGBE_FDIRCMD);
-               if (!(fdircmd & IXGBE_FDIRCMD_CMD_MASK))
-                       break;
-       }
-
-       if (!retry_count) {
-               PMD_INIT_LOG(ERR, "Timeout querying for flow director filter");
-               err = -EIO;
+       err = ixgbe_fdir_check_cmd_complete(hw, &fdircmd);
+       if (err < 0) {
+               PMD_INIT_LOG(ERR, "Timeout querying for flow director filter.");
+               return err;
        }
 
        /* if filter exists in hardware then remove it */
@@ -536,331 +872,176 @@ fdir_erase_filter_82599(struct ixgbe_hw *hw,
                IXGBE_WRITE_REG(hw, IXGBE_FDIRCMD,
                                IXGBE_FDIRCMD_CMD_REMOVE_FLOW);
        }
-
+       err = ixgbe_fdir_check_cmd_complete(hw, &fdircmd);
+       if (err < 0)
+               PMD_INIT_LOG(ERR, "Timeout erasing flow director filter.");
        return err;
+
 }
 
-int
-ixgbe_fdir_remove_signature_filter(struct rte_eth_dev *dev,
-               struct rte_fdir_filter *fdir_filter)
+/*
+ * ixgbe_add_del_fdir_filter - add or remove a flow diretor filter.
+ * @dev: pointer to the structure rte_eth_dev
+ * @fdir_filter: fdir filter entry
+ * @del: 1 - delete, 0 - add
+ * @update: 1 - update
+ */
+static int
+ixgbe_add_del_fdir_filter(struct rte_eth_dev *dev,
+                             const struct rte_eth_fdir_filter *fdir_filter,
+                             bool del,
+                             bool update)
 {
        struct ixgbe_hw *hw = IXGBE_DEV_PRIVATE_TO_HW(dev->data->dev_private);
+       uint32_t fdircmd_flags;
+       uint32_t fdirhash;
        union ixgbe_atr_input input;
+       uint8_t queue;
+       bool is_perfect = FALSE;
        int err;
 
-       PMD_INIT_FUNC_TRACE();
+       if (dev->data->dev_conf.fdir_conf.mode == RTE_FDIR_MODE_NONE)
+               return -ENOTSUP;
 
-       if (hw->mac.type != ixgbe_mac_82599EB && hw->mac.type !=ixgbe_mac_X540)
-               return -ENOSYS;
+       if (dev->data->dev_conf.fdir_conf.mode == RTE_FDIR_MODE_PERFECT)
+               is_perfect = TRUE;
 
-       err = fdir_filter_to_atr_input(fdir_filter, &input);
+       memset(&input, 0, sizeof(input));
+
+       err = ixgbe_fdir_filter_to_atr_input(fdir_filter, &input);
        if (err)
                return err;
 
-       return fdir_erase_filter_82599(hw, &input,
-                       atr_compute_sig_hash_82599(&input,
-                       dev->data->dev_conf.fdir_conf.pballoc));
-}
-
-/**
- * Reverse the bits in FDIR registers that store 2 x 16 bit masks.
- *
- *  @hi_dword: Bits 31:16 mask to be bit swapped.
- *  @lo_dword: Bits 15:0  mask to be bit swapped.
- *
- *  Flow director uses several registers to store 2 x 16 bit masks with the
- *  bits reversed such as FDIRTCPM, FDIRUDPM and FDIRIP6M. The LS bit of the
- *  mask affects the MS bit/byte of the target. This function reverses the
- *  bits in these masks.
- *  **/
-static uint32_t
-reverse_fdir_bitmasks(uint16_t hi_dword, uint16_t lo_dword)
-{
-       u32 mask = hi_dword << 16;
-       mask |= lo_dword;
-       mask = ((mask & 0x55555555) << 1) | ((mask & 0xAAAAAAAA) >> 1);
-       mask = ((mask & 0x33333333) << 2) | ((mask & 0xCCCCCCCC) >> 2);
-       mask = ((mask & 0x0F0F0F0F) << 4) | ((mask & 0xF0F0F0F0) >> 4);
-       return ((mask & 0x00FF00FF) << 8) | ((mask & 0xFF00FF00) >> 8);
-}
-
-/*
- * This macro exists in ixgbe/ixgbe_82599.c, however in that file it reverses
- * the bytes, and then reverses them again. So here it does nothing.
- */
-#define IXGBE_WRITE_REG_BE32 IXGBE_WRITE_REG
-
-/*
- * This is based on ixgbe_fdir_set_input_mask_82599() in ixgbe/ixgbe_82599.c,
- * but makes use of the rte_fdir_masks structure to see which bits to set.
- */
-static int
-fdir_set_input_mask_82599(struct ixgbe_hw *hw,
-               struct rte_fdir_masks *input_mask)
-{
-       /* mask VM pool since it is currently not supported */
-       u32 fdirm = IXGBE_FDIRM_POOL;
-       u32 fdirtcpm;  /* TCP source and destination port masks. */
-       u32 fdiripv6m; /* IPv6 source and destination masks. */
-
-       PMD_INIT_FUNC_TRACE();
-
-       /*
-        * Program the relevant mask registers.  If src/dst_port or src/dst_addr
-        * are zero, then assume a full mask for that field. Also assume that
-        * a VLAN of 0 is unspecified, so mask that out as well.  L4type
-        * cannot be masked out in this implementation.
-        */
-       if (input_mask->only_ip_flow) {
-               /* use the L4 protocol mask for raw IPv4/IPv6 traffic */
-               fdirm |= IXGBE_FDIRM_L4P;
-               if (input_mask->dst_port_mask || input_mask->src_port_mask) {
-                       PMD_INIT_LOG(ERR, " Error on src/dst port mask\n");
-                       return -EINVAL;
+       if (is_perfect) {
+               if (input.formatted.flow_type & IXGBE_ATR_L4TYPE_IPV6_MASK) {
+                       PMD_DRV_LOG(ERR, "IPv6 is not supported in"
+                                        " perfect mode!");
+                       return -ENOTSUP;
                }
+               fdirhash = atr_compute_perfect_hash_82599(&input,
+                               dev->data->dev_conf.fdir_conf.pballoc);
+               fdirhash |= fdir_filter->soft_id <<
+                               IXGBE_FDIRHASH_SIG_SW_INDEX_SHIFT;
+       } else
+               fdirhash = atr_compute_sig_hash_82599(&input,
+                               dev->data->dev_conf.fdir_conf.pballoc);
+
+       if (del) {
+               err = fdir_erase_filter_82599(hw, fdirhash);
+               if (err < 0)
+                       PMD_DRV_LOG(ERR, "Fail to delete FDIR filter!");
+               else
+                       PMD_DRV_LOG(DEBUG, "Success to delete FDIR filter!");
+               return err;
        }
-
-       if (!input_mask->comp_ipv6_dst)
-               /* mask DIPV6 */
-               fdirm |= IXGBE_FDIRM_DIPv6;
-
-       if (!input_mask->vlan_id)
-               /* mask VLAN ID*/
-               fdirm |= IXGBE_FDIRM_VLANID;
-
-       if (!input_mask->vlan_prio)
-               /* mask VLAN priority */
-               fdirm |= IXGBE_FDIRM_VLANP;
-
-       if (!input_mask->flexbytes)
-               /* Mask Flex Bytes */
-               fdirm |= IXGBE_FDIRM_FLEX;
-
-       IXGBE_WRITE_REG(hw, IXGBE_FDIRM, fdirm);
-
-       /* store the TCP/UDP port masks, bit reversed from port layout */
-       fdirtcpm = reverse_fdir_bitmasks(input_mask->dst_port_mask,
-                                        input_mask->src_port_mask);
-
-       /* write both the same so that UDP and TCP use the same mask */
-       IXGBE_WRITE_REG(hw, IXGBE_FDIRTCPM, ~fdirtcpm);
-       IXGBE_WRITE_REG(hw, IXGBE_FDIRUDPM, ~fdirtcpm);
-
-       if (!input_mask->set_ipv6_mask) {
-               /* Store source and destination IPv4 masks (big-endian) */
-               IXGBE_WRITE_REG_BE32(hw, IXGBE_FDIRSIP4M,
-                               IXGBE_NTOHL(~input_mask->src_ipv4_mask));
-               IXGBE_WRITE_REG_BE32(hw, IXGBE_FDIRDIP4M,
-                               IXGBE_NTOHL(~input_mask->dst_ipv4_mask));
-       }
-       else {
-               /* Store source and destination IPv6 masks (bit reversed) */
-               fdiripv6m = reverse_fdir_bitmasks(input_mask->dst_ipv6_mask,
-                                                 input_mask->src_ipv6_mask);
-
-               IXGBE_WRITE_REG(hw, IXGBE_FDIRIP6M, ~fdiripv6m);
-       }
-
-       return IXGBE_SUCCESS;
-}
-
-int
-ixgbe_fdir_set_masks(struct rte_eth_dev *dev, struct rte_fdir_masks *fdir_masks)
-{
-       struct ixgbe_hw *hw;
-       int err;
-
-       PMD_INIT_FUNC_TRACE();
-
-       hw = IXGBE_DEV_PRIVATE_TO_HW(dev->data->dev_private);
-
-       if (hw->mac.type != ixgbe_mac_82599EB && hw->mac.type !=ixgbe_mac_X540)
-               return -ENOSYS;
-
-       err = ixgbe_reinit_fdir_tables_82599(hw);
-       if (err) {
-               PMD_INIT_LOG(ERR, "reinit of fdir tables failed");
-               return -EIO;
-       }
-
-       return fdir_set_input_mask_82599(hw, fdir_masks);
-}
-
-static uint32_t
-atr_compute_perfect_hash_82599(union ixgbe_atr_input *input,
-               enum rte_fdir_pballoc_type pballoc)
-{
-       if (pballoc == RTE_FDIR_PBALLOC_256K)
-               return ixgbe_atr_compute_hash_82599(input,
-                               IXGBE_ATR_BUCKET_HASH_KEY) &
-                               PERFECT_BUCKET_256KB_HASH_MASK;
-       else if (pballoc == RTE_FDIR_PBALLOC_128K)
-               return ixgbe_atr_compute_hash_82599(input,
-                               IXGBE_ATR_BUCKET_HASH_KEY) &
-                               PERFECT_BUCKET_128KB_HASH_MASK;
+       /* add or update an fdir filter*/
+       fdircmd_flags = (update) ? IXGBE_FDIRCMD_FILTER_UPDATE : 0;
+       if (fdir_filter->action.behavior == RTE_ETH_FDIR_REJECT) {
+               if (is_perfect) {
+                       queue = dev->data->dev_conf.fdir_conf.drop_queue;
+                       fdircmd_flags |= IXGBE_FDIRCMD_DROP;
+               } else {
+                       PMD_DRV_LOG(ERR, "Drop option is not supported in"
+                               " signature mode.");
+                       return -EINVAL;
+               }
+       } else if (fdir_filter->action.rx_queue < IXGBE_MAX_RX_QUEUE_NUM)
+               queue = (uint8_t)fdir_filter->action.rx_queue;
        else
-               return ixgbe_atr_compute_hash_82599(input,
-                               IXGBE_ATR_BUCKET_HASH_KEY) &
-                               PERFECT_BUCKET_64KB_HASH_MASK;
-}
+               return -EINVAL;
 
-/*
- * This is based on ixgbe_fdir_write_perfect_filter_82599() in
- * ixgbe/ixgbe_82599.c, with the ability to set extra flags in FDIRCMD register
- * added, and IPv6 support also added. The hash value is also pre-calculated
- * as the pballoc value is needed to do it.
- */
-static void
-fdir_write_perfect_filter_82599(struct ixgbe_hw *hw, union ixgbe_atr_input *input,
-               uint16_t soft_id, uint8_t queue, uint32_t fdircmd,
-               uint32_t fdirhash)
-{
-       u32 fdirport, fdirvlan;
-
-       /* record the source address (big-endian) */
-       if (input->formatted.flow_type & IXGBE_ATR_L4TYPE_IPV6_MASK) {
-               IXGBE_WRITE_REG_BE32(hw, IXGBE_FDIRSIPv6(0), input->formatted.src_ip[0]);
-               IXGBE_WRITE_REG_BE32(hw, IXGBE_FDIRSIPv6(1), input->formatted.src_ip[1]);
-               IXGBE_WRITE_REG_BE32(hw, IXGBE_FDIRSIPv6(2), input->formatted.src_ip[2]);
-               IXGBE_WRITE_REG_BE32(hw, IXGBE_FDIRIPSA, input->formatted.src_ip[3]);
-       }
-       else {
-               IXGBE_WRITE_REG_BE32(hw, IXGBE_FDIRIPSA, input->formatted.src_ip[0]);
+       if (is_perfect) {
+               err = fdir_write_perfect_filter_82599(hw, &input, queue,
+                               fdircmd_flags, fdirhash);
+       } else {
+               err = fdir_add_signature_filter_82599(hw, &input, queue,
+                               fdircmd_flags, fdirhash);
        }
+       if (err < 0)
+               PMD_DRV_LOG(ERR, "Fail to add FDIR filter!");
+       else
+               PMD_DRV_LOG(DEBUG, "Success to add FDIR filter");
 
-       /* record the first 32 bits of the destination address (big-endian) */
-       IXGBE_WRITE_REG_BE32(hw, IXGBE_FDIRIPDA, input->formatted.dst_ip[0]);
-
-       /* record source and destination port (little-endian)*/
-       fdirport = IXGBE_NTOHS(input->formatted.dst_port);
-       fdirport <<= IXGBE_FDIRPORT_DESTINATION_SHIFT;
-       fdirport |= IXGBE_NTOHS(input->formatted.src_port);
-       IXGBE_WRITE_REG(hw, IXGBE_FDIRPORT, fdirport);
-
-       /* record vlan (little-endian) and flex_bytes(big-endian) */
-       fdirvlan = input->formatted.flex_bytes;
-       fdirvlan <<= IXGBE_FDIRVLAN_FLEX_SHIFT;
-       fdirvlan |= IXGBE_NTOHS(input->formatted.vlan_id);
-       IXGBE_WRITE_REG(hw, IXGBE_FDIRVLAN, fdirvlan);
-
-       /* configure FDIRHASH register */
-       fdirhash |= soft_id << IXGBE_FDIRHASH_SIG_SW_INDEX_SHIFT;
-       IXGBE_WRITE_REG(hw, IXGBE_FDIRHASH, fdirhash);
-
-       /*
-        * flush all previous writes to make certain registers are
-        * programmed prior to issuing the command
-        */
-       IXGBE_WRITE_FLUSH(hw);
-
-       /* configure FDIRCMD register */
-       fdircmd |= IXGBE_FDIRCMD_CMD_ADD_FLOW |
-                 IXGBE_FDIRCMD_LAST | IXGBE_FDIRCMD_QUEUE_EN;
-       fdircmd |= input->formatted.flow_type << IXGBE_FDIRCMD_FLOW_TYPE_SHIFT;
-       fdircmd |= (u32)queue << IXGBE_FDIRCMD_RX_QUEUE_SHIFT;
-       fdircmd |= (u32)input->formatted.vm_pool << IXGBE_FDIRCMD_VT_POOL_SHIFT;
-
-       IXGBE_WRITE_REG(hw, IXGBE_FDIRCMD, fdircmd);
+       return err;
 }
 
-/*
- * Adds or updates a perfect filter.
- *
- * dev: ethernet device to add filter to
- * fdir_filter: filter details
- * soft_id: software index for the filters
- * queue: queue index to direct traffic to
- * drop: non-zero if packets should be sent to the drop queue
- * update: 0 to add a new filter, otherwise update existing.
- */
 static int
-fdir_add_update_perfect_filter(struct rte_eth_dev *dev,
-               struct rte_fdir_filter *fdir_filter, uint16_t soft_id,
-               uint8_t queue, int drop, int update)
+ixgbe_fdir_flush(struct rte_eth_dev *dev)
 {
        struct ixgbe_hw *hw = IXGBE_DEV_PRIVATE_TO_HW(dev->data->dev_private);
-       uint32_t fdircmd_flags = (update) ? IXGBE_FDIRCMD_FILTER_UPDATE : 0;
-       uint32_t fdirhash;
-       union ixgbe_atr_input input;
-       int err;
-
-       if (hw->mac.type != ixgbe_mac_82599EB && hw->mac.type !=ixgbe_mac_X540)
-               return -ENOSYS;
-
-       err = fdir_filter_to_atr_input(fdir_filter, &input);
-       if (err)
-               return err;
+       struct ixgbe_hw_fdir_info *info =
+                       IXGBE_DEV_PRIVATE_TO_FDIR_INFO(dev->data->dev_private);
+       int ret;
 
-       if (drop) {
-               queue = dev->data->dev_conf.fdir_conf.drop_queue;
-               fdircmd_flags |= IXGBE_FDIRCMD_DROP;
+       ret = ixgbe_reinit_fdir_tables_82599(hw);
+       if (ret < 0) {
+               PMD_INIT_LOG(ERR, "Failed to re-initialize FD table.");
+               return ret;
        }
 
-       fdirhash = atr_compute_perfect_hash_82599(&input,
-                       dev->data->dev_conf.fdir_conf.pballoc);
-
-       fdir_write_perfect_filter_82599(hw, &input, soft_id, queue,
-                       fdircmd_flags, fdirhash);
-       return 0;
-}
+       info->f_add = 0;
+       info->f_remove = 0;
+       info->add = 0;
+       info->remove = 0;
 
-int
-ixgbe_fdir_add_perfect_filter(struct rte_eth_dev *dev,
-               struct rte_fdir_filter *fdir_filter, uint16_t soft_id,
-               uint8_t queue, uint8_t drop)
-{
-       PMD_INIT_FUNC_TRACE();
-       return fdir_add_update_perfect_filter(dev, fdir_filter, soft_id, queue,
-                       drop, 0);
+       return ret;
 }
 
-int
-ixgbe_fdir_update_perfect_filter(struct rte_eth_dev *dev,
-               struct rte_fdir_filter *fdir_filter, uint16_t soft_id,
-               uint8_t queue, uint8_t drop)
-{
-       PMD_INIT_FUNC_TRACE();
-       return fdir_add_update_perfect_filter(dev, fdir_filter, soft_id, queue,
-                       drop, 1);
-}
-
-int
-ixgbe_fdir_remove_perfect_filter(struct rte_eth_dev *dev,
-               struct rte_fdir_filter *fdir_filter,
-               uint16_t soft_id)
+#define FDIRENTRIES_NUM_SHIFT 10
+static void
+ixgbe_fdir_info_get(struct rte_eth_dev *dev, struct rte_eth_fdir_info *fdir_info)
 {
        struct ixgbe_hw *hw = IXGBE_DEV_PRIVATE_TO_HW(dev->data->dev_private);
-       union ixgbe_atr_input input;
-       uint32_t fdirhash;
-       int err;
-
-       PMD_INIT_FUNC_TRACE();
-
-       if (hw->mac.type != ixgbe_mac_82599EB && hw->mac.type !=ixgbe_mac_X540)
-               return -ENOSYS;
-
-       err = fdir_filter_to_atr_input(fdir_filter, &input);
-       if (err)
-               return err;
-
-       /* configure FDIRHASH register */
-       fdirhash = atr_compute_perfect_hash_82599(&input,
-                       dev->data->dev_conf.fdir_conf.pballoc);
-       fdirhash |= soft_id << IXGBE_FDIRHASH_SIG_SW_INDEX_SHIFT;
-
-       return fdir_erase_filter_82599(hw, &input, fdirhash);
+       struct ixgbe_hw_fdir_info *info =
+                       IXGBE_DEV_PRIVATE_TO_FDIR_INFO(dev->data->dev_private);
+       uint32_t fdirctrl, max_num;
+       uint8_t offset;
+
+       fdirctrl = IXGBE_READ_REG(hw, IXGBE_FDIRCTRL);
+       offset = ((fdirctrl & IXGBE_FDIRCTRL_FLEX_MASK) >>
+                       IXGBE_FDIRCTRL_FLEX_SHIFT) * sizeof(uint16_t);
+
+       fdir_info->mode = dev->data->dev_conf.fdir_conf.mode;
+       max_num = (1 << (FDIRENTRIES_NUM_SHIFT +
+                       (fdirctrl & FDIRCTRL_PBALLOC_MASK)));
+       if (fdir_info->mode == RTE_FDIR_MODE_PERFECT)
+               fdir_info->guarant_spc = max_num;
+       else if (fdir_info->mode == RTE_FDIR_MODE_SIGNATURE)
+               fdir_info->guarant_spc = max_num * 4;
+
+       fdir_info->mask.vlan_tci_mask = info->mask.vlan_tci_mask;
+       fdir_info->mask.ipv4_mask.src_ip = info->mask.src_ipv4_mask;
+       fdir_info->mask.ipv4_mask.dst_ip = info->mask.dst_ipv4_mask;
+       IPV6_MASK_TO_ADDR(info->mask.src_ipv6_mask,
+                       fdir_info->mask.ipv6_mask.src_ip);
+       IPV6_MASK_TO_ADDR(info->mask.dst_ipv6_mask,
+                       fdir_info->mask.ipv6_mask.dst_ip);
+       fdir_info->mask.src_port_mask = info->mask.src_port_mask;
+       fdir_info->mask.dst_port_mask = info->mask.dst_port_mask;
+       fdir_info->max_flexpayload = IXGBE_FDIR_MAX_FLEX_LEN;
+       fdir_info->flow_types_mask[0] = IXGBE_FDIR_FLOW_TYPES;
+       fdir_info->flex_payload_unit = sizeof(uint16_t);
+       fdir_info->max_flex_payload_segment_num = 1;
+       fdir_info->flex_payload_limit = 62;
+       fdir_info->flex_conf.nb_payloads = 1;
+       fdir_info->flex_conf.flex_set[0].type = RTE_ETH_RAW_PAYLOAD;
+       fdir_info->flex_conf.flex_set[0].src_offset[0] = offset;
+       fdir_info->flex_conf.flex_set[0].src_offset[1] = offset + 1;
+       fdir_info->flex_conf.nb_flexmasks = 1;
+       fdir_info->flex_conf.flex_mask[0].flow_type = RTE_ETH_FLOW_UNKNOWN;
+       fdir_info->flex_conf.flex_mask[0].mask[0] =
+                       (uint8_t)(info->mask.flex_bytes_mask & 0x00FF);
+       fdir_info->flex_conf.flex_mask[0].mask[1] =
+                       (uint8_t)((info->mask.flex_bytes_mask & 0xFF00) >> 8);
 }
 
-void
-ixgbe_fdir_info_get(struct rte_eth_dev *dev, struct rte_eth_fdir *fdir)
+static void
+ixgbe_fdir_stats_get(struct rte_eth_dev *dev, struct rte_eth_fdir_stats *fdir_stats)
 {
        struct ixgbe_hw *hw = IXGBE_DEV_PRIVATE_TO_HW(dev->data->dev_private);
        struct ixgbe_hw_fdir_info *info =
                        IXGBE_DEV_PRIVATE_TO_FDIR_INFO(dev->data->dev_private);
-       uint32_t reg;
-
-       if (hw->mac.type != ixgbe_mac_82599EB && hw->mac.type !=ixgbe_mac_X540)
-               return;
+       uint32_t reg, max_num;
 
        /* Get the information from registers */
        reg = IXGBE_READ_REG(hw, IXGBE_FDIRFREE);
@@ -888,12 +1069,76 @@ ixgbe_fdir_info_get(struct rte_eth_dev *dev, struct rte_eth_fdir *fdir)
                IXGBE_FDIRFSTAT_FADD_SHIFT;
 
        /*  Copy the new information in the fdir parameter */
-       fdir->collision = info->collision;
-       fdir->free = info->free;
-       fdir->maxhash = info->maxhash;
-       fdir->maxlen = info->maxlen;
-       fdir->remove = info->remove;
-       fdir->add = info->add;
-       fdir->f_remove = info->f_remove;
-       fdir->f_add = info->f_add;
+       fdir_stats->collision = info->collision;
+       fdir_stats->free = info->free;
+       fdir_stats->maxhash = info->maxhash;
+       fdir_stats->maxlen = info->maxlen;
+       fdir_stats->remove = info->remove;
+       fdir_stats->add = info->add;
+       fdir_stats->f_remove = info->f_remove;
+       fdir_stats->f_add = info->f_add;
+
+       reg = IXGBE_READ_REG(hw, IXGBE_FDIRCTRL);
+       max_num = (1 << (FDIRENTRIES_NUM_SHIFT +
+                       (reg & FDIRCTRL_PBALLOC_MASK)));
+       if (dev->data->dev_conf.fdir_conf.mode == RTE_FDIR_MODE_PERFECT)
+                       fdir_stats->guarant_cnt = max_num - fdir_stats->free;
+       else if (dev->data->dev_conf.fdir_conf.mode == RTE_FDIR_MODE_SIGNATURE)
+               fdir_stats->guarant_cnt = max_num * 4 - fdir_stats->free;
+
+}
+
+/*
+ * ixgbe_fdir_ctrl_func - deal with all operations on flow director.
+ * @dev: pointer to the structure rte_eth_dev
+ * @filter_op:operation will be taken
+ * @arg: a pointer to specific structure corresponding to the filter_op
+ */
+int
+ixgbe_fdir_ctrl_func(struct rte_eth_dev *dev,
+                       enum rte_filter_op filter_op, void *arg)
+{
+       struct ixgbe_hw *hw = IXGBE_DEV_PRIVATE_TO_HW(dev->data->dev_private);
+       int ret = 0;
+
+       if (hw->mac.type != ixgbe_mac_82599EB &&
+               hw->mac.type != ixgbe_mac_X540 &&
+               hw->mac.type != ixgbe_mac_X550 &&
+               hw->mac.type != ixgbe_mac_X550EM_x)
+               return -ENOTSUP;
+
+       if (filter_op == RTE_ETH_FILTER_NOP)
+               return 0;
+
+       if (arg == NULL && filter_op != RTE_ETH_FILTER_FLUSH)
+               return -EINVAL;
+
+       switch (filter_op) {
+       case RTE_ETH_FILTER_ADD:
+               ret = ixgbe_add_del_fdir_filter(dev,
+                       (struct rte_eth_fdir_filter *)arg, FALSE, FALSE);
+               break;
+       case RTE_ETH_FILTER_UPDATE:
+               ret = ixgbe_add_del_fdir_filter(dev,
+                       (struct rte_eth_fdir_filter *)arg, FALSE, TRUE);
+               break;
+       case RTE_ETH_FILTER_DELETE:
+               ret = ixgbe_add_del_fdir_filter(dev,
+                       (struct rte_eth_fdir_filter *)arg, TRUE, FALSE);
+               break;
+       case RTE_ETH_FILTER_FLUSH:
+               ret = ixgbe_fdir_flush(dev);
+               break;
+       case RTE_ETH_FILTER_INFO:
+               ixgbe_fdir_info_get(dev, (struct rte_eth_fdir_info *)arg);
+               break;
+       case RTE_ETH_FILTER_STATS:
+               ixgbe_fdir_stats_get(dev, (struct rte_eth_fdir_stats *)arg);
+               break;
+       default:
+               PMD_DRV_LOG(ERR, "unknown operation %u", filter_op);
+               ret = -EINVAL;
+               break;
+       }
+       return ret;
 }