Changeset 100791 in vbox for trunk

Timestamp:

Aug 4, 2023 9:00:00 AM (17 months ago)

Author:

vboxsync

Message:

VMM/IEM: VINF_IEM_REEXEC_MODE_CHANGED -> VINF_IEM_REEXEC_BREAK - more generic, no need for specificity here. bugref:10369

Location:

trunk

Files:

• include/VBox/err.h (modified) (1 diff)
• src/VBox/VMM/VMMAll/IEMAllThrdFuncsBltIn.cpp (modified) (6 diffs)
• src/VBox/VMM/VMMAll/IEMAllThrdRecompiler.cpp (modified) (1 diff)

trunk/include/VBox/err.h

@@ -2501 +2501 @@
 /** Recompiler: Translation block allocation failed. */
 #define VERR_IEM_TB_ALLOC_FAILED                    (-5309)
-/** Recompiled execution: Stop execution TB - Mode (fExec) changed. */
-#define VINF_IEM_REEXEC_MODE_CHANGED                (5310)
+/** Recompiled execution: Break out of current TB execution. */
+#define VINF_IEM_REEXEC_BREAK                       (5310)
 /** Recompilation: End translation block. */
 #define VINF_IEM_RECOMPILE_END_TB                   (5311)

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

@@ -74 +74 @@
 {
     iemThreadedTbObsolete(pVCpu, pVCpu->iem.s.pCurTbR3);
-    return VINF_IEM_REEXEC_MODE_CHANGED; /** @todo different status code... */
+    return VINF_IEM_REEXEC_BREAK;
 }
 
@@ -120 +120 @@
          pVCpu->cpum.GstCtx.cs.Sel, pVCpu->cpum.GstCtx.eip, fCpu,
          pVCpu->CTX_SUFF(pVM)->fGlobalForcedActions & VM_FF_ALL_MASK, pVCpu->cpum.GstCtx.rflags.Bits.u1IF));
-    return VINF_IEM_REEXEC_MODE_CHANGED;
+    return VINF_IEM_REEXEC_BREAK;
 }
 
@@ -139 +139 @@
              fExpectedExec, pVCpu->iem.s.fExec, fExpectedExec ^ pVCpu->iem.s.fExec));
     RT_NOREF(uParam1, uParam2);
-    return VINF_IEM_REEXEC_MODE_CHANGED;
+    return VINF_IEM_REEXEC_BREAK;
 }
 
@@ -245 +245 @@
                       pVCpu->iem.s.GCPhysInstrBuf + off, GCPhysRangePageWithOffset, pVCpu->iem.s.pbInstrBuf, __LINE__)); \
                 RT_NOREF(a_cbInstr); \
-                return VINF_IEM_REEXEC_MODE_CHANGED; /** @todo new status code? */ \
+                return VINF_IEM_REEXEC_BREAK; \
             } \
             else \
@@ -278 +278 @@
                       pVCpu->iem.s.GCPhysInstrBuf + offNew, GCPhysRangePageWithOffset, pVCpu->iem.s.pbInstrBuf, __LINE__)); \
                 RT_NOREF(a_cbInstr); \
-                return VINF_IEM_REEXEC_MODE_CHANGED; /** @todo new status code? */ \
+                return VINF_IEM_REEXEC_BREAK; \
             } \
             else \
@@ -311 +311 @@
                   pVCpu->iem.s.GCPhysInstrBuf + off, GCPhysRangePageWithOffset, pVCpu->iem.s.pbInstrBuf, __LINE__)); \
             RT_NOREF(a_cbInstr); \
-            return VINF_IEM_REEXEC_MODE_CHANGED; /** @todo new status code? */ \
+            return VINF_IEM_REEXEC_BREAK; \
         } \
     } while(0)

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

@@ -1475 +1475 @@
             /* Some status codes are just to get us out of this loop and
                continue in a different translation block. */
-            if (rcStrict == VINF_IEM_REEXEC_MODE_CHANGED)
+            if (rcStrict == VINF_IEM_REEXEC_BREAK)
                 return iemExecStatusCodeFiddling(pVCpu, VINF_SUCCESS);
             return iemExecStatusCodeFiddling(pVCpu, rcStrict);