net/liquidio: add API to setup Rx queue
[dpdk.git] / drivers / net / tap / tap_flow.c
index c32ed38..7f1693d 100644 (file)
 
 #include <sys/queue.h>
 
+#include <rte_byteorder.h>
+#include <rte_jhash.h>
 #include <rte_malloc.h>
 #include <rte_eth_tap.h>
 #include <tap_flow.h>
+#include <tap_autoconf.h>
+#include <tap_tcmsgs.h>
+
+#ifndef HAVE_TC_FLOWER
+/*
+ * For kernels < 4.2, this enum is not defined. Runtime checks will be made to
+ * avoid sending TC messages the kernel cannot understand.
+ */
+enum {
+       TCA_FLOWER_UNSPEC,
+       TCA_FLOWER_CLASSID,
+       TCA_FLOWER_INDEV,
+       TCA_FLOWER_ACT,
+       TCA_FLOWER_KEY_ETH_DST,         /* ETH_ALEN */
+       TCA_FLOWER_KEY_ETH_DST_MASK,    /* ETH_ALEN */
+       TCA_FLOWER_KEY_ETH_SRC,         /* ETH_ALEN */
+       TCA_FLOWER_KEY_ETH_SRC_MASK,    /* ETH_ALEN */
+       TCA_FLOWER_KEY_ETH_TYPE,        /* be16 */
+       TCA_FLOWER_KEY_IP_PROTO,        /* u8 */
+       TCA_FLOWER_KEY_IPV4_SRC,        /* be32 */
+       TCA_FLOWER_KEY_IPV4_SRC_MASK,   /* be32 */
+       TCA_FLOWER_KEY_IPV4_DST,        /* be32 */
+       TCA_FLOWER_KEY_IPV4_DST_MASK,   /* be32 */
+       TCA_FLOWER_KEY_IPV6_SRC,        /* struct in6_addr */
+       TCA_FLOWER_KEY_IPV6_SRC_MASK,   /* struct in6_addr */
+       TCA_FLOWER_KEY_IPV6_DST,        /* struct in6_addr */
+       TCA_FLOWER_KEY_IPV6_DST_MASK,   /* struct in6_addr */
+       TCA_FLOWER_KEY_TCP_SRC,         /* be16 */
+       TCA_FLOWER_KEY_TCP_DST,         /* be16 */
+       TCA_FLOWER_KEY_UDP_SRC,         /* be16 */
+       TCA_FLOWER_KEY_UDP_DST,         /* be16 */
+};
+#endif
+#ifndef HAVE_TC_VLAN_ID
+enum {
+       /* TCA_FLOWER_FLAGS, */
+       TCA_FLOWER_KEY_VLAN_ID = TCA_FLOWER_KEY_UDP_DST + 2, /* be16 */
+       TCA_FLOWER_KEY_VLAN_PRIO,       /* u8   */
+       TCA_FLOWER_KEY_VLAN_ETH_TYPE,   /* be16 */
+};
+#endif
 
 struct rte_flow {
        LIST_ENTRY(rte_flow) next; /* Pointer to the next rte_flow structure */
+       struct rte_flow *remote_flow; /* associated remote flow */
+       struct nlmsg msg;
+};
+
+struct convert_data {
+       uint16_t eth_type;
+       uint16_t ip_proto;
+       uint8_t vlan;
+       struct rte_flow *flow;
+};
+
+struct remote_rule {
+       struct rte_flow_attr attr;
+       struct rte_flow_item items[2];
+       int mirred;
 };
 
