Changeset 50291 in vbox for trunk/src/VBox/Main/src-server

Timestamp:

Jan 30, 2014 4:22:55 PM (11 years ago)

Author:

vboxsync

svn:sync-xref-src-repo-rev:

91946

Message:

Main/SystemProperties: Add GetStorageControllerHotplugCapable to check whether a given controller type is hotplug capable

Location:

trunk/src/VBox/Main/src-server

Files:

• MachineImpl.cpp (modified) (1 diff)
• SystemPropertiesImpl.cpp (modified) (1 diff)

trunk/src/VBox/Main/src-server/MachineImpl.cpp

@@ -12527 +12527 @@
 bool Machine::isControllerHotplugCapable(StorageControllerType_T enmCtrlType)
 {
-    switch (enmCtrlType)
-    {
-        case StorageControllerType_IntelAhci:
-        case StorageControllerType_USB:
-            return true;
-        case StorageControllerType_LsiLogic:
-        case StorageControllerType_LsiLogicSas:
-        case StorageControllerType_BusLogic:
-        case StorageControllerType_PIIX3:
-        case StorageControllerType_PIIX4:
-        case StorageControllerType_ICH6:
-        case StorageControllerType_I82078:
-        default:
-            return false;
-    }
+    ComPtr<ISystemProperties> systemProperties;
+    HRESULT rc = mParent->COMGETTER(SystemProperties)(systemProperties.asOutParam());
+    if (FAILED(rc))
+        return false;
+
+    BOOL aHotplugCapable = FALSE;
+    systemProperties->COMGETTER(StorageControllerHotplugCapable)(enmCtrlType, &aHotplugCapable);
+
+    return RT_BOOL(aHotplugCapable);
 }
 

trunk/src/VBox/Main/src-server/SystemPropertiesImpl.cpp

@@ -538 +538 @@
 }
 
+HRESULT SystemProperties::getStorageControllerHotplugCapable(StorageControllerType_T aControllerType,
+                                                             BOOL *aHotplugCapable)
+{
+    switch (aControllerType)
+    {
+        case StorageControllerType_IntelAhci:
+        case StorageControllerType_USB:
+            *aHotplugCapable = true;
+            break;
+        case StorageControllerType_LsiLogic:
+        case StorageControllerType_LsiLogicSas:
+        case StorageControllerType_BusLogic:
+        case StorageControllerType_PIIX3:
+        case StorageControllerType_PIIX4:
+        case StorageControllerType_ICH6:
+        case StorageControllerType_I82078:
+            *aHotplugCapable = false;
+            break;
+        default:
+            AssertMsgFailedReturn(("Invalid controller type %d\n", aControllerType), E_FAIL);
+    }
+
+    return S_OK;
+}
+
 HRESULT SystemProperties::getMaxInstancesOfUSBControllerType(ChipsetType_T aChipset,
                                                              USBControllerType_T aType,