Changeset 92227 in vbox

Timestamp:

Nov 4, 2021 9:40:52 PM (3 years ago)

Author:

vboxsync

Message:

Storage: On reset mark the medium as unlocked. Solves long standing medium lock misbehavior which can be annoying with VMs running off a DVD (image or host drive).

Location:

trunk/src/VBox/Devices/Storage

Files:

• DrvHostDVD.cpp (modified) (3 diffs)
• DrvVD.cpp (modified) (1 diff)

trunk/src/VBox/Devices/Storage/DrvHostDVD.cpp

@@ -47 +47 @@
 typedef struct DRVHOSTDVD
 {
-    /** Base drivr data. */
+    /** Base driver data. */
     DRVHOSTBASE             Core;
     /** The current tracklist of the loaded medium if passthrough is used. */
@@ -499 +499 @@
 }
 
+/**
+ * Reset a host dvd drive driver instance.
+ *
+ * @copydoc FNPDMDRVRESET
+ */
+static DECLCALLBACK(void) drvHostDvdReset(PPDMDRVINS pDrvIns)
+{
+    PDRVHOSTDVD pThis = PDMINS_2_DATA(pDrvIns, PDRVHOSTDVD);
+
+    if (pThis->pTrackList)
+    {
+        ATAPIPassthroughTrackListDestroy(pThis->pTrackList);
+        pThis->pTrackList = NULL;
+    }
+
+    int rc = drvHostBaseDoLockOs(&pThis->Core, false);
+    RT_NOREF(rc);
+}
+
 
 /**
@@ -534 +553 @@
     NULL,
     /* pfnReset */
-    NULL,
+    drvHostDvdReset,
     /* pfnSuspend */
     NULL,

trunk/src/VBox/Devices/Storage/DrvVD.cpp

@@ -4479 +4479 @@
         pThis->offDisk          = 0;
     }
+    pThis->fLocked = false;
 }