summaryrefslogtreecommitdiffstats
path: root/src
diff options
context:
space:
mode:
authorSimon Zhang <yuwei1.zhang@intel.com>2019-03-28 16:38:49 +0800
committerDave Wallace <dwallacelf@gmail.com>2019-05-17 12:57:27 +0000
commit84a53bc5fed2b0343a672e62b4790d5e5c1e1430 (patch)
treed136e63e288d589a71f6da4cf15d6916bcfe3796 /src
parentcfd30860fb1db3add1a2548d758eb72362416bbc (diff)
Add rte_flow_validate to verify the flow before calling rte_flow_create.
Change-Id: I817580963e38bafcc53cc6502b9c58f567070244 Signed-off-by: Simon Zhang <yuwei1.zhang@intel.com>
Diffstat (limited to 'src')
-rw-r--r--src/plugins/dpdk/device/flow.c17
1 files changed, 14 insertions, 3 deletions
diff --git a/src/plugins/dpdk/device/flow.c b/src/plugins/dpdk/device/flow.c
index 63f04b8323d..fe52bcb5ac8 100644
--- a/src/plugins/dpdk/device/flow.c
+++ b/src/plugins/dpdk/device/flow.c
@@ -209,12 +209,23 @@ dpdk_flow_add (dpdk_device_t * xd, vnet_flow_t * f, dpdk_flow_entry_t * fe)
vec_add2 (actions, action, 1);
action->type = RTE_FLOW_ACTION_TYPE_END;
+ rv = rte_flow_validate (xd->device_index, &ingress, items, actions,
+ &xd->last_flow_error);
+ if (rv != 0)
+ {
+ if (rv == -EINVAL)
+ rv = VNET_FLOW_ERROR_NOT_SUPPORTED;
+ else if (rv == -EEXIST)
+ rv = VNET_FLOW_ERROR_ALREADY_EXISTS;
+ else
+ rv = VNET_FLOW_ERROR_INTERNAL;
+
+ goto done;
+ }
+
fe->handle = rte_flow_create (xd->device_index, &ingress, items, actions,
&xd->last_flow_error);
- if (!fe->handle)
- rv = VNET_FLOW_ERROR_NOT_SUPPORTED;
-
done:
vec_free (items);
vec_free (actions);