Acked-by: Eitan Eliahu <[email protected]> Thanks, Eitan
-----Original Message----- From: dev [mailto:[email protected]] On Behalf Of Sorin Vinturis Sent: Wednesday, April 15, 2015 3:42 AM To: [email protected] Subject: [ovs-dev] [PATCH v2 2/2] datapath-windows: Removed assert from FilterNetPnPEvent handler Removed an inappropriate assert from the FilterNetPnPEvent routine, OvsExtNetPnPEvent. When NDIS calls the FilterNetPnPEvent routine, the extension is in paused state and, obviously, the switch is not active. The switch becomes active after FilterRestart routine is called and the restart is successfully complete. Signed-off-by: Sorin Vinturis <[email protected]> --- datapath-windows/ovsext/Switch.c | 1 - 1 file changed, 1 deletion(-) diff --git a/datapath-windows/ovsext/Switch.c b/datapath-windows/ovsext/Switch.c index 2b9a713..cfbb5a0 100644 --- a/datapath-windows/ovsext/Switch.c +++ b/datapath-windows/ovsext/Switch.c @@ -523,7 +523,6 @@ OvsExtNetPnPEvent(NDIS_HANDLE filterModuleContext, switchContext->isActivateFailed = TRUE; } else { ASSERT(switchContext->isActivated == FALSE); - ASSERT(switchActive == TRUE); if (switchContext->isActivated == FALSE && switchActive == TRUE) { status = OvsActivateSwitch(switchContext); OVS_LOG_TRACE("OvsExtNetPnPEvent: activated switch: %p " -- 1.9.0.msysgit.0 _______________________________________________ dev mailing list [email protected] https://urldefense.proofpoint.com/v2/url?u=http-3A__openvswitch.org_mailman_listinfo_dev&d=AwIGaQ&c=Sqcl0Ez6M0X8aeM67LKIiDJAXVeAw-YihVMNtXt-uEs&r=CWsgHUxi6ExLXY798tmo3LJ4e3geGYp56lkcH-5cLCY&m=CSXu-iIaADJIxM63f-MXIdiak6jSiIUo2oyeS7uqPJs&s=kjRXgtYuj-q0wGQqabL0U0U-XMev-5H0lYo5y2rU2LY&e= _______________________________________________ dev mailing list [email protected] http://openvswitch.org/mailman/listinfo/dev
