Changeset 102113 in vbox

Timestamp:

Nov 15, 2023 4:23:52 PM (14 months ago)

Author:

vboxsync

Message:

Main: Added IPlatformProperties::getSupportedBootDevices(), so that API clients know which boot devices actually are supported for a particular platform. This also returns the supported boot devices in the default order (was hardcoded in FE/Qt). bugref:10384

Location:

trunk/src/VBox/Main

Files:

• idl/VirtualBox.xidl (modified) (2 diffs)
• include/PlatformPropertiesImpl.h (modified) (1 diff)
• src-all/PlatformPropertiesImpl.cpp (modified) (1 diff)

trunk/src/VBox/Main/idl/VirtualBox.xidl

@@ -12723 +12723 @@
     name="IPlatformProperties"
     extends="$unknown"
-    uuid="25cf8416-1e5b-4d01-ae9d-1dc34ef737c8"
+    uuid="45def725-ab32-4a0a-a85e-fae56249d547"
     wsmap="managed"
     rest="managed"
@@ -12841 +12841 @@
         Returns an array of officially supported values for enum <link to="AudioControllerType"/>,
         in the sense of what is e.g. worth offering in the VirtualBox GUI.
+      </desc>
+    </attribute>
+
+    <attribute name="supportedBootDevices" type="DeviceType" safearray="yes" readonly="yes">
+      <desc>
+        Returns an array of officially supported boot devices for enum <link to="DeviceType"/>,
+        in the sense of what is e.g. worth offering in the VirtualBox GUI.
+
+        The returned array also defines the default boot device order for all supported devices.
       </desc>
     </attribute>

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

@@ -86 +86 @@
     HRESULT getSupportedUSBControllerTypes(std::vector<USBControllerType_T> &aSupportedUSBControllerTypes) RT_OVERRIDE;
     HRESULT getSupportedAudioControllerTypes(std::vector<AudioControllerType_T> &aSupportedAudioControllerTypes) RT_OVERRIDE;
+    HRESULT getSupportedBootDevices(std::vector<DeviceType_T> &aSupportedBootDevices);
     HRESULT getSupportedStorageBuses(std::vector<StorageBus_T> &aSupportedStorageBuses) RT_OVERRIDE;
     HRESULT getSupportedStorageControllerTypes(std::vector<StorageControllerType_T> &aSupportedStorageControllerTypes) RT_OVERRIDE;

trunk/src/VBox/Main/src-all/PlatformPropertiesImpl.cpp

@@ -859 +859 @@
 }
 
+HRESULT PlatformProperties::getSupportedBootDevices(std::vector<DeviceType_T> &aSupportedBootDevices)
+{
+    /* Note: This function returns the supported boot devices for the given architecture,
+             in the default order, to keep it simple for the caller. */
+
+    switch (mPlatformArchitecture)
+    {
+        case PlatformArchitecture_x86:
+        {
+            static const DeviceType_T aBootDevices[] =
+            {
+                DeviceType_Floppy,
+                DeviceType_DVD,
+                DeviceType_HardDisk,
+                DeviceType_Network
+            };
+            aSupportedBootDevices.assign(aBootDevices,
+                                         aBootDevices + RT_ELEMENTS(aBootDevices));
+            break;
+        }
+
+        case PlatformArchitecture_ARM:
+        {
+            static const DeviceType_T aBootDevices[] =
+            {
+                DeviceType_DVD,
+                DeviceType_HardDisk
+                /** @todo BUGBUG We need to test network booting via PXE on ARM first! */
+            };
+            aSupportedBootDevices.assign(aBootDevices,
+                                         aBootDevices + RT_ELEMENTS(aBootDevices));
+            break;
+        }
+
+        default:
+            AssertFailedStmt(aSupportedBootDevices.clear());
+            break;
+    }
+
+    return S_OK;
+}
+
 HRESULT PlatformProperties::getSupportedStorageBuses(std::vector<StorageBus_T> &aSupportedStorageBuses)
 {