diff mbox series

[ovs-dev,V4,07/17] netdev-offload-dpdk: Implement flow get method

Message ID 20191216151047.5967-8-elibr@mellanox.com
State Changes Requested
Delegated to: Ilya Maximets
Headers show
Series netdev datapath actions offload | expand

Commit Message

Eli Britstein Dec. 16, 2019, 3:10 p.m. UTC 
Implement the flow get method for DPDK, to get the statistics of the
provided ufid, towards reading statistics of fully offloaded flows.

Signed-off-by: Eli Britstein <elibr@mellanox.com>
Reviewed-by: Oz Shlomo <ozsh@mellanox.com>
---
 lib/netdev-offload-dpdk.c | 38 ++++++++++++++++++++++++++++++++++++++
 1 file changed, 38 insertions(+)
diff mbox series

Patch

diff --git a/lib/netdev-offload-dpdk.c b/lib/netdev-offload-dpdk.c
index 536071ac4..79a7d33fb 100644
--- a/lib/netdev-offload-dpdk.c
+++ b/lib/netdev-offload-dpdk.c
@@ -56,6 +56,7 @@  struct ufid_to_rte_flow_data {
     struct cmap_node node;
     ovs_u128 ufid;
     struct rte_flow *rte_flow;
+    struct dpif_flow_stats stats;
 };
 
 /* Find rte_flow with @ufid. */
@@ -885,9 +886,46 @@  netdev_offload_dpdk_init_flow_api(struct netdev *netdev)
     return netdev_dpdk_flow_api_supported(netdev) ? 0 : EOPNOTSUPP;
 }
 
+static int
+netdev_offload_dpdk_flow_get(struct netdev *netdev,
+                             struct match *match OVS_UNUSED,
+                             struct nlattr **actions OVS_UNUSED,
+                             const ovs_u128 *ufid,
+                             struct dpif_flow_stats *stats,
+                             struct dpif_flow_attrs *attrs OVS_UNUSED,
+                             struct ofpbuf *buf OVS_UNUSED)
+{
+    struct rte_flow_query_count query = { .reset = 1 };
+    struct ufid_to_rte_flow_data *rte_flow_data;
+    struct rte_flow_error error;
+    int ret = 0;
+
+    rte_flow_data = ufid_to_rte_flow_data_find(ufid);
+    if (!rte_flow_data || !rte_flow_data->rte_flow) {
+        ret = -1;
+        goto out;
+    }
+
+    ret = netdev_dpdk_rte_flow_query_count(netdev, rte_flow_data->rte_flow,
+                                           &query, &error);
+    if (ret) {
+        goto out;
+    }
+    rte_flow_data->stats.n_packets += (query.hits_set) ? query.hits : 0;
+    rte_flow_data->stats.n_bytes += (query.bytes_set) ? query.bytes : 0;
+    if (query.hits_set && query.hits) {
+        rte_flow_data->stats.used = time_usec() / 1000;
+    }
+    memcpy(stats, &rte_flow_data->stats, sizeof *stats);
+
+out:
+    return ret;
+}
+
 const struct netdev_flow_api netdev_offload_dpdk = {
     .type = "dpdk_flow_api",
     .flow_put = netdev_offload_dpdk_flow_put,
     .flow_del = netdev_offload_dpdk_flow_del,
     .init_flow_api = netdev_offload_dpdk_init_flow_api,
+    .flow_get = netdev_offload_dpdk_flow_get,
 };