Changeset 42761 in vbox

Timestamp:

Aug 10, 2012 6:23:20 PM (12 years ago)

Author:

vboxsync

Message:

IEM: Bail out on REP STOS and other string ops when handler bypassing is enabled. (Will fix later.)

File:

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

trunk/src/VBox/VMM/VMMAll/IEMAllCImplStrInstr.cpp.h

@@ -654 +654 @@
 
     /*
+     * Be careful with handle bypassing.
+     */
+    if (pIemCpu->fBypassHandlers)
+    {
+        Log(("%s: declining because we're bypassing handlers\n", __FUNCTION__));
+        return VERR_IEM_ASPECT_NOT_IMPLEMENTED;
+    }
+
+    /*
      * If we're reading back what we write, we have to let the verfication code
      * to prevent a false positive.
      * Note! This doesn't take aliasing or wrapping into account - lazy bird.
      */
-#ifdef IEM_VERIFICATION_MODE
+#ifdef IEM_VERIFICATION_MODE_FULL
     if (   IEM_VERIFICATION_ENABLED(pIemCpu)
         && (cbIncr > 0
@@ -804 +813 @@
 
     /*
+     * Be careful with handle bypassing.
+     */
+    /** @todo Permit doing a page if correctly aligned. */
+    if (pIemCpu->fBypassHandlers)
+    {
+        Log(("%s: declining because we're bypassing handlers\n", __FUNCTION__));
+        return VERR_IEM_ASPECT_NOT_IMPLEMENTED;
+    }
+
+    /*
      * The loop.
      */
@@ -1016 +1035 @@
     PCPUMCTX        pCtx = pIemCpu->CTX_SUFF(pCtx);
     VBOXSTRICTRC    rcStrict;
+
+    /*
+     * Be careful with handle bypassing.
+     */
+    if (pIemCpu->fBypassHandlers)
+    {
+        Log(("%s: declining because we're bypassing handlers\n", __FUNCTION__));
+        return VERR_IEM_ASPECT_NOT_IMPLEMENTED;
+    }
 
     /*
@@ -1087 +1115 @@
     int8_t const    cbIncr      = pCtx->eflags.Bits.u1DF ? -(OP_SIZE / 8) : (OP_SIZE / 8);
     ADDR_TYPE       uAddrReg    = pCtx->ADDR_rDI;
+
+    /*
+     * Be careful with handle bypassing.
+     */
+    if (pIemCpu->fBypassHandlers)
+    {
+        Log(("%s: declining because we're bypassing handlers\n", __FUNCTION__));
+        return VERR_IEM_ASPECT_NOT_IMPLEMENTED;
+    }
 
     /*