commit cd120f9bbb2101dfd7eca11d1a28e06ac5ace479
author Chenmin Sun <chenmin.sun@intel.com> Mon Jan 20 20:17:09 2020 +0800
committer Damjan Marion <dmarion@me.com> Fri Feb 14 13:11:51 2020 +0000
tree 18ff033a94005ed0f145107250ad7da453527a74
parent 360336fa56ae913ceb17a8e0b010e96bed78cfc3

dpdk: fix flow(with mark action) deletion crash issue

Type: fix

this patch fixes mark flow deletion crash issue, see below

test flow add src-ip any proto udp src-port 111 dst-port 222 mark 100
test flow enable index 0 1/1
test flow disable index 0 1/1
test flow enable index 0 1/1
test flow disable index 0 1/1 -> [crash]

This is because the code resets a wrong vector in flow lookup entry
recycle logic. See function dpdk_flow_ops_fn().

Signed-off-by: Chenmin Sun <chenmin.sun@intel.com>
Change-Id: I2b0a1e531931ab25541d672d88da18dc2289f1ce

src/plugins/dpdk/device/flow.c

diff --git a/src/plugins/dpdk/device/flow.c b/src/plugins/dpdk/device/flow.c
index d3ca435..85a62bb 100644
--- a/src/plugins/dpdk/device/flow.c
+++ b/src/plugins/dpdk/device/flow.c
@@ -573,7 +573,7 @@
 
       vec_foreach (fl_index, xd->parked_lookup_indexes)
 	pool_put_index (xd->flow_lookup_entries, *fl_index);
-      vec_reset_length (xd->flow_lookup_entries);
+      vec_reset_length (xd->parked_lookup_indexes);
     }
 
   if (op == VNET_FLOW_DEV_OP_DEL_FLOW)