Changeset 66463 in vbox for trunk/src/VBox/ValidationKit/bootsectors

Timestamp:

Apr 6, 2017 5:58:25 PM (8 years ago)

Author:

vboxsync

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

114448

Message:

IEM,bs3-cpu-generated-1: Made the current testcases pass on AMD.

File:

• trunk/src/VBox/ValidationKit/bootsectors/bs3-cpu-generated-1-template.c (modified) (9 diffs)

trunk/src/VBox/ValidationKit/bootsectors/bs3-cpu-generated-1-template.c

@@ -121 +121 @@
     /** The encoding. */
     BS3CG1ENC               enmEncoding;
+    /** The non-invalid encoding.  This differs from enmEncoding when
+     * Bs3Cg1CalcNoneIntelInvalidEncoding has been called. */
+    BS3CG1ENC               enmEncodingNonInvalid;
     /** The CPU test / CPU ID. */
     BS3CG1CPU               enmCpuTest;
@@ -2219 +2222 @@
     {
         case BS3CG1ENC_MODRM_Gb_Eb:
+        case BS3CG1ENC_MODRM_Gv_Ma:
         case BS3CG1ENC_FIXED:
             return BS3CG1ENC_FIXED;
@@ -2703 +2707 @@
 
                     default:
+                        if (pThis->enmEncoding != pThis->enmEncodingNonInvalid)
+                            goto l_advance_to_next;
                         return Bs3TestFailedF("Internal error: cbDst=%u idxField=%d (%d) offField=%#x: enmLocation=%u off=%#x idxField=%u",
                                               cbDst, idxField, idxOp, offField, pThis->aOperands[idxOp].enmLocation,
@@ -2917 +2923 @@
          * Advance to the next instruction.
          */
+l_advance_to_next:
         pbCode += cbValue;
         cbLeft -= cbValue;
@@ -3497 +3504 @@
          */
         PCBS3CG1INSTR pInstr = &g_aBs3Cg1Instructions[iInstr];
-        pThis->iInstr          = iInstr;
-        pThis->pTestHdr        = (PCBS3CG1TESTHDR)&g_abBs3Cg1Tests[pInstr->offTests];
-        pThis->fFlags          = pInstr->fFlags;
-        pThis->enmEncoding     = (BS3CG1ENC)pInstr->enmEncoding;
-        pThis->enmCpuTest      = (BS3CG1CPU)pInstr->enmCpuTest;
-        pThis->enmPrefixKind   = (BS3CG1PFXKIND)pInstr->enmPrefixKind;
-        pThis->enmXcptType     = (BS3CG1XCPTTYPE)pInstr->enmXcptType;
-        pThis->cchMnemonic     = pInstr->cchMnemonic;
+        pThis->iInstr                   = iInstr;
+        pThis->pTestHdr                 = (PCBS3CG1TESTHDR)&g_abBs3Cg1Tests[pInstr->offTests];
+        pThis->fFlags                   = pInstr->fFlags;
+        pThis->enmEncoding              = (BS3CG1ENC)pInstr->enmEncoding;
+        pThis->enmEncodingNonInvalid    = (BS3CG1ENC)pInstr->enmEncoding;
+        pThis->enmCpuTest               = (BS3CG1CPU)pInstr->enmCpuTest;
+        pThis->enmPrefixKind            = (BS3CG1PFXKIND)pInstr->enmPrefixKind;
+        pThis->enmXcptType              = (BS3CG1XCPTTYPE)pInstr->enmXcptType;
+        pThis->cchMnemonic              = pInstr->cchMnemonic;
         if (pThis->fAdvanceMnemonic)
             Bs3TestSubF("%s / %.*s", pThis->pszModeShort, pThis->cchMnemonic, pThis->pchMnemonic);
-        pThis->fAdvanceMnemonic = pInstr->fAdvanceMnemonic;
-        pThis->cOperands       = pInstr->cOperands;
-        pThis->cbOpcodes       = pInstr->cbOpcodes;
+        pThis->fAdvanceMnemonic         = pInstr->fAdvanceMnemonic;
+        pThis->cOperands                = pInstr->cOperands;
+        pThis->cbOpcodes                = pInstr->cbOpcodes;
         switch (pThis->cOperands)
         {
@@ -3517 +3525 @@
             case 1: pThis->aenmOperands[0] = (BS3CG1OP)pThis->pabOperands[0];
         }
-
         switch (pThis->cbOpcodes)
         {
@@ -3526 +3533 @@
         }
 
-        /* Switch the encoder for some of the invalid instructions on non-intel CPUs. */
-        if (   (pThis->fFlags & BS3CG1INSTR_F_INTEL_DECODES_INVALID)
-            && pThis->bCpuVendor != BS3CPUVENDOR_INTEL
-            && (   (pThis->fFlags & (BS3CG1INSTR_F_UNUSED | BS3CG1INSTR_F_INVALID))
-                || (BS3_MODE_IS_64BIT_CODE(pThis->bMode) && (pThis->fFlags & BS3CG1INSTR_F_INVALID_64BIT)) ) )
-            pThis->enmEncoding = Bs3Cg1CalcNoneIntelInvalidEncoding(pThis->enmEncoding);
-
         /*
          * Check if the CPU supports the instruction.
@@ -3539 +3539 @@
             || (pThis->fFlags & (BS3CG1INSTR_F_UNUSED | BS3CG1INSTR_F_INVALID)))
             fOuterInvalidInstr = true;
+
+        /* Switch the encoder for some of the invalid instructions on non-intel CPUs. */
+        if (   (pThis->fFlags & BS3CG1INSTR_F_INTEL_DECODES_INVALID)
+            && pThis->bCpuVendor != BS3CPUVENDOR_INTEL
+            && (   (pThis->fFlags & (BS3CG1INSTR_F_UNUSED | BS3CG1INSTR_F_INVALID))
+                || (BS3_MODE_IS_64BIT_CODE(pThis->bMode) && (pThis->fFlags & BS3CG1INSTR_F_INVALID_64BIT))
+                || fOuterInvalidInstr ) )
+            pThis->enmEncoding = Bs3Cg1CalcNoneIntelInvalidEncoding(pThis->enmEncoding);
 
         for (iCpuSetup = 0;; iCpuSetup++)
@@ -3705 +3713 @@
 #if 0
     /* (for debugging) */
-    if (bMode != BS3_MODE_PPV86)
+    if (bMode < BS3_MODE_LM16)
         return BS3TESTDOMODE_SKIPPED;
 #endif