diff options
| author | Nithin Raju <nithin@vmware.com> | 2015-12-10 11:16:51 -0800 |
|---|---|---|
| committer | Ben Pfaff <blp@ovn.org> | 2015-12-15 00:47:44 -0800 |
| commit | 398e182e365d1709a0d085bdee365ff3c19ca001 (patch) | |
| tree | 628d9fdb3562fe269b34a4651dff7c0909b448b3 /datapath-windows | |
| parent | 5d8b248c8af699d5f52567616f3bf5f125975df7 (diff) | |
| download | openvswitch-398e182e365d1709a0d085bdee365ff3c19ca001.tar.gz | |
datapath-windows: remove ASSERT in OvsDoFlowLookupOutput()
We needed this ASSERT earlier to catch unexpected cases. This code seems
to be fairly stable, and we can remove the ASSERT.
It is annoying to be hitting this ASSERT while changing the internal
adapter properties.
Signed-off-by: Nithin Raju <nithin@vmware.com>
Acked-by: Alin Gabriel Serdean <aserdean@cloudbasesolutions.com>
Signed-off-by: Ben Pfaff <blp@ovn.org>
Diffstat (limited to 'datapath-windows')
| -rw-r--r-- | datapath-windows/ovsext/Actions.c | 1 |
1 files changed, 0 insertions, 1 deletions
diff --git a/datapath-windows/ovsext/Actions.c b/datapath-windows/ovsext/Actions.c index c113a84eb..650658476 100644 --- a/datapath-windows/ovsext/Actions.c +++ b/datapath-windows/ovsext/Actions.c @@ -570,7 +570,6 @@ OvsDoFlowLookupOutput(OvsForwardingContext *ovsFwdCtx) POVS_VPORT_ENTRY vport = OvsFindVportByPortNo(ovsFwdCtx->switchContext, ovsFwdCtx->srcVportNo); if (vport == NULL || vport->ovsState != OVS_STATE_CONNECTED) { - ASSERT(FALSE); // XXX: let's catch this for now OvsCompleteNBLForwardingCtx(ovsFwdCtx, L"OVS-Dropped due to internal/tunnel port removal"); ovsActionStats.noVport++; |