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

Timestamp:

Dec 4, 2023 3:53:11 PM (14 months ago)

Author:

vboxsync

Message:

Main: Removed some more BUGBUG entries and documented empty / non-implemented code areas where needed. bugref:10384

Location:

trunk/src/VBox/Main/src-server

Files:

• FirmwareSettingsImpl.cpp (modified) (1 diff)
• MachineImpl.cpp (modified) (3 diffs)
• PlatformARMImpl.cpp (modified) (3 diffs)
• PlatformImpl.cpp (modified) (2 diffs)

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

@@ -685 +685 @@
     else
     {
-        m->bd->firmwareType   = FirmwareType_BIOS; /** @todo BUGBUG Handle ARM? */
+        m->bd->firmwareType   = FirmwareType_BIOS;
         m->bd->fIOAPICEnabled = true;
     }

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

@@ -236 +236 @@
 Machine::HWData::HWData()
 {
-    /* default values for a newly created machine */
+    /* default values for a newly created machine for x86. */
     mHWVersion.printf("%d", SchemaDefs::DefaultHardwareVersion);
     mMemorySize = 128;
@@ -259 +259 @@
     mDnDMode = DnDMode_Disabled;
 
-    mKeyboardHIDType = KeyboardHIDType_PS2Keyboard; /** @todo BUGBUG Assumes x86! */
-    mPointingHIDType = PointingHIDType_PS2Mouse; /** @todo BUGBUG Assumes x86! */
+    mKeyboardHIDType = KeyboardHIDType_PS2Keyboard;
+    mPointingHIDType = PointingHIDType_PS2Mouse;
     mParavirtProvider = ParavirtProvider_Default;
     mEmulatedUSBCardReaderEnabled = FALSE;
@@ -6311 +6311 @@
         if (FAILED(hrc)) return hrc;
 
-        ChipsetType_T aChipset = ChipsetType_PIIX3; /** @todo BUGBUG ASSUMES x86! */
+        ChipsetType_T aChipset = ChipsetType_PIIX3;
         hrc = mPlatform->COMGETTER(ChipsetType)(&aChipset);
         if (FAILED(hrc)) return hrc;

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

@@ -241 +241 @@
     RT_NOREF(data);
 
-    /** @todo BUGBUG Implement this form ARM! */
+    /* Nothing here yet. */
     return S_OK;
 }
@@ -257 +257 @@
     RT_NOREF(data);
 
-    /** @todo BUGBUG Implement this for ARM! */
+    /* Nothing here yet. */
     return S_OK;
 }
@@ -265 +265 @@
     RT_NOREF(aOsType);
 
-    /** @todo BUGBUG Implement this for ARM! */
-    return S_OK;
-}
+    /* Nothing here yet. */
+    return S_OK;
+}

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

@@ -243 +243 @@
     AutoWriteLock alock(this COMMA_LOCKVAL_SRC_POS);
 
-    /** @todo BUGBUG Implement this! */
     RT_NOREF(aArchitecture);
 
+    /* Currently we don't allow changing the platform architecture after the object was created.
+     * Mostly makes no sense, as this would render the VMs non-bootable and confuses users. */
     return E_NOTIMPL;
 }
@@ -510 +511 @@
     /* Allocates architecture-dependent stuff. */
     HRESULT hrc = i_initArchitecture(m->bd->architectureType);
-    AssertComRCReturnRC(hrc);
-
-    switch (m->bd->architectureType)
-    {
-        case PlatformArchitecture_x86:
-            return mX86->i_loadSettings(data.x86);
-
-#ifdef VBOX_WITH_VIRT_ARMV8
-        case PlatformArchitecture_ARM:
-            return mARM->i_loadSettings(data.arm);
-#endif
-        case PlatformArchitecture_None:
-            RT_FALL_THROUGH();
-        default:
-            break;
+    if (SUCCEEDED(hrc))
+    {
+        switch (m->bd->architectureType)
+        {
+            case PlatformArchitecture_x86:
+                return mX86->i_loadSettings(data.x86);
+
+#ifdef VBOX_WITH_VIRT_ARMV8
+            case PlatformArchitecture_ARM:
+                return mARM->i_loadSettings(data.arm);
+#endif
+            case PlatformArchitecture_None:
+                RT_FALL_THROUGH();
+            default:
+                break;
+        }
     }