Changeset 57064 in vbox for trunk/src/VBox

Timestamp:

Jul 23, 2015 4:22:04 PM (9 years ago)

Author:

vboxsync

Message:

Frontends/VBoxManage+Storage/VMDK: introduce support for creating raw partition VMDK images with readonly partitions (indicated by a 'r' suffix after the partition number)

Location:

trunk/src/VBox

Files:

• Frontends/VBoxManage/VBoxInternalManage.cpp (modified) (5 diffs)
• Storage/VMDK.cpp (modified) (7 diffs)

trunk/src/VBox/Frontends/VBoxManage/VBoxInternalManage.cpp

@@ -1502 +1502 @@
     if (!pszPartitions)
     {
-        RawDescriptor.fRawDisk = true;
+        RawDescriptor.uFlags = VBOXHDDRAW_DISK;
         RawDescriptor.pszRawDisk = rawdisk.c_str();
     }
     else
     {
-        RawDescriptor.fRawDisk = false;
+        RawDescriptor.uFlags = VBOXHDDRAW_NORMAL;
         RawDescriptor.pszRawDisk = NULL;
         RawDescriptor.cPartDescs = 0;
@@ -1513 +1513 @@
 
         uint32_t uPartitions = 0;
+        uint32_t uPartitionsRO = 0;
 
         const char *p = pszPartitions;
@@ -1527 +1528 @@
             uPartitions |= RT_BIT(u32);
             p = pszNext;
+            if (*p == 'r')
+            {
+                uPartitionsRO |= RT_BIT(u32);
+                p++;
+            }
             if (*p == ',')
                 p++;
@@ -1647 +1653 @@
             if (uPartitions & RT_BIT(partitions.aPartitions[i].uIndex))
             {
+                if (uPartitionsRO & RT_BIT(partitions.aPartitions[i].uIndex))
+                    pPartDesc->uFlags |= VBOXHDDRAW_READONLY;
+
                 if (fRelative)
                 {
@@ -1759 +1768 @@
 
 #ifdef DEBUG_klaus
-    if (!RawDescriptor.fRawDisk)
+    if (!(RawDescriptor.uFlags & VBOXHDDRAW_DISK))
     {
         RTPrintf("#            start         length    startoffset  partdataptr  device\n");

trunk/src/VBox/Storage/VMDK.cpp

@@ -3246 +3246 @@
                 case VMDKETYPE_HOSTED_SPARSE:
                     rc = vmdkFileOpen(pImage, &pExtent->pFile, pExtent->pszFullname,
-                                      VDOpenFlagsToFileOpenFlags(uOpenFlags,
+                                      VDOpenFlagsToFileOpenFlags(uOpenFlags | (pExtent->enmAccess == VMDKACCESS_READONLY) ? VD_OPEN_FLAGS_READONLY : 0,
                                                                  false /* fCreate */));
                     if (RT_FAILURE(rc))
@@ -3273 +3273 @@
                 case VMDKETYPE_FLAT:
                     rc = vmdkFileOpen(pImage, &pExtent->pFile, pExtent->pszFullname,
-                                      VDOpenFlagsToFileOpenFlags(uOpenFlags,
+                                      VDOpenFlagsToFileOpenFlags(uOpenFlags | (pExtent->enmAccess == VMDKACCESS_READONLY) ? VD_OPEN_FLAGS_READONLY : 0,
                                                                  false /* fCreate */));
                     if (RT_FAILURE(rc))
@@ -3365 +3365 @@
     PVMDKEXTENT pExtent;
 
-    if (pRaw->fRawDisk)
+    if (pRaw->uFlags & VBOXHDDRAW_DISK)
     {
         /* Full raw disk access. This requires setting up a descriptor
@@ -3394 +3394 @@
         pExtent->cNominalSectors = VMDK_BYTE2SECTOR(cbSize);
         pExtent->uSectorOffset = 0;
-        pExtent->enmAccess = VMDKACCESS_READWRITE;
+        pExtent->enmAccess = (pRaw->uFlags & VBOXHDDRAW_READONLY) ? VMDKACCESS_READONLY : VMDKACCESS_READWRITE;
         pExtent->fMetaDirty = false;
 
         /* Open flat image, the raw disk. */
         rc = vmdkFileOpen(pImage, &pExtent->pFile, pExtent->pszFullname,
-                          VDOpenFlagsToFileOpenFlags(pImage->uOpenFlags & ~VD_OPEN_FLAGS_READONLY,
+                          VDOpenFlagsToFileOpenFlags(pImage->uOpenFlags | (pExtent->enmAccess == VMDKACCESS_READONLY) ? VD_OPEN_FLAGS_READONLY : 0,
                                                      false /* fCreate */));
         if (RT_FAILURE(rc))
@@ -3507 +3507 @@
                 /* Create partition table flat image. */
                 rc = vmdkFileOpen(pImage, &pExtent->pFile, pExtent->pszFullname,
-                                  VDOpenFlagsToFileOpenFlags(pImage->uOpenFlags,
+                                  VDOpenFlagsToFileOpenFlags(pImage->uOpenFlags | (pExtent->enmAccess == VMDKACCESS_READONLY) ? VD_OPEN_FLAGS_READONLY : 0,
                                                              true /* fCreate */));
                 if (RT_FAILURE(rc))
@@ -3537 +3537 @@
                     pExtent->cNominalSectors = VMDK_BYTE2SECTOR(pPart->cbData);
                     pExtent->uSectorOffset = VMDK_BYTE2SECTOR(pPart->uStartOffset);
-                    pExtent->enmAccess = VMDKACCESS_READWRITE;
+                    pExtent->enmAccess = (pPart->uFlags & VBOXHDDRAW_READONLY) ? VMDKACCESS_READONLY : VMDKACCESS_READWRITE;
                     pExtent->fMetaDirty = false;
 
                     /* Open flat image, the raw partition. */
                     rc = vmdkFileOpen(pImage, &pExtent->pFile, pExtent->pszFullname,
-                                      VDOpenFlagsToFileOpenFlags(pImage->uOpenFlags & ~VD_OPEN_FLAGS_READONLY,
+                                      VDOpenFlagsToFileOpenFlags(pImage->uOpenFlags | (pExtent->enmAccess == VMDKACCESS_READONLY) ? VD_OPEN_FLAGS_READONLY : 0,
                                                                  false /* fCreate */));
                     if (RT_FAILURE(rc))
@@ -3574 +3574 @@
 
     rc = vmdkDescBaseSetStr(pImage, &pImage->Descriptor, "createType",
-                            pRaw->fRawDisk ?
+                            (pRaw->uFlags & VBOXHDDRAW_DISK) ?
                             "fullDevice" : "partitionedDevice");
     if (RT_FAILURE(rc))