nfp: flower-ct: add flow_pay to the offload table
authorLouis Peens <louis.peens@corigine.com>
Thu, 22 Jul 2021 07:58:06 +0000 (09:58 +0200)
committerDavid S. Miller <davem@davemloft.net>
Thu, 22 Jul 2021 09:22:22 +0000 (02:22 -0700)
Compile the offload flow metadata and add flow_pay to the offload
table. Also add in the delete paths. This does not include actual
offloading to the card yet, this will follow soon.

Signed-off-by: Louis Peens <louis.peens@corigine.com>
Signed-off-by: Yinjun Zhang <yinjun.zhang@corigine.com>
Signed-off-by: Simon Horman <simon.horman@corigine.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
drivers/net/ethernet/netronome/nfp/flower/conntrack.c
drivers/net/ethernet/netronome/nfp/flower/main.h
drivers/net/ethernet/netronome/nfp/flower/offload.c

index 41b1f9773d46a6dc28ca4742156ac2d814057f37..8ab7c7e8792d2fe98cffb9a68e3a32fc6da23b15 100644 (file)
@@ -2,6 +2,7 @@
 /* Copyright (C) 2021 Corigine, Inc. */
 
 #include "conntrack.h"
+#include "../nfp_port.h"
 
 const struct rhashtable_params nfp_tc_ct_merge_params = {
        .head_offset            = offsetof(struct nfp_fl_ct_tc_merge,
@@ -549,6 +550,7 @@ static int nfp_fl_ct_add_offload(struct nfp_fl_nft_tc_merge *m_entry)
 
        struct flow_rule *rules[_CT_TYPE_MAX];
        u8 *key, *msk, *kdata, *mdata;
+       struct nfp_port *port = NULL;
        struct net_device *netdev;
        bool qinq_sup;
        u32 port_id;
@@ -792,7 +794,40 @@ static int nfp_fl_ct_add_offload(struct nfp_fl_nft_tc_merge *m_entry)
        if (err)
                goto ct_offload_err;
 
+       /* Use the pointer address as the cookie, but set the last bit to 1.
+        * This is to avoid the 'is_merge_flow' check from detecting this as
+        * an already merged flow. This works since address alignment means
+        * that the last bit for pointer addresses will be 0.
+        */
+       flow_pay->tc_flower_cookie = ((unsigned long)flow_pay) | 0x1;
+       err = nfp_compile_flow_metadata(priv->app, flow_pay->tc_flower_cookie,
+                                       flow_pay, netdev, NULL);
+       if (err)
+               goto ct_offload_err;
+
+       if (nfp_netdev_is_nfp_repr(netdev))
+               port = nfp_port_from_netdev(netdev);
+
+       err = rhashtable_insert_fast(&priv->flow_table, &flow_pay->fl_node,
+                                    nfp_flower_table_params);
+       if (err)
+               goto ct_release_offload_meta_err;
+
+       m_entry->tc_flower_cookie = flow_pay->tc_flower_cookie;
+       m_entry->flow_pay = flow_pay;
+
+       if (port)
+               port->tc_offload_cnt++;
+
+       return err;
+
+ct_release_offload_meta_err:
+       nfp_modify_flow_metadata(priv->app, flow_pay);
 ct_offload_err:
+       if (flow_pay->nfp_tun_ipv4_addr)
+               nfp_tunnel_del_ipv4_off(priv->app, flow_pay->nfp_tun_ipv4_addr);
+       if (flow_pay->nfp_tun_ipv6)
+               nfp_tunnel_put_ipv6_off(priv->app, flow_pay->nfp_tun_ipv6);
        kfree(flow_pay->action_data);
        kfree(flow_pay->mask_data);
        kfree(flow_pay->unmasked_data);
@@ -803,7 +838,45 @@ ct_offload_err:
 static int nfp_fl_ct_del_offload(struct nfp_app *app, unsigned long cookie,
                                 struct net_device *netdev)
 {
-       return 0;
+       struct nfp_flower_priv *priv = app->priv;
+       struct nfp_fl_payload *flow_pay;
+       struct nfp_port *port = NULL;
+       int err = 0;
+
+       if (nfp_netdev_is_nfp_repr(netdev))
+               port = nfp_port_from_netdev(netdev);
+
+       flow_pay = nfp_flower_search_fl_table(app, cookie, netdev);
+       if (!flow_pay)
+               return -ENOENT;
+
+       err = nfp_modify_flow_metadata(app, flow_pay);
+       if (err)
+               goto err_free_merge_flow;
+
+       if (flow_pay->nfp_tun_ipv4_addr)
+               nfp_tunnel_del_ipv4_off(app, flow_pay->nfp_tun_ipv4_addr);
+
+       if (flow_pay->nfp_tun_ipv6)
+               nfp_tunnel_put_ipv6_off(app, flow_pay->nfp_tun_ipv6);
+
+       if (!flow_pay->in_hw) {
+               err = 0;
+               goto err_free_merge_flow;
+       }
+
+err_free_merge_flow:
+       nfp_flower_del_linked_merge_flows(app, flow_pay);
+       if (port)
+               port->tc_offload_cnt--;
+       kfree(flow_pay->action_data);
+       kfree(flow_pay->mask_data);
+       kfree(flow_pay->unmasked_data);
+       WARN_ON_ONCE(rhashtable_remove_fast(&priv->flow_table,
+                                           &flow_pay->fl_node,
+                                           nfp_flower_table_params));
+       kfree_rcu(flow_pay, rcu);
+       return err;
 }
 
 static int nfp_ct_do_nft_merge(struct nfp_fl_ct_zone_entry *zt,
index 226bcbf6e5b5d979ef978ee445484d17b9c7e537..9e933deabfe25e4122574f9e3bad236ab9048a22 100644 (file)
@@ -560,4 +560,7 @@ int nfp_flower_calculate_key_layers(struct nfp_app *app,
                                    struct flow_rule *flow,
                                    enum nfp_flower_tun_type *tun_type,
                                    struct netlink_ext_ack *extack);
+void
+nfp_flower_del_linked_merge_flows(struct nfp_app *app,
+                                 struct nfp_fl_payload *sub_flow);
 #endif
index 87a32e9fe4e51329a560f5258235e5cfaa05ad0f..e510711f6398ed74d19ca85ff66d757c923016f5 100644 (file)
@@ -1470,7 +1470,7 @@ err_free_links:
        kfree_rcu(merge_flow, rcu);
 }
 
-static void
+void
 nfp_flower_del_linked_merge_flows(struct nfp_app *app,
                                  struct nfp_fl_payload *sub_flow)
 {