Changeset 60185 in vbox for trunk/src/VBox/VMM/include

Timestamp:

Mar 24, 2016 5:39:40 PM (9 years ago)

Author:

vboxsync

Message:

IEM: Fixed a couple of edge cases and broken verification mode.

• Update enmCpuMode after loading hidden CS flags (prep for recompiling).
• Fixed retf in 64-bit mode where we would load CS.BASE with zero when returning to 16-bit or 32-bit code.
• Fixed ESP/SP handling for protected mode exception injection.
• Implemented the two string I/O notification functions that would assert in verification mode.
• The IEMExec* methods must call iemUninitExec to undo poisoning of decoding data members as it will otherwise interfere with verification mode opcode fetching optimizations and other stuff.

The above makes the current bs3-cpu-basic-2 code work in --execute-all-in-iem mode.

File:

• trunk/src/VBox/VMM/include/IEMInternal.h (modified) (5 diffs)

trunk/src/VBox/VMM/include/IEMInternal.h

@@ -175 +175 @@
     IEMVERIFYEVENT_IOPORT_READ,
     IEMVERIFYEVENT_IOPORT_WRITE,
+    IEMVERIFYEVENT_IOPORT_STR_READ,
+    IEMVERIFYEVENT_IOPORT_STR_WRITE,
     IEMVERIFYEVENT_RAM_WRITE,
     IEMVERIFYEVENT_RAM_READ
@@ -198 +200 @@
         {
             RTIOPORT    Port;
-            uint32_t    cbValue;
+            uint8_t     cbValue;
         } IOPortRead;
 
@@ -205 +207 @@
         {
             RTIOPORT    Port;
-            uint32_t    cbValue;
+            uint8_t     cbValue;
             uint32_t    u32Value;
         } IOPortWrite;
+
+        /** IEMVERIFYEVENT_IOPORT_STR_READ */
+        struct
+        {
+            RTIOPORT    Port;
+            uint8_t     cbValue;
+            RTGCUINTREG cTransfers;
+        } IOPortStrRead;
+
+        /** IEMVERIFYEVENT_IOPORT_STR_WRITE */
+        struct
+        {
+            RTIOPORT    Port;
+            uint8_t     cbValue;
+            RTGCUINTREG cTransfers;
+        } IOPortStrWrite;
 
         /** IEMVERIFYEVENT_RAM_READ */
@@ -303 +321 @@
      * This is used to skip past really slow bits.  */
     bool                    fNoRem;
+    /** Saved fNoRem flag used by #iemInitExec and #iemUninitExec. */
+    bool                    fNoRemSavedByExec;
     /** Indicates that RAX and RDX differences should be ignored since RDTSC
      *  and RDTSCP are timing sensitive.  */
@@ -488 +508 @@
 /** Used in aMemMappings to indicate that the entry is bounce buffered. */
 #define IEM_ACCESS_BOUNCE_BUFFERED      UINT32_C(0x00000200)
+/** Valid bit mask. */
+#define IEM_ACCESS_VALID_MASK           UINT32_C(0x000003ff)
 /** Read+write data alias. */
 #define IEM_ACCESS_DATA_RW              (IEM_ACCESS_TYPE_READ  | IEM_ACCESS_TYPE_WRITE | IEM_ACCESS_WHAT_DATA)