Add the corresponding logics to support the offload
of UDP item.

Signed-off-by: Chaoyong He <chaoyong...@corigine.com>
Reviewed-by: Niklas Söderlund <niklas.soderl...@corigine.com>
---
 doc/guides/nics/features/nfp.ini       |  1 +
 doc/guides/rel_notes/release_22_11.rst |  1 +
 drivers/net/nfp/nfp_flow.c             | 63 ++++++++++++++++++++++++++++++++--
 3 files changed, 63 insertions(+), 2 deletions(-)

diff --git a/doc/guides/nics/features/nfp.ini b/doc/guides/nics/features/nfp.ini
index 66ee03c..f6ba2eb 100644
--- a/doc/guides/nics/features/nfp.ini
+++ b/doc/guides/nics/features/nfp.ini
@@ -33,6 +33,7 @@ ipv6                 = Y
 ipv6_frag_ext        = Y
 port_id              = Y
 tcp                  = Y
+udp                  = Y
 vlan                 = Y
 
 [rte_flow actions]
diff --git a/doc/guides/rel_notes/release_22_11.rst 
b/doc/guides/rel_notes/release_22_11.rst
index 0ec1c07..7972725 100644
--- a/doc/guides/rel_notes/release_22_11.rst
+++ b/doc/guides/rel_notes/release_22_11.rst
@@ -170,6 +170,7 @@ New Features
     * IPv4
     * IPv6
     * TCP
+    * UDP
 
   Add the support of rte_flow actions as follow:
 
diff --git a/drivers/net/nfp/nfp_flow.c b/drivers/net/nfp/nfp_flow.c
index 739d980..dc917fc 100644
--- a/drivers/net/nfp/nfp_flow.c
+++ b/drivers/net/nfp/nfp_flow.c
@@ -549,6 +549,11 @@ struct nfp_mask_id_entry {
                        key_ls->key_layer |= NFP_FLOWER_LAYER_TP;
                        key_ls->key_size += sizeof(struct nfp_flower_tp_ports);
                        break;
+               case RTE_FLOW_ITEM_TYPE_UDP:
+                       PMD_DRV_LOG(DEBUG, "RTE_FLOW_ITEM_TYPE_UDP detected");
+                       key_ls->key_layer |= NFP_FLOWER_LAYER_TP;
+                       key_ls->key_size += sizeof(struct nfp_flower_tp_ports);
+                       break;
                default:
                        PMD_DRV_LOG(ERR, "Item type %d not supported.", 
item->type);
                        return -ENOTSUP;
@@ -850,6 +855,47 @@ struct nfp_mask_id_entry {
        return 0;
 }
 
+static int
+nfp_flow_merge_udp(struct rte_flow *nfp_flow,
+               char **mbuf_off,
+               const struct rte_flow_item *item,
+               const struct nfp_flow_item_proc *proc,
+               bool is_mask)
+{
+       char *ports_off;
+       struct nfp_flower_tp_ports *ports;
+       const struct rte_flow_item_udp *spec;
+       const struct rte_flow_item_udp *mask;
+       struct nfp_flower_meta_tci *meta_tci;
+
+       spec = item->spec;
+       if (spec == NULL) {
+               PMD_DRV_LOG(DEBUG, "nfp flow merge udp: no item->spec!");
+               return 0;
+       }
+
+       meta_tci = (struct nfp_flower_meta_tci 
*)nfp_flow->payload.unmasked_data;
+       if (meta_tci->nfp_flow_key_layer & NFP_FLOWER_LAYER_IPV4) {
+               ports_off = *mbuf_off - sizeof(struct nfp_flower_ipv4) -
+                       sizeof(struct nfp_flower_tp_ports);
+       } else {/* IPv6 */
+               ports_off = *mbuf_off - sizeof(struct nfp_flower_ipv6) -
+                       sizeof(struct nfp_flower_tp_ports);
+       }
+       ports = (struct nfp_flower_tp_ports *)ports_off;
+
+       mask = item->mask ? item->mask : proc->mask_default;
+       if (is_mask) {
+               ports->port_src = mask->hdr.src_port;
+               ports->port_dst = mask->hdr.dst_port;
+       } else {
+               ports->port_src = spec->hdr.src_port;
+               ports->port_dst = spec->hdr.dst_port;
+       }
+
+       return 0;
+}
+
 /* Graph of supported items and associated process function */
 static const struct nfp_flow_item_proc nfp_flow_item_proc_list[] = {
        [RTE_FLOW_ITEM_TYPE_END] = {
@@ -886,7 +932,8 @@ struct nfp_mask_id_entry {
                .merge = nfp_flow_merge_vlan,
        },
        [RTE_FLOW_ITEM_TYPE_IPV4] = {
-               .next_item = NEXT_ITEM(RTE_FLOW_ITEM_TYPE_TCP),
+               .next_item = NEXT_ITEM(RTE_FLOW_ITEM_TYPE_TCP,
+                       RTE_FLOW_ITEM_TYPE_UDP),
                .mask_support = &(const struct rte_flow_item_ipv4){
                        .hdr = {
                                .type_of_service = 0xff,
@@ -902,7 +949,8 @@ struct nfp_mask_id_entry {
                .merge = nfp_flow_merge_ipv4,
        },
        [RTE_FLOW_ITEM_TYPE_IPV6] = {
-               .next_item = NEXT_ITEM(RTE_FLOW_ITEM_TYPE_TCP),
+               .next_item = NEXT_ITEM(RTE_FLOW_ITEM_TYPE_TCP,
+                       RTE_FLOW_ITEM_TYPE_UDP),
                .mask_support = &(const struct rte_flow_item_ipv6){
                        .hdr = {
                                .vtc_flow   = RTE_BE32(0x0ff00000),
@@ -931,6 +979,17 @@ struct nfp_mask_id_entry {
                .mask_sz = sizeof(struct rte_flow_item_tcp),
                .merge = nfp_flow_merge_tcp,
        },
+       [RTE_FLOW_ITEM_TYPE_UDP] = {
+               .mask_support = &(const struct rte_flow_item_udp){
+                       .hdr = {
+                               .src_port = RTE_BE16(0xffff),
+                               .dst_port = RTE_BE16(0xffff),
+                       },
+               },
+               .mask_default = &rte_flow_item_udp_mask,
+               .mask_sz = sizeof(struct rte_flow_item_udp),
+               .merge = nfp_flow_merge_udp,
+       },
 };
 
 static int
-- 
1.8.3.1

Reply via email to