From d44296e70bb4f1b7fab42d454cef125d3d4479e3 Mon Sep 17 00:00:00 2001 From: Prabhjot Singh Sethi Date: Sat, 27 Feb 2016 23:21:58 +0530 Subject: [PATCH] Disable Eviction of flow in Vrouter Vrouter will track tcp flags, but will not evict the flow entry once the session is closed and let vrouter agent look at the flag to delete the flow eventually. This is required till vrouter agent handles evict event in a more stable way Partial-Bug: 1550759 Change-Id: Ic7074442b22fc2c11362274e4e390654f3f2400a --- dp-core/vr_flow.c | 11 +++++++++++ 1 file changed, 11 insertions(+) diff --git a/dp-core/vr_flow.c b/dp-core/vr_flow.c index 809615cbe..8110173ee 100644 --- a/dp-core/vr_flow.c +++ b/dp-core/vr_flow.c @@ -810,8 +810,19 @@ vr_flow_action(struct vrouter *router, struct vr_flow_entry *fe, break; } + /* + * Eviction of Flow in vrouter add additional events to vrouter + * agent state machine making it complex and unstable, till + * agent is handling vrouter eviction appropriately, disabling + * eviction from vrouter. + * NOTE: Vrouter agent will look at VR_FLOW_TCP_DEAD flag to + * clear/evict flow immediately + * TODO(prabhjot): need to re-enable, after stablizing agent + */ +#if 0 if (fe->fe_tcp_flags & VR_FLOW_TCP_DEAD) vr_flow_mark_evict(router, fe, index); +#endif return result; }