+static int tap_flow_create_eth(const struct rte_flow_item *item, void *data);
+static int tap_flow_create_vlan(const struct rte_flow_item *item, void *data);
+static int tap_flow_create_ipv4(const struct rte_flow_item *item, void *data);
+static int tap_flow_create_ipv6(const struct rte_flow_item *item, void *data);
+static int tap_flow_create_udp(const struct rte_flow_item *item, void *data);
+static int tap_flow_create_tcp(const struct rte_flow_item *item, void *data);
 static int
 tap_flow_validate(struct rte_eth_dev *dev,
                  const struct rte_flow_attr *attr,
@@ -67,6 +131,934 @@ static const struct rte_flow_ops tap_flow_ops = {
        .flush = tap_flow_flush,
 };
 
+/* Static initializer for items. */
+#define ITEMS(...) \
+       (const enum rte_flow_item_type []){ \
+               __VA_ARGS__, RTE_FLOW_ITEM_TYPE_END, \
+       }
+
+/* Structure to generate a simple graph of layers supported by the NIC. */
+struct tap_flow_items {
+       /* Bit-mask corresponding to what is supported for this item. */
+       const void *mask;
+       const unsigned int mask_sz; /* Bit-mask size in bytes. */
+       /*
+        * Bit-mask corresponding to the default mask, if none is provided
+        * along with the item.
+        */
+       const void *default_mask;
+       /**
+        * Conversion function from rte_flow to netlink attributes.
+        *
+        * @param item
+        *   rte_flow item to convert.
+        * @param data
+        *   Internal structure to store the conversion.
+        *
+        * @return
+        *   0 on success, negative value otherwise.
+        */
+       int (*convert)(const struct rte_flow_item *item, void *data);
+       /** List of possible following items.  */
+       const enum rte_flow_item_type *const items;
+};
+
+/* Graph of supported items and associated actions. */
+static const struct tap_flow_items tap_flow_items[] = {
+       [RTE_FLOW_ITEM_TYPE_END] = {
+               .items = ITEMS(RTE_FLOW_ITEM_TYPE_ETH),
+       },
+       [RTE_FLOW_ITEM_TYPE_ETH] = {
+               .items = ITEMS(
+                       RTE_FLOW_ITEM_TYPE_VLAN,
+                       RTE_FLOW_ITEM_TYPE_IPV4,
+                       RTE_FLOW_ITEM_TYPE_IPV6),
+               .mask = &(const struct rte_flow_item_eth){
+                       .dst.addr_bytes = "\xff\xff\xff\xff\xff\xff",
+                       .src.addr_bytes = "\xff\xff\xff\xff\xff\xff",
+                       .type = -1,
+               },
+               .mask_sz = sizeof(struct rte_flow_item_eth),
+               .default_mask = &rte_flow_item_eth_mask,
+               .convert = tap_flow_create_eth,
+       },
+       [RTE_FLOW_ITEM_TYPE_VLAN] = {
+               .items = ITEMS(RTE_FLOW_ITEM_TYPE_IPV4,
+                              RTE_FLOW_ITEM_TYPE_IPV6),
+               .mask = &(const struct rte_flow_item_vlan){
+                       .tpid = -1,
+                       /* DEI matching is not supported */
+#if RTE_BYTE_ORDER == RTE_LITTLE_ENDIAN
+                       .tci = 0xffef,
+#else
+                       .tci = 0xefff,
+#endif
+               },
+               .mask_sz = sizeof(struct rte_flow_item_vlan),
+               .default_mask = &rte_flow_item_vlan_mask,
+               .convert = tap_flow_create_vlan,
+       },
+       [RTE_FLOW_ITEM_TYPE_IPV4] = {
+               .items = ITEMS(RTE_FLOW_ITEM_TYPE_UDP,
+                              RTE_FLOW_ITEM_TYPE_TCP),
+               .mask = &(const struct rte_flow_item_ipv4){
+                       .hdr = {
+                               .src_addr = -1,
+                               .dst_addr = -1,
+                               .next_proto_id = -1,
+                       },
+               },
+               .mask_sz = sizeof(struct rte_flow_item_ipv4),
+               .default_mask = &rte_flow_item_ipv4_mask,
+               .convert = tap_flow_create_ipv4,
+       },
+       [RTE_FLOW_ITEM_TYPE_IPV6] = {
+               .items = ITEMS(RTE_FLOW_ITEM_TYPE_UDP,
+                              RTE_FLOW_ITEM_TYPE_TCP),
+               .mask = &(const struct rte_flow_item_ipv6){
+                       .hdr = {
+                               .src_addr = {
+                                       "\xff\xff\xff\xff\xff\xff\xff\xff"
+                                       "\xff\xff\xff\xff\xff\xff\xff\xff",
+                               },
+                               .dst_addr = {
+                                       "\xff\xff\xff\xff\xff\xff\xff\xff"
+                                       "\xff\xff\xff\xff\xff\xff\xff\xff",
+                               },
+                               .proto = -1,
+                       },
+               },
+               .mask_sz = sizeof(struct rte_flow_item_ipv6),
+               .default_mask = &rte_flow_item_ipv6_mask,
+               .convert = tap_flow_create_ipv6,
+       },
+       [RTE_FLOW_ITEM_TYPE_UDP] = {
+               .mask = &(const struct rte_flow_item_udp){
+                       .hdr = {
+                               .src_port = -1,
+                               .dst_port = -1,
+                       },
+               },
+               .mask_sz = sizeof(struct rte_flow_item_udp),
+               .default_mask = &rte_flow_item_udp_mask,
+               .convert = tap_flow_create_udp,
+       },
+       [RTE_FLOW_ITEM_TYPE_TCP] = {
+               .mask = &(const struct rte_flow_item_tcp){
+                       .hdr = {
+                               .src_port = -1,
+                               .dst_port = -1,
+                       },
+               },
+               .mask_sz = sizeof(struct rte_flow_item_tcp),
+               .default_mask = &rte_flow_item_tcp_mask,
+               .convert = tap_flow_create_tcp,
+       },
+};
+
+static struct remote_rule implicit_rte_flows[TAP_REMOTE_MAX_IDX] = {
+       [TAP_REMOTE_LOCAL_MAC] = {
+               .attr = {
+                       .group = MAX_GROUP,
+                       .priority = PRIORITY_MASK - TAP_REMOTE_LOCAL_MAC,
+                       .ingress = 1,
+               },
+               .items[0] = {
+                       .type = RTE_FLOW_ITEM_TYPE_ETH,
+                       .mask =  &(const struct rte_flow_item_eth){
+                               .dst.addr_bytes = "\xff\xff\xff\xff\xff\xff",
+                       },
+               },
+               .items[1] = {
+                       .type = RTE_FLOW_ITEM_TYPE_END,
+               },
+               .mirred = TCA_EGRESS_REDIR,
+       },
+       [TAP_REMOTE_BROADCAST] = {
+               .attr = {
+                       .group = MAX_GROUP,
+                       .priority = PRIORITY_MASK - TAP_REMOTE_BROADCAST,
+                       .ingress = 1,
+               },
+               .items[0] = {
+                       .type = RTE_FLOW_ITEM_TYPE_ETH,
+                       .mask =  &(const struct rte_flow_item_eth){
+                               .dst.addr_bytes = "\xff\xff\xff\xff\xff\xff",
+                       },
+                       .spec = &(const struct rte_flow_item_eth){
+                               .dst.addr_bytes = "\xff\xff\xff\xff\xff\xff",
+                       },
+               },
+               .items[1] = {
+                       .type = RTE_FLOW_ITEM_TYPE_END,
+               },
+               .mirred = TCA_EGRESS_MIRROR,
+       },
+       [TAP_REMOTE_BROADCASTV6] = {
+               .attr = {
+                       .group = MAX_GROUP,
+                       .priority = PRIORITY_MASK - TAP_REMOTE_BROADCASTV6,
+                       .ingress = 1,
+               },
+               .items[0] = {
+                       .type = RTE_FLOW_ITEM_TYPE_ETH,
+                       .mask =  &(const struct rte_flow_item_eth){
+                               .dst.addr_bytes = "\x33\x33\x00\x00\x00\x00",
+                       },
+                       .spec = &(const struct rte_flow_item_eth){
+                               .dst.addr_bytes = "\x33\x33\x00\x00\x00\x00",
+                       },
+               },
+               .items[1] = {
+                       .type = RTE_FLOW_ITEM_TYPE_END,
+               },
+               .mirred = TCA_EGRESS_MIRROR,
+       },
+       [TAP_REMOTE_PROMISC] = {
+               .attr = {
+                       .group = MAX_GROUP,
+                       .priority = PRIORITY_MASK - TAP_REMOTE_PROMISC,
+                       .ingress = 1,
+               },
+               .items[0] = {
+                       .type = RTE_FLOW_ITEM_TYPE_VOID,
+               },
+               .items[1] = {
+                       .type = RTE_FLOW_ITEM_TYPE_END,
+               },
+               .mirred = TCA_EGRESS_MIRROR,
+       },
+       [TAP_REMOTE_ALLMULTI] = {
+               .attr = {
+                       .group = MAX_GROUP,
+                       .priority = PRIORITY_MASK - TAP_REMOTE_ALLMULTI,
+                       .ingress = 1,
+               },
+               .items[0] = {
+                       .type = RTE_FLOW_ITEM_TYPE_ETH,
+                       .mask =  &(const struct rte_flow_item_eth){
+                               .dst.addr_bytes = "\x01\x00\x00\x00\x00\x00",
+                       },
+                       .spec = &(const struct rte_flow_item_eth){
+                               .dst.addr_bytes = "\x01\x00\x00\x00\x00\x00",
+                       },
+               },
+               .items[1] = {
+                       .type = RTE_FLOW_ITEM_TYPE_END,
+               },
+               .mirred = TCA_EGRESS_MIRROR,
+       },
+       [TAP_REMOTE_TX] = {
+               .attr = {
+                       .group = 0,
+                       .priority = TAP_REMOTE_TX,
+                       .egress = 1,
+               },
+               .items[0] = {
+                       .type = RTE_FLOW_ITEM_TYPE_VOID,
+               },
+               .items[1] = {
+                       .type = RTE_FLOW_ITEM_TYPE_END,
+               },
+               .mirred = TCA_EGRESS_MIRROR,
+       },
+};
+
+/**
+ * Make as much checks as possible on an Ethernet item, and if a flow is
+ * provided, fill it appropriately with Ethernet info.
+ *
+ * @param[in] item
+ *   Item specification.
+ * @param[in, out] data
+ *   Additional data structure to tell next layers we've been here.
+ *
+ * @return
+ *   0 if checks are alright, -1 otherwise.
+ */
+static int
+tap_flow_create_eth(const struct rte_flow_item *item, void *data)
+{
+       struct convert_data *info = (struct convert_data *)data;
+       const struct rte_flow_item_eth *spec = item->spec;
+       const struct rte_flow_item_eth *mask = item->mask;
+       struct rte_flow *flow = info->flow;
+       struct nlmsg *msg;
+
+       /* use default mask if none provided */
+       if (!mask)
+               mask = tap_flow_items[RTE_FLOW_ITEM_TYPE_ETH].default_mask;
+       /* TC does not support eth_type masking. Only accept if exact match. */
+       if (mask->type && mask->type != 0xffff)
+               return -1;
+       if (!spec)
+               return 0;
+       /* store eth_type for consistency if ipv4/6 pattern item comes next */
+       if (spec->type & mask->type)
+               info->eth_type = spec->type;
+       if (!flow)
+               return 0;
+       msg = &flow->msg;
+       if (spec->type & mask->type)
+               msg->t.tcm_info = TC_H_MAKE(msg->t.tcm_info,
+                                           (spec->type & mask->type));
+       if (!is_zero_ether_addr(&spec->dst)) {
+               nlattr_add(&msg->nh, TCA_FLOWER_KEY_ETH_DST, ETHER_ADDR_LEN,
+                          &spec->dst.addr_bytes);
+               nlattr_add(&msg->nh,
+                          TCA_FLOWER_KEY_ETH_DST_MASK, ETHER_ADDR_LEN,
+                          &mask->dst.addr_bytes);
+       }
+       if (!is_zero_ether_addr(&mask->src)) {
+               nlattr_add(&msg->nh, TCA_FLOWER_KEY_ETH_SRC, ETHER_ADDR_LEN,
+                          &spec->src.addr_bytes);
+               nlattr_add(&msg->nh,
+                          TCA_FLOWER_KEY_ETH_SRC_MASK, ETHER_ADDR_LEN,
+                          &mask->src.addr_bytes);
+       }
+       return 0;
+}
+
+/**
+ * Make as much checks as possible on a VLAN item, and if a flow is provided,
+ * fill it appropriately with VLAN info.
+ *
+ * @param[in] item
+ *   Item specification.
+ * @param[in, out] data
+ *   Additional data structure to tell next layers we've been here.
+ *
+ * @return
+ *   0 if checks are alright, -1 otherwise.
+ */
+static int
+tap_flow_create_vlan(const struct rte_flow_item *item, void *data)
+{
+       struct convert_data *info = (struct convert_data *)data;
+       const struct rte_flow_item_vlan *spec = item->spec;
+       const struct rte_flow_item_vlan *mask = item->mask;
+       struct rte_flow *flow = info->flow;
+       struct nlmsg *msg;
+
+       /* use default mask if none provided */
+       if (!mask)
+               mask = tap_flow_items[RTE_FLOW_ITEM_TYPE_VLAN].default_mask;
+       /* TC does not support tpid masking. Only accept if exact match. */
+       if (mask->tpid && mask->tpid != 0xffff)
+               return -1;
+       /* Double-tagging not supported. */
+       if (spec && mask->tpid && spec->tpid != htons(ETH_P_8021Q))
+               return -1;
+       info->vlan = 1;
+       if (!flow)
+               return 0;
+       msg = &flow->msg;
+       msg->t.tcm_info = TC_H_MAKE(msg->t.tcm_info, htons(ETH_P_8021Q));
+#define VLAN_PRIO(tci) ((tci) >> 13)
+#define VLAN_ID(tci) ((tci) & 0xfff)
+       if (!spec)
+               return 0;
+       if (spec->tci) {
+               uint16_t tci = ntohs(spec->tci) & mask->tci;
+               uint16_t prio = VLAN_PRIO(tci);
+               uint8_t vid = VLAN_ID(tci);
+
+               if (prio)
+                       nlattr_add8(&msg->nh, TCA_FLOWER_KEY_VLAN_PRIO, prio);
+               if (vid)
+                       nlattr_add16(&msg->nh, TCA_FLOWER_KEY_VLAN_ID, vid);
+       }
+       return 0;
+}
+
+/**
+ * Make as much checks as possible on an IPv4 item, and if a flow is provided,
+ * fill it appropriately with IPv4 info.
+ *
+ * @param[in] item
+ *   Item specification.
+ * @param[in, out] data
+ *   Additional data structure to tell next layers we've been here.
+ *
+ * @return
+ *   0 if checks are alright, -1 otherwise.
+ */
+static int
+tap_flow_create_ipv4(const struct rte_flow_item *item, void *data)
+{
+       struct convert_data *info = (struct convert_data *)data;
+       const struct rte_flow_item_ipv4 *spec = item->spec;
+       const struct rte_flow_item_ipv4 *mask = item->mask;
+       struct rte_flow *flow = info->flow;
+       struct nlmsg *msg;
+
+       /* use default mask if none provided */
+       if (!mask)
+               mask = tap_flow_items[RTE_FLOW_ITEM_TYPE_IPV4].default_mask;
+       /* check that previous eth type is compatible with ipv4 */
+       if (info->eth_type && info->eth_type != htons(ETH_P_IP))
+               return -1;
+       /* store ip_proto for consistency if udp/tcp pattern item comes next */
+       if (spec)
+               info->ip_proto = spec->hdr.next_proto_id;
+       if (!flow)
+               return 0;
+       msg = &flow->msg;
+       if (!info->eth_type)
+               info->eth_type = htons(ETH_P_IP);
+       if (!info->vlan)
+               msg->t.tcm_info = TC_H_MAKE(msg->t.tcm_info, htons(ETH_P_IP));
+       if (!spec)
+               return 0;
+       if (spec->hdr.dst_addr) {
+               nlattr_add32(&msg->nh, TCA_FLOWER_KEY_IPV4_DST,
+                            spec->hdr.dst_addr);
+               nlattr_add32(&msg->nh, TCA_FLOWER_KEY_IPV4_DST_MASK,
+                            mask->hdr.dst_addr);
+       }
+       if (spec->hdr.src_addr) {
+               nlattr_add32(&msg->nh, TCA_FLOWER_KEY_IPV4_SRC,
+                            spec->hdr.src_addr);
+               nlattr_add32(&msg->nh, TCA_FLOWER_KEY_IPV4_SRC_MASK,
+                            mask->hdr.src_addr);
+       }
+       if (spec->hdr.next_proto_id)
+               nlattr_add8(&msg->nh, TCA_FLOWER_KEY_IP_PROTO,
+                           spec->hdr.next_proto_id);
+       return 0;
+}
+
+/**
+ * Make as much checks as possible on an IPv6 item, and if a flow is provided,
+ * fill it appropriately with IPv6 info.
+ *
+ * @param[in] item
+ *   Item specification.
+ * @param[in, out] data
+ *   Additional data structure to tell next layers we've been here.
+ *
+ * @return
+ *   0 if checks are alright, -1 otherwise.
+ */
+static int
+tap_flow_create_ipv6(const struct rte_flow_item *item, void *data)
+{
+       struct convert_data *info = (struct convert_data *)data;
+       const struct rte_flow_item_ipv6 *spec = item->spec;
+       const struct rte_flow_item_ipv6 *mask = item->mask;
+       struct rte_flow *flow = info->flow;
+       uint8_t empty_addr[16] = { 0 };
+       struct nlmsg *msg;
+
+       /* use default mask if none provided */
+       if (!mask)
+               mask = tap_flow_items[RTE_FLOW_ITEM_TYPE_IPV6].default_mask;
+       /* check that previous eth type is compatible with ipv6 */
+       if (info->eth_type && info->eth_type != htons(ETH_P_IPV6))
+               return -1;
+       /* store ip_proto for consistency if udp/tcp pattern item comes next */
+       if (spec)
+               info->ip_proto = spec->hdr.proto;
+       if (!flow)
+               return 0;
+       msg = &flow->msg;
+       if (!info->eth_type)
+               info->eth_type = htons(ETH_P_IPV6);
+       if (!info->vlan)
+               msg->t.tcm_info = TC_H_MAKE(msg->t.tcm_info, htons(ETH_P_IPV6));
+       if (!spec)
+               return 0;
+       if (memcmp(spec->hdr.dst_addr, empty_addr, 16)) {
+               nlattr_add(&msg->nh, TCA_FLOWER_KEY_IPV6_DST,
+                          sizeof(spec->hdr.dst_addr), &spec->hdr.dst_addr);
+               nlattr_add(&msg->nh, TCA_FLOWER_KEY_IPV6_DST_MASK,
+                          sizeof(mask->hdr.dst_addr), &mask->hdr.dst_addr);
+       }
+       if (memcmp(spec->hdr.src_addr, empty_addr, 16)) {
+               nlattr_add(&msg->nh, TCA_FLOWER_KEY_IPV6_SRC,
+                          sizeof(spec->hdr.src_addr), &spec->hdr.src_addr);
+               nlattr_add(&msg->nh, TCA_FLOWER_KEY_IPV6_SRC_MASK,
+                          sizeof(mask->hdr.src_addr), &mask->hdr.src_addr);
+       }
+       if (spec->hdr.proto)
+               nlattr_add8(&msg->nh, TCA_FLOWER_KEY_IP_PROTO, spec->hdr.proto);
+       return 0;
+}
+
+/**
+ * Make as much checks as possible on a UDP item, and if a flow is provided,
+ * fill it appropriately with UDP info.
+ *
+ * @param[in] item
+ *   Item specification.
+ * @param[in, out] data
+ *   Additional data structure to tell next layers we've been here.
+ *
+ * @return
+ *   0 if checks are alright, -1 otherwise.
+ */
+static int
+tap_flow_create_udp(const struct rte_flow_item *item, void *data)
+{
+       struct convert_data *info = (struct convert_data *)data;
+       const struct rte_flow_item_udp *spec = item->spec;
+       const struct rte_flow_item_udp *mask = item->mask;
+       struct rte_flow *flow = info->flow;
+       struct nlmsg *msg;
+
+       /* use default mask if none provided */
+       if (!mask)
+               mask = tap_flow_items[RTE_FLOW_ITEM_TYPE_UDP].default_mask;
+       /* check that previous ip_proto is compatible with udp */
+       if (info->ip_proto && info->ip_proto != IPPROTO_UDP)
+               return -1;
+       if (!flow)
+               return 0;
+       msg = &flow->msg;
+       nlattr_add8(&msg->nh, TCA_FLOWER_KEY_IP_PROTO, IPPROTO_UDP);
+       if (!spec)
+               return 0;
+       if (spec->hdr.dst_port &&
+           (spec->hdr.dst_port & mask->hdr.dst_port) == spec->hdr.dst_port)
+               nlattr_add16(&msg->nh, TCA_FLOWER_KEY_UDP_DST,
+                            spec->hdr.dst_port);
+       if (spec->hdr.src_port &&
+           (spec->hdr.src_port & mask->hdr.src_port) == spec->hdr.src_port)
+               nlattr_add16(&msg->nh, TCA_FLOWER_KEY_UDP_SRC,
+                            spec->hdr.src_port);
+       return 0;
+}
+
+/**
+ * Make as much checks as possible on a TCP item, and if a flow is provided,
+ * fill it appropriately with TCP info.
+ *
+ * @param[in] item
+ *   Item specification.
+ * @param[in, out] data
+ *   Additional data structure to tell next layers we've been here.
+ *
+ * @return
+ *   0 if checks are alright, -1 otherwise.
+ */
+static int
+tap_flow_create_tcp(const struct rte_flow_item *item, void *data)
+{
+       struct convert_data *info = (struct convert_data *)data;
+       const struct rte_flow_item_tcp *spec = item->spec;
+       const struct rte_flow_item_tcp *mask = item->mask;
+       struct rte_flow *flow = info->flow;
+       struct nlmsg *msg;
+
+       /* use default mask if none provided */
+       if (!mask)
+               mask = tap_flow_items[RTE_FLOW_ITEM_TYPE_TCP].default_mask;
+       /* check that previous ip_proto is compatible with tcp */
+       if (info->ip_proto && info->ip_proto != IPPROTO_TCP)
+               return -1;
+       if (!flow)
+               return 0;
+       msg = &flow->msg;
+       nlattr_add8(&msg->nh, TCA_FLOWER_KEY_IP_PROTO, IPPROTO_TCP);
+       if (!spec)
+               return 0;
+       if (spec->hdr.dst_port &&
+           (spec->hdr.dst_port & mask->hdr.dst_port) == spec->hdr.dst_port)
+               nlattr_add16(&msg->nh, TCA_FLOWER_KEY_TCP_DST,
+                            spec->hdr.dst_port);
+       if (spec->hdr.src_port &&
+           (spec->hdr.src_port & mask->hdr.src_port) == spec->hdr.src_port)
+               nlattr_add16(&msg->nh, TCA_FLOWER_KEY_TCP_SRC,
+                            spec->hdr.src_port);
+       return 0;
+}
+
+/**
+ * Check support for a given item.
+ *
+ * @param[in] item
+ *   Item specification.
+ * @param size
+ *   Bit-Mask size in bytes.
+ * @param[in] supported_mask
+ *   Bit-mask covering supported fields to compare with spec, last and mask in
+ *   \item.
+ * @param[in] default_mask
+ *   Bit-mask default mask if none is provided in \item.
+ *
+ * @return
+ *   0 on success.
+ */
+static int
+tap_flow_item_validate(const struct rte_flow_item *item,
+                      unsigned int size,
+                      const uint8_t *supported_mask,
+                      const uint8_t *default_mask)
+{
+       int ret = 0;
+
+       /* An empty layer is allowed, as long as all fields are NULL */
+       if (!item->spec && (item->mask || item->last))
+               return -1;
+       /* Is the item spec compatible with what the NIC supports? */
+       if (item->spec && !item->mask) {
+               unsigned int i;
+               const uint8_t *spec = item->spec;
+
+               for (i = 0; i < size; ++i)
+                       if ((spec[i] | supported_mask[i]) != supported_mask[i])
+                               return -1;
+               /* Is the default mask compatible with what the NIC supports? */
+               for (i = 0; i < size; i++)
+                       if ((default_mask[i] | supported_mask[i]) !=
+                           supported_mask[i])
+                               return -1;
+       }
+       /* Is the item last compatible with what the NIC supports? */
+       if (item->last && !item->mask) {
+               unsigned int i;
+               const uint8_t *spec = item->last;
+
+               for (i = 0; i < size; ++i)
+                       if ((spec[i] | supported_mask[i]) != supported_mask[i])
+                               return -1;
+       }
+       /* Is the item mask compatible with what the NIC supports? */
+       if (item->mask) {
+               unsigned int i;
+               const uint8_t *spec = item->mask;
+
+               for (i = 0; i < size; ++i)
+                       if ((spec[i] | supported_mask[i]) != supported_mask[i])
+                               return -1;
+       }
+       /**
+        * Once masked, Are item spec and item last equal?
+        * TC does not support range so anything else is invalid.
+        */
+       if (item->spec && item->last) {
+               uint8_t spec[size];
+               uint8_t last[size];
+               const uint8_t *apply = default_mask;
+               unsigned int i;
+
+               if (item->mask)
+                       apply = item->mask;
+               for (i = 0; i < size; ++i) {
+                       spec[i] = ((const uint8_t *)item->spec)[i] & apply[i];
+                       last[i] = ((const uint8_t *)item->last)[i] & apply[i];
+               }
+               ret = memcmp(spec, last, size);
+       }
+       return ret;
+}
+
+/**
+ * Transform a DROP/PASSTHRU action item in the provided flow for TC.
+ *
+ * @param[in, out] flow
+ *   Flow to be filled.
+ * @param[in] action
+ *   Appropriate action to be set in the TCA_GACT_PARMS structure.
+ *
+ * @return
+ *   0 if checks are alright, -1 otherwise.
+ */
+static int
+add_action_gact(struct rte_flow *flow, int action)
+{
+       struct nlmsg *msg = &flow->msg;
+       size_t act_index = 1;
+       struct tc_gact p = {
+               .action = action
+       };
+
+       if (nlattr_nested_start(msg, TCA_FLOWER_ACT) < 0)
+               return -1;
+       if (nlattr_nested_start(msg, act_index++) < 0)
+               return -1;
+       nlattr_add(&msg->nh, TCA_ACT_KIND, sizeof("gact"), "gact");
+       if (nlattr_nested_start(msg, TCA_ACT_OPTIONS) < 0)
+               return -1;
+       nlattr_add(&msg->nh, TCA_GACT_PARMS, sizeof(p), &p);
+       nlattr_nested_finish(msg); /* nested TCA_ACT_OPTIONS */
+       nlattr_nested_finish(msg); /* nested act_index */
+       nlattr_nested_finish(msg); /* nested TCA_FLOWER_ACT */
+       return 0;
+}
+
+/**
+ * Transform a MIRRED action item in the provided flow for TC.
+ *
+ * @param[in, out] flow
+ *   Flow to be filled.
+ * @param[in] ifindex
+ *   Netdevice ifindex, where to mirror/redirect packet to.
+ * @param[in] action_type
+ *   Either TCA_EGRESS_REDIR for redirection or TCA_EGRESS_MIRROR for mirroring.
+ *
+ * @return
+ *   0 if checks are alright, -1 otherwise.
+ */
+static int
+add_action_mirred(struct rte_flow *flow, uint16_t ifindex, uint16_t action_type)
+{
+       struct nlmsg *msg = &flow->msg;
+       size_t act_index = 1;
+       struct tc_mirred p = {
+               .eaction = action_type,
+               .ifindex = ifindex,
+       };
+
+       if (nlattr_nested_start(msg, TCA_FLOWER_ACT) < 0)
+               return -1;
+       if (nlattr_nested_start(msg, act_index++) < 0)
+               return -1;
+       nlattr_add(&msg->nh, TCA_ACT_KIND, sizeof("mirred"), "mirred");
+       if (nlattr_nested_start(msg, TCA_ACT_OPTIONS) < 0)
+               return -1;
+       if (action_type == TCA_EGRESS_MIRROR)
+               p.action = TC_ACT_PIPE;
+       else /* REDIRECT */
+               p.action = TC_ACT_STOLEN;
+       nlattr_add(&msg->nh, TCA_MIRRED_PARMS, sizeof(p), &p);
+       nlattr_nested_finish(msg); /* nested TCA_ACT_OPTIONS */
+       nlattr_nested_finish(msg); /* nested act_index */
+       nlattr_nested_finish(msg); /* nested TCA_FLOWER_ACT */
+       return 0;
+}
+
+/**
+ * Transform a QUEUE action item in the provided flow for TC.
+ *
+ * @param[in, out] flow
+ *   Flow to be filled.
+ * @param[in] queue
+ *   Queue id to use.
+ *
+ * @return
+ *   0 if checks are alright, -1 otherwise.
+ */
+static int
+add_action_skbedit(struct rte_flow *flow, uint16_t queue)
+{
+       struct nlmsg *msg = &flow->msg;
+       size_t act_index = 1;
+       struct tc_skbedit p = {
+               .action = TC_ACT_PIPE
+       };
+
+       if (nlattr_nested_start(msg, TCA_FLOWER_ACT) < 0)
+               return -1;
+       if (nlattr_nested_start(msg, act_index++) < 0)
+               return -1;
+       nlattr_add(&msg->nh, TCA_ACT_KIND, sizeof("skbedit"), "skbedit");
+       if (nlattr_nested_start(msg, TCA_ACT_OPTIONS) < 0)
+               return -1;
+       nlattr_add(&msg->nh, TCA_SKBEDIT_PARMS, sizeof(p), &p);
+       nlattr_add16(&msg->nh, TCA_SKBEDIT_QUEUE_MAPPING, queue);
+       nlattr_nested_finish(msg); /* nested TCA_ACT_OPTIONS */
+       nlattr_nested_finish(msg); /* nested act_index */
+       nlattr_nested_finish(msg); /* nested TCA_FLOWER_ACT */
+       return 0;
+}
+
+/**
+ * Validate a flow supported by TC.
+ * If flow param is not NULL, then also fill the netlink message inside.
+ *
+ * @param pmd
+ *   Pointer to private structure.
+ * @param[in] attr
+ *   Flow rule attributes.
+ * @param[in] pattern
+ *   Pattern specification (list terminated by the END pattern item).
+ * @param[in] actions
+ *   Associated actions (list terminated by the END action).
+ * @param[out] error
+ *   Perform verbose error reporting if not NULL.
+ * @param[in, out] flow
+ *   Flow structure to update.
+ * @param[in] mirred
+ *   If set to TCA_EGRESS_REDIR, provided actions will be replaced with a
+ *   redirection to the tap netdevice, and the TC rule will be configured
+ *   on the remote netdevice in pmd.
+ *   If set to TCA_EGRESS_MIRROR, provided actions will be replaced with a
+ *   mirroring to the tap netdevice, and the TC rule will be configured
+ *   on the remote netdevice in pmd. Matching packets will thus be duplicated.
+ *   If set to 0, the standard behavior is to be used: set correct actions for
+ *   the TC rule, and apply it on the tap netdevice.
+ *
+ * @return
+ *   0 on success, a negative errno value otherwise and rte_errno is set.
+ */
+static int
+priv_flow_process(struct pmd_internals *pmd,
+                 const struct rte_flow_attr *attr,
+                 const struct rte_flow_item items[],
+                 const struct rte_flow_action actions[],
+                 struct rte_flow_error *error,
+                 struct rte_flow *flow,
+                 int mirred)
+{
+       const struct tap_flow_items *cur_item = tap_flow_items;
+       struct convert_data data = {
+               .eth_type = 0,
+               .ip_proto = 0,
+               .flow = flow,
+       };
+       int action = 0; /* Only one action authorized for now */
+
+       if (attr->group > MAX_GROUP) {
+               rte_flow_error_set(
+                       error, EINVAL, RTE_FLOW_ERROR_TYPE_ATTR_GROUP,
+                       NULL, "group value too big: cannot exceed 15");
+               return -rte_errno;
+       }
+       if (attr->priority > MAX_PRIORITY) {
+               rte_flow_error_set(
+                       error, EINVAL, RTE_FLOW_ERROR_TYPE_ATTR_PRIORITY,
+                       NULL, "priority value too big");
+               return -rte_errno;
+       } else if (flow) {
+               uint16_t group = attr->group << GROUP_SHIFT;
+               uint16_t prio = group | (attr->priority + PRIORITY_OFFSET);
+               flow->msg.t.tcm_info = TC_H_MAKE(prio << 16,
+                                                flow->msg.t.tcm_info);
+       }
+       if (flow) {
+               if (mirred) {
+                       /*
+                        * If attr->ingress, the rule applies on remote ingress
+                        * to match incoming packets
+                        * If attr->egress, the rule applies on tap ingress (as
+                        * seen from the kernel) to deal with packets going out
+                        * from the DPDK app.
+                        */
+                       flow->msg.t.tcm_parent = TC_H_MAKE(TC_H_INGRESS, 0);
+               } else {
+                       /* Standard rule on tap egress (kernel standpoint). */
+                       flow->msg.t.tcm_parent =
+                               TC_H_MAKE(MULTIQ_MAJOR_HANDLE, 0);
+               }
+               /* use flower filter type */
+               nlattr_add(&flow->msg.nh, TCA_KIND, sizeof("flower"), "flower");
+               if (nlattr_nested_start(&flow->msg, TCA_OPTIONS) < 0)
+                       goto exit_item_not_supported;
+       }
+       for (; items->type != RTE_FLOW_ITEM_TYPE_END; ++items) {
+               const struct tap_flow_items *token = NULL;
+               unsigned int i;
+               int err = 0;
+
+               if (items->type == RTE_FLOW_ITEM_TYPE_VOID)
+                       continue;
+               for (i = 0;
+                    cur_item->items &&
+                    cur_item->items[i] != RTE_FLOW_ITEM_TYPE_END;
+                    ++i) {
+                       if (cur_item->items[i] == items->type) {
+                               token = &tap_flow_items[items->type];
+                               break;
+                       }
+               }
+               if (!token)
+                       goto exit_item_not_supported;
+               cur_item = token;
+               err = tap_flow_item_validate(
+                       items, cur_item->mask_sz,
+                       (const uint8_t *)cur_item->mask,
+                       (const uint8_t *)cur_item->default_mask);
+               if (err)
+                       goto exit_item_not_supported;
+               if (flow && cur_item->convert) {
+                       if (!pmd->flower_vlan_support &&
+                           cur_item->convert == tap_flow_create_vlan)
+                               goto exit_item_not_supported;
+                       err = cur_item->convert(items, &data);
+                       if (err)
+                               goto exit_item_not_supported;
+               }
+       }
+       if (flow) {
+               if (pmd->flower_vlan_support && data.vlan) {
+                       nlattr_add16(&flow->msg.nh, TCA_FLOWER_KEY_ETH_TYPE,
+                                    htons(ETH_P_8021Q));
+                       nlattr_add16(&flow->msg.nh,
+                                    TCA_FLOWER_KEY_VLAN_ETH_TYPE,
+                                    data.eth_type ?
+                                    data.eth_type : htons(ETH_P_ALL));
+               } else if (data.eth_type) {
+                       nlattr_add16(&flow->msg.nh, TCA_FLOWER_KEY_ETH_TYPE,
+                                    data.eth_type);
+               }
+       }
+       if (mirred && flow) {
+               uint16_t if_index = pmd->if_index;
+
+               /*
+                * If attr->egress && mirred, then this is a special
+                * case where the rule must be applied on the tap, to
+                * redirect packets coming from the DPDK App, out
+                * through the remote netdevice.
+                */
+               if (attr->egress)
+                       if_index = pmd->remote_if_index;
+               if (add_action_mirred(flow, if_index, mirred) < 0)
+                       goto exit_action_not_supported;
+               else
+                       goto end;
+       }
+       for (; actions->type != RTE_FLOW_ACTION_TYPE_END; ++actions) {
+               int err = 0;
+
+               if (actions->type == RTE_FLOW_ACTION_TYPE_VOID) {
+                       continue;
+               } else if (actions->type == RTE_FLOW_ACTION_TYPE_DROP) {
+                       if (action)
+                               goto exit_action_not_supported;
+                       action = 1;
+                       if (flow)
+                               err = add_action_gact(flow, TC_ACT_SHOT);
+               } else if (actions->type == RTE_FLOW_ACTION_TYPE_PASSTHRU) {
+                       if (action)
+                               goto exit_action_not_supported;
+                       action = 1;
+                       if (flow)
+                               err = add_action_gact(flow, TC_ACT_UNSPEC);
+               } else if (actions->type == RTE_FLOW_ACTION_TYPE_QUEUE) {
+                       const struct rte_flow_action_queue *queue =
+                               (const struct rte_flow_action_queue *)
+                               actions->conf;
+                       if (action)
+                               goto exit_action_not_supported;
+                       action = 1;
+                       if (!queue || (queue->index >= pmd->nb_queues))
+                               goto exit_action_not_supported;
+                       if (flow)
+                               err = add_action_skbedit(flow, queue->index);
+               } else {
+                       goto exit_action_not_supported;
+               }
+               if (err)
+                       goto exit_action_not_supported;
+       }
+end:
+       if (flow)
+               nlattr_nested_finish(&flow->msg); /* nested TCA_OPTIONS */
+       return 0;
+exit_item_not_supported:
+       rte_flow_error_set(error, ENOTSUP, RTE_FLOW_ERROR_TYPE_ITEM,
+                          items, "item not supported");
+       return -rte_errno;
+exit_action_not_supported:
+       rte_flow_error_set(error, ENOTSUP, RTE_FLOW_ERROR_TYPE_ACTION,
+                          actions, "action not supported");
+       return -rte_errno;
+}
+
+
+
 /**
  * Validate a flow.
  *
@@ -74,15 +1066,46 @@ static const struct rte_flow_ops tap_flow_ops = {
  * @see rte_flow_ops
  */
 static int
-tap_flow_validate(struct rte_eth_dev *dev __rte_unused,
-                 const struct rte_flow_attr *attr __rte_unused,
-                 const struct rte_flow_item items[] __rte_unused,
-                 const struct rte_flow_action actions[] __rte_unused,
+tap_flow_validate(struct rte_eth_dev *dev,
+                 const struct rte_flow_attr *attr,
+                 const struct rte_flow_item items[],
+                 const struct rte_flow_action actions[],
                  struct rte_flow_error *error)
 {
-       return -rte_flow_error_set(error, ENOTSUP,
-                                  RTE_FLOW_ERROR_TYPE_UNSPECIFIED,
-                                  NULL, "not implemented yet");
+       struct pmd_internals *pmd = dev->data->dev_private;
+
+       return priv_flow_process(pmd, attr, items, actions, error, NULL, 0);
+}
+
+/**
+ * Set a unique handle in a flow.
+ *
+ * The kernel supports TC rules with equal priority, as long as they use the
+ * same matching fields (e.g.: dst mac and ipv4) with different values (and
+ * full mask to ensure no collision is possible).
+ * In those rules, the handle (uint32_t) is the part that would identify
+ * specifically each rule.
+ *
+ * On 32-bit architectures, the handle can simply be the flow's pointer address.
+ * On 64-bit architectures, we rely on jhash(flow) to find a (sufficiently)
+ * unique handle.
+ *
+ * @param[in, out] flow
+ *   The flow that needs its handle set.
+ */
+static void
+tap_flow_set_handle(struct rte_flow *flow)
+{
+       uint32_t handle = 0;
+
+       if (sizeof(flow) > 4)
+               handle = rte_jhash(&flow, sizeof(flow), 1);
+       else
+               handle = (uintptr_t)flow;
+       /* must be at least 1 to avoid letting the kernel choose one for us */
+       if (!handle)
+               handle = 1;
+       flow->msg.t.tcm_handle = handle;
 }
 
 /**
@@ -99,18 +1122,167 @@ tap_flow_create(struct rte_eth_dev *dev,
                struct rte_flow_error *error)
 {
        struct pmd_internals *pmd = dev->data->dev_private;
+       struct rte_flow *remote_flow = NULL;
        struct rte_flow *flow = NULL;
+       struct nlmsg *msg = NULL;
+       int err;
 
-       if (tap_flow_validate(dev, attr, items, actions, error))
-               return NULL;
+       if (!pmd->if_index) {
+               rte_flow_error_set(error, ENOTSUP, RTE_FLOW_ERROR_TYPE_HANDLE,
+                                  NULL,
+                                  "can't create rule, ifindex not found");
+               goto fail;
+       }
+       /*
+        * No rules configured through standard rte_flow should be set on the
+        * priorities used by implicit rules.
+        */
+       if ((attr->group == MAX_GROUP) &&
+           attr->priority > (MAX_PRIORITY - TAP_REMOTE_MAX_IDX)) {
+               rte_flow_error_set(
+                       error, ENOTSUP, RTE_FLOW_ERROR_TYPE_ATTR_PRIORITY,
+                       NULL, "priority value too big");
+               goto fail;
+       }
        flow = rte_malloc(__func__, sizeof(struct rte_flow), 0);
        if (!flow) {
                rte_flow_error_set(error, ENOMEM, RTE_FLOW_ERROR_TYPE_HANDLE,
                                   NULL, "cannot allocate memory for rte_flow");
-               return NULL;
+               goto fail;
+       }
+       msg = &flow->msg;
+       tc_init_msg(msg, pmd->if_index, RTM_NEWTFILTER,
+                   NLM_F_REQUEST | NLM_F_ACK | NLM_F_EXCL | NLM_F_CREATE);
+       msg->t.tcm_info = TC_H_MAKE(0, htons(ETH_P_ALL));
+       tap_flow_set_handle(flow);
+       if (priv_flow_process(pmd, attr, items, actions, error, flow, 0))
+               goto fail;
+       err = nl_send(pmd->nlsk_fd, &msg->nh);
+       if (err < 0) {
+               rte_flow_error_set(error, ENOTSUP, RTE_FLOW_ERROR_TYPE_HANDLE,
+                                  NULL, "couldn't send request to kernel");
+               goto fail;
+       }
+       err = nl_recv_ack(pmd->nlsk_fd);
+       if (err < 0) {
+               rte_flow_error_set(error, EEXIST, RTE_FLOW_ERROR_TYPE_HANDLE,
+                                  NULL, "overlapping rules");
+               goto fail;
        }
        LIST_INSERT_HEAD(&pmd->flows, flow, next);
+       /**
+        * If a remote device is configured, a TC rule with identical items for
+        * matching must be set on that device, with a single action: redirect
+        * to the local pmd->if_index.
+        */
+       if (pmd->remote_if_index) {
+               remote_flow = rte_malloc(__func__, sizeof(struct rte_flow), 0);
+               if (!remote_flow) {
+                       rte_flow_error_set(
+                               error, ENOMEM, RTE_FLOW_ERROR_TYPE_HANDLE, NULL,
+                               "cannot allocate memory for rte_flow");
+                       goto fail;
+               }
+               msg = &remote_flow->msg;
+               /* set the rule if_index for the remote netdevice */
+               tc_init_msg(
+                       msg, pmd->remote_if_index, RTM_NEWTFILTER,
+                       NLM_F_REQUEST | NLM_F_ACK | NLM_F_EXCL | NLM_F_CREATE);
+               msg->t.tcm_info = TC_H_MAKE(0, htons(ETH_P_ALL));
+               tap_flow_set_handle(remote_flow);
+               if (priv_flow_process(pmd, attr, items, NULL,
+                                     error, remote_flow, TCA_EGRESS_REDIR)) {
+                       rte_flow_error_set(
+                               error, ENOMEM, RTE_FLOW_ERROR_TYPE_HANDLE,
+                               NULL, "rte flow rule validation failed");
+                       goto fail;
+               }
+               err = nl_send(pmd->nlsk_fd, &msg->nh);
+               if (err < 0) {
+                       rte_flow_error_set(
+                               error, ENOMEM, RTE_FLOW_ERROR_TYPE_HANDLE,
+                               NULL, "Failure sending nl request");
+                       goto fail;
+               }
+               err = nl_recv_ack(pmd->nlsk_fd);
+               if (err < 0) {
+                       rte_flow_error_set(
+                               error, ENOMEM, RTE_FLOW_ERROR_TYPE_HANDLE,
+                               NULL, "overlapping rules");
+                       goto fail;
+               }
+               flow->remote_flow = remote_flow;
+       }
        return flow;
+fail:
+       if (remote_flow)
+               rte_free(remote_flow);
+       if (flow)
+               rte_free(flow);
+       return NULL;
+}
+
+/**
+ * Destroy a flow using pointer to pmd_internal.
+ *
+ * @param[in, out] pmd
+ *   Pointer to private structure.
+ * @param[in] flow
+ *   Pointer to the flow to destroy.
+ * @param[in, out] error
+ *   Pointer to the flow error handler
+ *
+ * @return 0 if the flow could be destroyed, -1 otherwise.
+ */
+static int
+tap_flow_destroy_pmd(struct pmd_internals *pmd,
+                    struct rte_flow *flow,
+                    struct rte_flow_error *error)
+{
+       struct rte_flow *remote_flow = flow->remote_flow;
+       int ret = 0;
+
+       LIST_REMOVE(flow, next);
+       flow->msg.nh.nlmsg_flags = NLM_F_REQUEST | NLM_F_ACK;
+       flow->msg.nh.nlmsg_type = RTM_DELTFILTER;
+
+       ret = nl_send(pmd->nlsk_fd, &flow->msg.nh);
+       if (ret < 0) {
+               rte_flow_error_set(error, ENOTSUP, RTE_FLOW_ERROR_TYPE_HANDLE,
+                                  NULL, "couldn't send request to kernel");
+               goto end;
+       }
+       ret = nl_recv_ack(pmd->nlsk_fd);
+       if (ret < 0) {
+               rte_flow_error_set(
+                       error, ENOTSUP, RTE_FLOW_ERROR_TYPE_HANDLE, NULL,
+                       "couldn't receive kernel ack to our request");
+               goto end;
+       }
+       if (remote_flow) {
+               remote_flow->msg.nh.nlmsg_flags = NLM_F_REQUEST | NLM_F_ACK;
+               remote_flow->msg.nh.nlmsg_type = RTM_DELTFILTER;
+
+               ret = nl_send(pmd->nlsk_fd, &remote_flow->msg.nh);
+               if (ret < 0) {
+                       rte_flow_error_set(
+                               error, ENOMEM, RTE_FLOW_ERROR_TYPE_HANDLE,
+                               NULL, "Failure sending nl request");
+                       goto end;
+               }
+               ret = nl_recv_ack(pmd->nlsk_fd);
+               if (ret < 0) {
+                       rte_flow_error_set(
+                               error, ENOMEM, RTE_FLOW_ERROR_TYPE_HANDLE,
+                               NULL, "Failure trying to receive nl ack");
+                       goto end;
+               }
+       }
+end:
+       if (remote_flow)
+               rte_free(remote_flow);
+       rte_free(flow);
+       return ret;
 }
 
 /**
@@ -120,13 +1292,13 @@ tap_flow_create(struct rte_eth_dev *dev,
  * @see rte_flow_ops
  */
 static int
-tap_flow_destroy(struct rte_eth_dev *dev __rte_unused,
+tap_flow_destroy(struct rte_eth_dev *dev,
                 struct rte_flow *flow,
-                struct rte_flow_error *error __rte_unused)
+                struct rte_flow_error *error)
 {
-       LIST_REMOVE(flow, next);
-       rte_free(flow);
-       return 0;
+       struct pmd_internals *pmd = dev->data->dev_private;
+
+       return tap_flow_destroy_pmd(pmd, flow, error);
 }
 
 /**
@@ -149,6 +1321,128 @@ tap_flow_flush(struct rte_eth_dev *dev, struct rte_flow_error *error)
        return 0;
 }
 
+/**
+ * Add an implicit flow rule on the remote device to make sure traffic gets to
+ * the tap netdevice from there.
+ *
+ * @param pmd
+ *   Pointer to private structure.
+ * @param[in] idx
+ *   The idx in the implicit_rte_flows array specifying which rule to apply.
+ *
+ * @return -1 if the rule couldn't be applied, 0 otherwise.
+ */
+int tap_flow_implicit_create(struct pmd_internals *pmd,
+                            enum implicit_rule_index idx)
+{
+       struct rte_flow_item *items = implicit_rte_flows[idx].items;
+       struct rte_flow_attr *attr = &implicit_rte_flows[idx].attr;
+       struct rte_flow_item_eth eth_local = { .type = 0 };
+       uint16_t if_index = pmd->remote_if_index;
+       struct rte_flow *remote_flow = NULL;
+       struct nlmsg *msg = NULL;
+       int err = 0;
+       struct rte_flow_item items_local[2] = {
+               [0] = {
+                       .type = items[0].type,
+                       .spec = &eth_local,
+                       .mask = items[0].mask,
+               },
+               [1] = {
+                       .type = items[1].type,
+               }
+       };
+
+       remote_flow = rte_malloc(__func__, sizeof(struct rte_flow), 0);
+       if (!remote_flow) {
+               RTE_LOG(ERR, PMD, "Cannot allocate memory for rte_flow");
+               goto fail;
+       }
+       msg = &remote_flow->msg;
+       if (idx == TAP_REMOTE_TX) {
+               if_index = pmd->if_index;
+       } else if (idx == TAP_REMOTE_LOCAL_MAC) {
+               /*
+                * eth addr couldn't be set in implicit_rte_flows[] as it is not
+                * known at compile time.
+                */
+               memcpy(&eth_local.dst, &pmd->eth_addr, sizeof(pmd->eth_addr));
+               items = items_local;
+       }
+       tc_init_msg(msg, if_index, RTM_NEWTFILTER,
+                   NLM_F_REQUEST | NLM_F_ACK | NLM_F_EXCL | NLM_F_CREATE);
+       msg->t.tcm_info = TC_H_MAKE(0, htons(ETH_P_ALL));
+       tap_flow_set_handle(remote_flow);
+       if (priv_flow_process(pmd, attr, items, NULL, NULL,
+                             remote_flow, implicit_rte_flows[idx].mirred)) {
+               RTE_LOG(ERR, PMD, "rte flow rule validation failed\n");
+               goto fail;
+       }
+       err = nl_send(pmd->nlsk_fd, &msg->nh);
+       if (err < 0) {
+               RTE_LOG(ERR, PMD, "Failure sending nl request");
+               goto fail;
+       }
+       err = nl_recv_ack(pmd->nlsk_fd);
+       if (err < 0) {
+               RTE_LOG(ERR, PMD,
+                       "Kernel refused TC filter rule creation");
+               goto fail;
+       }
+       LIST_INSERT_HEAD(&pmd->implicit_flows, remote_flow, next);
+       return 0;
+fail:
+       if (remote_flow)
+               rte_free(remote_flow);
+       return -1;
+}
+
+/**
+ * Remove specific implicit flow rule on the remote device.
+ *
+ * @param[in, out] pmd
+ *   Pointer to private structure.
+ * @param[in] idx
+ *   The idx in the implicit_rte_flows array specifying which rule to remove.
+ *
+ * @return -1 if one of the implicit rules couldn't be created, 0 otherwise.
+ */
+int tap_flow_implicit_destroy(struct pmd_internals *pmd,
+                             enum implicit_rule_index idx)
+{
+       struct rte_flow *remote_flow;
+       int cur_prio = -1;
+       int idx_prio = implicit_rte_flows[idx].attr.priority + PRIORITY_OFFSET;
+
+       for (remote_flow = LIST_FIRST(&pmd->implicit_flows);
+            remote_flow;
+            remote_flow = LIST_NEXT(remote_flow, next)) {
+               cur_prio = (remote_flow->msg.t.tcm_info >> 16) & PRIORITY_MASK;
+               if (cur_prio != idx_prio)
+                       continue;
+               return tap_flow_destroy_pmd(pmd, remote_flow, NULL);
+       }
+       return 0;
+}
+
+/**
+ * Destroy all implicit flows.
+ *
+ * @see rte_flow_flush()
+ */
+int
+tap_flow_implicit_flush(struct pmd_internals *pmd, struct rte_flow_error *error)
+{
+       struct rte_flow *remote_flow;
+
+       while (!LIST_EMPTY(&pmd->implicit_flows)) {
+               remote_flow = LIST_FIRST(&pmd->implicit_flows);
+               if (tap_flow_destroy_pmd(pmd, remote_flow, error) < 0)
+                       return -1;
+       }
+       return 0;
+}
+
 /**
  * Manage filter operations.
  *
@@ -170,6 +1464,10 @@ tap_dev_filter_ctrl(struct rte_eth_dev *dev,
                    enum rte_filter_op filter_op,
                    void *arg)
 {
+       struct pmd_internals *pmd = dev->data->dev_private;
+
+       if (!pmd->flower_support)
+               return -ENOTSUP;
        switch (filter_type) {
        case RTE_ETH_FILTER_GENERIC:
                if (filter_op != RTE_ETH_FILTER_GET)