Changeset 8113 in vbox

Timestamp:

Apr 17, 2008 4:31:58 PM (17 years ago)

Author:

vboxsync

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

29815

Message:

The recompiler must refresh its cpuid cache when we change a cpuid feature.

Location:

trunk

Files:

• include/VBox/cpum.h (modified) (1 diff)
• src/VBox/VMM/VMMAll/CPUMAllRegs.cpp (modified) (2 diffs)
• src/recompiler/VBoxRecompiler.c (modified) (2 diffs)

trunk/include/VBox/cpum.h

@@ -722 +722 @@
 #define CPUM_CHANGED_SYSENTER_MSR       RT_BIT(9)
 #define CPUM_CHANGED_HIDDEN_SEL_REGS    RT_BIT(10)
+#define CPUM_CHANGED_CPUID              RT_BIT(11)
 /** @} */
 

trunk/src/VBox/VMM/VMMAll/CPUMAllRegs.cpp

@@ -1079 +1079 @@
             break;
     }
+    pVM->cpum.s.fChanged |= CPUM_CHANGED_CPUID;
 }
 
@@ -1142 +1143 @@
             break;
     }
+    pVM->cpum.s.fChanged |= CPUM_CHANGED_CPUID;
 }
 

trunk/src/recompiler/VBoxRecompiler.c

@@ -1657 +1657 @@
     if (fFlags & (  CPUM_CHANGED_CR4  | CPUM_CHANGED_CR3  | CPUM_CHANGED_CR0
                   | CPUM_CHANGED_GDTR | CPUM_CHANGED_IDTR | CPUM_CHANGED_LDTR | CPUM_CHANGED_TR
-                  | CPUM_CHANGED_FPU_REM | CPUM_CHANGED_SYSENTER_MSR))
+                  | CPUM_CHANGED_FPU_REM | CPUM_CHANGED_SYSENTER_MSR | CPUM_CHANGED_CPUID))
     {
         if (fFlags & CPUM_CHANGED_FPU_REM)
@@ -1740 +1740 @@
             /** @note do_interrupt will fault if the busy flag is still set.... */
             pVM->rem.s.Env.tr.flags &= ~DESC_TSS_BUSY_MASK;
+        }
+
+        if (fFlags & CPUM_CHANGED_CPUID)
+        {
+            uint32_t u32Dummy;
+
+            /*
+            * Get the CPUID features.
+            */
+            CPUMGetGuestCpuId(pVM,          1, &u32Dummy, &u32Dummy, &pVM->rem.s.Env.cpuid_ext_features, &pVM->rem.s.Env.cpuid_features);
+            CPUMGetGuestCpuId(pVM, 0x80000001, &u32Dummy, &u32Dummy, &u32Dummy, &pVM->rem.s.Env.cpuid_ext2_features);
         }
     }