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