Changeset 8849

Timestamp:

05/15/08 15:16:14 (2 months ago)

Author:

vboxsync

Message:

load the modrm value before parsing it (CMPXCHG8B/16B). Updated s->pc comment.

Files:

• trunk/src/recompiler/target-i386/translate.c (modified) (3 diffs)

trunk/src/recompiler/target-i386/translate.c

@@ -3238 +3238 @@
-static bool is_invalid_lock_sequence(DisasContext *s, target_ulong pc_start, int b)
-{
-0
+1
-    target_ulong pc = s->pc;
-    int modrm, mod, op;
@@ -3338 +3338 @@
-                /* /1: CMPXCHG8B mem64 or CMPXCHG16B mem128 */
-                case 0xc7:
+                    modrm = ldub_code(pc++);
-                    op = (modrm >> 3) & 7;
-                    if (op != 1)
@@ -3346 +3347 @@
-    }
-
-
+
+
-    Log(("illegal lock sequence %VGv (b=%#x)\n", pc_start, b));
-    /* exception 6 (UD) is a fault, therefore the PC must not be changed */
-    return true;
-#else