Changeset 25836 in vbox for trunk/src/VBox/Main

Timestamp:

Jan 14, 2010 4:44:46 PM (15 years ago)

Author:

vboxsync

Message:

Main: locking order fixes; GUI comes up now with strict locking enabled

Location:

trunk/src/VBox/Main

Files:

• HostImpl.cpp (modified) (1 diff)
• MediumImpl.cpp (modified) (2 diffs)
• glue/AutoLock.cpp (modified) (1 diff)
• include/SnapshotImpl.h (modified) (2 diffs)

trunk/src/VBox/Main/HostImpl.cpp

@@ -152 +152 @@
     Data()
 #ifdef VBOX_WITH_USB
-        : usbListsLock(LOCKCLASS_HOSTLIST)
+        : usbListsLock(LOCKCLASS_USBLIST)
 #endif
     {};

trunk/src/VBox/Main/MediumImpl.cpp

@@ -1657 +1657 @@
         if (FAILED(autoCaller.rc())) return autoCaller.rc();
 
-        AutoReadLock alock(this COMMA_LOCKVAL_SRC_POS);
-
         /* we access mParent */
         AutoReadLock treeLock(m->pVirtualBox->hardDiskTreeLockHandle() COMMA_LOCKVAL_SRC_POS);
+
+        AutoReadLock alock(this COMMA_LOCKVAL_SRC_POS);
 
         if (m->pParent.isNull())
@@ -3008 +3008 @@
     AssertComRCReturn(autoCaller.rc(), false);
 
-    AutoReadLock alock(this COMMA_LOCKVAL_SRC_POS);
-
     /* we access children */
     AutoReadLock treeLock(m->pVirtualBox->hardDiskTreeLockHandle() COMMA_LOCKVAL_SRC_POS);
+
+    AutoReadLock alock(this COMMA_LOCKVAL_SRC_POS);
 
     switch (m->type)

trunk/src/VBox/Main/glue/AutoLock.cpp

@@ -74 +74 @@
         { LOCKCLASS_USBPROXYSERVICE,  "3-USBPROXYSERVICE" },
         { LOCKCLASS_HOSTOBJECT,       "4-HOSTOBJECT" },
-        { LOCKCLASS_HOSTLIST,         "5-HOSTLIST" },
-        { LOCKCLASS_OTHEROBJECT,      "6-OTHEROBJECT" },
-        { LOCKCLASS_OTHERLIST,        "7-OTHERLIST" },
-        { LOCKCLASS_OBJECTSTATE,      "8-OBJECTSTATE" }
+        { LOCKCLASS_OTHEROBJECT,      "5-OTHEROBJECT" },
+        { LOCKCLASS_OTHERLIST,        "6-OTHERLIST" },
+        { LOCKCLASS_SNAPSHOTOBJECT,   "7-SNAPSHOTOBJECT" },
+        { LOCKCLASS_USBLIST,          "8-USBLIST" },
+        { LOCKCLASS_OBJECTSTATE,      "9-OBJECTSTATE" }
     };
 

trunk/src/VBox/Main/include/SnapshotImpl.h

@@ -91 +91 @@
     // public methods only for internal purposes
 
-
     /**
      * Simple run-time type identification without having to enable C++ RTTI.
@@ -100 +99 @@
     {
         return clsidSnapshot;
+    }
+
+    /**
+     * Override of the default locking class to be used for validating lock
+     * order with the standard member lock handle.
+     */
+    virtual VBoxLockingClass getLockingClass() const
+    {
+        return LOCKCLASS_SNAPSHOTOBJECT;
     }