Changeset 76402 in vbox for trunk/src/VBox/VMM/VMMR3

Timestamp:

Dec 23, 2018 3:13:04 PM (6 years ago)

Author:

vboxsync

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

127701

Message:

VBox/vmm/apic.h: Try avoid dragging in pdmdev.h, so use function for APIC device registration rather than exposing g_DeviceAPIC. bugref:9344

Location:

trunk/src/VBox/VMM/VMMR3

Files:

• APIC.cpp (modified) (2 diffs)
• EM.cpp (modified) (1 diff)
• GIMHv.cpp (modified) (1 diff)
• PDMDevice.cpp (modified) (2 diffs)

trunk/src/VBox/VMM/VMMR3/APIC.cpp

@@ -1670 +1670 @@
  * APIC device registration structure.
  */
-const PDMDEVREG g_DeviceAPIC =
+static const PDMDEVREG g_DeviceAPIC =
 {
     /* u32Version */
@@ -1723 +1723 @@
 };
 
+
+/**
+ * Called by PDM to register the APIC device.
+ */
+VMMR3_INT_DECL(int) APICR3RegisterDevice(PPDMDEVREGCB pCallbacks)
+{
+    return pCallbacks->pfnRegister(pCallbacks, &g_DeviceAPIC);
+}
+
 #endif /* !VBOX_DEVICE_STRUCT_TESTCASE */
 

trunk/src/VBox/VMM/VMMR3/EM.cpp

@@ -67 +67 @@
 #include <VBox/dis.h>
 #include <VBox/disopcode.h>
+#include <VBox/err.h>
 #include "VMMTracing.h"
 

trunk/src/VBox/VMM/VMMR3/GIMHv.cpp

@@ -32 +32 @@
 #include <VBox/vmm/vm.h>
 
+#include <VBox/err.h>
 #include <VBox/version.h>
 
 #include <iprt/assert.h>
-#include <iprt/err.h>
 #include <iprt/string.h>
 #include <iprt/mem.h>

trunk/src/VBox/VMM/VMMR3/PDMDevice.cpp

@@ -495 +495 @@
      * Load the internal VMM APIC device.
      */
-    int rc2 = pdmR3DevReg_Register(&RegCB.Core, &g_DeviceAPIC);
-    AssertRCReturn(rc2, rc2);
+    int rc = APICR3RegisterDevice(&RegCB.Core);
+    AssertRCReturn(rc, rc);
 
     /*
@@ -503 +503 @@
     PCFGMNODE pDevicesNode = CFGMR3GetChild(CFGMR3GetRoot(pVM), "PDM/Devices");
     bool fLoadBuiltin;
-    int rc = CFGMR3QueryBool(pDevicesNode, "LoadBuiltin", &fLoadBuiltin);
+    rc = CFGMR3QueryBool(pDevicesNode, "LoadBuiltin", &fLoadBuiltin);
     if (rc == VERR_CFGM_VALUE_NOT_FOUND || rc == VERR_CFGM_NO_PARENT)
         fLoadBuiltin = true;