Skip to content

Commit

Permalink
Disable Eviction of flow in Vrouter
Browse files Browse the repository at this point in the history
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
  • Loading branch information
Prabhjot Singh Sethi committed Feb 27, 2016
1 parent 01dbe86 commit d44296e
Showing 1 changed file with 11 additions and 0 deletions.
11 changes: 11 additions & 0 deletions dp-core/vr_flow.c
Original file line number Diff line number Diff line change
Expand Up @@ -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;
}
Expand Down

0 comments on commit d44296e

Please sign in to comment.