Changeset 14771 in vbox for trunk/src/VBox/VMM/VMMSwitcher.cpp

Timestamp:

Nov 28, 2008 11:07:18 AM (16 years ago)

Author:

vboxsync

Message:

Fixed 64 bits switcher loading

File:

• trunk/src/VBox/VMM/VMMSwitcher.cpp (modified) (6 diffs)

trunk/src/VBox/VMM/VMMSwitcher.cpp

@@ -55 +55 @@
     &vmmR3Switcher32BitTo32Bit_Def,
     &vmmR3Switcher32BitToPAE_Def,
-    NULL, //&vmmR3Switcher32BitToAMD64_Def - disabled because it causes assertions.
+    &vmmR3Switcher32BitToAMD64_Def,
     &vmmR3SwitcherPAETo32Bit_Def,
     &vmmR3SwitcherPAEToPAE_Def,
-    NULL, //&vmmR3SwitcherPAEToAMD64_Def - disabled because it causes assertions.
+    &vmmR3SwitcherPAEToAMD64_Def,
     NULL,   //&vmmR3SwitcherPAETo32Bit_Def,
 # ifdef VBOX_WITH_HYBIRD_32BIT_KERNEL
@@ -348 +348 @@
             case FIX_GC_2_ID_NEAR_REL:
             {
-                Assert(offSrc - pSwitcher->offGCCode < pSwitcher->cbGCCode);
+                AssertMsg(offSrc - pSwitcher->offGCCode < pSwitcher->cbGCCode, ("%x - %x < %x\n", offSrc, pSwitcher->offGCCode, pSwitcher->cbGCCode));
                 uint32_t offTrg = *u.pu32++;
                 Assert(offTrg - pSwitcher->offIDCode0 < pSwitcher->cbIDCode0 || offTrg - pSwitcher->offIDCode1 < pSwitcher->cbIDCode1);
@@ -610 +610 @@
 #if defined(RT_ARCH_AMD64) || defined(VBOX_WITH_HYBIRD_32BIT_KERNEL)
             /*
-             * 64-bit HC pointer fixup to (HC) target within the code (32-bit offset).
-             */
-            case FIX_HC_64BIT:
-            {
-                uint32_t offTrg = *u.pu32++;
-                Assert(offSrc < pSwitcher->cbCode);
-                Assert(offTrg - pSwitcher->offHCCode0 < pSwitcher->cbHCCode0 || offTrg - pSwitcher->offHCCode1 < pSwitcher->cbHCCode1);
-                *uSrc.pu64 = R0PtrCode + offTrg;
-                break;
-            }
-
-            /*
              * 64-bit HC Code Selector (no argument).
              */
@@ -636 +624 @@
 #endif
             /*
+             * 64-bit HC pointer fixup to (HC) target within the code (32-bit offset).
+             */
+            case FIX_HC_64BIT:
+            {
+                uint32_t offTrg = *u.pu32++;
+                Assert(offSrc < pSwitcher->cbCode);
+                Assert(offTrg - pSwitcher->offHCCode0 < pSwitcher->cbHCCode0 || offTrg - pSwitcher->offHCCode1 < pSwitcher->cbHCCode1);
+                *uSrc.pu64 = R0PtrCode + offTrg;
+                break;
+            }
+
+            /*
              * 64-bit HC pointer to the CPUM instance data (no argument).
              */
@@ -661 +661 @@
              */
             case FIX_ID_64BIT:
+            case FIX_HC_64BIT_NOCHECK:
             {
                 uint32_t offTrg = *u.pu32++;
                 Assert(offSrc < pSwitcher->cbCode);
-                Assert(offTrg - pSwitcher->offIDCode0 < pSwitcher->cbIDCode0 || offTrg - pSwitcher->offIDCode1 < pSwitcher->cbIDCode1);
+                Assert(u8 == FIX_HC_64BIT_NOCHECK || offTrg - pSwitcher->offIDCode0 < pSwitcher->cbIDCode0 || offTrg - pSwitcher->offIDCode1 < pSwitcher->cbIDCode1);
                 *uSrc.pu64 = u32IDCode + offTrg;
                 break;
@@ -883 +884 @@
 }
 
+/**
+ * Relocator for the PAE to AMD64 world switcher.
+ */
+DECLCALLBACK(void) vmmR3SwitcherPAEToAMD64_Relocate(PVM pVM, PVMMSWITCHERDEF pSwitcher, RTR0PTR R0PtrCode, uint8_t *pu8CodeR3, RTGCPTR GCPtrCode, uint32_t u32IDCode)
+{
+    vmmR3SwitcherGenericRelocate(pVM, pSwitcher, R0PtrCode, pu8CodeR3, GCPtrCode, u32IDCode,
+                                 SELMGetHyperCS(pVM), SELMGetHyperDS(pVM), SELMGetHyperTSS(pVM), SELMGetHyperGDT(pVM), SELMGetHyperCS64(pVM));
+}
+
 
 /**