Changeset 36068 in vbox

Timestamp:

Feb 24, 2011 10:04:44 AM (14 years ago)

Author:

vboxsync

svn:sync-xref-src-repo-rev:

70188

Message:

VBoxUSBMon.sys: initialize RemoveLock once.

Location:

trunk/src/VBox/HostDrivers/VBoxUSB/win/Monitor

Files:

• USBMon.cpp (modified) (1 diff)
• USBMon.h (modified) (1 diff)

trunk/src/VBox/HostDrivers/VBoxUSB/win/Monitor/USBMon.cpp

@@ -313 +313 @@
         }
 
-        //
-        // Let us use remove lock to keep count of IRPs so that we don't
-        // detach and delete our deviceobject until all pending I/Os in our
-        // devstack are completed. Remlock is required to protect us from
-        // various race conditions where our driver can get unloaded while we
-        // are still running dispatch or completion code.
-        //
-
-        IoInitializeRemoveLock (&pDevExt->RemoveLock , POOL_TAG,
-                                1,      // MaxLockedMinutes
-                                100);   // HighWatermark, this parameter is
-                                        // used only on checked build. Specifies
-                                        // the maximum number of outstanding
-                                        // acquisitions allowed on the lock
-
-        DebugPrint(("VBoxUSBMon: remove lock = %x\n", pDevExt->RemoveLock.Common.RemoveEvent));
+        if (InterlockedCompareExchange(&pDevExt->fRemoveLockInitialized, 1, 0))
+        {
+            //
+            // Let us use remove lock to keep count of IRPs so that we don't
+            // detach and delete our deviceobject until all pending I/Os in our
+            // devstack are completed. Remlock is required to protect us from
+            // various race conditions where our driver can get unloaded while we
+            // are still running dispatch or completion code.
+            //
+
+            IoInitializeRemoveLock (&pDevExt->RemoveLock , POOL_TAG,
+                                    1,      // MaxLockedMinutes
+                                    100);   // HighWatermark, this parameter is
+                                            // used only on checked build. Specifies
+                                            // the maximum number of outstanding
+                                            // acquisitions allowed on the lock
+
+            DebugPrint(("VBoxUSBMon: remove lock = %x\n", pDevExt->RemoveLock.Common.RemoveEvent));
+        }
     }
 

trunk/src/VBox/HostDrivers/VBoxUSB/win/Monitor/USBMon.h

@@ -83 +83 @@
     IO_REMOVE_LOCK      RemoveLock;
 
+    /* Whether the lock has been initialized. */
+    LONG                fRemoveLockInitialized;
 
     /* Number of times the device was opened. */