Changeset 42482 in vbox

Timestamp:

Jul 31, 2012 1:42:04 PM (13 years ago)

Author:

vboxsync

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

79634

Message:

helper_rdmsr: Don't try to check if the value is the same for
MSR_IA32_TSC by re-reading it, as it *will* be different. Fixes bogus
and annoying assertion for debug builds.

File:

• trunk/src/recompiler/target-i386/op_helper.c (modified) (1 diff)

trunk/src/recompiler/target-i386/op_helper.c

@@ -3933 +3933 @@
 
 # ifdef VBOX_STRICT
-    if (cpu_rdmsr(env, (uint32_t)ECX, &val) != 0)
-        val = 0;
-    AssertMsg(val == RT_MAKE_U64(EAX, EDX), ("idMsr=%#x val=%#llx eax:edx=%#llx\n", (uint32_t)ECX, val, RT_MAKE_U64(EAX, EDX)));
+    if ((uint32_t)ECX != MSR_IA32_TSC) {
+        if (cpu_rdmsr(env, (uint32_t)ECX, &val) != 0)
+            val = 0;
+        AssertMsg(val == RT_MAKE_U64(EAX, EDX), ("idMsr=%#x val=%#llx eax:edx=%#llx\n", (uint32_t)ECX, val, RT_MAKE_U64(EAX, EDX)));
+    }
 # endif
 }