Changeset 47996 in vbox for trunk/src/VBox

Timestamp:

Aug 22, 2013 3:09:32 PM (11 years ago)

Author:

vboxsync

Message:

More MSRs fixes on AMD64. MSR_K8_NB_CFG is for recent linux kernels (64-bit), and the MSR_IA32_BIOS_SIGN_ID is for windows 7.

File:

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

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

@@ -1091 +1091 @@
             /* no break */
 #endif
+        /*
+         * The BIOS_SIGN_ID MSR exists on AMD64 as well, at least bulldozer
+         * have it. Windows 7 is querying it. Just fake it.
+         */
+        case MSR_IA32_BIOS_SIGN_ID:         /* fam/mod >= 6_01 */
+            *puValue = 0;
+            break;
+
 
         /*
@@ -1096 +1104 @@
          */
         case MSR_IA32_PLATFORM_ID:          /* fam/mod >= 6_01 */
-        case MSR_IA32_BIOS_SIGN_ID:         /* fam/mod >= 6_01 */
         /*case MSR_IA32_BIOS_UPDT_TRIG: - write-only? */
         case MSR_IA32_MCP_CAP:              /* fam/mod >= 6_01 */
@@ -1130 +1137 @@
         case MSR_K8_SYSCFG:
         case MSR_K8_INT_PENDING:
+        case MSR_K8_NB_CFG:             /* (All known values are 0 on reset.) */
             *puValue = 0;
             if (CPUMGetGuestCpuVendor(pVCpu->CTX_SUFF(pVM)) != CPUMCPUVENDOR_AMD)
@@ -1387 +1395 @@
         case MSR_K8_SYSCFG:      /** @todo can be written, but we ignore that for now. */
         case MSR_K8_INT_PENDING: /** @todo can be written, but we ignore that for now. */
+        case MSR_K8_NB_CFG:      /** @todo can be written; the apicid swapping might be used and would need saving, but probably unnecessary. */
             if (CPUMGetGuestCpuVendor(pVCpu->CTX_SUFF(pVM)) != CPUMCPUVENDOR_AMD)
             {