Changeset 66165 in vbox

Timestamp:

Mar 20, 2017 11:06:26 AM (8 years ago)

Author:

vboxsync

Message:

DrvHostBase: Implement basic version of the IMEDIA region descriptor API

File:

• trunk/src/VBox/Devices/Storage/DrvHostBase.cpp (modified) (2 diffs)

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

@@ -337 +337 @@
     PDRVHOSTBASE pThis = RT_FROM_MEMBER(pInterface, DRVHOSTBASE, IMedia);
     return pThis->fBiosVisible;
+}
+
+
+/** @interface_method_impl{PDMIMEDIA,pfnGetRegionCount} */
+static DECLCALLBACK(uint32_t) drvHostBaseGetRegionCount(PPDMIMEDIA pInterface)
+{
+    RT_NOREF1(pInterface);
+
+    LogFlowFunc(("\n"));
+    uint32_t cRegions = 1;
+
+    /* For now just return one region for all devices. */
+    /** @todo: Handle CD/DVD passthrough properly. */
+
+    LogFlowFunc(("returns %u\n", cRegions));
+    return cRegions;
+}
+
+/** @interface_method_impl{PDMIMEDIA,pfnQueryRegionProperties} */
+static DECLCALLBACK(int) drvHostBaseQueryRegionProperties(PPDMIMEDIA pInterface, uint32_t uRegion, uint64_t *pu64LbaStart,
+                                                          uint64_t *pcBlocks, uint64_t *pcbBlock,
+                                                          PPDMMEDIAREGIONDATAFORM penmDataForm)
+{
+    LogFlowFunc(("\n"));
+    int rc = VINF_SUCCESS;
+    PDRVHOSTBASE pThis = RT_FROM_MEMBER(pInterface, DRVHOSTBASE, IMedia);
+
+    if (uRegion < 1)
+    {
+        uint64_t cbMedia;
+        rc = drvHostBaseGetMediaSizeOs(pThis, &cbMedia);
+        if (RT_SUCCESS(rc))
+        {
+            uint64_t cbBlock = 0;
+
+            if (pThis->enmType == PDMMEDIATYPE_DVD)
+                cbBlock = 2048;
+            else
+                cbBlock = 512; /* Floppy. */
+
+            if (pu64LbaStart)
+                *pu64LbaStart = 0;
+            if (pcBlocks)
+                *pcBlocks = cbMedia / cbBlock;
+            if (pcbBlock)
+                *pcbBlock = cbBlock;
+            if (penmDataForm)
+                *penmDataForm = PDMMEDIAREGIONDATAFORM_RAW;
+        }
+    }
+    else
+        rc = VERR_NOT_FOUND;
+
+    LogFlowFunc(("returns %Rrc\n", rc));
+    return rc;
+}
+
+/** @interface_method_impl{PDMIMEDIA,pfnQueryRegionPropertiesForLba} */
+static DECLCALLBACK(int) drvHostBaseQueryRegionPropertiesForLba(PPDMIMEDIA pInterface, uint64_t u64LbaStart,
+                                                                uint64_t *pcBlocks, uint64_t *pcbBlock,
+                                                                PPDMMEDIAREGIONDATAFORM penmDataForm)
+{
+    LogFlowFunc(("\n"));
+    int rc = VINF_SUCCESS;
+    PDRVHOSTBASE pThis = RT_FROM_MEMBER(pInterface, DRVHOSTBASE, IMedia);
+    uint64_t cbMedia;
+    uint64_t cbBlock = 0;
+
+    if (pThis->enmType == PDMMEDIATYPE_DVD)
+        cbBlock = 2048;
+    else
+        cbBlock = 512; /* Floppy. */
+
+    rc = drvHostBaseGetMediaSizeOs(pThis, &cbMedia);
+    if (   RT_SUCCESS(rc)
+        && u64LbaStart < cbMedia / cbBlock)
+    {
+        rc = VERR_NOT_FOUND;
+
+        if (pcBlocks)
+            *pcBlocks = cbMedia / cbBlock;
+        if (pcbBlock)
+            *pcbBlock = cbBlock;
+        if (penmDataForm)
+            *penmDataForm = PDMMEDIAREGIONDATAFORM_RAW;
+    }
+    else
+        rc = VERR_NOT_FOUND;
+
+    LogFlowFunc(("returns %Rrc\n", rc));
+    return rc;
 }
 
@@ -1185 +1276 @@
 
     /* IMedia. */
-    pThis->IMedia.pfnRead                   = drvHostBaseRead;
-    pThis->IMedia.pfnWrite                  = drvHostBaseWrite;
-    pThis->IMedia.pfnFlush                  = drvHostBaseFlush;
-    pThis->IMedia.pfnIsReadOnly             = drvHostBaseIsReadOnly;
-    pThis->IMedia.pfnIsNonRotational        = drvHostBaseIsNonRotational;
-    pThis->IMedia.pfnGetSize                = drvHostBaseGetSize;
-    pThis->IMedia.pfnGetType                = drvHostBaseGetType;
-    pThis->IMedia.pfnGetUuid                = drvHostBaseGetUuid;
-    pThis->IMedia.pfnBiosGetPCHSGeometry    = drvHostBaseGetPCHSGeometry;
-    pThis->IMedia.pfnBiosSetPCHSGeometry    = drvHostBaseSetPCHSGeometry;
-    pThis->IMedia.pfnBiosGetLCHSGeometry    = drvHostBaseGetLCHSGeometry;
-    pThis->IMedia.pfnBiosSetLCHSGeometry    = drvHostBaseSetLCHSGeometry;
-    pThis->IMedia.pfnBiosIsVisible          = drvHostBaseIsVisible;
+    pThis->IMedia.pfnRead                        = drvHostBaseRead;
+    pThis->IMedia.pfnWrite                       = drvHostBaseWrite;
+    pThis->IMedia.pfnFlush                       = drvHostBaseFlush;
+    pThis->IMedia.pfnIsReadOnly                  = drvHostBaseIsReadOnly;
+    pThis->IMedia.pfnIsNonRotational             = drvHostBaseIsNonRotational;
+    pThis->IMedia.pfnGetSize                     = drvHostBaseGetSize;
+    pThis->IMedia.pfnGetType                     = drvHostBaseGetType;
+    pThis->IMedia.pfnGetUuid                     = drvHostBaseGetUuid;
+    pThis->IMedia.pfnBiosGetPCHSGeometry         = drvHostBaseGetPCHSGeometry;
+    pThis->IMedia.pfnBiosSetPCHSGeometry         = drvHostBaseSetPCHSGeometry;
+    pThis->IMedia.pfnBiosGetLCHSGeometry         = drvHostBaseGetLCHSGeometry;
+    pThis->IMedia.pfnBiosSetLCHSGeometry         = drvHostBaseSetLCHSGeometry;
+    pThis->IMedia.pfnBiosIsVisible               = drvHostBaseIsVisible;
+    pThis->IMedia.pfnGetRegionCount              = drvHostBaseGetRegionCount;
+    pThis->IMedia.pfnQueryRegionProperties       = drvHostBaseQueryRegionProperties;
+    pThis->IMedia.pfnQueryRegionPropertiesForLba = drvHostBaseQueryRegionPropertiesForLba;
 
     /* IMediaEx */