Changeset 49091 in vbox for trunk/src

Timestamp:

Oct 14, 2013 9:15:46 PM (11 years ago)

Author:

vboxsync

Message:

UIWizardNewVM.cpp: Always SetPortCount on AHCI when used, whether it has one or two devices attached. Old code only did it if it was only used for HD, which meant OS X guests got the max port count since they now have both HD and DVD on AHCI.

File:

• trunk/src/VBox/Frontends/VirtualBox/src/wizards/newvm/UIWizardNewVM.cpp (modified) (2 diffs)

trunk/src/VBox/Frontends/VirtualBox/src/wizards/newvm/UIWizardNewVM.cpp

@@ -142 +142 @@
         hdCtr = m_machine.GetStorageControllerByName(strHDName);
         hdCtr.SetControllerType(hdStorageControllerType);
-
-        /* Set the port count to 1 if SATA is used. */
-        if (hdStorageControllerType == KStorageControllerType_IntelAhci)
-            hdCtr.SetPortCount(1);
     }
     else
@@ -153 +149 @@
         strHDName = strDVDName;
     }
+
+    /* Liomit the AHCI port count if it's used because windows has trouble with
+       too many ports and other guest (OS X in particular) may take extra long
+       to boot: */
+    if (hdStorageControllerType == KStorageControllerType_IntelAhci)
+        hdCtr.SetPortCount(1 + (dvdStorageControllerType == KStorageControllerType_IntelAhci));
+    else if (dvdStorageControllerType == KStorageControllerType_IntelAhci)
+        dvdCtr.SetPortCount(1);
 
     /* Turn on PAE, if recommended: */