Changeset 48024 in vbox for trunk/src/VBox/VMM/VMMAll

Timestamp:

Aug 23, 2013 12:24:00 PM (11 years ago)

Author:

vboxsync

Message:

Since AMD has MSR_IA32_MCP_CAP, we have to fake the other machine check MSRs on AMD.

File:

• trunk/src/VBox/VMM/VMMAll/CPUMAllRegs.cpp (modified) (2 diffs)

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

@@ -1092 +1092 @@
 #endif
         /*
-         * The BIOS_SIGN_ID MSR and MSR_IA32_MCP_CAP exist on AMD64 as well, at 
-         * least bulldozer have it. Windows 7 is querying them.  Just fake them.
+         * The BIOS_SIGN_ID MSR and MSR_IA32_MCP_CAP et al exist on AMD64 as
+         * well, at least bulldozer have them.  Windows 7 is querying them.
+         * XP has been observed querying MSR_IA32_MC0_CTL.
          */
         case MSR_IA32_BIOS_SIGN_ID:         /* fam/mod >= 6_01 */
         case MSR_IA32_MCP_CAP:              /* fam/mod >= 6_01 */
+        /*case MSR_IA32_MCP_STATUS:     - indicated as not present in CAP */
+        /*case MSR_IA32_MCP_CTRL:       - indicated as not present in CAP */
+        case MSR_IA32_MC0_CTL:
+        case MSR_IA32_MC0_STATUS:
             *puValue = 0;
             break;
@@ -1106 +1111 @@
         case MSR_IA32_PLATFORM_ID:          /* fam/mod >= 6_01 */
         /*case MSR_IA32_BIOS_UPDT_TRIG: - write-only? */
-        /*case MSR_IA32_MCP_STATUS:     - indicated as not present in CAP */
-        /*case MSR_IA32_MCP_CTRL:       - indicated as not present in CAP */
-        case MSR_IA32_MC0_CTL:
-        case MSR_IA32_MC0_STATUS:
         case MSR_RAPL_POWER_UNIT:
             *puValue = 0;