Changeset 8984 in vbox for trunk

Timestamp:

May 20, 2008 8:59:45 PM (17 years ago)

Author:

vboxsync

Message:

Removed the dtrace experiment. the invalid lock sequences now ends up as OP_ILLUD2 after disassembly.

File:

• trunk/src/VBox/VMM/PATM/PATM.cpp (modified) (3 diffs)

trunk/src/VBox/VMM/PATM/PATM.cpp

@@ -1376 +1376 @@
         return VINF_SUCCESS;
 
-#ifdef DTRACE_EXPERIMENT
-    case OP_MOV: /* dtrace changes push ebp; mov ebp, esp; into lock mov ebp, esp */
-        if (pCpu->prefix & PREFIX_LOCK)
-        {
-            Log(("illegal lock sequence -> return here\n"));
-            return VINF_SUCCESS;
-        }
-#endif
-
     case OP_STI:
     case OP_POPF:
@@ -1524 +1515 @@
         return VINF_SUCCESS;
 
-#ifdef DTRACE_EXPERIMENT
-    case OP_MOV: /* dtrace changes push ebp; mov ebp, esp; into lock mov ebp, esp */
-        if (pCpu->prefix & PREFIX_LOCK)
-        {
-            Log(("illegal lock sequence -> return here\n"));
-            return VINF_SUCCESS;
-        }
-#endif
-
     case OP_IRET:
     case OP_SYSEXIT: /* will fault or emulated in GC */
@@ -1671 +1653 @@
 
     case OP_MOV:
-#ifdef DTRACE_EXPERIMENT
-        /* dtrace changes push ebp; mov ebp, esp; into lock mov ebp, esp */
-        if (pCpu->prefix & PREFIX_LOCK)
-        {
-            rc = patmPatchGenIllegalInstr(pVM, pPatch);
-            Log(("illegal lock sequence -> return here\n"));
-            if (VBOX_SUCCESS(rc))
-                rc = VINF_SUCCESS;  /* exit point by definition */
-            break;
-        }
-#endif
         if (pCpu->pCurInstr->optype & OPTYPE_POTENTIALLY_DANGEROUS)
         {