Changeset 87048 in vbox for trunk/src/VBox/Devices/Storage/DevATA.cpp

Timestamp:

Dec 7, 2020 2:44:34 PM (4 years ago)

Author:

vboxsync

Message:

CUE, DevATA: Added support for MODE2/2352 tracks in CUE sheets and IDE/ATAPI emulation (see bugref:6975).

File:

• trunk/src/VBox/Devices/Storage/DevATA.cpp (modified) (3 diffs)

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

@@ -2003 +2003 @@
     int rc;
     uint64_t cbBlockRegion = 0;
+    VDREGIONDATAFORM enmDataForm;
 
     Assert(s->uTxDir == PDMMEDIATXDIR_FROM_DEVICE);
@@ -2016 +2017 @@
 
     rc = pDevR3->pDrvMedia->pfnQueryRegionPropertiesForLba(pDevR3->pDrvMedia, iATAPILBA, NULL, NULL,
-                                                      &cbBlockRegion, NULL);
+                                                      &cbBlockRegion, &enmDataForm);
     if (RT_SUCCESS(rc))
     {
@@ -2069 +2070 @@
                 {
                     uint8_t abTmp[2352];
+                    uint8_t cbSkip;
+
                     rc = pDevR3->pDrvMedia->pfnRead(pDevR3->pDrvMedia, (uint64_t)i * 2352, &abTmp[0], 2352);
                     if (RT_FAILURE(rc))
                         break;
 
-                    memcpy(pbBuf, &abTmp[16], 2048);
+                    /* Mode 2 has an additional subheader before user data; we need to
+                     * skip 16 bytes for Mode 1 (sync + header) and 20 bytes for Mode 2       +
+                     * (sync + header + subheader).
+                     */
+                    switch (enmDataForm) {
+                    case VDREGIONDATAFORM_MODE2_2352:
+                    case VDREGIONDATAFORM_XA_2352:
+                        cbSkip = 24;
+                        break;
+                    case VDREGIONDATAFORM_MODE1_2352:
+                        cbSkip = 16;
+                        break;
+                    default:
+                        AssertMsgFailed(("Unexpected region form (%#u), using default skip value\n", enmDataForm));
+                        cbSkip = 16;
+                    }
+                    memcpy(pbBuf, &abTmp[cbSkip], 2048);
                     pbBuf += 2048;
                 }