Hey Nithin,

AFAIK OvsCleanupOpenInstance is called by OvsCleanupDevice, which is a callback 
called by NDIS when an IO is pending and the file must be closed.
FreeUserDumpState is only for dump operations.

Is it possible that an IO to be pending (packet queueing) while at the same 
time a dump operation to be in progress, for the same file?

Sam
________________________________________
Date: Wed, 10 Sep 2014 13:17:17 -0700
From: Nithin Raju <nit...@vmware.com>
To: dev@openvswitch.org
Subject: [ovs-dev] [PATCH] datapath-windows: cleanup dump state during
        instance cleanup
Message-ID: <1410380237-28794-1-git-send-email-nit...@vmware.com>

Signed-off-by: Nithin Raju <nit...@vmware.com>
---
 datapath-windows/ovsext/Datapath.c |    1 +
 1 files changed, 1 insertions(+), 0 deletions(-)

diff --git a/datapath-windows/ovsext/Datapath.c 
b/datapath-windows/ovsext/Datapath.c
index c145d00..d1f4b9d 100644
--- a/datapath-windows/ovsext/Datapath.c
+++ b/datapath-windows/ovsext/Datapath.c
@@ -417,6 +417,7 @@ OvsCleanupOpenInstance(PFILE_OBJECT fileObject)
     POVS_OPEN_INSTANCE instance = (POVS_OPEN_INSTANCE)fileObject->FsContext;
     ASSERT(instance);
     ASSERT(fileObject == instance->fileObject);
+    FreeUserDumpState(instance);
     OvsCleanupEvent(instance);
     OvsCleanupPacketQueue(instance);
 }
--
1.7.4.1
_______________________________________________
dev mailing list
dev@openvswitch.org
http://openvswitch.org/mailman/listinfo/dev

Reply via email to